slachiewicz opened a new pull request, #2273: URL: https://github.com/apache/maven-resolver/pull/2273
`HttpTransporterTest` is at checkstyle's 2000-line file limit (1999 lines on master), so any change to it, including the deprecation cleanup for #2272, fails `checkstyle:check`. `HttpServer` also read its key store paths from the test class, which is the wrong direction for a server used by the test. The key store paths, the copy of the stores from the test resources into the build directory and the client `SSLContext` move into a new `SslTestSupport` class in the same package; the test and the server both use it. No behaviour change: the method bodies are moved verbatim, and the test class drops to 1907 lines. Nothing outside the module referenced the moved members. Verified: `mvn -pl maven-resolver-transport-jetty -Dtest=JettyTransporterTest test` on JDK 21 (macOS) → 113 tests, 0 failures, 5 skipped (as on master). --- - [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]
