Sonar Plugins

DuplicatedSourceException in EunitXmlSensor in some erlang projects

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: ERLANG-0.1
  • Component/s: Erlang
  • Labels:
    None
  • Number of attachments :
    0

Description

Sometimes the name attribute of the testsuite in the junit like xml is not the name of the test module but the name of the tested module. So instead of getting this value we should try to find the file with name: name+"_test[s]" or "_eunit"

or even better, use the name of the testcase and get the filename from that:
testmodulename:testmethod/number_number

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Tamas Kende added a comment -

Cannot do that, it comes from the AbstractSurefireSensor... need to handle the possible test names somehow

Show
Tamas Kende added a comment - Cannot do that, it comes from the AbstractSurefireSensor... need to handle the possible test names somehow
Hide
Permalink
Tamas Kende added a comment -

The problem is when the test src file is called *_tests and not *_eunit the module name in the xml won't contain the _tests in its name. So if this classkey does not contain _eunit or _tests we have to append _tests to it and search for the file based on this name.

Show
Tamas Kende added a comment - The problem is when the test src file is called *_tests and not *_eunit the module name in the xml won't contain the _tests in its name. So if this classkey does not contain _eunit or _tests we have to append _tests to it and search for the file based on this name.
Hide
Permalink
Tamas Kende added a comment -

it is fixed

Show
Tamas Kende added a comment - it is fixed
Hide
Permalink
Evgeny Mandrikov added a comment -

Don't know how I should test this, so if you say that this was fixed, then ticket closed.

Show
Evgeny Mandrikov added a comment - Don't know how I should test this, so if you say that this was fixed, then ticket closed.

People

  • Assignee:
    Tamas Kende
    Reporter:
    Tamas Kende
Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: