Repository navigation
feat: perch coverage says what test to add - #348
Closed
jbrown9513 wants to merge 1 commit into
Closed
jbrown9513 wants to merge 1 commit into
jbrown9513 wants to merge 1 commit into
Conversation
… what to add The summary, each file's page and the terminal listed one row per survived mutant, with the same eight test names under every one, and the code came after the rows. They now rank methods with survived mutants, most first, each with a sentence from its surest mutant saying what a new test has to tell apart, and the mutants fold under the row. A file's page shows its code first with a count in the gutter where mutants survived, one note open at a time, and next and previous survived. Source files are a tree worst first with scores coloured against 60 and 80 as Stryker colours them; test files are listed only when something needs fixing.
Contributor
Author
|
Stacked on #345, which it is based on; merge that first, then this one retargets to main. |
Contributor
|
Perch Scan: no issues in 137 methods at |
This was referenced Oct 9, 2026
Contributor
Author
|
Folded into #355. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The report listed mutants where a reader acts on methods. The summary and each file's page were one row per survived mutant, so a method with five edits on one line was five rows, and on perch the graph.js page put 613 rows before its code. Every note under a line repeated the same eight test names, which is most of why the page was 53 MB. Scores had no thresholds, the file table was flat and alphabetical, the test table was 47 rows of 100%, and the only call to action was behind a Copy prompt button. The terminal printed the same things in the same order.
The unit is now the method, and each one says what test to add.
Add a test at the boundary of \>=` on line 12, where `>=` and `>` give different results.The sentence is fixed wording per kind of edit around the report's own facts (src/coverage-advice.js`); the mutants fold out under the row, and one Copy prompt covers all of them. The terminal prints the sentence whole on its own line and wraps it rather than cut it.46 test files have nothing to fix.On perch the page goes from 53 MB to 33 MB.
--allon the terminal lists every method and every survived mutant with the idperch closetakes; without it the count line says so.Closes #346