SonarQube Plugins
  1. SonarQube Plugins
  2. SONARPLUGINS-2555

DuplicatedSourceException in EunitXmlSensor in some erlang projects

    Details

    • 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

        Hide
        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
        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
        Tamas Kende added a comment -

        it is fixed

        Show
        Tamas Kende added a comment - it is fixed
        Hide
        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
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: