Changeset 10946 in josm for trunk/test/unit/org/openstreetmap/josm/data/osm

Timestamp:

2016-09-03T20:01:46+02:00 (8 years ago)

Author:

Don-vip

Message:

fix #13395 - fix serialization issue causing bugs in copy/paste

File:

• trunk/test/unit/org/openstreetmap/josm/data/osm/NodeDataTest.java (modified) (3 diffs)

trunk/test/unit/org/openstreetmap/josm/data/osm/NodeDataTest.java

@@ -2 +2 @@
 package org.openstreetmap.josm.data.osm;
 
+import static org.junit.Assert.assertNull;
+import static org.junit.Assert.assertTrue;
+
 import java.io.ByteArrayInputStream;
 import java.io.ByteArrayOutputStream;
+import java.io.IOException;
 import java.io.ObjectInputStream;
 import java.io.ObjectOutputStream;
@@ -12 +16 @@
 
 public class NodeDataTest {
+
+    private static NodeData serializeUnserialize(NodeData data) throws IOException, ClassNotFoundException {
+        try (ByteArrayOutputStream bytes = new ByteArrayOutputStream();
+             ObjectOutputStream out = new ObjectOutputStream(bytes)) {
+            out.writeObject(data);
+            try (ObjectInputStream in = new ObjectInputStream(new ByteArrayInputStream(bytes.toByteArray()))) {
+                return (NodeData) in.readObject();
+            }
+        }
+    }
+
     @Test
     public void testSerializationForDragAndDrop() throws Exception {
@@ -19 +34 @@
         data.setVersion(14);
         data.setChangesetId(314159);
-        final Object readData;
-        try (ByteArrayOutputStream bytes = new ByteArrayOutputStream();
-             ObjectOutputStream out = new ObjectOutputStream(bytes)) {
-            out.writeObject(data);
-            try (ObjectInputStream in = new ObjectInputStream(new ByteArrayInputStream(bytes.toByteArray()))) {
-                readData = in.readObject();
-            }
-        }
+        final NodeData readData = serializeUnserialize(data);
         Assert.assertEquals(data.toString(), readData.toString());
     }
+
+    /**
+     * Non-regression test for ticket <a href="https://josm.openstreetmap.de/ticket/13395">#13395</a>.
+     * @throws Exception if an error occurs
+     */
+    @Test
+    public void testTicket13395() throws Exception {
+        Node n = new Node(1925320646, 1);
+        n.setCoor(null);
+        assertNull(n.getCoor());
+        assertTrue(n.isIncomplete());
+
+        NodeData data = n.save();
+        assertNull(data.getCoor());
+        assertTrue(data.isIncomplete());
+
+        NodeData readData = serializeUnserialize(data);
+        assertNull(readData.getCoor());
+        assertTrue(readData.isIncomplete());
+    }
 }