skrcode opened a new pull request, #764: URL: https://github.com/apache/maven-indexer/pull/764
Completes the Jetty 11 dependency update in #762 by moving the embedded-server test fixture from the removed `javax.servlet` namespace to Jakarta Servlet 5's `jakarta.servlet` namespace. Only imports change. The servlet implementations, mappings, and test behavior remain unchanged. Verification on Maven 3.9.9 and JDK 21: - `mvn -pl indexer-core -Dit.test=DefaultIndexUpdaterEmbeddingIT verify` — 225 unit tests and 5 embedded-server integration tests passed - `mvn -Prun-its verify` — all 11 reactor modules passed, including RAT, Checkstyle, Spotless, Modernizer, and CLI integration tests - `git diff --check` Following this checklist to help incorporate the contribution: - [x] This pull request addresses only the failed Jetty 11 update in #762. - [x] The description explains what changed and why. - [x] The commit has a meaningful subject and body. - [x] No new test was added because this is a test-fixture namespace migration with no runtime behavior change; the existing five `DefaultIndexUpdaterEmbeddingIT` cases exercise the migrated servlets before and after the upgrade. - [x] `mvn verify` passed as part of the full verification command. - [x] The integration tests passed with `mvn -Prun-its verify`. - [x] I hereby declare this contribution to be licensed under the [Apache License Version 2.0, January 2004](https://www.apache.org/licenses/LICENSE-2.0). -- 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]
