[PR] Report every test of a skipped container as skipped (BZ 69683) [ant]

slachiewicz opened a new pull request, #234:
URL: https://github.com/apache/ant/pull/234

   The `legacy-xml` listener recorded a skipped container as one skip and left the tests it holds out of the report, so a disabled class produced `tests="0" skipped="1"` with no `testcase` elements. `junitreport` selects `.//testcase[skipped]`, so those tests were invisible to it.
   
   Each test of a skipped container is now reported individually, and skipped tests count towards the total the way the `junit` task did with `res.runCount() + res.ignoredCount()`. A container holding no tests still contributes to the skipped count alone, since the format cannot represent a container.
   
   The added tests are the reproducer asked for in the bug.
   
   Bugzilla Report 69683
   
   Verified: `ant -Dtestcase=org.apache.tools.ant.taskdefs.optional.junitlauncher.LegacyXmlResultFormatterTest test` → 5 tests, 0 failures; the 3 new tests fail against master. `JUnitLauncherTaskTest` and `OutputDirLocationTest` also green, 32 tests total.
   
   *This change was created with AI assistance.*


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]