Uploaded image for project: 'SonarPHP'
  1. SonarPHP
  2. SONARPHP-656

Deprecate IT/Overall coverage and accept many coverage reports

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.10
    • Component/s: Coverage
    • Labels:
      None

      Description

      Since SonarQube 6.2, the concept of coverage type (unit/IT/overall) was dropped. Instead a Sensor can save multiple coverage reports (with no specific type) per file.
      As a result, the PHP plugin should be updated. When the runtime is SonarQube 6.2+:

      • log a warning when properties sonar.php.coverage.itReportPath or sonar.php.coverage.overallReportPath are used
      • deprecate sonar.php.coverage.reportPath and introduce sonar.php.coverage.reportPaths to accept a comma separated list of coverage reports

      Behavior

      The behavior should be

        6.1- 6.2+
      reportPath, itReportPath, overallReportPath honor honor & print deprecation notice
      reportPaths ignore; no coverage honor

      Note that "honor" behavior is specified for SQ versions 6.1 and lower to not break backward compatibility with the LTS version.

      If both reportPaths and the legacy properties are used in 6.2+ the legacy properties are not used and a skipped notice is printed in the logs.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                carlo.bottiglieri Carlo Bottiglieri (Inactive)
                Reporter:
                julien.henry Julien Henry
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Due:
                  Created:
                  Updated:
                  Resolved: