Changeset 17430 in josm for trunk/test

Timestamp:

2021-01-02T09:37:50+01:00 (3 years ago)

Author:

GerdP

Message:

fix #20330: Crossing landuse=residential and natural=water should not produce a warning

• ignore crossing ways when one is landuse=residential and the other is natural=water or landuse=reservoir

(both should be implemented in mapcss rules if wanted)

• add unit test

File:

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

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

@@ -47 +47 @@
     private static void testMessage(int code, CrossingWays test, String tags1, String tags2) {
         assertEquals(code, test.createMessage(TestUtils.newWay(tags1), TestUtils.newWay(tags2)).code);
+    }
+
+    private static void testIgnore(boolean expected, CrossingWays test, String tags1, String tags2) {
+        // test both possible combinations
+        assertEquals(expected, test.ignoreWaySegmentCombination(TestUtils.newWay(tags1), TestUtils.newWay(tags2)));
+        assertEquals(expected, test.ignoreWaySegmentCombination(TestUtils.newWay(tags2), TestUtils.newWay(tags1)));
     }
 
@@ -156 +162 @@
         testMessage(665, test, "barrier=hedge", "natural=water");
 
+        testIgnore(true, test, "landuse=residential", "natural=water");
+        testIgnore(false, test, "landuse=residential", "building=yes");
+
         assertFalse(test.isPrimitiveUsable(newUsableWay("amenity=restaurant")));
         assertFalse(test.isPrimitiveUsable(TestUtils.newWay("barrier=yes"))); // Unusable (0 node)