FYI, NAE is a IAE so this has no effect only that if the IAE type is not a NAE that this test will fail with an unexpected error.

--jason


On Saturday, August 16, 2003, at 10:08 PM, [EMAIL PROTECTED] wrote:

bsnyder     2003/08/16 08:08:53

Modified: modules/common/src/test/org/apache/geronimo/common
StringsTest.java
Log:
Fixed the StringsTest - It was catching IllegalArgumentException instead of NullArgumentException.


Revision Changes Path
1.2 +4 -4 incubator-geronimo/modules/common/src/test/org/apache/geronimo/common/ StringsTest.java


Index: StringsTest.java
===================================================================
RCS file: /home/cvs/incubator-geronimo/modules/common/src/test/org/apache/ geronimo/common/StringsTest.java,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -r1.1 -r1.2
--- StringsTest.java 16 Aug 2003 14:06:48 -0000 1.1
+++ StringsTest.java 16 Aug 2003 15:08:53 -0000 1.2
@@ -89,8 +89,8 @@


           try {
               Strings.isEmpty(null);
  -            fail("Expected IllegalArgumnetException to be thrown");
  -        } catch (IllegalArgumentException ignore) {}
  +            fail("Expected NullArgumentException to be thrown");
  +        } catch (NullArgumentException ignore) {}
       }

       public void testCompare() {
  @@ -190,4 +190,4 @@
           assertEquals("four", strArray[3]);
           assertEquals("five", strArray[4]);
       }
  -}
  \ No newline at end of file
  +}







Reply via email to