gnodet opened a new pull request, #13169:
URL: https://github.com/apache/maven/pull/13169
## Problem
The `TestSuiteOrdering` class on `maven-4.0.x` uses a static `Map<Class<?>,
Integer>` registry. Every IT class must be explicitly added to this map — if it
is missing, the framework prints a warning and the test is **not run** at all:
```
Test class org.apache.maven.it.MavenITgh12522NonExtensionPluginTest is not
present in TestSuiteOrdering
- please add it or annotate with @Tag("disabled")
```
This caused all recently backported ITs (33+ classes) to be silently
excluded from CI on this branch.
## Fix
Replace the static registry with the dynamic `ClassOrderer` already used on
`master`: tests are ordered automatically by their numeric suffix (`gh-XXXXX` >
`mng-XXXXX` > `it-XXXXX` in descending numeric order), with no explicit
registration required. New IT classes on this branch will be picked up
automatically going forward.
The only adaptation from master:
- Keep the `String`-based `Verifier` constructor (maven-4.0.x does not have
the `Path`-based one)
- Keep the `ExecutorHelper` mode logging already present on this branch
--
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]
