Changeset 7070 in josm for trunk

Timestamp:

2014-05-07T02:04:32+02:00 (11 years ago)

Author:

Don-vip

Message:

see #9990 - show problem with unit tests

Location:

trunk

Files:

• src/org/openstreetmap/josm/io/session/OsmDataSessionExporter.java (modified) (1 diff)
• src/org/openstreetmap/josm/io/session/SessionWriter.java (modified) (6 diffs)
• test/unit/org/openstreetmap/josm/io/session (added)
• test/unit/org/openstreetmap/josm/io/session/SessionWriterTest.java (added)

trunk/src/org/openstreetmap/josm/io/session/OsmDataSessionExporter.java

@@ -56 +56 @@
     }
 
-    public OsmDataSessionExporter() {
-    }
-
     @Override
     public Collection<Layer> getDependencies() {

trunk/src/org/openstreetmap/josm/io/session/SessionWriter.java

@@ -28 +28 @@
 import javax.xml.transform.stream.StreamResult;
 
-import org.w3c.dom.Document;
-import org.w3c.dom.Element;
-import org.w3c.dom.Text;
-
 import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.data.coor.EastNorth;
@@ -45 +41 @@
 import org.openstreetmap.josm.tools.MultiMap;
 import org.openstreetmap.josm.tools.Utils;
+import org.w3c.dom.Document;
+import org.w3c.dom.Element;
+import org.w3c.dom.Text;
 
 public class SessionWriter {
@@ -80 +79 @@
 
     private final List<Layer> layers;
-    private final int active; 
+    private final int active;
     private final Map<Layer, SessionLayerExporter> exporters;
     private final MultiMap<Layer, Layer> dependencies;
@@ -91 +90 @@
      * @param layers The ordered list of layers to save
      * @param active The index of active layer in {@code layers} (starts to 0). Ignored if set to -1
-     * @param exporters The exprters to use to save layers
+     * @param exporters The exporters to use to save layers
      * @param zip {@code true} if a joz archive has to be created, {@code false otherwise}
      * @since 6271
@@ -208 +207 @@
             }
             Set<Layer> deps = dependencies.get(layer);
-            if (!deps.isEmpty()) {
+            if (deps != null && !deps.isEmpty()) {
                 List<Integer> depsInt = new ArrayList<>();
                 for (Layer depLayer : deps) {
@@ -247 +246 @@
     }
 
-    public void write (OutputStream out) throws IOException {
+    public void write(OutputStream out) throws IOException {
         if (zip) {
             zipOut = new ZipOutputStream(new BufferedOutputStream(out));