SonarQube Plugins

Cannot analyse project with multiple test files

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Duplicate
  • Affects Version/s: PHP-0.4
  • Fix Version/s: PHP-1.0
  • Component/s: PHP
  • Labels:
    None
  • Environment:
    windows xp
  • Number of attachments :
    0

Description

I have a project with a test directory which have several test files, when I run mvn sonar:sonar with following options (in my pom.xml file):
sonar.language=php
sonar.dynamicAnalysis=true
(sonar.phpPmd.shouldRun=false
sonar.phpDepend.shouldRun=false
sonar.phpcpd.shouldRun=false)
sonar.phpUnit.coverage.shouldRun=true
sonar.phpUnit.shouldRun=true

I have the following error :

Embedded error: Can not add twice the same measure:
org.sonar.api.measures.Measure@10bed9c[
id=<null>,
metricKey=test_execution_time,
metric=org.sonar.api.measures.Metric@1bb3e9a[
key=test_execution_time,
name=Unit tests duration,
type=MILLISEC,enabled=true,
qualitative=false,
direction=-1,
domain=Tests,
worstValue=<null>,
bestValue=<null>,
optimizedBestValue=<null>,
hidden=false],
value=891.0,data=<null>,
description=<null>,
alertStatus=<null>,
alertText=<null>,
tendency=<null>,
date=<null>,
variation1=<null>,
variation2=<null>,
variation3=<null>,
variation4=<null>,
variation5=<null>,
url=<null>,
characteristic=<null>
persistenceMode=FULL]

The generated phpunit.xml file is formatted as following (phpunit command format)

<testsuites>
<testsuite ...>
<testsuite ...>
</testsuite>
<testsuite ...>
</testsuite>
</testsuite>
</testsuites>

Issue Links

depends upon

Task - A task that needs to be done. SONARPLUGINS-1391 Refactor the way PHP plugin handles resources

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Cyril AKNINE added a comment -

Cloned from http://jira.codehaus.org/browse/SONARPLUGINS-916
I have the same issue with 0.4 and sonar.dynamicAnalysis=false
If my phpunit.xml log file has this kind of structure :
<testsuites>
<testsuite>
<testsuite>
<testcase />
</testsuite>
<testsuite>
<testcase />
</testsuite>
</testsuite>
</testsuites>

Then, I have "Embedded error: Can not add twice the same measure on org.sonar.api.resources" error. But if I keep only 1 test file (commenting testcases in the other tests files, but keeping the files !), I have no error. Here is the kind of phpunit.xml log file generated that works :

<testsuites>
<testsuite>
<testsuite /><!-- no testcase ! -->
<testsuite>
<testcase />
</testsuite>
</testsuite>
</testsuites>

Show
Cyril AKNINE added a comment - Cloned from http://jira.codehaus.org/browse/SONARPLUGINS-916 I have the same issue with 0.4 and sonar.dynamicAnalysis=false If my phpunit.xml log file has this kind of structure : <testsuites> <testsuite> <testsuite> <testcase /> </testsuite> <testsuite> <testcase /> </testsuite> </testsuite> </testsuites> Then, I have " Embedded error: Can not add twice the same measure on org.sonar.api.resources " error. But if I keep only 1 test file (commenting testcases in the other tests files, but keeping the files !), I have no error. Here is the kind of phpunit.xml log file generated that works : <testsuites> <testsuite> <testsuite /><!-- no testcase ! --> <testsuite> <testcase /> </testsuite> </testsuite> </testsuites>
Hide
Permalink
Gennadiy Litvinyuk added a comment -

Akram Ben Aissi, Sebastian Marek, this issue was fixed in the patch, that I sent to you.

Show
Gennadiy Litvinyuk added a comment - Akram Ben Aissi , Sebastian Marek , this issue was fixed in the patch, that I sent to you.
Hide
Permalink
Cyril AKNINE added a comment -

Can you send me this patch too ?

Show
Cyril AKNINE added a comment - Can you send me this patch too ?
Hide
Permalink
Gennadiy Litvinyuk added a comment -

Do you pass Tests Directory to Sonar?

Show
Gennadiy Litvinyuk added a comment - Do you pass Tests Directory to Sonar?
Hide
Permalink
Cyril AKNINE added a comment -

No, my unit tests are already executed by Hudson, I just give the phpunit.xml report to Sonar.

Show
Cyril AKNINE added a comment - No, my unit tests are already executed by Hudson, I just give the phpunit.xml report to Sonar.
Hide
Permalink
Gennadiy Litvinyuk added a comment -

I do it the same way, but you should tell Sonar, where your tests are.

For example in pom.xml

<testSourceDirectory><![CDATA[tests]]></testSourceDirectory>

If you use Jenkins Sonar plugin (to generate pom.xml), it is not possible to do it, please see my patch https://github.com/gennadiylitvinyuk/jenkins-sonar-plugin/commit/df18afa2b0d794706619415f23f45338ca777fa3

Show
Gennadiy Litvinyuk added a comment - I do it the same way, but you should tell Sonar, where your tests are. For example in pom.xml <testSourceDirectory> <![CDATA[tests]]> </testSourceDirectory> If you use Jenkins Sonar plugin (to generate pom.xml), it is not possible to do it, please see my patch https://github.com/gennadiylitvinyuk/jenkins-sonar-plugin/commit/df18afa2b0d794706619415f23f45338ca777fa3
Hide
Permalink
Fabrice Bellingard added a comment -

Gennadiy Litvinyuk, do you know what the status of this ticket is? You talked about a patch that you sent to Akram and Sebastian: is it already applied?

Show
Fabrice Bellingard added a comment - Gennadiy Litvinyuk , do you know what the status of this ticket is? You talked about a patch that you sent to Akram and Sebastian: is it already applied?
Hide
Permalink
Gennadiy Litvinyuk added a comment -

Sorry, I don't know and I can't check this now.

Cyril AKNINE, can you give us a feedback, whether this issue is still present with version 0.6?

Show
Gennadiy Litvinyuk added a comment - Sorry, I don't know and I can't check this now. Cyril AKNINE , can you give us a feedback, whether this issue is still present with version 0.6?
Hide
Permalink
Fabrice Bellingard added a comment -

As Cyril hasn't shown up for quite a while, I remove the 'fixFor" version of this ticket.

Show
Fabrice Bellingard added a comment - As Cyril hasn't shown up for quite a while, I remove the 'fixFor" version of this ticket.
Hide
Permalink
Fabrice Bellingard added a comment -

This issue is duplicated by SONAR-1588 that has just been fixed.

Show
Fabrice Bellingard added a comment - This issue is duplicated by SONAR-1588 that has just been fixed.
Hide
Permalink
Fabrice Bellingard added a comment -

Sorry, this issue is duplicated by SONARPLUGINS-1588.

Show
Fabrice Bellingard added a comment - Sorry, this issue is duplicated by SONARPLUGINS-1588 .

People

  • Assignee:
    Fabrice Bellingard
    Reporter:
    Cyril AKNINE
Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: