[ros_bridge][py] Add back CollisionDetector as an option.

Now that RTCs to activate is selectable from the end client (e.g. hironx.py), we need to expose all RTCs available. [ros_bridge][test][py] Refactoring.
start-jsk · Apr 29, 2017 · 83e50bc · 83e50bc
1 parent 94196e7
commit 83e50bc
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 14 deletions.
diff --git a/hironx_ros_bridge/src/hironx_ros_bridge/hironx_client.py b/hironx_ros_bridge/src/hironx_ros_bridge/hironx_client.py
@@ -316,7 +316,7 @@ class via the link above; nicely formatted api doc web page
             ['fk', "ForwardKinematics"],
             ['ic', "ImpedanceController"],
             ['el', "SoftErrorLimiter"],
-            # ['co', "CollisionDetector"],
+            ['co', "CollisionDetector"],
             ['sc', "ServoController"],
             ['log', "DataLogger"],
         ]

diff --git a/hironx_ros_bridge/src/hironx_ros_bridge/testutil/test_hironx_client.py b/hironx_ros_bridge/src/hironx_ros_bridge/testutil/test_hironx_client.py
@@ -41,18 +41,6 @@
 
 class TestHiroClient(TestHiro):
 
-    _RTC_LIST = [
-            ['seq', "SequencePlayer"],
-            ['sh', "StateHolder"],
-            ['fk', "ForwardKinematics"],
-            ['ic', "ImpedanceController"],
-            ['el', "SoftErrorLimiter"],
-            ['sc', "ServoController"],
-            ['log', "DataLogger"],
-            # rmfo will be automatically added in getRTCList.
-            ['rmfo', 'RemoveForceSensorLinkOffset']
-        ]
-
     _RTC_LIST_CUSTOM = [
             ['seq', "SequencePlayer"],
             ['sh', "StateHolder"],
@@ -80,7 +68,9 @@ def test_getRTCList(self):
         '''
         self.assertTrue(
             self._compare_2dlist(
-                self.robot.getRTCList(), self._RTC_LIST))
+                self.robot.getRTCList(),
+                # Accessing a private member var only for testing purpose. 
+                self.robot._RTC_list))
 
     def test_getRTCList_customrtcs_args_correct(self):
         '''