Issue #34: Add tests for suppress annotations

policeman-tools · Mar 29, 2015 · 5128cc5 · 5128cc5
1 parent 6981f7e
commit 5128cc5
Show file tree

Hide file tree

Showing 5 changed files with 38 additions and 2 deletions.
diff --git a/src/test/antunit/Java5DeprecatedAnnotation.class b/src/test/antunit/Java5DeprecatedAnnotation.class
diff --git a/src/test/antunit/Java5DeprecatedAnnotation.java b/src/test/antunit/Java5DeprecatedAnnotation.java
@@ -21,6 +21,7 @@ class Java5DeprecatedAnnotation {
 
   /** @deprecated **/
   public Java5DeprecatedAnnotation() {
+    new StringBuilder();
   }
 
 }
diff --git a/src/test/antunit/Java8Annotations.class b/src/test/antunit/Java8Annotations.class
diff --git a/src/test/antunit/Java8Annotations.java b/src/test/antunit/Java8Annotations.java
@@ -52,7 +52,7 @@ public InnerClassWithCtorParam(@FooBar X param) {
   public int testField2;
 
   @ClassFileOnly
-  public int testField3;
+  public StringBuilder testField3;
 
   @Retention(value=RetentionPolicy.RUNTIME)
   @Target({ElementType.TYPE_USE, ElementType.PARAMETER, ElementType.METHOD, ElementType.TYPE})

diff --git a/src/test/antunit/TestAnnotations.xml b/src/test/antunit/TestAnnotations.xml
@@ -34,12 +34,47 @@
   <target name="testMissingDeprecatedAnnotation">
     <au:expectfailure expectedMessage="Check for forbidden API calls failed, see log">
       <forbiddenapis failOnMissingClasses="true" classpath="${basedir}">
-        <fileset file="Java5DeprecatedAnnotation.class"/>
+        <file file="Java5DeprecatedAnnotation.class"/>
         java.lang.Deprecated @ Deprecated annotation
       </forbiddenapis>
     </au:expectfailure>
     <au:assertLogContains level="error" text="java.lang.Deprecated [Deprecated annotation]"/> 
     <au:assertLogContains level="error" text=" 1 error(s)"/> 
   </target>
 
+  <target name="testSuppressAnnotationsDeprecated1">
+    <forbiddenapis failOnMissingClasses="true" classpath="${basedir}" suppressAnnotation="java.lang.Deprecated">
+      <file file="Java5DeprecatedAnnotation.class"/>
+      java.lang.StringBuilder
+    </forbiddenapis>
+  </target>
+
+  <target name="testSuppressAnnotationsDeprecated2">
+    <forbiddenapis failOnMissingClasses="true" classpath="${basedir}">
+      <suppressAnnotation classname="java.lang.Deprecated"/>
+      <file file="Java5DeprecatedAnnotation.class"/>
+      java.lang.StringBuilder
+    </forbiddenapis>
+  </target>
+
+  <target name="testSuppressAnnotationsJava8">
+    <forbiddenapis failOnMissingClasses="true" classpath="${basedir}">
+      <!-- this should disable whole class scanning: -->
+      <suppressAnnotation classname="Java8Annotations$FooBar"/>
+      <!-- normally, this should produce similar errors like 'testAnnotations': -->
+      <file file="Java8Annotations.class"/>
+      Java8Annotations$ClassFileOnly @ Forbidden class-only annotation
+      Java8Annotations$FooBar @ Forbidden annotation
+      java.lang.Deprecated @ Deprecated annotation
+    </forbiddenapis>
+  </target>
+
+  <target name="testSuppressAnnotationsJava8ClassFileOnly">
+    <forbiddenapis failOnMissingClasses="true" classpath="${basedir}">
+      <suppressAnnotation classname="Java8Annotations$ClassFileOnly"/>
+      <file file="Java8Annotations.class"/>
+      java.lang.StringBuilder
+    </forbiddenapis>
+  </target>
+
 </project>