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

cargo test --all --lib doesn't work on virtual manifests #4387

Closed
behnam opened this issue Aug 9, 2017 · 7 comments
Closed

cargo test --all --lib doesn't work on virtual manifests #4387

behnam opened this issue Aug 9, 2017 · 7 comments
Labels
A-workspaces Area: workspaces C-bug Category: bug Command-test

Comments

@behnam
Copy link
Contributor

behnam commented Aug 9, 2017

The

cargo test --all

variation works fine in a virtual manifest, but surprisingly

cargo test --all --lib

doesn't, erring with:

error: no library targets found

I believe this is a bug, as the filters do make complete sense.

@alexcrichton
Copy link
Member

Hm yeah I think I'd imagine a better error just saying you can't compose --all with more filters?

@behnam
Copy link
Contributor Author

behnam commented Aug 10, 2017

Oh, I'm doing that all the time! Specifically, test --all --lib works fine with a non-virtual workspace, it just panics in a virtual workspace.

@alexcrichton
Copy link
Member

Hm ok, then yeah I guess we just need to implement and/or fix this, and interpret --all --lib as "test all libraries"?

@behnam
Copy link
Contributor Author

behnam commented Aug 10, 2017

Yeah, I think falls into the not-still-implemented---all category.

@stale
Copy link

stale bot commented Sep 18, 2018

As there hasn't been any activity here in over 6 months I've marked this as stale and if no further activity happens for 7 days I will close it.

I'm a bot so this may be in error! If this issue should remain open, could someone (the author, a team member, or any interested party) please comment to that effect?

The team would be especially grateful if such a comment included details such as:

  • Is this still relevant?
  • If so, what is blocking it?
  • Is it known what could be done to help move this forward?

Thank you for contributing!

(The cargo team is currently evaluating the use of Stale bot, and using #6035 as the tracking issue to gather feedback.)

If you're reading this comment from the distant future, fear not if this was closed automatically. If you believe it's still an issue please leave a comment and a team member can reopen this issue. Opening a new issue is also acceptable!

@stale stale bot added the stale label Sep 18, 2018
@ehuss
Copy link
Contributor

ehuss commented Sep 18, 2018

I believe this is fixed by #5873.

@ehuss ehuss closed this as completed Sep 18, 2018
@inferiorhumanorgans
Copy link

I'm not 100% sure I'm not doing something wrong, but cargo test --all --lib doesn't appear to hit all the libraries defined in a virtual manifest.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-workspaces Area: workspaces C-bug Category: bug Command-test
Projects
None yet
Development

No branches or pull requests

5 participants