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.

Sign up for GitHub

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

Jump to bottom

integrate testing #11

Closed
straszheim opened this issue Dec 5, 2011 · 4 comments
Closed

integrate testing #11

straszheim opened this issue Dec 5, 2011 · 4 comments
Assignees
@gerkey

Comments

@straszheim
Copy link

integration of unit tests. test results go only in build directory. Nothing is ever written to home directory. Python unit tests TBD... do we just pivot off of some definition of tests in setup.py and leave it at that?
@straszheim
Copy link
Author

first we need a cmake interface, then Brian and Ken make the ros utils work.

@ghost ghost assigned gerkey Jan 5, 2012
@kwc
Copy link
Member

kwc commented Jan 6, 2012

nosetests, gtest, and pyunit integrated. Brian is now working on rostest. Current implementation only allows tests at the top-level, which is not ideal.

@gerkey
Copy link
Contributor

gerkey commented Jan 11, 2012

all types of tests (nose,gtest,pyunit,rostest) are now enabled, with post-test verification of the generation of result files (to catch compile failures and program crashes).

The top-level-only 'make test' target remains an issue.

@gerkey
Copy link
Contributor

gerkey commented Jan 15, 2012

There's already a per-project test target. You can do this:
make (package_or_stack)_run_tests
And you'll get just those tests.

At this point, testing integration is basically done. Ticket specific problem separately.

@gerkey gerkey closed this as completed Jan 15, 2012
cwecht pushed a commit to cwecht/catkin that referenced this issue Mar 20, 2018
@wohe
Fix support for 2D SLAM without IMU. (ros#11)
d2b583d 
Do not subscribe to the IMU topic and expect IMU data
if expect_imu_data is set to false in the Lua configuration.
johnsonshih added a commit to johnsonshih/catkin that referenced this issue Sep 20, 2018
@johnsonshih
Provide top level devel space environment and setup bat files for Win…
de1beee 
…dows (ros#11)
dirk-thomas pushed a commit that referenced this issue Jan 28, 2019
@kejxu @dirk-thomas
update builder.py to add Windows support (#982)
b2f73e0 
* Create env.bat and setup.bat in build_cmake_package for Windows (#9)

* Skip the drive letter if run on Windows (#10)

* Skip the drive letter if run on Windows

* Remove unnecessary check for win32

* Provide top level devel space environment and setup bat files for Windows (#11)

* use os.pathsep to join path components

* revert whitespace change

* Fix path backslashes in catkin_package.cmake (#25)

* Update catkin_package.cmake

* Update catkin_package.cmake

* Update builder.py

* Normalize the path from subs['cmake_prefix_path']

* Revert the unrelated changes from my bad merge in the previous commit.

Revert the unrelated changes from my bad merge in the previous commit to keep this PR clean.

* use underscope to exclude drive letter, and clarify there's no effect on Linux (#31)

* Define string templates in separate steps and update comment (#32)

* Create env.bat and setup.bat in build_cmake_package for Windows (#9)

* define string templates in a separate step

* change script writer function names

* fix indentation for script template

* fix env script name typo (#33)

* minor nitpicks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gerkey gerkey

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@straszheim @kwc @gerkey