Remove assertions

Signed-off-by: Shane Loretz <[email protected]>
ros2 · Oct 14, 2022 · 76fd188 · 76fd188
1 parent 5a0f651
commit 76fd188
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 8 deletions.
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)