From c7e02ba008565c1f432ab05b82eac7a6277158dd Mon Sep 17 00:00:00 2001
From: Olivier Lamy <olamy@apache.org>
Date: Wed, 31 Jul 2024 08:07:55 +1000
Subject: [PATCH] invoker:install should include test dependencies as well as
 they are used by the tests, this issue can happen only when not snapshots
 have been deployed of demos new versions

Signed-off-by: Olivier Lamy <olamy@apache.org>
---
 jetty-ee10/jetty-ee10-runner/pom.xml | 8 --------
 jetty-ee8/jetty-ee8-runner/pom.xml   | 8 --------
 jetty-ee9/jetty-ee9-runner/pom.xml   | 8 --------
 pom.xml                              | 1 +
 4 files changed, 1 insertion(+), 24 deletions(-)

diff --git a/jetty-ee10/jetty-ee10-runner/pom.xml b/jetty-ee10/jetty-ee10-runner/pom.xml
index 08ad7649d13..97c900f5eab 100644
--- a/jetty-ee10/jetty-ee10-runner/pom.xml
+++ b/jetty-ee10/jetty-ee10-runner/pom.xml
@@ -131,15 +131,7 @@
         <configuration>
           <debug>${it.debug}</debug>
           <addTestClassPath>true</addTestClassPath>
-          <extraArtifacts>
-            <extraArtifact>org.eclipse.jetty:jetty-client:${project.version}</extraArtifact>
-          </extraArtifacts>
           <junitPackageName>org.eclipse.jetty.maven.its.ee10.runner</junitPackageName>
-          <scriptVariables>
-            <maven.dependency.plugin.version>${maven.dependency.plugin.version}</maven.dependency.plugin.version>
-            <maven.surefire.plugin.version>${maven.surefire.plugin.version}</maven.surefire.plugin.version>
-            <hamcrest.version>${hamcrest.version}</hamcrest.version>
-          </scriptVariables>
           <goals>
             <goal>clean</goal>
           </goals>
diff --git a/jetty-ee8/jetty-ee8-runner/pom.xml b/jetty-ee8/jetty-ee8-runner/pom.xml
index 7d6973fdbdb..247ed87961f 100644
--- a/jetty-ee8/jetty-ee8-runner/pom.xml
+++ b/jetty-ee8/jetty-ee8-runner/pom.xml
@@ -135,15 +135,7 @@
         <configuration>
           <debug>${it.debug}</debug>
           <addTestClassPath>true</addTestClassPath>
-          <extraArtifacts>
-            <extraArtifact>org.eclipse.jetty:jetty-client:${project.version}</extraArtifact>
-          </extraArtifacts>
           <junitPackageName>org.eclipse.jetty.maven.its.ee8.runner</junitPackageName>
-          <scriptVariables>
-            <maven.dependency.plugin.version>${maven.dependency.plugin.version}</maven.dependency.plugin.version>
-            <maven.surefire.plugin.version>${maven.surefire.plugin.version}</maven.surefire.plugin.version>
-            <hamcrest.version>${hamcrest.version}</hamcrest.version>
-          </scriptVariables>
           <goals>
             <goal>clean</goal>
           </goals>
diff --git a/jetty-ee9/jetty-ee9-runner/pom.xml b/jetty-ee9/jetty-ee9-runner/pom.xml
index 981c02d2a3e..f5c4ef3e4fc 100644
--- a/jetty-ee9/jetty-ee9-runner/pom.xml
+++ b/jetty-ee9/jetty-ee9-runner/pom.xml
@@ -133,15 +133,7 @@
         <configuration>
           <debug>${it.debug}</debug>
           <addTestClassPath>true</addTestClassPath>
-          <extraArtifacts>
-            <extraArtifact>org.eclipse.jetty:jetty-client:${project.version}</extraArtifact>
-          </extraArtifacts>
           <junitPackageName>org.eclipse.jetty.maven.its.ee9.runner</junitPackageName>
-          <scriptVariables>
-            <maven.dependency.plugin.version>${maven.dependency.plugin.version}</maven.dependency.plugin.version>
-            <maven.surefire.plugin.version>${maven.surefire.plugin.version}</maven.surefire.plugin.version>
-            <hamcrest.version>${hamcrest.version}</hamcrest.version>
-          </scriptVariables>
           <goals>
             <goal>clean</goal>
           </goals>
diff --git a/pom.xml b/pom.xml
index 6b4d7081ff2..f0ca25ed6bc 100644
--- a/pom.xml
+++ b/pom.xml
@@ -1440,6 +1440,7 @@
             <timeoutInSeconds>300</timeoutInSeconds>
             <cloneProjectsTo>${project.build.directory}/it</cloneProjectsTo>
             <localRepositoryPath>${localRepoPath}</localRepositoryPath>
+            <scope>test</scope>
             <settingsFile>${settingsPath}</settingsFile>
             <skipInvocation>${skipTests}</skipInvocation>
             <streamLogsOnFailures>true</streamLogsOnFailures>