svn commit: r16594 - trunk/src/argouml-app/tests/org/argouml/TestDependencies.java

Michiel van der Wulp <[email protected]>
Newsgroups gmane.comp.lang.uml.argouml.cvs
Message-ID <[email protected]>
Author: mvw
Date: 2009-01-14 02:18:07-0800
New Revision: 16594

Modified:
   trunk/src/argouml-app/tests/org/argouml/TestDependencies.java

Log:
Fix the failing test - I hope, since I can not test.
There is no need to disallow any dependency from org.argouml.ui to org.argouml.moduleloader, since e.g. there is no dependency the other way.
This problem was caused by commit 16584, which solved issue 5509: The AboutBox asks the Module loader about the modules.

Modified: trunk/src/argouml-app/tests/org/argouml/TestDependencies.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/tests/org/argouml/TestDependencies.java?view=diff&pathrev=16594&r1=16593&r2=16594
==============================================================================
--- trunk/src/argouml-app/tests/org/argouml/TestDependencies.java	(original)
+++ trunk/src/argouml-app/tests/org/argouml/TestDependencies.java	2009-01-14 02:18:07-0800
@@ -1,5 +1,5 @@
 // $Id$
-// Copyright (c) 2007 The Regents of the University of California. All
+// Copyright (c) 2007-2009 The Regents of the University of California. All
 // Rights Reserved. Permission to use, copy, modify, and distribute this
 // software and its documentation without fee, and without a written
 // agreement is hereby granted, provided that the above copyright notice
@@ -155,7 +155,6 @@
             {"org.argouml.uml.diagram", "org.argouml.ui"},
             {"org.argouml.ui", "org.argouml.notation.ui"},
             {"org.argouml.util", "org.argouml.ui.cmd"},
-            {"org.argouml.moduleloader", "org.argouml.ui"},
             //{"org.argouml.kernel", "org.argouml.uml.diagram.ui"},//why does this fail?
         };
         for (int i = 0; i < dep.length; i++) {

------------------------------------------------------
http://argouml.tigris.org/ds/viewMessage.do?dsForumId=5905&dsMessageId=1023916

To unsubscribe from this discussion, e-mail: [[email protected]].
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.