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

Run tests in debug mode #301

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

edef1c
Copy link

@edef1c edef1c commented Oct 10, 2023

Previously, crate2nix tests were run equivalently to cargo test --release, which is a somewhat unusual default configuration.

It might be worth having a flag to tweak this behaviour, similar to Cargo. A more ambitious version yet would permit running both debug and release versions of the tests. I'm not sure what the best interface for either of those is.

Previously, crate2nix tests were run equivalently to `cargo test --release`,
which is a somewhat unusual default configuration.
@kolloch
Copy link
Collaborator

kolloch commented Oct 11, 2023

Hi @edef1c!

I also think it should be configurable. Usually, I execute tests locally with cargo in debug mode but then in CI in release mode. Depends quite on the test. Obviously, model tests or the like profit from release mode more than others.

I haven't worked on crate2nix for a while and will first invest in flakifying it and making it work on Mac OS again (which I am using currently). But I'd be interested in hearing how such an interface could look like!

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 this pull request may close these issues.

2 participants