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

feat(warn slow): warn users for slow runs #3490

Merged
merged 3 commits into from
May 2, 2022
Merged

feat(warn slow): warn users for slow runs #3490

merged 3 commits into from
May 2, 2022

Conversation

nicojs
Copy link
Member

@nicojs nicojs commented Apr 30, 2022

Warn users for slow runs because a relatively big chunk of their mutants being static mutants.

  • Implement unit tests
  • Decide how to warn when estimated time is 50%, but static mutants are also 50%

Fixes #3435

Warn users for slow runs because of a relatively big chunk of their mutants being _static mutants_.
@nicojs nicojs requested a review from simondel April 30, 2022 00:37
@nicojs nicojs marked this pull request as draft April 30, 2022 00:37
@nicojs
Copy link
Member Author

nicojs commented Apr 30, 2022

@simondel what do you think about this message?

02:33:39 (4545) INFO InputFileResolver Found 113 of 1037 file(s) to be mutated.
02:33:40 (4545) INFO Instrumenter Instrumented 113 source file(s) with 3309 mutant(s)
02:33:40 (4545) INFO ConcurrencyTokenProvider Creating 1 checker process(es) and 1 test runner process(es).
02:33:41 (4545) INFO Sandbox Running build command "tsc -b" in "/home/nicojs/github/stryker-js/packages/core/.stryker-tmp/sandbox1282307".
02:34:00 (4545) INFO DryRunExecutor Starting initial test run (mocha test runner with "perTest" coverage analysis). This may take a while.
02:34:04 (4545) INFO DryRunExecutor Initial test run succeeded. Ran 693 tests in 4 seconds (net 3420 ms, overhead 591 ms).
02:34:04 (4545) WARN MutantTestPlanner Detected 255 static mutants (8% of total) that are estimated to take 85% of the time running the tests!
  You might want to enable "ignoreStatic" to ignore these static mutants for your next run. 
  For more information about static mutants visit: https://stryker-mutator.io/docs/mutation-testing-elements/static-mutants.
  (disable "warnings.slow" to ignore this warning)
Mutation testing  [                                                  ] 0% (elapsed: <1m, remaining: n/a) 0/3305 Mutants tested (0 survived, 0 timed out)

@nicojs nicojs marked this pull request as ready for review May 2, 2022 15:01
@nicojs nicojs merged commit 1103958 into master May 2, 2022
@nicojs nicojs deleted the feat/warn-slow branch May 2, 2022 16:11
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.

Idea: --ignoreStatic by default
1 participant