From 8f0bb8149ca43d9df1f23d06058cf72bfcd93002 Mon Sep 17 00:00:00 2001
From: olivier lamy <oliver.lamy@gmail.com>
Date: Wed, 19 May 2021 15:43:34 +1000
Subject: [PATCH] align names with jetty-10

Signed-off-by: olivier lamy <oliver.lamy@gmail.com>
---
 tests/test-distribution/pom.xml                    |  2 +-
 .../tests/distribution/DistributionTester.java     | 14 +++++++-------
 .../eclipse/jetty/tests/distribution/CDITests.java |  4 ++--
 3 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/tests/test-distribution/pom.xml b/tests/test-distribution/pom.xml
index 26e74893cf0..6e2771a7f22 100644
--- a/tests/test-distribution/pom.xml
+++ b/tests/test-distribution/pom.xml
@@ -172,7 +172,7 @@
               <artifactId>maven-surefire-plugin</artifactId>
               <configuration>
                 <systemPropertyVariables>
-                  <javaOptions>--add-opens java.base/jdk.internal.misc=ALL-UNNAMED --add-opens java.base/java.lang=ALL-UNNAMED</javaOptions>
+                  <cdi.tests.jvmArgs>--add-opens java.base/jdk.internal.misc=ALL-UNNAMED --add-opens java.base/java.lang=ALL-UNNAMED</cdi.tests.jvmArgs>
                 </systemPropertyVariables>
               </configuration>
             </plugin>
diff --git a/tests/test-distribution/src/main/java/org/eclipse/jetty/tests/distribution/DistributionTester.java b/tests/test-distribution/src/main/java/org/eclipse/jetty/tests/distribution/DistributionTester.java
index 55f90ea1fb1..c38993baa84 100644
--- a/tests/test-distribution/src/main/java/org/eclipse/jetty/tests/distribution/DistributionTester.java
+++ b/tests/test-distribution/src/main/java/org/eclipse/jetty/tests/distribution/DistributionTester.java
@@ -150,7 +150,7 @@ public class DistributionTester
 
         List<String> commands = new ArrayList<>();
         commands.add(getJavaExecutable());
-        commands.addAll(config.javaOptions == null ? Collections.emptyList() : config.javaOptions);
+        commands.addAll(config.jvmArgs);
         commands.add("-Djava.io.tmpdir=" + workDir.toAbsolutePath().toString());
         commands.add("-jar");
         commands.add(config.jettyHome.toAbsolutePath() + "/start.jar");
@@ -382,18 +382,18 @@ public class DistributionTester
         private String jettyVersion;
         private String mavenLocalRepository = System.getProperty("user.home") + "/.m2/repository";
         private Map<String, String> mavenRemoteRepositories = new HashMap<>();
-        private List<String> javaOptions;
+        private List<String> jvmArgs = new ArrayList<>();
 
         @Override
         public String toString()
         {
-            return String.format("%s@%x{jettyBase=%s, jettyHome=%s, jettyVersion=%s, javaOptions=%s, mavenLocalRepository=%s, mavenRemoteRepositories=%s}",
+            return String.format("%s@%x{jettyBase=%s, jettyHome=%s, jettyVersion=%s, jvmArgs=%s, mavenLocalRepository=%s, mavenRemoteRepositories=%s}",
                 getClass().getSimpleName(),
                 hashCode(),
                 jettyBase,
                 jettyHome,
                 jettyVersion,
-                javaOptions,
+                jvmArgs,
                 mavenLocalRepository,
                 mavenRemoteRepositories);
         }
@@ -576,12 +576,12 @@ public class DistributionTester
         }
 
         /**
-         * @param javaOptions to use java options such <code></code>
+         * @param jvmArgs the jvm args to add
          * @return the {@link Builder}
          */
-        public Builder javaOptions(List<String> javaOptions)
+        public Builder jvmArgs(List<String> jvmArgs)
         {
-            config.javaOptions = javaOptions;
+            config.jvmArgs = jvmArgs;
             return this;
         }
 
diff --git a/tests/test-distribution/src/test/java/org/eclipse/jetty/tests/distribution/CDITests.java b/tests/test-distribution/src/test/java/org/eclipse/jetty/tests/distribution/CDITests.java
index 097e462e057..0d0e5b96ea3 100644
--- a/tests/test-distribution/src/test/java/org/eclipse/jetty/tests/distribution/CDITests.java
+++ b/tests/test-distribution/src/test/java/org/eclipse/jetty/tests/distribution/CDITests.java
@@ -84,10 +84,10 @@ public class CDITests extends AbstractDistributionTest
     public void testCDIIncludedInWebapp(String implementation, String integration, Consumer<DistributionTester> configure) throws Exception
     {
         String jettyVersion = System.getProperty("jettyVersion");
-        String javaOptions = System.getProperty("javaOptions");
+        String jvmArgs = System.getProperty("cdi.tests.jvmArgs");
         DistributionTester distribution = DistributionTester.Builder.newInstance()
             .jettyVersion(jettyVersion)
-            .javaOptions(javaOptions == null ? Collections.emptyList() : Arrays.asList(javaOptions.split("\\s+")))
+            .jvmArgs(jvmArgs == null ? Collections.emptyList() : Arrays.asList(jvmArgs.split("\\s+")))
             .mavenLocalRepository(System.getProperty("mavenRepoPath"))
             .build();