Cleanups for PR #86: Fix file name, change error message a bit

policeman-tools · Nov 14, 2015 · 34b5f1c · 34b5f1c
1 parent b9ea33e
commit 34b5f1c
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/src/main/resources/de/thetaphi/forbiddenapis/signatures/jdk-reflection.txt b/src/main/resources/de/thetaphi/forbiddenapis/signatures/jdk-reflection.txt
@@ -14,5 +14,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-java.lang.reflect.AccessibleObject#setAccessible(java.lang.reflect.AccessibleObject[], boolean) @ Reflection usage fails with SecurityManagers and likely will not work any more in Java 9
-java.lang.reflect.AccessibleObject#setAccessible(boolean) @ Reflection usage fails with SecurityManagers and likely will not work any more in Java 9
+@defaultMessage Reflection usage to work around access flags fails with SecurityManagers and likely will not work anymore on runtime classes in Java 9
+
+java.lang.reflect.AccessibleObject#setAccessible(java.lang.reflect.AccessibleObject[], boolean)
+java.lang.reflect.AccessibleObject#setAccessible(boolean)
diff --git a/src/test/antunit/Java6Reflection.class b/src/test/antunit/Java6Reflection.class
diff --git a/src/test/antunit/TestJava6Reflection.xml b/src/test/antunit/TestJava6Reflection.xml
@@ -20,10 +20,10 @@
     <au:expectfailure expectedMessage="Check for forbidden API calls failed, see log">
       <forbiddenapis>
         <bundledsignatures name="jdk-reflection"/>
-        <file file="OracleReflection.class"/>
+        <file file="Java6Reflection.class"/>
       </forbiddenapis>
     </au:expectfailure>
-    <au:assertLogContains level="error" text="java.lang.reflect.AccessibleObject#setAccessible(boolean) [Reflection usage fails with SecurityManagers"/>
+    <au:assertLogContains level="error" text="java.lang.reflect.AccessibleObject#setAccessible(boolean) [Reflection usage to work around access flags"/>
     <au:assertLogContains level="error" text=" 2 error(s)"/> 
   </target>