From 1f902f6371992b43ae99b40ed461835c772c2cf1 Mon Sep 17 00:00:00 2001
From: Ludovic Orban <lorban@bitronix.be>
Date: Thu, 16 Jun 2022 15:59:58 +0200
Subject: [PATCH] Disable H3 tests by default with a system property to
 explicitly enable them (#8150)

Fixes #8149 disable H3 tests by default with a system property to explicitly enable them

Signed-off-by: Ludovic Orban <lorban@bitronix.be>
---
 tests/test-http-client-transport/pom.xml           | 14 ++++++++++++++
 .../jetty/http/client/TransportProvider.java       |  7 ++++++-
 2 files changed, 20 insertions(+), 1 deletion(-)

diff --git a/tests/test-http-client-transport/pom.xml b/tests/test-http-client-transport/pom.xml
index e58686350731..f356289bc601 100644
--- a/tests/test-http-client-transport/pom.xml
+++ b/tests/test-http-client-transport/pom.xml
@@ -13,9 +13,23 @@
 
   <properties>
     <bundle-symbolic-name>${project.groupId}.client.http</bundle-symbolic-name>
+    <h3.test.enabled>false</h3.test.enabled>
   </properties>
 
   <build>
+    <pluginManagement>
+      <plugins>
+        <plugin>
+          <groupId>org.apache.maven.plugins</groupId>
+          <artifactId>maven-surefire-plugin</artifactId>
+          <configuration>
+            <systemPropertyVariables>
+              <org.eclipse.jetty.http.client.TransportProvider.H3.enable>${h3.test.enabled}</org.eclipse.jetty.http.client.TransportProvider.H3.enable>
+            </systemPropertyVariables>
+          </configuration>
+        </plugin>
+      </plugins>
+    </pluginManagement>
     <plugins>
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
diff --git a/tests/test-http-client-transport/src/test/java/org/eclipse/jetty/http/client/TransportProvider.java b/tests/test-http-client-transport/src/test/java/org/eclipse/jetty/http/client/TransportProvider.java
index a857e58380b8..ae64567e7f0b 100644
--- a/tests/test-http-client-transport/src/test/java/org/eclipse/jetty/http/client/TransportProvider.java
+++ b/tests/test-http-client-transport/src/test/java/org/eclipse/jetty/http/client/TransportProvider.java
@@ -14,6 +14,7 @@
 package org.eclipse.jetty.http.client;
 
 import java.util.Arrays;
+import java.util.EnumSet;
 import java.util.stream.Stream;
 
 import org.eclipse.jetty.util.StringUtil;
@@ -30,7 +31,11 @@ public static Stream<Transport> getActiveTransports()
         if (!StringUtil.isBlank(transports))
             return Arrays.stream(transports.split("\\s*,\\s*")).map(Transport::valueOf);
 
-        return Arrays.stream(Transport.values());
+        EnumSet<Transport> ts = EnumSet.allOf(Transport.class);
+        // Disable H3 tests unless explicitly enabled with a system property.
+        if (!Boolean.getBoolean(Transport.class.getName() + ".H3.enable"))
+            ts.remove(Transport.H3);
+        return ts.stream();
     }
 
     @Override