Changeset 271 in josm for test

Timestamp:

2007-07-04T00:34:16+02:00 (18 years ago)

Author:

imi

Message:

• fixed bug in 'combine way' that did not collect all properties.

Location:

test/org/openstreetmap/josm

Files:

• actions (added)
• actions/CombineWayActionTest.java (added)
• testframework/MainMock.java (modified) (3 diffs)
• testframework/MotherObject.java (modified) (4 diffs)

test/org/openstreetmap/josm/testframework/MainMock.java

@@ -1 +1 @@
 package org.openstreetmap.josm.testframework;
 
+import java.awt.AWTEvent;
+import java.awt.AWTException;
+import java.awt.Toolkit;
+import java.awt.event.AWTEventListener;
 import java.io.IOException;
 import java.util.Collection;
 import java.util.Collections;
 
+import javax.swing.JButton;
+import javax.swing.JDialog;
 import javax.swing.JFrame;
+import javax.swing.JOptionPane;
+import javax.swing.SwingUtilities;
 
+import org.junit.Before;
 import org.junit.BeforeClass;
 import org.openstreetmap.josm.Main;
@@ -15 +24 @@
 public class MainMock {
 
+        private static JDialog lastPopup;
+
+        @Before public void clearFoundPopup() {
+                lastPopup = null;
+        }
+        
         @BeforeClass public static void mockMain() throws Exception {
                 Main.pref = new Preferences(){
@@ -27 +42 @@
                 Main.main = new Main(){};
         }
+
+        @BeforeClass public static void startPopupKiller() throws AWTException {
+                Toolkit.getDefaultToolkit().addAWTEventListener(new AWTEventListener(){
+                        public void eventDispatched(AWTEvent event) {
+                                if (event.getSource() instanceof JButton) {
+                                        JButton b = (JButton)event.getSource();
+                                        if (b.getParent().getParent() instanceof JOptionPane) {
+                                                lastPopup = (JDialog)SwingUtilities.getRoot(b);
+                                                b.doClick();
+                                        }
+                                }
+            }
+                }, AWTEvent.FOCUS_EVENT_MASK);
+    }
+
+        public void assertPopup() {
+                waitForPopup();
+                lastPopup = null;
+        }
+
+        public JDialog waitForPopup() {
+            for (int i = 0; i < 100; ++i) {
+                        if (lastPopup != null)
+                                return lastPopup;
+                        try {Thread.sleep(10);} catch (InterruptedException e) {}
+                }
+                throw new AssertionError("Expected Popup dialog");
+    }
 }

test/org/openstreetmap/josm/testframework/MotherObject.java

@@ -16 +16 @@
 import org.openstreetmap.josm.data.projection.Epsg4326;
 
-public class MotherObject extends TestCase {
+abstract public class MotherObject extends TestCase {
 
+        public static DataSet dataSet;
+        
         @Override protected void setUp() throws Exception {
             super.setUp();
@@ -23 +25 @@
     }
 
-        public Node createNode(int id) {
+        public static Node createNode(int id) {
                 return createNode(id, 0, 0);
         }
         
-        public Node createNode(int id, double lat, double lon) {
+        public static Node createNode(int id, double lat, double lon) {
                 Node n = createNode(lat, lon);
                 n.id = id;
@@ -33 +35 @@
         }
 
-        public Node createNode() {
+        public static Node createNode() {
                 return createNode(Math.random()*360-180, Math.random()*180-90);
         }
 
-        public Node createNode(double lat, double lon) {
-            return new Node(new LatLon(lat,lon));
+        public static Node createNode(double lat, double lon) {
+            Node node = new Node(new LatLon(lat,lon));
+            if (dataSet != null)
+                dataSet.nodes.add(node);
+                return node;
     }
         
         
-        public Segment createSegment(long id) {
+        public static Segment createSegment(long id) {
                 Segment s = createSegment();
                 s.id = id;
                 return s;
         }
-        public Segment createSegment(long id, Node from, Node to) {
+        public static Segment createSegment(long id, Node from, Node to) {
                 Segment s = new Segment(from, to);
                 s.id = id;
                 return s;
         }
-        public Segment createSegment() {
-                return new Segment(createNode(), createNode());
+        public static Segment createSegment() {
+                Segment segment = new Segment(createNode(), createNode());
+                if (dataSet != null)
+                        dataSet.segments.add(segment);
+                return segment;
         }
         
         
-        public Way createWay() {
+        public static Way createWay() {
                 return createWay(0);
         }
-        public Way createWay(Segment... segments) {
+        public static Way createWay(Segment... segments) {
                 return createWay(0, segments);
         }
-        public Way createWay(long id, Segment... segments) {
+        public static Way createWay(long id, Segment... segments) {
                 Way way = new Way();
                 way.segments.addAll(Arrays.asList(segments));
                 way.id = id;
+                if (dataSet != null)
+                        dataSet.ways.add(way);
                 return way;
         }
         
-        public DataSet createDataSet() {
+        public static DataSet createDataSet() {
             DataSet ds = new DataSet();
                 Node node1 = createNode();
@@ -85 +95 @@
     }
 
-        public void assertContainsSame(Collection<OsmPrimitive> data, OsmPrimitive... all) {
+        public static void assertContainsSame(Collection<OsmPrimitive> data, OsmPrimitive... all) {
                 Collection<OsmPrimitive> copy = new LinkedList<OsmPrimitive>(data);
                 copy.removeAll(Arrays.asList(all));
                 assertEquals(0, copy.size());
     }
-        
-        /**
-         * To have JUnit shut up.
-         */
-        public void test() {
-        }
 }