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

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

      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

          [JENKINS-56123] JaCoCo Coverage Statistics Grid Portlet reports 100% coverage in case there is no coverage collected at all

          Sergey Zhemzhitsky created issue -
          Sergey Zhemzhitsky made changes -
          Description Original: JaCoCo Coverage Statistics Grid Portlet reports 100% coverage in case there no coverage collected at all.

          Here is the coverage reported by the portlet

          !image-2019-02-13-16-31-29-943.png|width=600!

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

           !image-2019-02-13-16-34-36-731.png|width=600!
          New: JaCoCo Coverage Statistics Grid Portlet reports 100% coverage in case there are no coverage collected at all.

          Here is the coverage reported by the portlet

          !image-2019-02-13-16-31-29-943.png|width=600!

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

          !image-2019-02-13-16-34-36-731.png|width=600!
          Sergey Zhemzhitsky made changes -
          Description Original: JaCoCo Coverage Statistics Grid Portlet reports 100% coverage in case there are no coverage collected at all.

          Here is the coverage reported by the portlet

          !image-2019-02-13-16-31-29-943.png|width=600!

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

          !image-2019-02-13-16-34-36-731.png|width=600!
          New: JaCoCo Coverage Statistics Grid Portlet reports 100% coverage in case there are no coverage collected at all.

          Here is the coverage reported by the portlet

          !image-2019-02-13-16-31-29-943.png|width=600!

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

          !image-2019-02-13-16-34-36-731.png|width=500!
          Sergey Zhemzhitsky made changes -
          Description Original: JaCoCo Coverage Statistics Grid Portlet reports 100% coverage in case there are no coverage collected at all.

          Here is the coverage reported by the portlet

          !image-2019-02-13-16-31-29-943.png|width=600!

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

          !image-2019-02-13-16-34-36-731.png|width=500!
          New: JaCoCo Coverage Statistics Grid Portlet reports 100% coverage in case there are no coverage collected at all.

          Here is the coverage reported by the portlet

          !image-2019-02-13-16-31-29-943.png|width=600!

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

          !image-2019-02-13-16-34-36-731.png|width=400!
          Sergey Zhemzhitsky made changes -
          Summary Original: JaCoCo Coverage Statistics Grid Portlet reports 100% coverage in case there no coverage collected at all New: JaCoCo Coverage Statistics Grid Portlet reports 100% coverage in case there is no coverage collected at all
          Sergey Zhemzhitsky made changes -
          Description Original: JaCoCo Coverage Statistics Grid Portlet reports 100% coverage in case there are no coverage collected at all.

          Here is the coverage reported by the portlet

          !image-2019-02-13-16-31-29-943.png|width=600!

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

          !image-2019-02-13-16-34-36-731.png|width=400!
          New: 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

          !image-2019-02-13-16-31-29-943.png|width=600!

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

          !image-2019-02-13-16-34-36-731.png|width=400!

          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 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 added a comment -

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

          centic added a comment - See JENKINS-25076 and JENKINS-29117 for previous issues about 100%/0% coverage.
          centic made changes -
          Link New: This issue is related to JENKINS-29117 [ JENKINS-29117 ]
          centic made changes -
          Link New: This issue is related to JENKINS-25076 [ JENKINS-25076 ]

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

              Created:
              Updated: