init (#1358)

Signed-off-by: Michael Carlstrom <[email protected]>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Author: InvincibleRMC

rclpy/test/test_action_client.py

@@ -31,7 +31,7 @@
 TIME_FUDGE = 0.3
 
 
-class MockActionServer():
+class MockActionServer:
 
     def __init__(self, node):
         self.goal_srv = node.create_service(
@@ -82,7 +82,7 @@ def tearDownClass(cls):
         cls.node.destroy_node()
         rclpy.shutdown(context=cls.context)
 
-    def setUp(self):
+    def setUp(self) -> None:
         self.feedback = None
 
     def feedback_callback(self, feedback):
@@ -93,12 +93,12 @@ def timed_spin(self, duration):
         while (time.time() - start_time) < duration:
             rclpy.spin_once(self.node, executor=self.executor, timeout_sec=0.1)
 
-    def test_constructor_defaults(self):
+    def test_constructor_defaults(self) -> None:
         # Defaults
         ac = ActionClient(self.node, Fibonacci, 'fibonacci')
         ac.destroy()
 
-    def test_constructor_no_defaults(self):
+    def test_constructor_no_defaults(self) -> None:
         ac = ActionClient(
             self.node,
             Fibonacci,
@@ -111,7 +111,7 @@ def test_constructor_no_defaults(self):
         )
         ac.destroy()
 
-    def test_get_num_entities(self):
+    def test_get_num_entities(self) -> None:
         ac = ActionClient(self.node, Fibonacci, 'fibonacci')
         num_entities = ac.get_num_entities()
         self.assertEqual(num_entities.num_subscriptions, 2)
@@ -121,7 +121,7 @@ def test_get_num_entities(self):
         self.assertEqual(num_entities.num_services, 0)
         ac.destroy()
 
-    def test_wait_for_server_nowait(self):
+    def test_wait_for_server_nowait(self) -> None:
         ac = ActionClient(self.node, Fibonacci, 'not_fibonacci')
         try:
             start = time.monotonic()
@@ -132,7 +132,7 @@ def test_wait_for_server_nowait(self):
         finally:
             ac.destroy()
 
-    def test_wait_for_server_timeout(self):
+    def test_wait_for_server_timeout(self) -> None:
         ac = ActionClient(self.node, Fibonacci, 'not_fibonacci')
         try:
             start = time.monotonic()
@@ -143,7 +143,7 @@ def test_wait_for_server_timeout(self):
         finally:
             ac.destroy()
 
-    def test_wait_for_server_exists(self):
+    def test_wait_for_server_exists(self) -> None:
         ac = ActionClient(self.node, Fibonacci, 'fibonacci')
         try:
             start = time.monotonic()
@@ -154,7 +154,7 @@ def test_wait_for_server_exists(self):
         finally:
             ac.destroy()
 
-    def test_send_goal_async(self):
+    def test_send_goal_async(self) -> None:
         ac = ActionClient(self.node, Fibonacci, 'fibonacci')
         try:
             self.assertTrue(ac.wait_for_server(timeout_sec=2.0))
@@ -166,7 +166,7 @@ def test_send_goal_async(self):
         finally:
             ac.destroy()
 
-    def test_send_goal_async_with_feedback_after_goal(self):
+    def test_send_goal_async_with_feedback_after_goal(self) -> None:
         ac = ActionClient(self.node, Fibonacci, 'fibonacci')
         try:
             self.assertTrue(ac.wait_for_server(timeout_sec=2.0))
@@ -186,7 +186,7 @@ def test_send_goal_async_with_feedback_after_goal(self):
         finally:
             ac.destroy()
 
-    def test_send_goal_async_with_feedback_before_goal(self):
+    def test_send_goal_async_with_feedback_before_goal(self) -> None:
         ac = ActionClient(self.node, Fibonacci, 'fibonacci')
         try:
             self.assertTrue(ac.wait_for_server(timeout_sec=2.0))
@@ -209,7 +209,7 @@ def test_send_goal_async_with_feedback_before_goal(self):
         finally:
             ac.destroy()
 
-    def test_send_goal_async_with_feedback_after_goal_result_requested(self):
+    def test_send_goal_async_with_feedback_after_goal_result_requested(self) -> None:
         ac = ActionClient(self.node, Fibonacci, 'fibonacci')
         try:
             self.assertTrue(ac.wait_for_server(timeout_sec=2.0))
@@ -235,7 +235,7 @@ def test_send_goal_async_with_feedback_after_goal_result_requested(self):
         finally:
             ac.destroy()
 
-    def test_send_goal_async_with_feedback_for_another_goal(self):
+    def test_send_goal_async_with_feedback_for_another_goal(self) -> None:
         ac = ActionClient(self.node, Fibonacci, 'fibonacci')
         try:
             self.assertTrue(ac.wait_for_server(timeout_sec=2.0))
@@ -266,7 +266,7 @@ def test_send_goal_async_with_feedback_for_another_goal(self):
         finally:
             ac.destroy()
 
-    def test_send_goal_async_with_feedback_for_not_a_goal(self):
+    def test_send_goal_async_with_feedback_for_not_a_goal(self) -> None:
         ac = ActionClient(self.node, Fibonacci, 'fibonacci')
         try:
             self.assertTrue(ac.wait_for_server(timeout_sec=2.0))
@@ -286,7 +286,7 @@ def test_send_goal_async_with_feedback_for_not_a_goal(self):
         finally:
             ac.destroy()
 
-    def test_send_goal_multiple(self):
+    def test_send_goal_multiple(self) -> None:
         ac = ActionClient(
             self.node,
             Fibonacci,
@@ -310,7 +310,7 @@ def test_send_goal_multiple(self):
         finally:
             ac.destroy()
 
-    def test_send_goal_async_no_server(self):
+    def test_send_goal_async_no_server(self) -> None:
         ac = ActionClient(self.node, Fibonacci, 'not_fibonacci')
         try:
             future = ac.send_goal_async(Fibonacci.Goal())
@@ -319,7 +319,7 @@ def test_send_goal_async_no_server(self):
         finally:
             ac.destroy()
 
-    def test_send_cancel_async(self):
+    def test_send_cancel_async(self) -> None:
         ac = ActionClient(self.node, Fibonacci, 'fibonacci')
         try:
             self.assertTrue(ac.wait_for_server(timeout_sec=2.0))
@@ -340,7 +340,7 @@ def test_send_cancel_async(self):
         finally:
             ac.destroy()
 
-    def test_get_result_async(self):
+    def test_get_result_async(self) -> None:
         ac = ActionClient(self.node, Fibonacci, 'fibonacci')
         try:
             self.assertTrue(ac.wait_for_server(timeout_sec=2.0))
@@ -358,7 +358,7 @@ def test_get_result_async(self):
         finally:
             ac.destroy()
 
-    def test_different_type_raises(self):
+    def test_different_type_raises(self) -> None:
         ac = ActionClient(self.node, Fibonacci, 'fibonacci')
         try:
             with self.assertRaises(TypeError):

rclpy/test/test_action_graph.py

@@ -87,7 +87,7 @@ def get_names_and_types(
             end = time.monotonic()
         assert len(nat) == expected_num_names
 
-    def test_get_action_client_names_and_types_by_node(self):
+    def test_get_action_client_names_and_types_by_node(self) -> None:
         self.get_names_and_types(
             get_action_client_names_and_types_by_node,
             self.node1,
@@ -128,7 +128,7 @@ def test_get_action_client_names_and_types_by_node(self):
         assert TEST_NAMESPACE2 + '/' + TEST_ACTION0 in [name20, name21]
         assert TEST_NAMESPACE2 + '/' + TEST_ACTION1 in [name20, name21]
 
-    def test_get_action_server_names_and_types_by_node(self):
+    def test_get_action_server_names_and_types_by_node(self) -> None:
         self.get_names_and_types(
             get_action_server_names_and_types_by_node,
             self.node1,
@@ -170,7 +170,7 @@ def test_get_action_server_names_and_types_by_node(self):
         assert TEST_NAMESPACE2 + '/' + TEST_ACTION0 in [name20, name21]
         assert TEST_NAMESPACE2 + '/' + TEST_ACTION1 in [name20, name21]
 
-    def test_get_action_names_and_types(self):
+    def test_get_action_names_and_types(self) -> None:
         names_and_types = self.get_names_and_types(
             get_action_names_and_types,
             self.node0,