From 6c611acda84db366d9f2e5ab7185074447b1f926 Mon Sep 17 00:00:00 2001 From: Ludovic Orban Date: Mon, 2 Dec 2024 11:51:30 +0100 Subject: [PATCH] Fix incorrect maven coordinates Signed-off-by: Ludovic Orban --- .../jetty-ee10-demo-embedded/pom.xml | 24 +++++++++---------- .../tests/distribution/DistributionTests.java | 17 ++++++++----- 2 files changed, 23 insertions(+), 18 deletions(-) diff --git a/jetty-ee10/jetty-ee10-demos/jetty-ee10-demo-embedded/pom.xml b/jetty-ee10/jetty-ee10-demos/jetty-ee10-demo-embedded/pom.xml index 4de8383bcd4a..48746fc67918 100644 --- a/jetty-ee10/jetty-ee10-demos/jetty-ee10-demo-embedded/pom.xml +++ b/jetty-ee10/jetty-ee10-demos/jetty-ee10-demo-embedded/pom.xml @@ -80,36 +80,36 @@ jetty-ee10-webapp - org.eclipse.jetty.ee10.demos - jetty-ee10-demo-async-rest-webapp + org.eclipse.jetty.demos + jetty-servlet5-demo-async-rest-webapp ${project.version} war - org.eclipse.jetty.ee10.demos - jetty-ee10-demo-jndi-webapp + org.eclipse.jetty.demos + jetty-servlet5-demo-jndi-webapp ${project.version} war - org.eclipse.jetty.ee10.demos - jetty-ee10-demo-jsp-webapp + org.eclipse.jetty.demos + jetty-servlet5-demo-jsp-webapp ${project.version} war - org.eclipse.jetty.ee10.demos - jetty-ee10-demo-mock-resources + org.eclipse.jetty.demos + jetty-servlet5-demo-mock-resources - org.eclipse.jetty.ee10.demos - jetty-ee10-demo-simple-webapp + org.eclipse.jetty.demos + jetty-servlet5-demo-simple-webapp ${project.version} war - org.eclipse.jetty.ee10.demos - jetty-ee10-demo-spec-webapp + org.eclipse.jetty.demos + jetty-servlet5-demo-spec-webapp ${project.version} war diff --git a/tests/test-distribution/test-distribution-common/src/test/java/org/eclipse/jetty/tests/distribution/DistributionTests.java b/tests/test-distribution/test-distribution-common/src/test/java/org/eclipse/jetty/tests/distribution/DistributionTests.java index 31c6398e8162..83863ad24cfb 100644 --- a/tests/test-distribution/test-distribution-common/src/test/java/org/eclipse/jetty/tests/distribution/DistributionTests.java +++ b/tests/test-distribution/test-distribution-common/src/test/java/org/eclipse/jetty/tests/distribution/DistributionTests.java @@ -1696,12 +1696,14 @@ public void testXmlDeployWarNotInWebapps(String env) throws Exception }; // Put war into ${jetty.base}/wars/ directory - Path srcWar = ("ee8".equals(env) - ? distribution.resolveArtifact("org.eclipse.jetty.demos:jetty-servlet4-demo-simple-webapp:war:" + jettyVersion) : distribution.resolveArtifact("org.eclipse.jetty.demos:jetty-servlet5-demo-simple-webapp:war:" + jettyVersion)); + String coordinates = "org.eclipse.jetty.demos:jetty-%s-demo-simple-webapp:war:%s".formatted( + "ee8".equals(env) ? "servlet4" : "servlet5", + jettyVersion + ); Path warsDir = jettyBase.resolve("wars"); FS.ensureDirExists(warsDir); Path destWar = warsDir.resolve("demo.war"); - Files.copy(srcWar, destWar); + Files.copy(distribution.resolveArtifact(coordinates), destWar); // Create XML for deployable String xml = """ @@ -2169,7 +2171,7 @@ public void testEagerMultiPartContentHandler(HttpVersion httpVersion) throws Exc } @ParameterizedTest - @ValueSource(strings = {"ee8", "ee9", "ee10"}) + @ValueSource(strings = {"ee8", "ee9", "ee10", "ee11"}) public void testLimitHandlers(String env) throws Exception { String jettyVersion = System.getProperty("jettyVersion"); @@ -2196,8 +2198,11 @@ public void testLimitHandlers(String env) throws Exception """; Files.writeString(jettyLogging, loggingConfig, StandardOpenOption.TRUNCATE_EXISTING); - Path war = distribution.resolveArtifact("org.eclipse.jetty." + env + ".demos:jetty-" + env + "-demo-simple-webapp:war:" + jettyVersion); - distribution.installWar(war, "test"); + String coordinates = "org.eclipse.jetty.demos:jetty-%s-demo-simple-webapp:war:%s".formatted( + "ee8".equals(env) ? "servlet4" : "servlet5", + jettyVersion + ); + distribution.installWar(distribution.resolveArtifact(coordinates), "test"); int port = Tester.freePort(); try (JettyHomeTester.Run run2 = distribution.start("jetty.http.selectors=1", "jetty.http.port=" + port))