Issue #5401 - Fixing test failure due to optional JPMS dep.

Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
jetty · Oct 6, 2020 · cf0c141 · cf0c141
1 parent a22c41e
commit cf0c141
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/jetty-http-tools/pom.xml b/jetty-http-tools/pom.xml
@@ -10,6 +10,20 @@
   <artifactId>jetty-http-tools</artifactId>
   <name>Jetty :: HTTP Testing Utilities</name>
 
+  <build>
+    <plugins>
+      <plugin>
+        <artifactId>maven-surefire-plugin</artifactId>
+        <configuration>
+          <argLine>
+            @{argLine} ${jetty.surefire.argLine}
+            --add-modules org.hamcrest
+          </argLine>
+        </configuration>
+      </plugin>
+    </plugins>
+  </build>
+
   <dependencies>
     <dependency>
       <groupId>org.eclipse.jetty</groupId>

diff --git a/jetty-http-tools/src/main/java/module-info.java b/jetty-http-tools/src/main/java/module-info.java
@@ -22,6 +22,7 @@
     exports org.eclipse.jetty.http.tools.matchers;
 
     requires transitive org.eclipse.jetty.http;
+
     // Optional - only required if you use hamcrest too.
     requires static org.hamcrest;
 }