Remove assertions

sloretz · sloretz · commit 76fd188c4392 · 2022-10-13T18:51:37.000-07:00
Signed-off-by: Shane Loretz &lt;sloretz@osrfoundation.org&gt;
diff --git a/rclpy/test/benchmark/test_benchmark_guard_condition.py b/rclpy/test/benchmark/test_benchmark_guard_condition.py
@@ -42,9 +42,8 @@ def bm():
             while num_calls < ONE_THOUSAND:
                 gc.trigger()
                 executor.spin_once(timeout_sec=0)
-            return num_calls == ONE_THOUSAND
 
-        assert benchmark(bm)
+        benchmark(bm)
 
         executor.shutdown()
         node.destroy_guard_condition(gc)
diff --git a/rclpy/test/benchmark/test_benchmark_pub_sub.py b/rclpy/test/benchmark/test_benchmark_pub_sub.py
@@ -53,9 +53,8 @@ def bm():
             pub.publish(EmptyMsg())
             while num_calls < ONE_THOUSAND:
                 executor.spin_once(timeout_sec=0)
-            return num_calls == ONE_THOUSAND
 
-        assert benchmark(bm)
+        benchmark(bm)
 
         executor.shutdown()
         node.destroy_publisher(pub)
diff --git a/rclpy/test/benchmark/test_benchmark_timer.py b/rclpy/test/benchmark/test_benchmark_timer.py
@@ -41,9 +41,8 @@ def bm():
             num_calls = 0
             while num_calls < ONE_THOUSAND:
                 executor.spin_once(timeout_sec=0)
-            return num_calls == ONE_THOUSAND
 
-        assert benchmark(bm)
+        benchmark(bm)
 
         executor.shutdown()
         node.destroy_timer(timer)
@@ -74,9 +73,8 @@ def bm():
             num_calls = 0
             while num_calls < ONE_THOUSAND:
                 executor.spin_once(timeout_sec=0)
-            return num_calls == ONE_THOUSAND
 
-        assert benchmark(bm)
+        benchmark(bm)
 
         executor.shutdown()
         node.destroy_timer(timer)
