Merge pull request #1664 from arjantijms/gfrunner_tx_2

Added dependency to EJB 30 TCK, on which tests seem to depend.
jakartaee · Nov 19, 2024 · e4d16ea · e4d16ea
2 parents e117193 + 18aea70
commit e4d16ea
Showing 1 changed file with 13 additions and 3 deletions.
diff --git a/glassfish-runner/transactions-tck/transactions-tck-run/pom.xml b/glassfish-runner/transactions-tck/transactions-tck-run/pom.xml
@@ -72,6 +72,11 @@
             <artifactId>jta-tck</artifactId>
             <version>${tck.version}</version>
         </dependency>
+        <dependency>
+            <groupId>jakarta.tck</groupId>
+            <artifactId>ejb30</artifactId>
+            <version>${tck.version}</version>
+        </dependency>
 
         <!-- Junit5 -->
         <dependency>
@@ -103,8 +108,6 @@
             <scope>test</scope>
         </dependency>
 
-
-
         <!--
             The Arquillian protocol to use the application client container for testing
         -->
@@ -154,7 +157,7 @@
                 </exclusion>
             </exclusions>
         </dependency>
-        
+
         <dependency>
             <groupId>jakarta.tck.arquillian</groupId>
             <artifactId>arquillian-protocol-common</artifactId>
@@ -208,6 +211,8 @@
                         <phase>generate-resources</phase>
                         <configuration>
                             <artifactItems>
+
+                                <!-- The application client container needs these two dependencies -->
                                 <artifactItem>
                                     <groupId>jakarta.tck.arquillian</groupId>
                                     <artifactId>tck-porting-lib</artifactId>
@@ -222,6 +227,11 @@
                                     <outputDirectory>${project.build.directory}/lib</outputDirectory>
                                     <destFileName>arquillian-protocol-lib.jar</destFileName>
                                 </artifactItem>
+
+                                <!-- 
+                                    AppClientDeploymentPackager needs this on the Eclipse CI since it can't fully resolve
+                                    jakarta.tck.arquillian:arquillian-protocol-lib in code for some reason.
+                                -->
                                 <artifactItem>
                                     <groupId>jakarta.tck.arquillian</groupId>
                                     <artifactId>arquillian-protocol-lib</artifactId>