search

ArnaudBuchholz / ui5-test-runner

A test runner for UI5 applications enabling parallel execution of tests.
https://arnaudbuchholz.github.io/ui5-test-runner/
MIT License
22 stars 9 forks source link

Generated junit XML report currently missing required property 'classname' for each testcase #88

Closed heimwege closed 7 months ago

heimwege commented 7 months ago

Hey hey 👋🏻

according to the junit spec a property classname for each testcase is 'required' (e,g, containing the value of the package property of the testsuite aka. the journey name). This is currently not part of the xml file generated by $/junit-xml-report.js

actual:

<testsuites>
    <testsuite
        name="http://localhost:8080/test/opaTests.qunit.html"
        package="Foo Journey"
        tests="1"
      >
        <testcase
              name="Test Bar" 
              time="3.374"
        >
        </testcase>
    </testsuite>
</testsuites>

expected:

<testsuites>
    <testsuite
        name="http://localhost:8080/test/opaTests.qunit.html"
        package="Foo Journey"
        tests="1"
      >
        <testcase
              name="Test Bar" 
              time="3.374"
              classname="Foo Journey"
        >
        </testcase>
    </testsuite>
</testsuites>
ArnaudBuchholz commented 7 months ago

Good catch, thx for reporting. Will be added in next release.

ArnaudBuchholz commented 7 months ago

Fixed in 5.0.0