CAUSEWAY-3855: surefire configuration to allow dynamic agent loading

apache · Feb 26, 2025 · 43a5f3f · 43a5f3f
1 parent 8aeebea
commit 43a5f3f
Show file tree

Hide file tree

Showing 11 changed files with 7 additions and 26 deletions.
diff --git a/bom/pom.xml b/bom/pom.xml
@@ -192,7 +192,10 @@ identified
         <snakeyaml.version>2.0</snakeyaml.version>
         <spring-boot.version>3.4.3</spring-boot.version>
 
-        <surefire-plugin.argLine>-Xmx384m</surefire-plugin.argLine>
+        <surefire-plugin.argLine>
+            -Xmx384m
+            -XX:+EnableDynamicAgentLoading 
+        </surefire-plugin.argLine>
         <surefire.useModulePath>false</surefire.useModulePath> <!-- disable
         module-path for testing, that is, put everything on the class-path -->
         <swagger-core.version>2.2.28</swagger-core.version>

diff --git a/...useway/core/metamodel/facets/object/domainobjectlayout/DomainObjectLayoutFactoryTest.java b/...useway/core/metamodel/facets/object/domainobjectlayout/DomainObjectLayoutFactoryTest.java
@@ -123,7 +123,6 @@ public void whenSpecified() {
                 facetFactory.process(processClassContext);
 
                 final Facet facet = facetHolder.getFacet(CssClassFacet.class);
-                System.err.printf("%s%n", facet);
                 assertNotNull(facet);
                 assertTrue(facet instanceof CssClassFacetForDomainObjectLayoutAnnotation);
 

diff --git a/...el/src/test/java/org/apache/causeway/core/metamodel/services/menubars/BSMenuBarsTest.java b/...el/src/test/java/org/apache/causeway/core/metamodel/services/menubars/BSMenuBarsTest.java
@@ -53,7 +53,8 @@ void happy_case() throws Exception {
         final Map<String, String> schemas = jaxbService.toXsd(menuBars, CausewaySchemas.INCLUDE);
 
         menuBars.visit(data->{
-            System.err.printf("%s%n", data);
+            //debug
+            //System.err.printf("%s%n", data);
         });
 
         for (Map.Entry<String, String> entry : schemas.entrySet()) {
@@ -63,7 +64,7 @@ void happy_case() throws Exception {
     }
 
     private void println(final String string) {
-        //for test debugging only
+        //debug
         //System.err.printf("%s%n", string);
     }
 

diff --git a/...e/causeway/core/metamodel/valuesemantics/temporal/TemporalValueSemanticsProviderTest.java b/...e/causeway/core/metamodel/valuesemantics/temporal/TemporalValueSemanticsProviderTest.java
@@ -75,8 +75,6 @@ void timeFormats(final TimePrecision timePrecision) {
 
         var formattedTemporal = formatter.format(localTime);
         assertNotNull(formattedTemporal);
-
-        System.out.println(formattedTemporal);
     }
 
     // -- HELPER

diff --git a/extensions/pom.xml b/extensions/pom.xml
@@ -43,9 +43,6 @@
 	<properties>
 		<jar-plugin.automaticModuleName>org.apache.causeway.extensions</jar-plugin.automaticModuleName>
 		<git-plugin.propertiesDir>org/apache/causeway/extensions</git-plugin.propertiesDir>
-
-		<!-- override surefire defaults from bom, as this is not a Java module yet -->
-		<surefire-plugin.argLine>-Xmx384m</surefire-plugin.argLine>
 	</properties>
 
 	<build>

diff --git a/persistence/jdo/datanucleus/pom.xml b/persistence/jdo/datanucleus/pom.xml
@@ -30,9 +30,6 @@
 	<properties>
 		<jar-plugin.automaticModuleName>org.apache.causeway.persistence.jdo.datanucleus</jar-plugin.automaticModuleName>
 		<git-plugin.propertiesDir>org/apache/causeway/persistence/jdo/datanucleus</git-plugin.propertiesDir>
-
-		<!-- override surefire defaults from bom, as this is not a Java module yet -->
-		<surefire-plugin.argLine>-Xmx384m</surefire-plugin.argLine>
 	</properties>
 
 	<dependencies>

diff --git a/regressiontests/pom.xml b/regressiontests/pom.xml
@@ -48,9 +48,6 @@
 
 		<jar-plugin.automaticModuleName>org.apache.causeway.regressiontests</jar-plugin.automaticModuleName>
 		<git-plugin.propertiesDir>org/apache/causeway/regressiontests</git-plugin.propertiesDir>
-
-		<!-- override surefire defaults from bom, as this is not a Java module yet -->
-		<surefire-plugin.argLine>-Xmx384m</surefire-plugin.argLine>
 	</properties>
 
 

diff --git a/retired/shiro/security/shiro/pom.xml b/retired/shiro/security/shiro/pom.xml
@@ -39,9 +39,6 @@
     	<!-- apache shiro is not migrated to JPMS yet, so no point in converting ours to JPMS -->
         <jar-plugin.automaticModuleName>org.apache.causeway.security.shiro</jar-plugin.automaticModuleName>
         <git-plugin.propertiesDir>org/apache/causeway/security/shiro</git-plugin.propertiesDir>
-
-        <!-- override surefire defaults from bom, as this is not a Java module yet -->
-		<surefire-plugin.argLine>-Xmx384m</surefire-plugin.argLine>
     </properties>
 
     <dependencies>

diff --git a/testing/pom.xml b/testing/pom.xml
@@ -44,9 +44,6 @@
 
 		<jar-plugin.automaticModuleName>org.apache.causeway.testing</jar-plugin.automaticModuleName>
 		<git-plugin.propertiesDir>org/apache/causeway/testing</git-plugin.propertiesDir>
-
-		<!-- override surefire defaults from bom, as this is not a Java module yet -->
-		<surefire-plugin.argLine>-Xmx384m</surefire-plugin.argLine>
 	</properties>
 
 	<build>

diff --git a/viewers/restfulobjects/jaxrs-resteasy/pom.xml b/viewers/restfulobjects/jaxrs-resteasy/pom.xml
@@ -30,9 +30,6 @@
 	<properties>
 		<jar-plugin.automaticModuleName>org.apache.causeway.viewer.restfulobjects.jaxrsresteasy</jar-plugin.automaticModuleName>
 		<git-plugin.propertiesDir>org/apache/causeway/viewer.restfulobjects/jaxrsresteasy</git-plugin.propertiesDir>
-
-		<!-- override surefire defaults from bom, as this is not a Java module yet -->
-		<surefire-plugin.argLine>-Xmx384m</surefire-plugin.argLine>
 	</properties>
 
 	<build>

diff --git a/viewers/wicket/viewer/pom.xml b/viewers/wicket/viewer/pom.xml
@@ -34,8 +34,6 @@
     <properties>
         <jar-plugin.automaticModuleName>org.apache.causeway.viewer.wicket.viewer</jar-plugin.automaticModuleName>
         <git-plugin.propertiesDir>org/apache/causeway/viewer/wicket/viewer</git-plugin.propertiesDir>
-		<!-- override surefire defaults from bom, as this is not a Java module yet -->
-		<surefire-plugin.argLine>-Xmx384m</surefire-plugin.argLine>
     </properties>
 
     <build>