1 | // License: GPL. For details, see LICENSE file.
|
---|
2 | package org.openstreetmap.josm.data.validation.tests;
|
---|
3 |
|
---|
4 | import static org.junit.jupiter.api.Assertions.assertEquals;
|
---|
5 | import static org.junit.jupiter.api.Assertions.assertNotNull;
|
---|
6 | import static org.junit.jupiter.api.Assertions.assertNull;
|
---|
7 | import static org.openstreetmap.josm.data.coor.LatLon.NORTH_POLE;
|
---|
8 | import static org.openstreetmap.josm.data.coor.LatLon.SOUTH_POLE;
|
---|
9 | import static org.openstreetmap.josm.data.coor.LatLon.ZERO;
|
---|
10 |
|
---|
11 | import java.util.List;
|
---|
12 |
|
---|
13 | import org.junit.jupiter.api.extension.RegisterExtension;
|
---|
14 | import org.junit.jupiter.api.Test;
|
---|
15 | import org.openstreetmap.josm.TestUtils;
|
---|
16 | import org.openstreetmap.josm.data.coor.LatLon;
|
---|
17 | import org.openstreetmap.josm.data.osm.DataSet;
|
---|
18 | import org.openstreetmap.josm.data.osm.Node;
|
---|
19 | import org.openstreetmap.josm.data.osm.RelationMember;
|
---|
20 | import org.openstreetmap.josm.data.validation.Severity;
|
---|
21 | import org.openstreetmap.josm.data.validation.TestError;
|
---|
22 | import org.openstreetmap.josm.testutils.JOSMTestRules;
|
---|
23 |
|
---|
24 | import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
|
---|
25 |
|
---|
26 | /**
|
---|
27 | * JUnit Test of {@link Addresses} validation test.
|
---|
28 | */
|
---|
29 | class AddressesTest {
|
---|
30 |
|
---|
31 | /**
|
---|
32 | * Setup test.
|
---|
33 | */
|
---|
34 | @RegisterExtension
|
---|
35 | @SuppressFBWarnings(value = "URF_UNREAD_PUBLIC_OR_PROTECTED_FIELD")
|
---|
36 | public JOSMTestRules test = new JOSMTestRules();
|
---|
37 |
|
---|
38 | private static Node createAddressNode(String nodeTags, String wayTags, String relationTags) {
|
---|
39 | DataSet ds = new DataSet();
|
---|
40 | Node n = TestUtils.newNode(nodeTags);
|
---|
41 | ds.addPrimitive(n);
|
---|
42 | if (wayTags != null) {
|
---|
43 | ds.addPrimitive(TestUtils.newWay(wayTags, n));
|
---|
44 | }
|
---|
45 | if (relationTags != null) {
|
---|
46 | ds.addPrimitive(TestUtils.newRelation(relationTags, new RelationMember(null, n)));
|
---|
47 | }
|
---|
48 | return n;
|
---|
49 | }
|
---|
50 |
|
---|
51 | private static TestError doTestHouseNumberWithoutStreet(String nodeTags, String wayTags, String relationTags) {
|
---|
52 | return new Addresses().checkHouseNumbersWithoutStreet(createAddressNode(nodeTags, wayTags, relationTags));
|
---|
53 | }
|
---|
54 |
|
---|
55 | /**
|
---|
56 | * Unit test of {@link Addresses#HOUSE_NUMBER_WITHOUT_STREET}
|
---|
57 | */
|
---|
58 | @Test
|
---|
59 | void testHouseNumberWithoutStreet() {
|
---|
60 | assertNull(doTestHouseNumberWithoutStreet("", null, null));
|
---|
61 | assertNotNull(doTestHouseNumberWithoutStreet("addr:housenumber=1", null, null));
|
---|
62 | assertNull(doTestHouseNumberWithoutStreet("addr:housenumber=1 addr:street=Foo", null, null));
|
---|
63 | assertNull(doTestHouseNumberWithoutStreet("addr:housenumber=1 addr:place=Foo", null, null));
|
---|
64 | assertNull(doTestHouseNumberWithoutStreet("addr:housenumber=1 addr:neighbourhood=Foo", null, null));
|
---|
65 | assertNotNull(doTestHouseNumberWithoutStreet("addr:housenumber=1", null, "type=enforcement"));
|
---|
66 | assertNull(doTestHouseNumberWithoutStreet("addr:housenumber=1", null, "type=associatedStreet"));
|
---|
67 | assertNotNull(doTestHouseNumberWithoutStreet("addr:housenumber=1", "building=yes", null));
|
---|
68 | assertNull(
|
---|
69 | doTestHouseNumberWithoutStreet("addr:housenumber=1", "addr:interpolation=odd addr:street=Foo", null));
|
---|
70 | }
|
---|
71 |
|
---|
72 | private static void doTestDuplicateHouseNumber(String tags1, LatLon ll1, String tags2, LatLon ll2,
|
---|
73 | Severity expected) {
|
---|
74 | DataSet ds = new DataSet();
|
---|
75 | Node n1 = TestUtils.newNode(tags1);
|
---|
76 | n1.setCoor(ll1);
|
---|
77 | ds.addPrimitive(n1);
|
---|
78 | Node n2 = TestUtils.newNode(tags2);
|
---|
79 | n2.setCoor(ll2);
|
---|
80 | ds.addPrimitive(n2);
|
---|
81 | List<TestError> errors = new Addresses().checkForDuplicate(n2);
|
---|
82 | assertEquals(expected != null ? 1 : 0, errors.size());
|
---|
83 | if (expected != null) {
|
---|
84 | assertEquals(expected, errors.get(0).getSeverity());
|
---|
85 | }
|
---|
86 | }
|
---|
87 |
|
---|
88 | /**
|
---|
89 | * Unit test of {@link Addresses#DUPLICATE_HOUSE_NUMBER}
|
---|
90 | */
|
---|
91 | @Test
|
---|
92 | void testDuplicateHouseNumber() {
|
---|
93 | String num1 = "addr:housenumber=1 addr:street=Foo ";
|
---|
94 | String num2 = "addr:housenumber=2 addr:street=Foo ";
|
---|
95 | String city1 = "addr:city=Gotham ";
|
---|
96 | String city2 = "addr:city=Metropolis ";
|
---|
97 | String suburb1 = "addr:suburb=Queens ";
|
---|
98 | String suburb2 = "addr:suburb=Bronx ";
|
---|
99 | // Warning for same addresses at close distance
|
---|
100 | doTestDuplicateHouseNumber(num1, ZERO, num1, ZERO, Severity.WARNING);
|
---|
101 | // Info for same addresses at long distance
|
---|
102 | doTestDuplicateHouseNumber(num1, SOUTH_POLE, num1, NORTH_POLE, Severity.OTHER);
|
---|
103 | // Nothing for different addresses
|
---|
104 | doTestDuplicateHouseNumber(num1, ZERO, num2, ZERO, null);
|
---|
105 | // Info for same address in different cities, warning if same city
|
---|
106 | doTestDuplicateHouseNumber(num1 + city1, ZERO, num1 + city2, ZERO, Severity.OTHER);
|
---|
107 | doTestDuplicateHouseNumber(num1 + city1, ZERO, num1 + city1, ZERO, Severity.WARNING);
|
---|
108 | // Info for same address in same city but different suburbs, warning if same
|
---|
109 | // suburb
|
---|
110 | doTestDuplicateHouseNumber(num1 + city1 + suburb1, ZERO, num1 + city1 + suburb2, ZERO, Severity.OTHER);
|
---|
111 | doTestDuplicateHouseNumber(num1 + city1 + suburb1, ZERO, num1 + city1 + suburb1, ZERO, Severity.WARNING);
|
---|
112 | }
|
---|
113 |
|
---|
114 | /**
|
---|
115 | * Unit test of {@link Addresses#expandHouseNumber}
|
---|
116 | */
|
---|
117 | @Test
|
---|
118 | void testMultiAddressDuplicates() {
|
---|
119 | String num1 = "addr:housenumber=1,3 addr:street=Foo";
|
---|
120 | String num2 = "addr:housenumber=1 addr:street=Foo";
|
---|
121 | String num3 = "addr:housenumber=3 addr:street=Foo";
|
---|
122 | String num4 = "addr:housenumber=4 addr:street=Foo";
|
---|
123 |
|
---|
124 | doTestDuplicateHouseNumber(num1, ZERO, num2, ZERO, Severity.WARNING);
|
---|
125 | doTestDuplicateHouseNumber(num1, ZERO, num3, ZERO, Severity.WARNING);
|
---|
126 | doTestDuplicateHouseNumber(num1, ZERO, num4, ZERO, null);
|
---|
127 |
|
---|
128 | num1 = num1.replace(",", ";");
|
---|
129 |
|
---|
130 | doTestDuplicateHouseNumber(num1, ZERO, num2, ZERO, Severity.WARNING);
|
---|
131 | doTestDuplicateHouseNumber(num1, ZERO, num3, ZERO, Severity.WARNING);
|
---|
132 | doTestDuplicateHouseNumber(num1, ZERO, num4, ZERO, null);
|
---|
133 | }
|
---|
134 | }
|
---|