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.

Sign up for GitHub

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

Jump to bottom

SQ Plugin: Remove deprecated import of integration test coverage from plugins #4685

Closed
andrei-epure-sonarsource opened this issue Jul 15, 2021 · 1 comment · Fixed by #9561
Closed

SQ Plugin: Remove deprecated import of integration test coverage from plugins #4685

andrei-epure-sonarsource opened this issue Jul 15, 2021 · 1 comment · Fixed by #9561
Assignees
@mary-georgiou-sonarsource
Labels
Area: SQ Plugin Java plugin related issues. Sprint: Hardening Fix FPs/FNs/improvements Type: Cleanup Improve and cleanup code base
Milestone
9.31

Comments

@andrei-epure-sonarsource
Copy link
Contributor

andrei-epure-sonarsource commented Jul 15, 2021 •
edited
Loading

We should remove them because they have been deprecated since SonarQube 6.2!

    if (this.isIntegrationTest) {
      LOG.warn("Starting with SonarQube 6.2 separation between Unit Tests and Integration Tests Coverage" +
        " reports is deprecated. Please move all reports specified from *.it.reportPaths into *.reportPaths.");
    }

Repro steps:

  1. Open a project in SonarCloud/SonarQube
  2. Go to Administration -> General Settings -> Languages
  3. Choose C#

It shows for example
image

However coverage for integration tests is deprecated.

The same, it shows NCover3
image
@andrei-epure-sonarsource andrei-epure-sonarsource added the Area: SQ Plugin Java plugin related issues. label Jul 15, 2021
@andrei-epure-sonarsource andrei-epure-sonarsource changed the title Should mention in UI what settings have been deprecated Should remove import of integration test coverage from plugins Jul 15, 2021
@pavel-mikula-sonarsource
Copy link
Contributor

This should remove the properties that our plugins publish, and their processing.

@pavel-mikula-sonarsource pavel-mikula-sonarsource changed the title Should remove import of integration test coverage from plugins Remove deprecated import of integration test coverage from plugins Jun 11, 2024
@pavel-mikula-sonarsource pavel-mikula-sonarsource added Type: Cleanup Improve and cleanup code base and removed Type: Improvement labels Jun 13, 2024
@mary-georgiou-sonarsource mary-georgiou-sonarsource added this to the 9.31 milestone Jul 23, 2024
@mary-georgiou-sonarsource mary-georgiou-sonarsource changed the title Remove deprecated import of integration test coverage from plugins SQ Plugin: Remove deprecated import of integration test coverage from plugins Jul 25, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mary-georgiou-sonarsource mary-georgiou-sonarsource

Labels
Area: SQ Plugin Java plugin related issues. Sprint: Hardening Fix FPs/FNs/improvements Type: Cleanup Improve and cleanup code base
Projects
None yet
Milestone
9.31
4 participants
@andrei-epure-sonarsource @pavel-mikula-sonarsource @mary-georgiou-sonarsource @zsolt-kolbay-sonarsource