lestarch: fixing Fw::Time to allow comparisons across different time … (

#803) * lestarch: fixing Fw::Time to allow comparisons across different time bases * lestarch: changing to proc time for zero time test
nasa · Jul 7, 2021 · 927d2f0 · 927d2f0
1 parent 4e6d413
commit 927d2f0
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 3 deletions.
diff --git a/Fw/Time/Time.cpp b/Fw/Time/Time.cpp
@@ -161,10 +161,14 @@ namespace Fw {
       )
     {
 #if FW_USE_TIME_BASE
-      FW_ASSERT(time1.getTimeBase() == time2.getTimeBase(), time1.getTimeBase(), time2.getTimeBase() );
+      if (time1.getTimeBase() != time2.getTimeBase()) {
+          return INCOMPARABLE;
+      }
 #endif
 #if FW_USE_TIME_CONTEXT
-      FW_ASSERT(time1.getContext() == time2.getContext(), time1.getContext(), time2.getContext() );
+      if (time1.getContext() != time2.getContext()) {
+          return INCOMPARABLE;
+      }
 #endif
       const U32 s1 = time1.getSeconds();
       const U32 s2 = time2.getSeconds();

diff --git a/Fw/Time/Time.hpp b/Fw/Time/Time.hpp
@@ -46,7 +46,8 @@ namespace Fw {
             typedef enum {
               LT = -1,
               EQ = 0,
-              GT = 1
+              GT = 1,
+              INCOMPARABLE = 2
             } Comparison;
 
             //! \return time zero

diff --git a/Fw/Time/test/ut/TimeTest.cpp b/Fw/Time/test/ut/TimeTest.cpp
@@ -89,6 +89,13 @@ TEST(TimeTestNominal,CopyTest) {
 
 }
 
+TEST(TimeTestNominal,ZeroTimeEquality) {
+    Fw::Time time(TB_PROC_TIME,1,2);
+    ASSERT_NE(time, Fw::ZERO_TIME);
+    Fw::Time time2;
+    ASSERT_EQ(time2, Fw::ZERO_TIME);
+}
+
 int main(int argc, char* argv[]) {
     ::testing::InitGoogleTest(&argc, argv);
     return RUN_ALL_TESTS();