This is an automated email from the ASF dual-hosted git repository. kwin pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/maven-resolver.git
commit 84d02cc6e9768bbcd53b69ef4ac5da297488eb89 Author: Konrad Windszus <[email protected]> AuthorDate: Fri Oct 10 16:17:28 2025 +0200 Add test for HTTP compression support Disable for now for Java Http Client This relates to #1620 --- .../aether/internal/test/util/http/HttpTransporterTest.java | 12 ++++++++++++ .../org/eclipse/aether/transport/jdk/JdkTransporterTest.java | 5 +++++ 2 files changed, 17 insertions(+) diff --git a/maven-resolver-test-http/src/main/java/org/eclipse/aether/internal/test/util/http/HttpTransporterTest.java b/maven-resolver-test-http/src/main/java/org/eclipse/aether/internal/test/util/http/HttpTransporterTest.java index 776bcc381..920922e07 100644 --- a/maven-resolver-test-http/src/main/java/org/eclipse/aether/internal/test/util/http/HttpTransporterTest.java +++ b/maven-resolver-test-http/src/main/java/org/eclipse/aether/internal/test/util/http/HttpTransporterTest.java @@ -184,6 +184,7 @@ public class HttpTransporterTest { factory = transporterFactorySupplier.get(); repoDir = TestFileUtils.createTempDir(); TestFileUtils.writeString(new File(repoDir, "file.txt"), "test"); + TestFileUtils.writeString(new File(repoDir, "artifact.pom"), "<xml>pom</xml>"); TestFileUtils.writeString(new File(repoDir, "dir/file.txt"), "test"); TestFileUtils.writeString(new File(repoDir, "dir/oldFile.txt"), "oldTest", OLD_FILE_TIMESTAMP); TestFileUtils.writeString(new File(repoDir, "empty.txt"), ""); @@ -391,6 +392,17 @@ public class HttpTransporterTest { assertEquals(OLD_FILE_TIMESTAMP, file.lastModified()); } + @Test + protected void testGet_CompressionUsedWithPom() throws Exception { + File file = TestFileUtils.createTempFile("pom"); + GetTask task = new GetTask(URI.create("repo/artifact.pom")).setDataPath(file.toPath()); + transporter.get(task); + String acceptEncoding = httpServer.getLogEntries().get(0).getHeaders().get("Accept-Encoding"); + assertNotNull(acceptEncoding, "Missing Accept-Encoding header when retrieving pom"); + // support either gzip or deflate as the transporter implementation may vary + assertTrue(acceptEncoding.contains("gzip") || acceptEncoding.contains("deflate")); + } + @Test protected void testGet_EmptyResource() throws Exception { File file = TestFileUtils.createTempFile("failure"); diff --git a/maven-resolver-transport-jdk-parent/maven-resolver-transport-jdk-11/src/test/java/org/eclipse/aether/transport/jdk/JdkTransporterTest.java b/maven-resolver-transport-jdk-parent/maven-resolver-transport-jdk-11/src/test/java/org/eclipse/aether/transport/jdk/JdkTransporterTest.java index 1f2976072..625c092b8 100644 --- a/maven-resolver-transport-jdk-parent/maven-resolver-transport-jdk-11/src/test/java/org/eclipse/aether/transport/jdk/JdkTransporterTest.java +++ b/maven-resolver-transport-jdk-parent/maven-resolver-transport-jdk-11/src/test/java/org/eclipse/aether/transport/jdk/JdkTransporterTest.java @@ -104,6 +104,11 @@ class JdkTransporterTest extends HttpTransporterTest { @Test protected void testRequestTimeout() throws Exception {} + @Override + @Disabled("https://github.com/apache/maven-resolver/issues/1620") + @Test + protected void testGet_CompressionUsedWithPom() throws Exception {} + public JdkTransporterTest() { super(() -> new JdkTransporterFactory(standardChecksumExtractor(), new DefaultPathProcessor())); }
