Changeset 16616 in josm for trunk/test

Timestamp:

2020-06-14T11:24:35+02:00 (4 years ago)

Author:

simon04

Message:

fix #19380 - RelationChecker: improve validation message "Role verification problem - Role 'route' is not among expected values '<empty>/guidepost'"

File:

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

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

@@ -10 +10 @@
 import org.junit.Rule;
 import org.junit.Test;
+import org.openstreetmap.josm.data.osm.Node;
+import org.openstreetmap.josm.data.osm.OsmUtils;
 import org.openstreetmap.josm.data.osm.Relation;
 import org.openstreetmap.josm.data.osm.RelationMember;
@@ -78 +80 @@
         List<TestError> errors = testRelation(r);
         assertEquals(1, errors.size());
-        assertEquals("Role 'outer2' is not in templates 'outer/inner'", errors.get(0).getDescription());
+        assertEquals("Role 'outer2' is not among expected values 'outer/inner'", errors.get(0).getDescription());
     }
 
@@ -154 +156 @@
         List<TestError> errors = testRelation(r);
         assertEquals(1, errors.size());
-        assertEquals("Role 'level_x' is not in templates 'outline/part/ridge/edge/entrance/level_-?\\d+'", errors.get(0).getDescription());
+        assertEquals("Role 'level_x' is not among expected values 'outline/part/ridge/edge/entrance/level_-?\\d+'",
+                errors.get(0).getDescription());
+    }
+
+    @Test
+    public void testHikingRouteMembers() {
+        Relation r = createRelation("type=route route=hiking");
+        r.addMember(new RelationMember("", OsmUtils.createPrimitive("way highway=path")));
+        r.addMember(new RelationMember("route", OsmUtils.createPrimitive("way highway=path"))); // fails
+        r.addMember(new RelationMember("guidepost", new Node())); // fails
+
+        List<TestError> errors = testRelation(r);
+        assertEquals(2, errors.size());
+        assertEquals("Role of relation member does not match template expression 'information=guidepost' in preset Hiking Route",
+                errors.get(0).getDescription());
+        assertEquals("Role 'route' is not among expected values '<empty>/guidepost'", errors.get(1).getDescription());
     }