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

Restructure coverage report generated #171

Merged
merged 7 commits into from
Aug 3, 2021

Conversation

LorenzoBettini
Copy link
Collaborator

Companion of #167 for SonarCloud analysis

Jonas Kutscha and others added 6 commits July 29, 2021 10:08
removed two coverageReportGenerated where no report is generated
in the newest PIT version
We relied on the console to contain a certain String to check wether pit
is done or not. Now we rely on the extension point which was made for
that and don't rely on the console view anymore. This should be alot
more stable, even if pit changes it output messages or the console view
is not open.

Also we waited sometimes several times for pit to finish in the same
test, example:
@test
public void emptyClassAndEmptyTest() throws CoreException {
    removeMethods(FOO_CLASS, FOO_BAR_PACKAGE, TEST_PROJECT);
    removeMethods(FOO_TEST_CLASS, FOO_BAR_PACKAGE, TEST_PROJECT);
    runTest(FOO_TEST_CLASS, FOO_BAR_PACKAGE, TEST_PROJECT);
    consoleContains(0, 0, 100, 0, 0);
    mutationsAre(Collections.emptyList());
    coverageReportGenerated(0,100,100);
}

In this test we waited for pit in the methods consoleContains(...),
mutationsAre(...) and in coverageReportGenerated(...).
Use pit result to parse killed and generated mutatns
@sonarcloud
Copy link

sonarcloud bot commented Aug 3, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@LorenzoBettini LorenzoBettini merged commit a834596 into master Aug 3, 2021
@LorenzoBettini LorenzoBettini deleted the Restructure_coverageReportGenerated branch August 3, 2021 12:20
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