ppkarwasz commented on PR #90:
URL: 
https://github.com/apache/commons-secure-xml/pull/90#issuecomment-5653997248

   Correction to my previous comment: the 87 aborts do **not** fail the build. 
`connectedDebugAndroidTest` — the task CI runs — reports BUILD SUCCESSFUL under 
AGP 9.4.0 exactly as it does under AGP 8.6.1, and CI here is green. I read 
`failures="87"` out of the result XML and wrote "stays red" without checking 
Gradle's own outcome.
   
   What is actually different is the report AGP 9 writes and the job uploads: 
the 87 JUnit 5 assumption aborts are recorded as failures there, where AGP 
8.6.1 recorded them as skips. So the migration is not blocked; what we would 
still want from AGP 9 is that its aggregated test XML label an aborted test as 
skipped, so the uploaded reports — and anything that parses them — do not read 
87 phantom failures.
   
   The two lists in the previous comment stand otherwise, with the first item 
of case 1 downgraded from a blocker to a reporting defect.


-- 
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]

Reply via email to