You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Related to #240, this package really needs some more intense workspace-level integration testing. One road forward on this could be to beef up the existing toy test workspace.
Unfortunately, several of the behaviours which matter the most come down to how packages like catkin itself, genmsg, etc. interact with each other, so it seems like it's going to be a bit of a mess basically re-implementing those behaviours inside test workspaces to test them.
I'm wondering about setting up another kind of test regime, which actually downloads sources for some of these external packages, and then exercises the different build configurations (devel, isolated devel, install, isolated install). This could be integrated with the existing nosetests, or left as a separate framework (for example, to be executed under travis, etc).
The text was updated successfully, but these errors were encountered:
A ton of workspace-level tests have been added with the pre-0.4.0 PRs, so I'm going to move this to the 1.0.0 milestone. We should look at doing some real coverage as well, and we can always have more tests.
Related to #240, this package really needs some more intense workspace-level integration testing. One road forward on this could be to beef up the existing toy test workspace.
Unfortunately, several of the behaviours which matter the most come down to how packages like catkin itself, genmsg, etc. interact with each other, so it seems like it's going to be a bit of a mess basically re-implementing those behaviours inside test workspaces to test them.
I'm wondering about setting up another kind of test regime, which actually downloads sources for some of these external packages, and then exercises the different build configurations (devel, isolated devel, install, isolated install). This could be integrated with the existing nosetests, or left as a separate framework (for example, to be executed under travis, etc).
The text was updated successfully, but these errors were encountered: