commit d5d14833ca01ffefd2a81a4eacf6eff5fb2ba2ea
author William A. Kennington III <wak@google.com> Wed Feb 28 09:54:08 2018 -0800
committer William A. Kennington III <wak@google.com> Wed Feb 28 15:38:41 2018 -0800
tree 18785f5b6ebf06137e46ef2d5ebfef031ddd1af8
parent 12926435b6f4880cb178d1d43295f7638a603ab8

tests: Convert EXPECT_EQ(bool -> EXPECT_{TRUE,FALSE}

This is more in line with normal gtest code style.

Change-Id: If49f700f756ad8d278927baae4e934ef7aa399d7
Signed-off-by: William A. Kennington III <wak@google.com>

test/watchdog_test.cpp

diff --git a/test/watchdog_test.cpp b/test/watchdog_test.cpp
index 57f32e9..a76a914 100644
--- a/test/watchdog_test.cpp
+++ b/test/watchdog_test.cpp
@@ -5,17 +5,17 @@
 /** @brief Make sure that watchdog is started and not enabled */
 TEST_F(WdogTest, createWdogAndDontEnable)
 {
-    EXPECT_EQ(false, wdog.enabled());
+    EXPECT_FALSE(wdog.enabled());
     EXPECT_EQ(0, wdog.timeRemaining());
-    EXPECT_EQ(false, wdog.timerExpired());
+    EXPECT_FALSE(wdog.timerExpired());
 }
 
 /** @brief Make sure that watchdog is started and enabled */
 TEST_F(WdogTest, createWdogAndEnable)
 {
     // Enable and then verify
-    EXPECT_EQ(true, wdog.enabled(true));
-    EXPECT_EQ(false, wdog.timerExpired());
+    EXPECT_TRUE(wdog.enabled(true));
+    EXPECT_FALSE(wdog.timerExpired());
 
     // Get the configured interval
     auto remaining = milliseconds(wdog.timeRemaining());
@@ -25,7 +25,7 @@
     EXPECT_TRUE((remaining >= defaultInterval - defaultDrift) &&
                 (remaining <= defaultInterval));
 
-    EXPECT_EQ(false, wdog.timerExpired());
+    EXPECT_FALSE(wdog.timerExpired());
 }
 
 /** @brief Make sure that watchdog is started and enabled.
@@ -34,11 +34,11 @@
 TEST_F(WdogTest, createWdogAndEnableThenDisable)
 {
     // Enable and then verify
-    EXPECT_EQ(true, wdog.enabled(true));
+    EXPECT_TRUE(wdog.enabled(true));
 
     // Disable and then verify
-    EXPECT_EQ(false, wdog.enabled(false));
-    EXPECT_EQ(false, wdog.enabled());
+    EXPECT_FALSE(wdog.enabled(false));
+    EXPECT_FALSE(wdog.enabled());
     EXPECT_EQ(0, wdog.timeRemaining());
 }
 
@@ -49,7 +49,7 @@
 TEST_F(WdogTest, enableWdogAndWait5Seconds)
 {
     // Enable and then verify
-    EXPECT_EQ(true, wdog.enabled(true));
+    EXPECT_TRUE(wdog.enabled(true));
 
     // Sleep for 5 seconds
     auto sleepTime = seconds(5s);
@@ -64,7 +64,7 @@
     // how we get scheduled. So checking a range here.
     EXPECT_TRUE((remaining >= expected - defaultDrift) &&
                 (remaining <= expected));
-    EXPECT_EQ(false, wdog.timerExpired());
+    EXPECT_FALSE(wdog.timerExpired());
 }
 
 /** @brief Make sure that watchdog is started and enabled.
@@ -74,7 +74,7 @@
 TEST_F(WdogTest, enableWdogAndResetTo5Seconds)
 {
     // Enable and then verify
-    EXPECT_EQ(true, wdog.enabled(true));
+    EXPECT_TRUE(wdog.enabled(true));
 
     // Sleep for 1 second
     std::this_thread::sleep_for(1s);
@@ -95,11 +95,11 @@
             count++;
         }
     }
-    EXPECT_EQ(true, wdog.timerExpired());
+    EXPECT_TRUE(wdog.timerExpired());
     EXPECT_EQ(expireTime.count() - 1 , count);
 
     // Make sure secondary callback was not called.
-    EXPECT_EQ(false, expired);
+    EXPECT_FALSE(expired);
 }
 
 /** @brief Make sure the Interval can be updated directly.
@@ -120,7 +120,7 @@
 TEST_F(WdogTest, enableWdogAndWaitTillEnd)
 {
     // Enable and then verify
-    EXPECT_EQ(true, wdog.enabled(true));
+    EXPECT_TRUE(wdog.enabled(true));
     auto expireTime = duration_cast<seconds>(
                         milliseconds(defaultInterval));
 
@@ -135,8 +135,8 @@
             count++;
         }
     }
-    EXPECT_EQ(true, wdog.enabled());
+    EXPECT_TRUE(wdog.enabled());
     EXPECT_EQ(0, wdog.timeRemaining());
-    EXPECT_EQ(true, wdog.timerExpired());
+    EXPECT_TRUE(wdog.timerExpired());
     EXPECT_EQ(expireTime.count() - 1, count);
 }