Skip to content

Commit 8362f9f

Browse files
committed
Clean up test description generators
1 parent 4137ee4 commit 8362f9f

File tree

2 files changed

+5
-12
lines changed

2 files changed

+5
-12
lines changed

ur_robot_driver/test/test_common.py

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -376,7 +376,6 @@ def generate_mock_hardware_test_description(
376376
tf_prefix="",
377377
initial_joint_controller="scaled_joint_trajectory_controller",
378378
controller_spawner_timeout=TIMEOUT_WAIT_SERVICE_INITIAL,
379-
use_mock_hardware="false",
380379
):
381380

382381
ur_type = LaunchConfiguration("ur_type")
@@ -390,8 +389,8 @@ def generate_mock_hardware_test_description(
390389
"headless_mode": "true",
391390
"launch_dashboard_client": "true",
392391
"start_joint_controller": "false",
393-
"use_mock_hardware": use_mock_hardware,
394-
"mock_sensor_commands": use_mock_hardware,
392+
"use_mock_hardware": "true",
393+
"mock_sensor_commands": "true",
395394
}
396395
if tf_prefix:
397396
launch_arguments["tf_prefix"] = tf_prefix
@@ -412,7 +411,6 @@ def generate_driver_test_description(
412411
tf_prefix="",
413412
initial_joint_controller="scaled_joint_trajectory_controller",
414413
controller_spawner_timeout=TIMEOUT_WAIT_SERVICE_INITIAL,
415-
use_mock_hardware="false",
416414
):
417415
ur_type = LaunchConfiguration("ur_type")
418416

@@ -425,8 +423,6 @@ def generate_driver_test_description(
425423
"headless_mode": "true",
426424
"launch_dashboard_client": "true",
427425
"start_joint_controller": "false",
428-
"use_mock_hardware": use_mock_hardware,
429-
"mock_sensor_commands": use_mock_hardware,
430426
}
431427
if tf_prefix:
432428
launch_arguments["tf_prefix"] = tf_prefix

ur_robot_driver/test/test_mock_hardware.py

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -56,11 +56,9 @@
5656

5757

5858
@pytest.mark.launch_test
59-
@launch_testing.parametrize("tf_prefix, use_mock_hardware", [("", "true"), ("my_ur_", "true")])
60-
def generate_test_description(tf_prefix, use_mock_hardware):
61-
return generate_mock_hardware_test_description(
62-
tf_prefix=tf_prefix, use_mock_hardware=use_mock_hardware
63-
)
59+
@launch_testing.parametrize("tf_prefix", [(""), ("my_ur_")])
60+
def generate_test_description(tf_prefix):
61+
return generate_mock_hardware_test_description(tf_prefix=tf_prefix)
6462

6563

6664
class RobotDriverTest(unittest.TestCase):
@@ -70,7 +68,6 @@ def setUpClass(cls, use_mock_hardware):
7068
rclpy.init()
7169
cls.node = Node("mock_hardware_test")
7270
time.sleep(1)
73-
cls.mock_hardware = use_mock_hardware == "true"
7471
cls.init_robot(cls)
7572

7673
@classmethod

0 commit comments

Comments
 (0)