Merge pull request #1207 from hcoles/feature/annotation_interface

allow removal of class annotations
hcoles · May 12, 2023 · 49b2327 · 49b2327
2 parents 8b5af8e + f60422a
commit 49b2327
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 3 deletions.
diff --git a/pitest-maven-verification/pom.xml b/pitest-maven-verification/pom.xml
@@ -106,7 +106,7 @@
 		<dependency>
 			<groupId>org.pitest</groupId>
 			<artifactId>pitest-entry</artifactId>
-			<version>1.0.0-SNAPSHOT</version>
+			<version>${project.version}</version>
 			<type>test-jar</type>
 			<scope>test</scope>
 		</dependency>

diff --git a/pitest/src/main/java/org/pitest/mutationtest/engine/gregor/MethodMutatorFactory.java b/pitest/src/main/java/org/pitest/mutationtest/engine/gregor/MethodMutatorFactory.java
@@ -50,6 +50,10 @@ default AnnotationVisitor createForAnnotation(NoMethodContext context, Annotatio
     return null;
   }
 
+  default boolean skipAnnotation(NoMethodContext nonMethodContext, AnnotationInfo annotationInfo) {
+    return false;
+  }
+
   default FieldVisitor createForField(NoMethodContext context, FieldInfo fieldInfo, FieldVisitor fieldVisitor) {
     return null;
   }

diff --git a/pitest/src/main/java/org/pitest/mutationtest/engine/gregor/MutatingClassVisitor.java b/pitest/src/main/java/org/pitest/mutationtest/engine/gregor/MutatingClassVisitor.java
@@ -60,8 +60,13 @@ public void visitSource(final String source, final String debug) {
 
   @Override
   public AnnotationVisitor visitAnnotation(String descriptor, boolean visible) {
-    AnnotationVisitor next = super.visitAnnotation(descriptor, visible);
     AnnotationInfo annotationInfo = new AnnotationInfo(descriptor, visible);
+    for (final MethodMutatorFactory each : this.mutators) {
+      if (each.skipAnnotation(this.nonMethodContext, annotationInfo)) {
+        return null;
+      }
+    }
+    AnnotationVisitor next = super.visitAnnotation(descriptor, visible);
     for (final MethodMutatorFactory each : this.mutators) {
       AnnotationVisitor fv = each.createForAnnotation(this.nonMethodContext, annotationInfo, next);
       if (fv != null) {

diff --git a/pom.xml b/pom.xml
@@ -22,7 +22,7 @@
     are used for few artifacts -->
 	<properties>
 		<!-- this revision number used only for local builds -->
-		<revision>1.0.0-SNAPSHOT</revision>
+		<revision>dev-SNAPSHOT</revision>
 		<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
 		<project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>