search

gazebosim / gazebo-classic

Gazebo classic. For the latest version, see https://github.com/gazebosim/gz-sim
http://classic.gazebosim.org/
Other
1.2k stars 484 forks source link

Add tests to conda-forge job #3264

Closed traversaro closed 2 years ago

traversaro commented 2 years ago

In https://github.com/conda-forge/gazebo-feedstock/issues/148 we debugged quite a tricky issue that only appeared at runtime. To avoid having problems like this in the future, I tried to enable tests (skipping the one that are currently failing) in the conda-forge job.

As the conda-forge CI job is already quite slow (~1 hour and 15 minutes), we can also think of changing the job to run the tests only on a given schedule, but for the time being I just implemented the simplest solution.

traversaro commented 2 years ago

It seems that there is some sort of cross-talking, so even if after one round I disable all failing test, and the following round more test fail. So probably for the moment it is a better idea just to run a few specified tests.

traversaro commented 2 years ago

Ok, by running just the INTEGRATION_transport_msg_count I reproduced the problem that we have in https://github.com/conda-forge/gazebo-feedstock/pull/152 .

traversaro commented 2 years ago

Apparently the problem in INTEGRATION_transport_msg_count on macOS will be solved by https://github.com/gazebosim/gazebo-classic/pull/3269 , but even with that fix the test is not passing in CI for some reason. Anyhow, we can close the PR for now.