Skip to content

Commit

Permalink
mutants: Move config from CI command lines to config file.
Browse files Browse the repository at this point in the history
  • Loading branch information
kpreid committed Jan 15, 2024
1 parent 819c9bd commit 24df1f1
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 5 deletions.
10 changes: 10 additions & 0 deletions .cargo/mutants.toml
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
# Cannot specify a package list here

exclude_globs = [
"tests.rs",
"all-is-cubes/src/content/**",# TODO: maybe un-exclude content once we are out of more important things (it's hard to test thoroughly)
]
exclude_re = [
"::fmt ->", # easy to break, hard to test
"<impl Arbitrary for ", # near-impossible to test
]
4 changes: 2 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -317,8 +317,8 @@ jobs:
git diff origin/${{ github.base_ref }}.. | tee git.diff
- name: Mutants
# See mutants.yml for notes on the exclusions
run: cargo mutants --in-diff git.diff --package all-is-cubes --exclude-re '::fmt ->' --exclude tests.rs -- --all-features
# See mutants.yml for notes on the package selection
run: cargo mutants --in-diff git.diff --package all-is-cubes -- --all-features

- uses: actions/upload-artifact@v3
if: always()
Expand Down
4 changes: 1 addition & 3 deletions .github/workflows/mutants.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,9 +26,7 @@ jobs:

- name: Run mutation testing for all-is-cubes
timeout-minutes: 60
# TODO: maybe un-exclude fmt()
# TODO: maybe un-exclude content once we are out of more important things (it's hard to test thoroughly)
run: cargo mutants --package all-is-cubes --exclude-re '::content::' --exclude-re '<impl Arbitrary for ' --exclude-re '::fmt ->' --exclude tests.rs -- --all-features
run: cargo mutants --package all-is-cubes -- --all-features
# TODO: Other crates' reasons for non-inclusion:
# all-is-cubes-content: insufficient test coverage
# all-is-cubes-desktop: insufficient test coverage, unclear if even feasible
Expand Down

0 comments on commit 24df1f1

Please sign in to comment.