ros-simulation#408 The require isn't supported, the wait for publishe…

…rs is redundant, also commenting out the test since it can't be run_tested
lucasw · Mar 22, 2016 · 7ba4409 · 7ba4409
1 parent 2c2ba68
commit 7ba4409
Show file tree

Hide file tree

Showing 8 changed files with 49 additions and 13 deletions.
diff --git a/gazebo_plugins/CMakeLists.txt b/gazebo_plugins/CMakeLists.txt
@@ -344,8 +344,9 @@ if (CATKIN_ENABLE_TESTING)
                     test/set_model_state_test/set_model_state_test.cpp)
   target_link_libraries(set_model_state-test ${catkin_LIBRARIES})
 
-  add_rostest_gtest(camera-test
-                    test/camera/camera.test
-                    test/camera/camera.cpp)
-  target_link_libraries(camera-test ${catkin_LIBRARIES})
+  # Can't run this and the above test together
+  # add_rostest_gtest(camera-test
+  #                   test/camera/camera.test
+  #                   test/camera/camera.cpp)
+  # target_link_libraries(camera-test ${catkin_LIBRARIES})
 endif()
diff --git a/gazebo_plugins/test/camera/camera.cpp b/gazebo_plugins/test/camera/camera.cpp
@@ -1,5 +1,5 @@
-#include <image_transport/image_transport.h>
 #include <gtest/gtest.h>
+#include <image_transport/image_transport.h>
 #include <ros/ros.h>
 
 class CameraTest : public testing::Test
@@ -31,6 +31,7 @@ TEST_F(CameraTest, cameraSubscribeTest)
                           &CameraTest::imageCallback,
                           dynamic_cast<CameraTest*>(this));
 
+#if 0
   // wait for gazebo to start publishing
   // TODO(lucasw) this isn't really necessary since this test
   // is purely passive
@@ -45,6 +46,7 @@ TEST_F(CameraTest, cameraSubscribeTest)
        wait_for_topic = false;
     ros::Duration(0.5).sleep();
   }
+#endif
 
   while (!has_new_image_)
   {

diff --git a/gazebo_plugins/test/camera/camera.test b/gazebo_plugins/test/camera/camera.test
@@ -7,6 +7,5 @@
       args="-r $(find gazebo_plugins)/test/camera/camera.world" />
 
   <test test-name="camera" pkg="gazebo_plugins" type="camera-test"
-      clear_params="true" time-limit="30.0" 
-      require="true" />
+      clear_params="true" time-limit="30.0" />
 </launch>
diff --git a/gazebo_ros/test/empty.world b/gazebo_ros/test/empty.world
@@ -0,0 +1,13 @@
+<?xml version="1.0" ?>
+<sdf version="1.5">
+  <world name="default">
+    <!-- A global light source -->
+    <include>
+      <uri>model://sun</uri>
+    </include>
+    <!-- A ground plane -->
+    <include>
+      <uri>model://ground_plane</uri>
+    </include>
+  </world>
+</sdf>
diff --git a/gazebo_ros/test/gazebo_ros.test b/gazebo_ros/test/gazebo_ros.test
@@ -4,8 +4,9 @@
 
   <node name="gazebo" pkg="gazebo_ros" type="gzserver"
       respawn="false" output="screen"
-      args="-r $(find gazebo_plugins)/test/camera/empty.world" />
+      args="-r $(find gazebo_ros)/test/empty.world" />
 
   <test test-name="gazebo_ros" pkg="gazebo_ros" type="gazebo_ros-test"
-      clear_params="true" time-limit="30.0" />
+      time-limit="10.0" />
+   <!-- clear_params="true" -->
 </launch>
diff --git a/gazebo_ros/test/gazebo_ros2.test b/gazebo_ros/test/gazebo_ros2.test
@@ -4,8 +4,9 @@
 
   <node name="gazebo" pkg="gazebo_ros" type="gzserver"
       respawn="false" output="screen"
-      args="-r $(find gazebo_plugins)/test/camera/empty.world" />
+      args="-r $(find gazebo_ros)/test/empty.world" />
 
   <test test-name="gazebo_ros2" pkg="gazebo_ros" type="gazebo_ros-test"
-      clear_params="true" time-limit="30.0" />
+      time-limit="10.0" />
+  <!-- clear_params="true" -->
 </launch>
diff --git a/gazebo_ros/test/gazebo_ros3.test b/gazebo_ros/test/gazebo_ros3.test
@@ -0,0 +1,19 @@
+<?xml version="1.0"?>
+<launch>
+  <param name="/use_sim_time" value="true" />
+
+  <!-- this world file doesn't exist, gazebo handles this poorly 
+  
+ [ INFO] /gazebo /home/lucasw/catkin_ws/src/gazebo_ros_pkgs/gazebo_ros/src/gazebo_ros_api_plugin.cpp:155 : Finished loading Gazebo ROS API Plugin.
+[ INFO] /gazebo /tmp/binarydeb/ros-jade-roscpp-1.11.16/src/libros/service.cpp:80 : waitForService: Service [/gazebo/set_physics_properties] has not been advertised, waiting...
+gzserver: /usr/include/boost/thread/pthread/recursive_mutex.hpp:101: boost::recursive_mutex::~recursive_mutex(): Assertion `!pthread_mutex_destroy(&m)' failed.
+Aborted (core dumped)
+[gazebo-2] process has died [pid 16643, exit code 134, cmd /home/lucasw/catkin_ws/src/gazebo_ros_pkgs/gazebo_ros/scripts/gzserver -r /home/lucasw/catkin_ws/src/gazebo_ros_pkgs/gazebo_ros/test/empty2.world __name:=gazebo __log:=/home/lucasw/.ros/log/1a07e234-efdc-11e5-962b-00dbdf11ceac/gazebo-2.log].
+log file: /home/lucasw/.ros/log/1a07e234-efdc-11e5-962b-00dbdf11ceac/gazebo-2*.log 
+  -->
+  <node name="gazebo" pkg="gazebo_ros" type="gzserver"
+      respawn="false" output="screen"
+      args="-r $(find gazebo_ros)/test/empty2.world" />
+
+  <!-- clear_params="true" -->
+</launch>
diff --git a/gazebo_ros/test/gazebo_ros_test.cpp b/gazebo_ros/test/gazebo_ros_test.cpp
@@ -23,9 +23,9 @@ class GazeboRosTest : public testing::Test
   }
 };
 
-// Test if the camera image is published at all, and that the timestamp
+// Test if the clock is published at all, and that the timestamp
 // is not too long in the past.
-TEST_F(GazeboRosTest, cameraSubscribeTest)
+TEST_F(GazeboRosTest, clockSubscribeTest)
 {
   model_states_sub_ = nh_.subscribe("clock", 1,
                           &GazeboRosTest::clockCallback,