> Some HttpClient tests - like `test/jdk/java/net/httpclient/SmokeTest.java`, > depend on non-library test classes that were developed for testing the > HttpServer, such as `../../../com/sun/net/httpserver/FileServerHandler.java` > which requires compiling using `@compile`. > > Instead of depending on HttpServer test classes, these tests should use > functionalities from the common servers provided by the `HttpServerAdapters`, > offered by the `test/jdk/java/net/httpclient/lib` library. An adapted version > of the `../../../com/sun/net/httpserver/FileServerHandler.java` could be > provided by the `HttpServerAdapters` helper. > > There are also 5 implementations of stand-alone echo handlers in the > httpclient test base. This change remove 4 of them, keep only the > implementation in `HttpServerAdapters`, and updates all tests that were > relying on the removed implementations to use that provided by > `HttpServerAdapters`. > > This is a test-only change.
Daniel Fuchs has updated the pull request incrementally with eight additional commits since the last revision: - Update test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/common/HttpServerAdapters.java Co-authored-by: Daniel Jelinski <[email protected]> - Update test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/common/HttpServerAdapters.java Co-authored-by: Daniel Jelinski <[email protected]> - Update test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/common/HttpServerAdapters.java Co-authored-by: Daniel Jelinski <[email protected]> - Update test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/common/HttpServerAdapters.java Co-authored-by: Daniel Jelinski <[email protected]> - Update test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/common/HttpServerAdapters.java Co-authored-by: Daniel Jelinski <[email protected]> - Update test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/common/HttpServerAdapters.java Co-authored-by: Daniel Jelinski <[email protected]> - Update test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/common/HttpServerAdapters.java Co-authored-by: Daniel Jelinski <[email protected]> - Update test/jdk/java/net/httpclient/lib/jdk/httpclient/test/lib/common/HttpServerAdapters.java Co-authored-by: Daniel Jelinski <[email protected]> ------------- Changes: - all: https://git.openjdk.org/jdk/pull/29702/files - new: https://git.openjdk.org/jdk/pull/29702/files/91ee1b7b..fae5304b Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=29702&range=03 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=29702&range=02-03 Stats: 8 lines in 1 file changed: 0 ins; 0 del; 8 mod Patch: https://git.openjdk.org/jdk/pull/29702.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/29702/head:pull/29702 PR: https://git.openjdk.org/jdk/pull/29702
