phosphor-time-manager/test, branch master

clang-format: Update to match docs repo

2018-10-08T06:07:30+00:00

Update the .clang-format file and run clang-format-6.0. This .clang-format matches the example one in https://github.com/openbmc/docs/blob/master/cpp-style-and-conventions.md#clang-formatting Change-Id: Ic2c462525eb27b8295c2b298871e04268d93faf2 Signed-off-by: Gunnar Mills

Enable link time optimization

2018-09-19T02:01:45+00:00

To enable link time optimization, several configures needs to be changed: 1. Add -flto in Makefiles 2. Use C++17 to get inline constexpr 3. Use gcc-ar and gcc-ranlib to generate static library, otherwise it gets link errors. Tested: Build OK and unit tests passes. Change-Id: Ic58cbfb631c97f43c82d5669cbf6ac1ebc5e0ab9 Signed-off-by: Lei YU

Use proper errors when failing to set time

2018-09-12T02:45:04+00:00

Previously it reports InsufficientPermission when it is not allowed to set time. Now phosphor-dbus-interfaces defines proper errors for such case, so report NotAllowed error when it is not allowed to set time, and report Failed error when it failed to set time. Tested: Get NotAllowed and Failed error with expected metadata from journal log. Change-Id: I53610bf27ffc3f62608cea6fd0e66ca859d94675 Signed-off-by: Lei YU

Use correct gtest/gmock flags

2018-09-12T02:44:55+00:00

The test Makefile.am was using -lgmock -lgmock_main directly, which cause link issues on compiling test. Fix this by using the flags specified by gtest package. Tested: Verify the CI passes build. Change-Id: Id51134d7e2696e18f7439f15f8d899edba51856e Signed-off-by: Lei YU

Throw excpetion when it is not allowed to set time

2018-06-13T03:30:20+00:00

When it is not allowed to set time depending on the time setting, previously we only log an error and continue. Now sdbusplus supports errors on properties, so we can throw exception on such case. Tested: Verify in unittest that exception is thrown when it is not allowed to set time. Verify in BMC that busctl gets the error message when it is not allowed to set time. Change-Id: I4a04d1aa8c081abf0f9fd449118dc1107e12f689 Signed-off-by: Lei YU

Spelling fixes

2017-10-26T01:45:50+00:00

Change-Id: Ia840360c00cc8d7baaafd797ae2411bc5cc03ed3 Signed-off-by: Gunnar Mills

Use host state object to check if host is on

2017-10-16T12:41:02+00:00

The code was using pgood to determine if host is on or off. Now we have host state object, which is a more appropriate way to check host on/off. So change pgood related code to use host state object. Change-Id: I553c1a40922ca2e8bc6904688c55e85971bd4720 Signed-off-by: Lei YU

Let timedate1 to decide on setting time in NTP mode

2017-10-16T12:41:02+00:00

In NTP mode, let timedate1 to set BMC time, for now, it results in error. Change-Id: I2d78cdcfe6868642f43198dd932b20b13fd8523d Signed-off-by: Lei YU

Use new settings API

2017-10-16T12:41:01+00:00

The new time manager code was using the old settings daemon. Now it uses the new settings API. Change-Id: Id551d97c28a6cfbb81c87118b26292b1b5574e93 Signed-off-by: Lei YU

Use time mode owner enums in dbus interface

2017-10-16T12:40:04+00:00

Dbus interface defines time mode owners in xyz::openbmc_project::Time. Use the enums from the interface instead of repo defined enums and use the generated code to do convertions between strings and enums. Update unit tests accordingly. Change-Id: Ic304aa3b4137375d208bb1702e0f64df512fc5a0 Signed-off-by: Lei YU