Fix bug on time sync test (#118)

Acuadros95 · web-flow · commit e57577661974 · 2021-03-26T10:31:30.000+01:00
* Fix bug on time sync test

* Add check to time diff
diff --git a/rmw_microxrcedds_c/test/test_rmw.cpp b/rmw_microxrcedds_c/test/test_rmw.cpp
@@ -80,7 +80,9 @@ TEST(rmw_microxrcedds, sync_session)
 
     std::time_t timestamp_seconds = std::time(nullptr);
     ASSERT_EQ(rmw_uros_sync_session(&result_ns, 500), RMW_RET_OK);
-    ASSERT_EQ(result_ns / 1000000000, timestamp_seconds);
+
+    int64_t time_diff = abs(result_ns / 1000000000 - timestamp_seconds);
+    ASSERT_LE(time_diff, 5);
 
     ASSERT_EQ(rmw_shutdown(&test_context), RMW_RET_OK);
 }

Original file line number	Diff line number	Diff line change
`@@ -80,7 +80,9 @@ TEST(rmw_microxrcedds, sync_session)`
`80`	`80`
`81`	`81`	`std::time_t timestamp_seconds = std::time(nullptr);`
`82`	`82`	`ASSERT_EQ(rmw_uros_sync_session(&result_ns, 500), RMW_RET_OK);`
`83`		`- ASSERT_EQ(result_ns / 1000000000, timestamp_seconds);`
	`83`	`+`
	`84`	`+ int64_t time_diff = abs(result_ns / 1000000000 - timestamp_seconds);`
	`85`	`+ ASSERT_LE(time_diff, 5);`
`84`	`86`
`85`	`87`	`ASSERT_EQ(rmw_shutdown(&test_context), RMW_RET_OK);`
`86`	`88`	`}`