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

Timestamp:

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

Author:

Don-vip

Message:

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

Location:

trunk/test/unit/org/openstreetmap/josm/data/validation/tests

Files:

• InternetTagsTest.java (modified) (3 diffs)
• MapCSSTagCheckerTest.java (modified) (2 diffs)
• TagCheckerTest.java (modified) (2 diffs)

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();
     }