Merge pull request #1407 from hcoles/feature/classloader_logging

Feature/classloader logging

Author: hcoles

.github/workflows/ci.yml

@@ -25,7 +25,7 @@ jobs:
       - name: 'Checkout from Git'
         uses: actions/checkout@v3
       - name: 'Set up JDK ${{ matrix.java }}'
-        uses: actions/setup-java@v3
+        uses: actions/setup-java@v4
         with:
           java-version: ${{ matrix.java }}
           distribution: adopt

pitest/src/main/java/org/pitest/mutationtest/execute/CatchNewClassLoadersTransformer.java

@@ -2,12 +2,14 @@
 
 import org.pitest.boot.HotSwapAgent;
 import org.pitest.classinfo.ClassName;
+import org.pitest.util.Log;
 
 import java.lang.instrument.ClassFileTransformer;
 import java.security.ProtectionDomain;
 import java.util.Collections;
 import java.util.Map;
 import java.util.WeakHashMap;
+import java.util.logging.Logger;
 
 /**
  * Pitest mainly inserts mutants by calling Instrumentation.redefineClasses using
@@ -28,6 +30,8 @@
  */
 public class CatchNewClassLoadersTransformer implements ClassFileTransformer {
 
+    private static final Logger LOG = Log.getLogger();
+
     private static String targetClass;
     private static byte[] currentMutant;
 
@@ -38,6 +42,9 @@ public class CatchNewClassLoadersTransformer implements ClassFileTransformer {
     public static synchronized void setMutant(String className, byte[] mutant) {
         targetClass = className;
         currentMutant = mutant;
+
+        logClassloaders();
+
         for (ClassLoader each : CLASS_LOADERS.keySet()) {
             final Class<?> clazz = checkClassForLoader(each, className);
             if (clazz != null) {
@@ -83,4 +90,10 @@ private boolean shouldTransform(ClassLoader loader) {
         return !loader.getClass().getName().startsWith("com.google.gwtmockito.");
     }
 
+    private static void logClassloaders() {
+        if (CLASS_LOADERS.size() > 1) {
+            LOG.fine("Accumulated " + CLASS_LOADERS.size() + " classloaders");
+        }
+    }
+
 }

pom.xml

@@ -99,7 +99,7 @@
 				</property>
 			</activation>
 			<modules>
-                            <module>pitest-maven-verification</module>
+				<module>pitest-maven-verification</module>
 			</modules>
 		</profile>
 		<profile>
@@ -404,4 +404,11 @@
 		</plugins>
 	</build>
 
+	<distributionManagement>
+		<snapshotRepository>
+			<id>central</id>
+			<url>https://central.sonatype.com/repository/maven-snapshots/</url>
+		</snapshotRepository>
+	</distributionManagement>
+
 </project>