diff options
| author | Vitaly Buka <vitalybuka@google.com> | 2015-12-22 13:14:16 -0800 |
|---|---|---|
| committer | Vitaly Buka <vitalybuka@google.com> | 2015-12-23 01:53:36 +0000 |
| commit | 3d6b55295d49240a8e3575ffd8561231074714a9 (patch) | |
| tree | 36ede6964a9931e008ce3d8d28c7a33d48f32c1b /include | |
| parent | 66a01e0cd687c1e078b9166b61bf9112c70c1615 (diff) | |
| download | libweave-3d6b55295d49240a8e3575ffd8561231074714a9.tar.gz | |
Hide Config::Load method
Constructor is going to load data instead.
This makes clear that ::Load is one time operations.
Change-Id: Id7b2275544cc034d30737bea21b3d8cc68e4a8a9
Reviewed-on: https://weave-review.googlesource.com/2080
Reviewed-by: Vitaly Buka <vitalybuka@google.com>
Diffstat (limited to 'include')
| -rw-r--r-- | include/weave/provider/test/mock_config_store.h | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/include/weave/provider/test/mock_config_store.h b/include/weave/provider/test/mock_config_store.h index 01e1ef9..3873251 100644 --- a/include/weave/provider/test/mock_config_store.h +++ b/include/weave/provider/test/mock_config_store.h @@ -35,7 +35,10 @@ class MockConfigStore : public ConfigStore { return true; })); EXPECT_CALL(*this, LoadSettings()) - .WillRepeatedly(Return(R"({"device_id": "TEST_DEVICE_ID"})")); + .WillRepeatedly(Return(R"({ + "version": 1, + "device_id": "TEST_DEVICE_ID" + })")); EXPECT_CALL(*this, SaveSettings(_)).WillRepeatedly(Return()); } MOCK_METHOD1(LoadDefaults, bool(Settings*)); |