Merge pull request #847 from kopp/fix_docu_nosetests_846

Minor changes in docu: Build messages for test
ros · Jan 20, 2017 · 1843178 · 1843178
2 parents 51ef314 + 2119c2f
commit 1843178
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 0 deletions.
diff --git a/doc/howto/format1/python_nose_configuration.rst b/doc/howto/format1/python_nose_configuration.rst
@@ -26,6 +26,20 @@ You can also let nosetest find all tests recursively::
     catkin_add_nosetests(tests)
   endif()
 
+If you used a message of the current package in the nosetest, make sure to
+specify the nosetest like this::
+
+  catkin_add_nosetests(tests/test_your_node.py
+                       DEPENDENCIES ${${PROJECT_NAME}_EXPORTED_TARGETS})
+
+If you used messages from other packages, use::
+
+  catkin_add_nosetests(tests/test_your_node.py
+                       DEPENDENCIES ${catkin_EXPORTED_TARGETS})
+
+The test will then make sure that the messages used in the test are built
+before they are used.
+
 For more info, please have a look at the :ref:`API <catkin_add_nosetests_ref>`.
 
 .. _Nosetest: http://www.ros.org/wiki/nosetest

diff --git a/doc/howto/format2/python_nose_configuration.rst b/doc/howto/format2/python_nose_configuration.rst
@@ -26,6 +26,20 @@ You can also let nosetest find all tests recursively::
     catkin_add_nosetests(tests)
   endif()
 
+If you used a message of the current package in the nosetest, make sure to
+specify the nosetest like this::
+
+  catkin_add_nosetests(tests/test_your_node.py
+                       DEPENDENCIES ${${PROJECT_NAME}_EXPORTED_TARGETS})
+
+If you used messages from other packages, use::
+
+  catkin_add_nosetests(tests/test_your_node.py
+                       DEPENDENCIES ${catkin_EXPORTED_TARGETS})
+
+The test will then make sure that the messages used in the test are built
+before they are used.
+
 For more info, please have a look at the :ref:`API <catkin_add_nosetests_ref>`.
 
 .. _Nosetest: http://www.ros.org/wiki/nosetest