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(report): report test states #2868

Merged
merged 6 commits into from
May 5, 2021
Merged

feat(report): report test states #2868

merged 6 commits into from
May 5, 2021

Conversation

nicojs
Copy link
Member

@nicojs nicojs commented May 3, 2021

This PR adds test reporting to both the clear-text and html reporter (it was already supported by the JSON reporter).

image

There are a couple of things to keep in mind:

  • Stryker will only be able to produce the Covering test state with "coverageAnalysis": "perTest"
    image
  • Currently, only the @stryker-mutator/jest-runner supports marking the test files for each test. This means that all other test runners will simply produce a big list of tests:
    image

Closes #2464
Closes #2636

@nicojs nicojs added this to the 5.0 milestone May 4, 2021
@nicojs nicojs changed the title feat(report): report test states in the html reporter feat(report): report test states in the reporters May 4, 2021
@nicojs
Copy link
Member Author

nicojs commented May 4, 2021

@HonzaMac what do you think?

We, unfortunately, don't have the information about the layering of the test suites, Stryker uses a flat test name structure. I personally don't think it is that big of a deal though.

@nicojs nicojs changed the title feat(report): report test states in the reporters feat(report): report test states May 5, 2021
@HonzaMac
Copy link

HonzaMac commented Jun 5, 2021

@nicojs Thank you for implementing this. It really works! I was able to delete useless tests, so running stryker is faster thanks for removal of redundant.

Looking forward to try it on bigger suite on another project.

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