-
-
Notifications
You must be signed in to change notification settings - Fork 90
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
Including workspace binaries in archive #423
Comments
Hi Guilhem! Hope you're doing well :) Good question -- currently, nextest includes any non-test binaries which have corresponding integration tests in their crates. So if you can include an integration test at all, even one without any tests, that would be ideal. However I think it makes sense to support this in some form, and I'm wondering whether we should have the ability in configuration to include additional files (or globs) in archives. Do you want to work on this? (One thing which I've seen people do is run cargo build within tests, so it's not completely unreasonable tbh.) |
Hi, I'm fine thank you. I need to see if I can manage to dedicate some time to it. In any case, what do you see by "in configuration" ? In the Or something else ? |
Actually -- for now I think maybe pass it in as an extra argument to |
Hi @Guiguiprim @sunshowers , I have the same problem. I'm working on a project called Torrust Tracker. @da2ce7 was working on the Cantainerfile and he's using I was trying to reproduce a minimal example of how to use torrust/torrust-website#95 (comment) And this is the example I can't make it work: I'm going to confirm it but I guess I have the same problem because I do not have any integration test. I suppose this could be a common pattern, since we consider it a good practice to take the binary that passes the tests. We could get the binary from previous stages (where we build the app) but this way seems to be cleaner and safer because you are sure you are using the binary that passed the tests. |
Using `cargo nextest archive` to build the application and pass it to the next stage did not work. The binary was missing. The problems was decribed here: nextest-rs/nextest#423 You need to have at least one integration test.
I have a use case that is maybe half a hack.
In more details:
cargo build
at the workspace level builds all those binariescargo test
works, as the binaries have already being built and can be foundcargo test
withoutcargo build
before fails, as the binaries are missingCurrently
cargo nextest archive
does not include those binaries even ifcargo build
was called before. Is it possible to include in the archive all the binaries in the target folder ? I guess the exact list would be hard to get other than just listing the folder content...As a workaround, I can:
archive
feature (which is a be sad)cargo test
to build themSo the question is: Is it reasonable to try to add the binaries inside the archive, or is my use-case too sketchy (which I would understand) ?
The text was updated successfully, but these errors were encountered: