Petals ESB CLI

Few unit tests fail when executed through Sonar

Details

  • Type: Task Task
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.1.1
  • Fix Version/s: 2.2.0
  • Component/s: Command interpretor
  • Security Level: Public
  • Description:

    Few unit tests fail when executed through Sonar

  • Environment:
    -

Activity

Hide
Christophe DENEUX added a comment - Fri, 27 Sep 2013 - 15:06:37 +0200

SONARJAVA-40 can be a solution .

Show
Christophe DENEUX added a comment - Fri, 27 Sep 2013 - 15:06:37 +0200 SONARJAVA-40 can be a solution .
Hide
Christophe DENEUX added a comment - Fri, 27 Sep 2013 - 15:08:45 +0200

SONARJAVA-40 seems resolve the problem: all tests are viewed as succeded by Sonar, but coverage is 0%. The workaround was test only an a development station, perhaps when executed with Jenkins the coverage will be right.

Show
Christophe DENEUX added a comment - Fri, 27 Sep 2013 - 15:08:45 +0200 SONARJAVA-40 seems resolve the problem: all tests are viewed as succeded by Sonar, but coverage is 0%. The workaround was test only an a development station, perhaps when executed with Jenkins the coverage will be right.
Hide
Christophe DENEUX added a comment - Fri, 27 Sep 2013 - 15:59:48 +0200

The workaround tells Sonar to reuse reports generated by maven-surefire-plugin during the test phase. During this phase, unit tests were not instrumented by JaCoCo, so no coverage information is available.

The workaround is not sufficient.

A mail was sent to the sonar mailing list: http://markmail.org/message/gblgtx3zsb73n2j7?q=list:org%2Ecodehaus%2Esonar%2Euser+date:201309+order:date-backward&page=1

Show
Christophe DENEUX added a comment - Fri, 27 Sep 2013 - 15:59:48 +0200 The workaround tells Sonar to reuse reports generated by maven-surefire-plugin during the test phase. During this phase, unit tests were not instrumented by JaCoCo, so no coverage information is available. The workaround is not sufficient. A mail was sent to the sonar mailing list: http://markmail.org/message/gblgtx3zsb73n2j7?q=list:org%2Ecodehaus%2Esonar%2Euser+date:201309+order:date-backward&page=1
Hide
Christophe DENEUX added a comment - Mon, 30 Sep 2013 - 19:21:06 +0200

According to the mailing list reply, the workaround is working. Only the execution of the JaCoCo report was missing and MUST be done directly by the project.

Fixed in trunk

Show
Christophe DENEUX added a comment - Mon, 30 Sep 2013 - 19:21:06 +0200 According to the mailing list reply, the workaround is working. Only the execution of the JaCoCo report was missing and MUST be done directly by the project. Fixed in trunk

People

Dates

  • Created:
    Fri, 27 Sep 2013 - 15:05:11 +0200
    Updated:
    Mon, 5 May 2014 - 11:51:18 +0200
    Resolved:
    Mon, 30 Sep 2013 - 19:21:06 +0200