diff options
| author | William A. Kennington III <wak@google.com> | 2018-03-01 10:53:25 -0800 |
|---|---|---|
| committer | Brad Bishop <bradleyb@fuzziesquirrel.com> | 2018-03-07 11:22:33 +0000 |
| commit | 0650a3f09da8899a742988ff55d02eebd6e796aa (patch) | |
| tree | b110719075d9f46ef608d8ffedd05d9cf91cc001 /test | |
| parent | 5e3f877179a5b56d098173afb60753b1d1e6c9af (diff) | |
| download | phosphor-watchdog-0650a3f09da8899a742988ff55d02eebd6e796aa.tar.gz phosphor-watchdog-0650a3f09da8899a742988ff55d02eebd6e796aa.zip | |
watchdog: Add a function to tell us if the timer is running
This helps us refactor some of our existing code and will be useful for
future changes.
Change-Id: Ifa1547f09997d6824f726fc5f46e15eed4c1e8c1
Signed-off-by: William A. Kennington III <wak@google.com>
Diffstat (limited to 'test')
| -rw-r--r-- | test/watchdog_test.cpp | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/test/watchdog_test.cpp b/test/watchdog_test.cpp index f846532..cb22a83 100644 --- a/test/watchdog_test.cpp +++ b/test/watchdog_test.cpp @@ -8,6 +8,7 @@ TEST_F(WdogTest, createWdogAndDontEnable) EXPECT_FALSE(wdog->enabled()); EXPECT_EQ(0, wdog->timeRemaining()); EXPECT_FALSE(wdog->timerExpired()); + EXPECT_FALSE(wdog->timerEnabled()); } /** @brief Make sure that watchdog is started and enabled */ @@ -16,6 +17,7 @@ TEST_F(WdogTest, createWdogAndEnable) // Enable and then verify EXPECT_TRUE(wdog->enabled(true)); EXPECT_FALSE(wdog->timerExpired()); + EXPECT_TRUE(wdog->timerEnabled()); // Get the configured interval auto remaining = milliseconds(wdog->timeRemaining()); @@ -26,6 +28,7 @@ TEST_F(WdogTest, createWdogAndEnable) (remaining <= defaultInterval)); EXPECT_FALSE(wdog->timerExpired()); + EXPECT_TRUE(wdog->timerEnabled()); } /** @brief Make sure that watchdog is started and enabled. @@ -40,6 +43,8 @@ TEST_F(WdogTest, createWdogAndEnableThenDisable) EXPECT_FALSE(wdog->enabled(false)); EXPECT_FALSE(wdog->enabled()); EXPECT_EQ(0, wdog->timeRemaining()); + EXPECT_FALSE(wdog->timerExpired()); + EXPECT_FALSE(wdog->timerEnabled()); } /** @brief Make sure that watchdog is started and enabled. @@ -65,6 +70,7 @@ TEST_F(WdogTest, enableWdogAndWait5Seconds) EXPECT_TRUE((remaining >= expected - defaultDrift) && (remaining <= expected)); EXPECT_FALSE(wdog->timerExpired()); + EXPECT_TRUE(wdog->timerEnabled()); } /** @brief Make sure that watchdog is started and enabled. @@ -96,7 +102,8 @@ TEST_F(WdogTest, enableWdogAndResetTo5Seconds) } } EXPECT_TRUE(wdog->timerExpired()); - EXPECT_EQ(expireTime.count() - 1 , count); + EXPECT_FALSE(wdog->timerEnabled()); + EXPECT_EQ(expireTime.count() - 1, count); // Make sure secondary callback was not called. EXPECT_FALSE(expired); @@ -138,5 +145,6 @@ TEST_F(WdogTest, enableWdogAndWaitTillEnd) EXPECT_TRUE(wdog->enabled()); EXPECT_EQ(0, wdog->timeRemaining()); EXPECT_TRUE(wdog->timerExpired()); + EXPECT_FALSE(wdog->timerEnabled()); EXPECT_EQ(expireTime.count() - 1, count); } |

