Changeset 16377 in josm for trunk/test/unit/org

Timestamp:

2020-04-24T12:06:05+02:00 (5 years ago)

Author:

GerdP

Message:

fix #19145: some unclosed ways are not found

• fix typo amenties -> amenity
• only ignore multipolygon members when checking unclosed boundary=* ways, others would be old style multipolygons

File:

• trunk/test/unit/org/openstreetmap/josm/data/validation/tests/UnclosedWaysTest.java (modified) (3 diffs)

trunk/test/unit/org/openstreetmap/josm/data/validation/tests/UnclosedWaysTest.java

@@ -79 +79 @@
 
     /**
-     * Test to make sure the multipolygon ways are ignored
+     * Test to make sure the multipolygon ways are not ignored
+     * See #19136, #19145
      * @throws Exception if an exception occurs
      */
@@ -96 +97 @@
         uwTest.visit(w);
         assertTrue(ElemStyles.hasAreaElemStyle(w, false));
+        assertEquals(1, uwTest.getErrors().size());
+
+        uwTest.endTest();
+    }
+
+    /**
+     * Test to make sure the boundary ways are ignored when member of a boundary relation
+     * See #19136, #19145
+     * @throws Exception if an exception occurs
+     */
+    @Test
+    public void testWayInBoundary() throws Exception {
+        UnclosedWays uwTest = new UnclosedWays();
+        uwTest.initialize();
+        uwTest.startTest(null);
+        DataSet ds = new DataSet();
+
+        // Erroneous tag
+        Way w = createUnclosedWay("boundary=administrative", ds);
+        Relation r = (Relation) OsmUtils.createPrimitive("relation type=boundary");
+        r.addMember(new RelationMember("inner", w));
+        ds.addPrimitive(r);
+        uwTest.visit(w);
+        assertFalse(ElemStyles.hasAreaElemStyle(w, false));
         assertEquals(0, uwTest.getErrors().size());
 
@@ -101 +126 @@
     }
 
+    /**
+     * Test to make sure that amenity=* is closed.
+     * See #19145
+     * @throws Exception if an exception occurs
+     */
+    @Test
+    public void testAmenity() throws Exception {
+        UnclosedWays uwTest = new UnclosedWays();
+        uwTest.initialize();
+        uwTest.startTest(null);
+        DataSet ds = new DataSet();
+
+        // Erroneous tag
+        Way w = createUnclosedWay("amenity=school", ds);
+        uwTest.visit(w);
+        assertTrue(ElemStyles.hasAreaElemStyle(w, false));
+        assertEquals(1, uwTest.getErrors().size());
+        assertEquals(1103, uwTest.getErrors().iterator().next().getCode());
+
+        uwTest.endTest();
+    }
+
  }