Tests disabled for hpp-pinocchio, hpp-core & hpp-constraints

humanoid-path-planner / hpp-doc

Documentation for project Humanoid Path Planner

https://humanoid-path-planner.github.io/hpp-doc

ISC License

29 stars 23 forks source link

Tests disabled for hpp-pinocchio, hpp-core & hpp-constraints #29

Closed nim65s closed 5 years ago

nim65s commented 5 years ago

Hi,

The commit 1e88828ada012d5d8df4fc39b854f7ccf4ebe69a disabled tests for hpp-pinocchio, hpp-core & hpp-constraints. What was the reason behind that ? Shouldn't we run them again ?

jmirabel commented 5 years ago

They are disabled because they are useful only to developers and their compilation is very time-consuming.

Developers are expected to enable them by hand, although it is not specified anywhere. When developers do make test, the unit tests are first compiled.

nim65s commented 5 years ago

Ok, but shouldn't they be enabled for the CI ?

jmirabel commented 5 years ago

They are. The compilation is only delayed to target test instead of target all.

nim65s commented 5 years ago

I see, thanks :)