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

Suppress the parallel warning during coverage calculation #92

Closed
wants to merge 3 commits into from

Conversation

Vampire
Copy link
Contributor

@Vampire Vampire commented May 12, 2023

Depends on #93 and PIT release with hcoles/pitest#1208 included.

As we properly serialize the coverage calculation, the warning
that is logged after detecting multiple threads can be suppressed.

@Vampire Vampire force-pushed the suppress-parallel-warning branch 7 times, most recently from 18df4a5 to 3138d20 Compare May 19, 2023 12:35
Vampire added 3 commits May 19, 2023 15:21
This generically fixes hcoles/pitest#760 and pitest#73 for all platform engines,
removing the Jupiter specific work-around from pitest#74 and serializing test execution
during coverage calculation using locks. This way the tests can also properly run
in parallel later on during mutant hunting.
As we properly serialize the coverage calculation, the warning
that is logged after detecting multiple threads can be suppressed.
@Vampire Vampire force-pushed the suppress-parallel-warning branch from 3138d20 to 8e6bfa4 Compare May 19, 2023 13:22
@Vampire
Copy link
Contributor Author

Vampire commented May 19, 2023

As 1.14.0 is already a logical dependency, this PR is now integrated directly into the parent PRs.

@Vampire Vampire closed this May 19, 2023
@Vampire Vampire deleted the suppress-parallel-warning branch May 19, 2023 14:01
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.

1 participant