You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@commons.apache.org by br...@apache.org on 2014/08/23 11:30:12 UTC

svn commit: r1619994 - /commons/proper/lang/trunk/src/test/java/org/apache/commons/lang3/reflect/FieldUtilsTest.java

Author: britter
Date: Sat Aug 23 09:30:12 2014
New Revision: 1619994

URL: http://svn.apache.org/r1619994
Log:
Add missing fail calls. Errors would be silently ignored if those calls are missing.

Modified:
    commons/proper/lang/trunk/src/test/java/org/apache/commons/lang3/reflect/FieldUtilsTest.java

Modified: commons/proper/lang/trunk/src/test/java/org/apache/commons/lang3/reflect/FieldUtilsTest.java
URL: http://svn.apache.org/viewvc/commons/proper/lang/trunk/src/test/java/org/apache/commons/lang3/reflect/FieldUtilsTest.java?rev=1619994&r1=1619993&r2=1619994&view=diff
==============================================================================
--- commons/proper/lang/trunk/src/test/java/org/apache/commons/lang3/reflect/FieldUtilsTest.java (original)
+++ commons/proper/lang/trunk/src/test/java/org/apache/commons/lang3/reflect/FieldUtilsTest.java Sat Aug 23 09:30:12 2014
@@ -1084,12 +1084,14 @@ public class FieldUtilsTest {
         field = parentClass.getDeclaredField("i");
         try {
             FieldUtils.writeField(field, publicChild, Integer.valueOf(Integer.MAX_VALUE));
+            fail("Expected IllegalAccessException");
         } catch (final IllegalAccessException e) {
             // pass
         }
         field = parentClass.getDeclaredField("d");
         try {
             FieldUtils.writeField(field, publicChild, Double.valueOf(Double.MAX_VALUE));
+            fail("Expected IllegalAccessException");
         } catch (final IllegalAccessException e) {
             // pass
         }