Changeset 53 in josm for test/org

Timestamp:

2006-02-14T21:07:29+01:00 (20 years ago)

Author:

imi

Message:

fixed bug in osm writer/reader: they did not saved the modified state of an object.

Location:

test/org/openstreetmap/josm/test

Files:

• OsmWriterTest.java (modified) (4 diffs)
• framework/DataSetTestCaseHelper.java (modified) (2 diffs)

test/org/openstreetmap/josm/test/OsmWriterTest.java

@@ -14 +14 @@
 import org.jdom.input.SAXBuilder;
 import org.openstreetmap.josm.data.osm.DataSet;
+import org.openstreetmap.josm.data.osm.LineSegment;
+import org.openstreetmap.josm.data.osm.Node;
+import org.openstreetmap.josm.data.osm.OsmPrimitive;
+import org.openstreetmap.josm.data.osm.Track;
 import org.openstreetmap.josm.io.OsmWriter;
 import org.openstreetmap.josm.test.framework.Bug;
@@ -24 +28 @@
 public class OsmWriterTest extends TestCase {
 
+        private Node n1;
+        private Node n2;
+        private Node n3;
+        private Node n4;
+        private Node n5;
+        private LineSegment ls1;
+        private LineSegment ls2;
+        private LineSegment ls3;
+        private Track t;
+        
         private DataSet ds;
         private Element osm;
@@ -79 +93 @@
 
         
+        /**
+         * Verify that action tag is set correctly.
+         */
+        public void testActionTag() throws IOException, JDOMException {
+                int id = 1;
+                for (OsmPrimitive osm : ds.allPrimitives())
+                        osm.id = id++; // make all objects "old".
+                n1.setDeleted(true);
+                ls1.modified = true;
+                ls1.modifiedProperties = true;
+                ls3.modified = true;
+                t.modifiedProperties = true;
+                reparse();
+                
+                boolean foundNode = false;
+                for (Element n : nodes) {
+                        if (n.getAttributeValue("uid").equals(""+n1.id)) {
+                                assertEquals("delete", n.getAttributeValue("action"));
+                                foundNode = true;
+                        }
+                }
+                assertTrue("Node found in output", foundNode);
+
+                boolean foundLs1 = false;
+                boolean foundLs3 = false;
+                for (Element lsElem : lineSegments) {
+                        String idStr = lsElem.getAttributeValue("uid");
+                        String action = lsElem.getAttributeValue("action");
+                        if (idStr.equals(""+ls1.id)) {
+                                assertEquals("Attribute action on modified data is ok", "modify", action);
+                                foundLs1 = true;
+                        } else if (idStr.equals(""+ls3.id)) {
+                                assertEquals("Attribute action on modified/object data is ok", "modify/object", action);
+                                foundLs3 = true;
+                        }
+                }
+                assertTrue("LineSegments found in output", foundLs1 && foundLs3);
+                
+                assertEquals("Track found in output", 1, tracks.size());
+                assertEquals("Attribute action on modifiedProperty data is ok", "modify/property", tracks.get(0).getAttributeValue("action"));
+        }
         
         
@@ -85 +140 @@
         protected void setUp() throws Exception {
                 super.setUp();
+                
                 // create some data
-                ds = DataSetTestCaseHelper.createCommon();
+                ds = new DataSet();
+                n1 = DataSetTestCaseHelper.createNode(ds);
+                n2 = DataSetTestCaseHelper.createNode(ds);
+                n3 = DataSetTestCaseHelper.createNode(ds);
+                n4 = DataSetTestCaseHelper.createNode(ds);
+                n5 = DataSetTestCaseHelper.createNode(ds);
+                ls1 = DataSetTestCaseHelper.createLineSegment(ds, n1, n2);
+                ls2 = DataSetTestCaseHelper.createLineSegment(ds, n2, n3);
+                ls3 = DataSetTestCaseHelper.createLineSegment(ds, n4, n5);
+                t = DataSetTestCaseHelper.createTrack(ds, ls1, ls2);
+                
                 reparse();
         }

test/org/openstreetmap/josm/test/framework/DataSetTestCaseHelper.java

@@ -1 +1 @@
 package org.openstreetmap.josm.test.framework;
+
+import java.util.Arrays;
 
 import org.openstreetmap.josm.data.GeoPoint;
@@ -33 +35 @@
                 LineSegment ls2 = createLineSegment(ds, n2, n3);
                 createLineSegment(ds, n4, n5);
+                createTrack(ds, ls1, ls2);
+                return ds;
+        }
+
+        public static Track createTrack(DataSet ds, LineSegment... lineSegments) {
                 Track t = new Track();
-                t.segments.add(ls1);
-                t.segments.add(ls2);
+                t.segments.addAll(Arrays.asList(lineSegments));
                 ds.tracks.add(t);
-                return ds;
+                return t;
         }