svn commit: r17134 - trunk/src/argouml-app/tests/org/argouml/uml/diagram/TestDiagramFactory.java

Author: tfmorris
Date: 2009-04-26 13:23:10-0700
New Revision: 17134

Modified:
   trunk/src/argouml-app/tests/org/argouml/uml/diagram/TestDiagramFactory.java

Log:
Issue 5780: Disable sequence diagram test since it's testing wrong code anyway
http://argouml.tigris.org/issues/show_bug.cgi?id=5780

Modified: trunk/src/argouml-app/tests/org/argouml/uml/diagram/TestDiagramFactory.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/tests/org/argouml/uml/diagram/TestDiagramFactory.java?view=diff&pathrev=17134&r1=17133&r2=17134
==============================================================================
--- trunk/src/argouml-app/tests/org/argouml/uml/diagram/TestDiagramFactory.java	(original)
+++ trunk/src/argouml-app/tests/org/argouml/uml/diagram/TestDiagramFactory.java	2009-04-26 13:23:10-0700
@@ -24,6 +24,7 @@
 
 package org.argouml.uml.diagram;
 
+import org.apache.log4j.Logger;
 import org.argouml.model.InitializeModel;
 import org.argouml.model.Model;
 import org.argouml.profile.init.InitProfileSubsystem;
@@ -38,6 +39,9 @@
  */
 public class TestDiagramFactory extends TestCase {
 
+    private static final Logger LOG = 
+        Logger.getLogger(TestDiagramFactory.class);
+    
     protected void setUp() throws Exception {
         super.setUp();
         InitializeModel.initializeDefault();
@@ -70,7 +74,11 @@
         DiagramSettings settings = new DiagramSettings();
         for (DiagramType type : DiagramType.values()) {
             ArgoDiagram diagram;
-            if (type == DiagramType.State) {
+            if (type == DiagramType.Sequence) {
+                // TODO: Fix this so that new sequence diagrams are tested
+                LOG.warn("Skipping Sequence Diagram test because they are in a separate module");
+                return;
+            } else if (type == DiagramType.State) {
                 Object context = Model.getCoreFactory().buildClass(model);
                 Object machine = Model.getStateMachinesFactory()
                         .buildStateMachine(context);

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

To unsubscribe from this discussion, e-mail: [[email protected]].