svn commit: r14609 - trunk/src/argouml-app: src/org/argouml/profile tests/org/argouml/profile

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