Author: mvw
Date: 2007-06-16 00:00:05-0700
New Revision: 12856
Modified:
trunk/tests/org/argouml/ui/TestDetailsPane.java
Log:
Fix for issue 1925: "Selected tab is remembered per fig". Correct failing test. This test needs some more work.
Modified: trunk/tests/org/argouml/ui/TestDetailsPane.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/tests/org/argouml/ui/TestDetailsPane.java?view=diff&rev=12856&p1=trunk/tests/org/argouml/ui/TestDetailsPane.java&p2=trunk/tests/org/argouml/ui/TestDetailsPane.java&r1=12855&r2=12856
==============================================================================
--- trunk/tests/org/argouml/ui/TestDetailsPane.java (original)
+++ trunk/tests/org/argouml/ui/TestDetailsPane.java 2007-06-16 00:00:05-0700
@@ -27,9 +27,9 @@
import javax.swing.JPanel;
import junit.framework.TestCase;
-import org.argouml.model.InitializeModel;
import org.argouml.cognitive.ui.TabToDo;
+import org.argouml.model.InitializeModel;
import org.argouml.model.Model;
import org.argouml.ui.targetmanager.TargetEvent;
import org.argouml.ui.targetmanager.TargetManager;
@@ -66,6 +66,7 @@
DetailsPane pane = new DetailsPane("detail", Horizontal.getInstance());
JPanel todoPane = pane.getTab(TabToDo.class);
JPanel propertyPane = pane.getTab(TabProps.class);
+// JPanel docPane = pane.getTab(TabDocumentation.class);
assertNotNull(todoPane);
assertNotNull(propertyPane);
@@ -82,7 +83,7 @@
o,
});
pane.targetSet(e);
- assertEquals("1:", todoPane, pane.getTabs().getSelectedComponent());
+ assertEquals("1:", propertyPane, pane.getTabs().getSelectedComponent());
UMLDiagram diagram = new UMLClassDiagram();
e =
new TargetEvent(
@@ -120,9 +121,9 @@
* commented out next piece to remove failure of testcase. The testcase
* is probably correct but the implementation of DetailsPane is not
- _pane.getTabs().setSelectedComponent(_docPane);
- _pane.targetSet(e);
- assertEquals(_docPane, _pane.getTabs().getSelectedComponent());
+ pane.getTabs().setSelectedComponent(docPane);
+ pane.targetSet(e);
+ assertEquals(docPane, pane.getTabs().getSelectedComponent());
*/
}
}
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.