Author: tfmorris Date: 2008-05-03 17:00:16-0700 New Revision: 14609 Modified: trunk/src/argouml-app/src/org/argouml/profile/CoreProfileReference.java trunk/src/argouml-app/tests/org/argouml/profile/TestCoreProfileReference.java
Log: Issue 5063 - Fix more failing profile tests. Not sure how these worked before. Modified: trunk/src/argouml-app/src/org/argouml/profile/CoreProfileReference.java Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/profile/CoreProfileReference.java?view=diff&rev=14609&p1=trunk/src/argouml-app/src/org/argouml/profile/CoreProfileReference.java&p2=trunk/src/argouml-app/src/org/argouml/profile/CoreProfileReference.java&r1=14608&r2=14609 ============================================================================== --- trunk/src/argouml-app/src/org/argouml/profile/CoreProfileReference.java (original) +++ trunk/src/argouml-app/src/org/argouml/profile/CoreProfileReference.java 2008-05-03 17:00:16-0700 @@ -56,5 +56,7 @@ new URL(PROFILES_BASE_URL + fileName)); assert fileName != null : "null isn't acceptable as the profile file name."; + assert !"".equals(fileName) + : "the empty string isn't acceptable as the profile file name."; } } Modified: trunk/src/argouml-app/tests/org/argouml/profile/TestCoreProfileReference.java Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/tests/org/argouml/profile/TestCoreProfileReference.java?view=diff&rev=14609&p1=trunk/src/argouml-app/tests/org/argouml/profile/TestCoreProfileReference.java&p2=trunk/src/argouml-app/tests/org/argouml/profile/TestCoreProfileReference.java&r1=14608&r2=14609 ============================================================================== --- trunk/src/argouml-app/tests/org/argouml/profile/TestCoreProfileReference.java (original) +++ trunk/src/argouml-app/tests/org/argouml/profile/TestCoreProfileReference.java 2008-05-03 17:00:16-0700 @@ -60,6 +60,8 @@ throws MalformedURLException { try { new CoreProfileReference(""); + // TODO: This requires that the tests be run with assertions + // enabled which isn't guaranteed in all environments. fail("Expecting AssertionError due to empty file name."); } catch (AssertionError e) { // expected @@ -75,6 +77,8 @@ throws MalformedURLException { try { new CoreProfileReference(null); + // TODO: This requires that the tests be run with assertions + // enabled which isn't guaranteed in all environments. fail("Expecting AssertionError due to null file name."); } catch (AssertionError e) { // expected --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
