Added bootstrap dependency for tests (#755)

* Added bootstrap dependency for tests * Properly propagated Surefire plugin arguments
windup · Oct 5, 2022 · 9ae5a81 · 9ae5a81
1 parent 77b7895
commit 9ae5a81
Showing 1 changed file with 8 additions and 14 deletions.
diff --git a/rules/pom.xml b/rules/pom.xml
@@ -93,6 +93,11 @@
             <artifactId>windup-module-spec</artifactId>
             <scope>test</scope>
         </dependency>
+        <dependency>
+            <groupId>org.jboss.windup</groupId>
+            <artifactId>windup-bootstrap-windup</artifactId>
+            <scope>test</scope>
+        </dependency>
 
         <!-- Workaround for addon dependency issue - see WINDUP-3229 -->
         <dependency>
@@ -154,20 +159,8 @@
                 <jdk>[9,)</jdk>
             </activation>
             <properties>
-                <arguments>-Xms512m -Xmx4096m -XX:ReservedCodeCacheSize=128m</arguments>
+                <arguments>-Xms512m -Xmx4096m -XX:ReservedCodeCacheSize=128m --add-modules java.se</arguments>
             </properties>
-            <build>
-                <plugins>
-                    <plugin>
-                        <groupId>org.apache.maven.plugins</groupId>
-                        <artifactId>maven-surefire-plugin</artifactId>
-                        <configuration>
-                            <!-- http://maven.apache.org/surefire/maven-surefire-plugin/faq.html#late-property-evaluation -->
-                            <argLine>--add-modules java.se</argLine>
-                        </configuration>
-                    </plugin>
-                </plugins>
-            </build>
         </profile>
 
         <profile>
@@ -317,7 +310,8 @@
                 <groupId>org.apache.maven.plugins</groupId>
                	<artifactId>maven-surefire-plugin</artifactId>
                 <configuration>
-                    <argLine>${arguments}</argLine>
+                    <!-- http://maven.apache.org/surefire/maven-surefire-plugin/faq.html#late-property-evaluation -->
+                    <argLine>@{arguments}</argLine>
                     <excludes>
                         <exclude>**/WindupRulesMultipleTests*</exclude>
                     </excludes>