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.

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

CI doesn't run all tests #9

Closed
pcpthm opened this issue Aug 17, 2020 · 0 comments · Fixed by #10
Closed

CI doesn't run all tests #9

pcpthm opened this issue Aug 17, 2020 · 0 comments · Fixed by #10

Comments

@pcpthm
Copy link
Contributor

pcpthm commented Aug 17, 2020

When a Cargo workspace contains both the root package and other workspace members, cargo test by default runs tests only of the root package. In fact, i3ipc-types test fails since #7.

cargo test --workspace can be used to run tests on all workspace members as well as the root package but there is an issue. Trying to build the whole workspace at once fails due to a compilation error. That is because Cargo tries to build the whole workspace with both async-traits and async-std-traits features enabled, and because those features are not additive (see rust-lang/cargo#4328) it breaks. It should probably be a separate issue, though.

As a workaround, running cargo test in each directory will build each package separately, and doesn't merge features.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant