time: For tests, subtract in the right order

This was a bug in the test rather than a bug in the code the test was testing.

Use the ternary operator to replace labs.

Signed-off-by: Adam C. Emerson <aemerson@redhat.com>

diff --git a/src/test/common/test_time.cc b/src/test/common/test_time.cc
index 2e6ad4bbcbd2bdc4c9d8bebcd6d3834ce51bb61e..a54fe90b83a3af596581e60b624d2c838f793a8d 100644 (file)
--- a/src/test/common/test_time.cc
+++ b/src/test/common/test_time.cc
@@ -122,7 +122,9 @@ static void system_clock_conversions() {
 
   ASSERT_EQ(Clock::to_double(brt), bd);
   // Fudge factor
-  ASSERT_LT(abs((Clock::from_double(bd) - brt).count()), 30);
+  ASSERT_LT((Clock::from_double(bd) >  brt ?
+            Clock::from_double(bd) - brt :
+            brt - Clock::from_double(bd)).count(), 30U);
 }
 
 TEST(RealClock, Sanity) {