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

JaCoCo Coverage Statistics Grid Portlet reports 100% coverage in case there is no coverage collected at all

    XMLWordPrintable

Details

    • Bug
    • Status: Open (View Workflow)
    • Major
    • Resolution: Unresolved
    • jacoco-plugin
    • None
    • Ubuntu Server 14.04
      Java 1.8.0_101
      Jenkins 2.161
      JaCoCo Plugin 3.0.4
      Dashboard View 2.10

    Description

      JaCoCo Coverage Statistics Grid Portlet reports 100% coverage in case there is no coverage collected at all.

      Here is the coverage reported by the portlet

      ... and here is the coverage trend as it is shown when clicking and navigating to the corresponding item from the portlet

      Attachments

        1. image-2019-02-13-16-31-29-943.png
          324 kB
          Sergey Zhemzhitsky
        2. image-2019-02-13-16-34-36-731.png
          20 kB
          Sergey Zhemzhitsky

        Issue Links

          Activity

            centic centic added a comment -

            There are currently two cases that are hard to distinguish:

            a) There are no lines/branches to cover, e.g. a interface class without any statics and default methods, this should be shown as "100% covered" as there is nothing more that can be covered
            b) There are lines/branches to cover, but no coverage was reported at all for the class. In this case it should be shown as "0%" covered

            Unfortunately it seems both cases currently look exactly the same when the jacoco-plugin see coverage-results, at least I could not find a way to distinguish between those two cases.

            If anybody has an idea how this can be done technically, it should be possible to fix this, otherwise we would be playing change-ping-pong if we just change the current code to show 0% as then other cases are again reported incorrectly.

            Code-location for the actual computation is at hudson/plugins/jacoco/model/Coverage.java:80

            centic centic added a comment - There are currently two cases that are hard to distinguish: a) There are no lines/branches to cover, e.g. a interface class without any statics and default methods, this should be shown as "100% covered" as there is nothing more that can be covered b) There are lines/branches to cover, but no coverage was reported at all for the class. In this case it should be shown as "0%" covered Unfortunately it seems both cases currently look exactly the same when the jacoco-plugin see coverage-results, at least I could not find a way to distinguish between those two cases. If anybody has an idea how this can be done technically, it should be possible to fix this, otherwise we would be playing change-ping-pong if we just change the current code to show 0% as then other cases are again reported incorrectly. Code-location for the actual computation is at hudson/plugins/jacoco/model/Coverage.java:80
            centic centic added a comment -

            See JENKINS-25076 and JENKINS-29117 for previous issues about 100%/0% coverage.

            centic centic added a comment - See JENKINS-25076 and JENKINS-29117 for previous issues about 100%/0% coverage.

            The jacoco-plugin brings its own dashboard portlet, removing dashboard-view-plugin from components, because this must be fixed in the jacoco-plugin.

             

            tgr Tobias Gruetzmacher added a comment - The jacoco-plugin brings its own dashboard portlet, removing dashboard-view-plugin from components, because this must be fixed in the jacoco-plugin.  

            People

              ognjenb Ognjen Bubalo
              szhem Sergey Zhemzhitsky
              Votes:
              2 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated: