geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jason Dillon <ja...@coredevelopers.net>
Subject Re: cvs commit: incubator-geronimo/modules/common/src/test/org/apache/geronimo/common StringsTest.java
Date Sat, 16 Aug 2003 16:20:32 GMT
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, bsnyder@apache.org 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
>   +}
>
>
>
>


Mime
View raw message