Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-20233

CloverPHP coverage metric targets are being ignored

      Steps to reproduce:
      1. Go into job configuration and change the coverage metric targets to something lower than the default of 70 and 80
      2. Run a build that has code coverage over the newly specified target

      Expected result:

      • The health report should show up as sunny

      Actual result:

      • The health report is calculated based on the default 70% and 80%

          [JENKINS-20233] CloverPHP coverage metric targets are being ignored

          Weather report is ignoring coverage targets

          Kenneth Skertchly added a comment - Weather report is ignoring coverage targets

          I just opened a PR to fix this issue: https://github.com/jenkinsci/cloverphp-plugin/pull/5

          Patrick Brückner added a comment - I just opened a PR to fix this issue: https://github.com/jenkinsci/cloverphp-plugin/pull/5

          I reassigned this to you, sogabe, since the fix needs your approval to be merged.

          Patrick Brückner added a comment - I reassigned this to you, sogabe , since the fix needs your approval to be merged.

          Code changed in jenkins
          User: Patrick Brueckner
          Path:
          src/main/java/org/jenkinsci/plugins/cloverphp/CloverBuildAction.java
          src/test/java/org/jenkinsci/plugins/cloverphp/CloverBuildActionTest.java
          http://jenkins-ci.org/commit/cloverphp-plugin/4b72095547da2d0cd1f3d0c9ebfe6230ea27ab1f
          Log:
          pay tribute to healthyTarget as amplification factor for HealthReport [FIXES JENKINS-20233]

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Patrick Brueckner Path: src/main/java/org/jenkinsci/plugins/cloverphp/CloverBuildAction.java src/test/java/org/jenkinsci/plugins/cloverphp/CloverBuildActionTest.java http://jenkins-ci.org/commit/cloverphp-plugin/4b72095547da2d0cd1f3d0c9ebfe6230ea27ab1f Log: pay tribute to healthyTarget as amplification factor for HealthReport [FIXES JENKINS-20233]

            sogabe sogabe
            kskertchly Kenneth Skertchly
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: