Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[hironx_ros_bridge] Add test to disable/enable RTCs from conf #537

Merged
merged 2 commits into from
Dec 20, 2019

Conversation

pazeshun
Copy link
Contributor

@pazeshun pazeshun commented Nov 23, 2019

Depends on #536
Test part of #535
Travis test of this PR should fail.

@pazeshun pazeshun changed the title Rtc on off tes [hironx_ros_bridge] Add test to disable/enable RTCs from conf Nov 23, 2019
@pazeshun
Copy link
Contributor Author

Travis test correctly failed:

=== /home/travis/ros/ws_rtmros_hironx/build/test_results/hironx_ros_bridge/rosunit-test_hironx_rtc_on_off.xml ===

<?xml version="1.0" encoding="utf-8"?>

<testsuite errors="0" failures="1" name="unittest.suite.TestSuite" tests="1" time="1.628">

  <testcase classname="__main__.TestHironxRtcOnOff" name="test_rtc_on_off" time="0.0020">

    <failure type="AssertionError">

  File "/usr/lib/python2.7/unittest/case.py", line 331, in run

    testMethod()

  File "/home/travis/ros/ws_rtmros_hironx/src/rtmros_hironx/hironx_ros_bridge/test/test_hironx_rtc_on_off.py", line 9, in test_rtc_on_off

    self.assertTrue(['co', "CollisionDetector"] in self.robot.getRTCList())

  File "/home/travis/build/start-jsk/rtmros_hironx/hironx_ros_bridge/test/test_hironx.py", line 227, in assertTrue

    assert(a)

    </failure>

  </testcase>

(from ROS_DISTRO=indigo ROSWS=wstool BUILDER=catkin USE_DEB=true ROS_REPOSITORY_PATH=http://packages.ros.org/ros/ubuntu)

@k-okada k-okada merged commit 2d1b1b3 into start-jsk:indigo-devel Dec 20, 2019
@pazeshun pazeshun deleted the rtc-on-off-test branch December 20, 2019 03:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants