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.

Sign up for GitHub

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

Jump to bottom

cargo insta test cli: add --disable-nextest-doctest option #438

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

cargo insta test cli: add --disable-nextest-doctest option #438

wants to merge 2 commits into from
+6 −0

Commits on Sep 16, 2024

  1. cargo insta test cli: add --disable-nextest-doctest option 

    This adds an option to `cargo insta test` that prevents running doctests
with `cargo test` when Nextest is the test runner, even if the given
targets would ortherwise include doctests.

I usually use `cargo insta test` with `--test-runner nextest
--workspace` options. This always runs `cargo test --doc` after nextest
is done, and this takes a bit of time. This is an annoyance for me. I
think the project I work on might have a doctest or two, but they are
not very relevant, so I'm happy to rely on CI to catch any doctest
errors.

I only have a vague understanding of `cargo test` options, but I have
not found an existing set of test specifiers that's equivalent to
`--workspace --no-doctests-pretty-please`.
    @ilyagr
    ilyagr committed Sep 16, 2024
    Configuration menu
    Copy the full SHA
    4cde386 View commit details
    Browse the repository at this point in the history

  2. cargo-insta --disable-nextest-doctest: Add -N short option 

    I'm not sure how appropriate this is for general use, but it's
convenient if, like me, you have a project where you use this all the
time.
    @ilyagr
    ilyagr committed Sep 16, 2024
    Configuration menu
    Copy the full SHA
    7c193c6 View commit details
    Browse the repository at this point in the history