Changeset 13489 in josm for trunk/test/unit/org/openstreetmap

Timestamp:

2018-03-03T20:33:09+01:00 (7 years ago)

Author:

Don-vip

Message:

fix #16044 - Autofix invalid URL with the wrong type of slashes

Location:

trunk/test/unit/org/openstreetmap/josm

Files:

• TestUtils.java (modified) (1 diff)
• command/MoveCommandTest.java (modified) (1 diff)
• data/validation/tests/InternetTagsTest.java (modified) (3 diffs)
• data/validation/tests/MapCSSTagCheckerTest.java (modified) (2 diffs)
• data/validation/tests/TagCheckerTest.java (modified) (2 diffs)
• gui/dialogs/relation/GenericRelationEditorTest.java (modified) (2 diffs)

trunk/test/unit/org/openstreetmap/josm/TestUtils.java

@@ -260 +260 @@
 
     /**
+     * Makes sure the given primitive belongs to a data set.
+     * @param <T> OSM primitive type
+     * @param osm OSM primitive
+     * @return OSM primitive, attached to a new {@code DataSet}
+     */
+    public static <T extends OsmPrimitive> T addFakeDataSet(T osm) {
+        new DataSet(osm);
+        return osm;
+    }
+
+    /**
      * Creates a new node with the given tags (see {@link OsmUtils#createPrimitive(java.lang.String)})
      *

trunk/test/unit/org/openstreetmap/josm/command/MoveCommandTest.java

@@ -247 +247 @@
     @Test
     public void testDescription() {
-        Node node = new Node(LatLon.ZERO);
+        Node node = TestUtils.addFakeDataSet(new Node(LatLon.ZERO));
         node.put("name", "xy");
-        new DataSet(node);
         List<OsmPrimitive> nodeList = Arrays.<OsmPrimitive>asList(node);
         assertTrue(new MoveCommand(nodeList, 1, 2).getDescriptionText().matches("Move 1 node"));

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

@@ -2 +2 @@
 package org.openstreetmap.josm.data.validation.tests;
 
+import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertNull;
+import static org.openstreetmap.josm.tools.I18n.tr;
 
 import org.junit.Rule;
 import org.junit.Test;
-import org.openstreetmap.josm.data.osm.OsmUtils;
+import org.openstreetmap.josm.TestUtils;
 import org.openstreetmap.josm.data.validation.TestError;
 import org.openstreetmap.josm.data.validation.routines.AbstractValidator;
@@ -75 +77 @@
     }
 
-    private static void testKey(String key, String value, boolean valid, AbstractValidator validator, int code) {
-        TestError error = TEST.validateTag(OsmUtils.createPrimitive("node "+key+"="+value+""), key, validator, code);
+    /**
+     * Test of invalid slashes.
+     */
+    @Test
+    public void testInvalidSlashes() {
+        TestError error = testUrl("website", "http:\\\\www.sjoekurs.no", false);
+        assertEquals(tr("''{0}'': {1}", "website", tr("URL contains backslashes instead of slashes")), error.getDescription());
+        assertNotNull(error.getFix());
+    }
+
+    private static TestError testKey(String key, String value, boolean valid, AbstractValidator validator, int code) {
+        TestError error = TEST.validateTag(TestUtils.addFakeDataSet(TestUtils.newNode(key+"="+value+"")), key, validator, code);
         if (valid) {
             assertNull(error != null ? error.getMessage() : null, error);
@@ -82 +94 @@
             assertNotNull(error);
         }
+        return error;
     }
 
-    private static void testUrl(String key, String value, boolean valid) {
-        testKey(key, value, valid, UrlValidator.getInstance(), InternetTags.INVALID_URL);
+    private static TestError testUrl(String key, String value, boolean valid) {
+        return testKey(key, value, valid, UrlValidator.getInstance(), InternetTags.INVALID_URL);
     }
 
-    private static void testEmail(String key, String value, boolean valid) {
-        testKey(key, value, valid, EmailValidator.getInstance(), InternetTags.INVALID_EMAIL);
+    private static TestError testEmail(String key, String value, boolean valid) {
+        return testKey(key, value, valid, EmailValidator.getInstance(), InternetTags.INVALID_EMAIL);
     }
 }

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

@@ -102 +102 @@
     @Test
     public void testTicket10913() throws ParseException {
-        final OsmPrimitive p = OsmUtils.createPrimitive("way highway=tertiary construction=yes");
+        final OsmPrimitive p = TestUtils.addFakeDataSet(TestUtils.newWay("highway=tertiary construction=yes"));
         final TagCheck check = TagCheck.readMapCSS(new StringReader("way {" +
                 "throwError: \"error\";" +
@@ -108 +108 @@
                 "fixAdd: \"highway=construction\";\n" +
                 "}")).parseChecks.get(0);
-        new DataSet(p);
         final Command command = check.fixPrimitive(p);
         assertTrue(command instanceof SequenceCommand);

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

@@ -12 +12 @@
 import org.junit.Rule;
 import org.junit.Test;
-import org.openstreetmap.josm.data.osm.DataSet;
+import org.openstreetmap.josm.TestUtils;
 import org.openstreetmap.josm.data.osm.OsmPrimitive;
 import org.openstreetmap.josm.data.osm.OsmUtils;
@@ -34 +34 @@
 
     List<TestError> test(OsmPrimitive primitive) throws IOException {
-        new DataSet(primitive);
         final TagChecker checker = new TagChecker();
         checker.initialize();
         checker.startTest(null);
-        checker.check(primitive);
+        checker.check(TestUtils.addFakeDataSet(primitive));
         return checker.getErrors();
     }

trunk/test/unit/org/openstreetmap/josm/gui/dialogs/relation/GenericRelationEditorTest.java

@@ -11 +11 @@
 import org.junit.Rule;
 import org.junit.Test;
+import org.openstreetmap.josm.TestUtils;
 import org.openstreetmap.josm.data.osm.DataSet;
 import org.openstreetmap.josm.data.osm.Node;
@@ -91 +92 @@
     @Test
     public void testAddPrimitivesToRelation() {
-        Relation r = new Relation(1);
-        new DataSet(r);
+        Relation r = TestUtils.addFakeDataSet(new Relation(1));
         assertNull(GenericRelationEditor.addPrimitivesToRelation(r, Collections.<OsmPrimitive>emptyList()));
         assertNull(GenericRelationEditor.addPrimitivesToRelation(r, Collections.singleton(new Relation(1))));