source: josm/trunk/test/unit/org/openstreetmap/josm/data/validation/tests/OpeningHourTestTest.java@ 13145

Last change on this file since 13145 was 13145, checked in by Don-vip, 6 years ago

see #15582 - simplify our usage of opening_hours.js library (drop mode: it is deduced from tag key) + make the parse method public so it can be called from opening_hours plugin

  • Property svn:eol-style set to native
File size: 11.5 KB
Line 
1// License: GPL. For details, see LICENSE file.
2package org.openstreetmap.josm.data.validation.tests;
3
4import static org.CustomMatchers.hasSize;
5import static org.CustomMatchers.isEmpty;
6import static org.hamcrest.CoreMatchers.not;
7import static org.junit.Assert.assertEquals;
8import static org.junit.Assert.assertFalse;
9import static org.junit.Assert.assertThat;
10import static org.junit.Assert.assertTrue;
11
12import java.util.Arrays;
13import java.util.Collection;
14import java.util.LinkedHashSet;
15import java.util.List;
16import java.util.Set;
17
18import org.junit.Before;
19import org.junit.Rule;
20import org.junit.Test;
21import org.openstreetmap.josm.data.osm.Tag;
22import org.openstreetmap.josm.data.validation.Severity;
23import org.openstreetmap.josm.gui.tagging.presets.TaggingPreset;
24import org.openstreetmap.josm.gui.tagging.presets.TaggingPresetItem;
25import org.openstreetmap.josm.gui.tagging.presets.TaggingPresetReader;
26import org.openstreetmap.josm.gui.tagging.presets.items.KeyedItem;
27import org.openstreetmap.josm.testutils.JOSMTestRules;
28
29import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
30
31/**
32 * JUnit Test of "Opening hours" validation test.
33 * @see OpeningHourTest
34 */
35public class OpeningHourTestTest {
36 /**
37 * We need prefs for this. We check strings so we need i18n.
38 */
39 @Rule
40 @SuppressFBWarnings(value = "URF_UNREAD_PUBLIC_OR_PROTECTED_FIELD")
41 public JOSMTestRules test = new JOSMTestRules().preferences().i18n();
42
43 private OpeningHourTest openingHourTest;
44
45 /**
46 * Setup test.
47 * @throws Exception if test cannot be initialized
48 */
49 @Before
50 public void setUp() throws Exception {
51 openingHourTest = new OpeningHourTest();
52 openingHourTest.initialize();
53 }
54
55 /**
56 * Test #1 of opening_hours syntax.
57 */
58 @Test
59 public void testCheckOpeningHourSyntax1() {
60 final String key = "opening_hours";
61 // frequently used tags according to https://taginfo.openstreetmap.org/keys/opening_hours#values
62 assertThat(openingHourTest.checkOpeningHourSyntax(key, "24/7"), isEmpty());
63 assertThat(openingHourTest.checkOpeningHourSyntax(key, "Mo-Fr 08:30-20:00"), isEmpty());
64 assertThat(openingHourTest.checkOpeningHourSyntax(key, "Mo-Fr sunrise-sunset"), isEmpty());
65 assertThat(openingHourTest.checkOpeningHourSyntax(key, "09:00-21:00"), isEmpty());
66 assertThat(openingHourTest.checkOpeningHourSyntax(key, "Su-Th sunset-24:00,04:00-sunrise; Fr-Sa sunset-sunrise"), isEmpty());
67 assertThat(openingHourTest.checkOpeningHourSyntax(key, "Su-Th sunset-24:00, 04:00-sunrise; Fr-Sa sunset-sunrise"), hasSize(1));
68 assertEquals(Severity.OTHER, openingHourTest.checkOpeningHourSyntax(
69 key, "Su-Th sunset-24:00, 04:00-sunrise; Fr-Sa sunset-sunrise").get(0).getSeverity());
70 assertEquals("Su-Th sunset-24:00,04:00-sunrise; Fr-Sa sunset-sunrise", openingHourTest.checkOpeningHourSyntax(
71 key, "Su-Th sunset-24:00, 04:00-sunrise; Fr-Sa sunset-sunrise").get(0).getPrettifiedValue());
72 }
73
74 /**
75 * Test translated messages.
76 */
77 @Test
78 public void testI18n() {
79 assertTrue(openingHourTest.checkOpeningHourSyntax("opening_hours", ".", false, "de")
80 .get(0).toString().contains("Unerwartetes Zeichen"));
81 assertFalse(openingHourTest.checkOpeningHourSyntax("opening_hours", ".", false, "en")
82 .get(0).toString().contains("Unerwartetes Zeichen"));
83 }
84
85 /**
86 * Test #2 of opening_hours syntax.
87 */
88 @Test
89 public void testCheckOpeningHourSyntax2() {
90 final String key = "opening_hours";
91 final List<OpeningHourTest.OpeningHoursTestError> errors = openingHourTest.checkOpeningHourSyntax(key, "Mo-Tue");
92 assertThat(errors, hasSize(2));
93 assertEquals(key + " - Mo-Tue <--- (Please use the abbreviation \"Tu\" for \"tue\".)", errors.get(0).getMessage());
94 assertEquals(Severity.WARNING, errors.get(0).getSeverity());
95 assertEquals(key +
96 " - Mo-Tue <--- (This rule is not very explicit because there is no time selector being used."+
97 " Please add a time selector to this rule or use a comment to make it more explicit.)", errors.get(1).getMessage());
98 assertEquals(Severity.WARNING, errors.get(1).getSeverity());
99 }
100
101 /**
102 * Test #3 of opening_hours syntax.
103 */
104 @Test
105 public void testCheckOpeningHourSyntax3() {
106 final String key = "opening_hours";
107 final List<OpeningHourTest.OpeningHoursTestError> errors = openingHourTest.checkOpeningHourSyntax(key, "Sa-Su 10.00-20.00");
108 assertThat(errors, hasSize(2));
109 assertEquals(key + " - Sa-Su 10. <--- (Please use \":\" as hour/minute-separator)", errors.get(0).getMessage());
110 assertEquals(Severity.WARNING, errors.get(0).getSeverity());
111 assertEquals("Sa-Su 10:00-20:00", errors.get(0).getPrettifiedValue());
112 assertEquals(key + " - Sa-Su 10.00-20. <--- (Please use \":\" as hour/minute-separator)", errors.get(1).getMessage());
113 assertEquals(Severity.WARNING, errors.get(1).getSeverity());
114 }
115
116 /**
117 * Test #4 of opening_hours syntax.
118 */
119 @Test
120 public void testCheckOpeningHourSyntax4() {
121 assertThat(openingHourTest.checkOpeningHourSyntax(null, null), isEmpty());
122 assertThat(openingHourTest.checkOpeningHourSyntax(null, ""), isEmpty());
123 assertEquals("opening_hours - The value contains nothing meaningful which can be parsed.",
124 openingHourTest.checkOpeningHourSyntax("opening_hours", " ").get(0).getMessage());
125 assertEquals("null - The optional_conf_parm[\"tag_key\"] parameter is of unknown type. Given object, expected string.",
126 openingHourTest.checkOpeningHourSyntax(null, " ").get(0).getMessage());
127 }
128
129 /**
130 * Test #5 of opening_hours syntax.
131 */
132 @Test
133 public void testCheckOpeningHourSyntax5() {
134 final String key = "opening_hours";
135 assertThat(openingHourTest.checkOpeningHourSyntax(key, "badtext"), hasSize(1));
136 assertEquals(key + " - ba <--- (Unexpected token: \"b\" Invalid/unsupported syntax.)",
137 openingHourTest.checkOpeningHourSyntax(key, "badtext").get(0).getMessage());
138 assertThat(openingHourTest.checkOpeningHourSyntax(key, "5.00 p.m-11.00 p.m"), hasSize(1));
139 assertEquals(key + " - 5.00 p <--- (hyphen (-) or open end (+) in time range expected. "
140 + "For working with points in time, the mode for opening_hours.js has to be altered. Maybe wrong tag?)",
141 openingHourTest.checkOpeningHourSyntax(key, "5.00 p.m-11.00 p.m").get(0).getMessage());
142 }
143
144 /**
145 * Test #6 of opening_hours syntax.
146 */
147 @Test
148 public void testCheckOpeningHourSyntax6() {
149 final String key = "opening_hours";
150 assertThat(openingHourTest.checkOpeningHourSyntax(key, "PH open \"always open on public holidays\""), isEmpty());
151 }
152
153 /**
154 * Test #7 of opening_hours syntax.
155 */
156 @Test
157 public void testCheckOpeningHourSyntax7() {
158 final String key = "opening_hours";
159 assertThat(openingHourTest.checkOpeningHourSyntax(key, "9:00-18:00"), hasSize(1));
160 assertEquals(Severity.OTHER, openingHourTest.checkOpeningHourSyntax(key, "9:00-18:00").get(0).getSeverity());
161 assertEquals("09:00-18:00", openingHourTest.checkOpeningHourSyntax(key, "9:00-18:00").get(0).getPrettifiedValue());
162 }
163
164 /**
165 * Non-regression Test of opening_hours syntax for bug #9367.
166 */
167 @Test
168 public void testCheckOpeningHourSyntaxTicket9367() {
169 final String key = "opening_hours";
170 assertEquals(Severity.WARNING, openingHourTest.checkOpeningHourSyntax(key, "Mo,Tu 04-17").get(0).getSeverity());
171 assertEquals(key + " - Mo,Tu 04-17 <--- (Time range without minutes specified. "
172 + "Not very explicit! Please use this syntax instead \"04:00-17:00\".)",
173 openingHourTest.checkOpeningHourSyntax(key, "Mo,Tu 04-17").get(0).getMessage());
174 assertEquals("Mo,Tu 04:00-17:00", openingHourTest.checkOpeningHourSyntax(key, "Mo,Tu 04-17").get(0).getPrettifiedValue());
175 }
176
177 /**
178 * Test #1 of service_times syntax.
179 */
180 @Test
181 public void testCheckServiceTimeSyntax1() {
182 final String key = "service_times";
183 // frequently used tags according to https://taginfo.openstreetmap.org/keys/service_times#values
184 assertThat(openingHourTest.checkOpeningHourSyntax(key, "Su 10:00"), isEmpty());
185 assertThat(openingHourTest.checkOpeningHourSyntax(key, "automatic"), not(isEmpty()));
186 assertThat(openingHourTest.checkOpeningHourSyntax(key, "Mo-Sa 09:00-18:00"), isEmpty());
187 assertThat(openingHourTest.checkOpeningHourSyntax(key, "Su 09:30; We 19:30"), isEmpty());
188 assertThat(openingHourTest.checkOpeningHourSyntax(key, "Mo-Fr 00:00-00:30,04:00-00:30; Sa,Su,PH 00:00-24:00"), isEmpty());
189 assertThat(openingHourTest.checkOpeningHourSyntax(key, "Mo-Fr 0:00-0:30,4:00-00:30; Sa,Su,PH 0:00-24:00"), hasSize(1));
190 assertEquals("Mo-Fr 00:00-00:30,04:00-00:30; Sa,Su,PH 00:00-24:00",
191 openingHourTest.checkOpeningHourSyntax(key, "Mo-Fr 0:00-0:30,4:00-00:30; Sa,Su,PH 0:00-24:00").get(0).getPrettifiedValue());
192 assertEquals("Mo-Fr 00:00-00:30,04:00-00:30; Sa,Su,PH 00:00-24:00",
193 openingHourTest.checkOpeningHourSyntax(key, "Mo-Fr 0:00-0:30,4:00-00:30; Sa,Su,PH 0:00-24:00").get(0).getPrettifiedValue());
194 }
195
196 /**
197 * Test #1 of collection_times syntax.
198 */
199 @Test
200 public void testCheckCollectionTimeSyntax1() {
201 final String key = "collection_times";
202 // frequently used tags according to https://taginfo.openstreetmap.org/keys/collection_times#values
203 assertThat(openingHourTest.checkOpeningHourSyntax(key, "Mo-Sa 09:00"), isEmpty());
204 assertThat(openingHourTest.checkOpeningHourSyntax(key, "fixme"), not(isEmpty()));
205 assertThat(openingHourTest.checkOpeningHourSyntax(key, "daily"), not(isEmpty()));
206 assertThat(openingHourTest.checkOpeningHourSyntax(key, "Mo-Fr 13:30,17:45,19:00; Sa 15:00; Su 11:00"), isEmpty());
207 assertThat(openingHourTest.checkOpeningHourSyntax(key, "Mo-Fr 13:30, 17:45, 19:00; Sa 15:00; Su 11:00"), hasSize(1));
208 assertEquals(Severity.OTHER,
209 openingHourTest.checkOpeningHourSyntax(key, "Mo-Fr 13:30, 17:45, 19:00; Sa 15:00; Su 11:00").get(0).getSeverity());
210 assertEquals("Mo-Fr 13:30,17:45,19:00; Sa 15:00; Su 11:00",
211 openingHourTest.checkOpeningHourSyntax(key, "Mo-Fr 13:30, 17:45, 19:00; Sa 15:00; Su 11:00").get(0).getPrettifiedValue());
212 }
213
214 /**
215 * Tests that predefined values in presets are correct.
216 */
217 @Test
218 public void testPresetValues() {
219 final Collection<TaggingPreset> presets = TaggingPresetReader.readFromPreferences(false, false);
220 final Set<Tag> values = new LinkedHashSet<>();
221 for (final TaggingPreset p : presets) {
222 for (final TaggingPresetItem i : p.data) {
223 if (i instanceof KeyedItem &&
224 Arrays.asList("opening_hours", "service_times", "collection_times").contains(((KeyedItem) i).key)) {
225 for (final String v : ((KeyedItem) i).getValues()) {
226 values.add(new Tag(((KeyedItem) i).key, v));
227 }
228 }
229 }
230 }
231 for (final Tag t : values) {
232 final List<OpeningHourTest.OpeningHoursTestError> errors = openingHourTest.checkOpeningHourSyntax(t.getKey(), t.getValue());
233 assertThat(t + " is valid", errors, isEmpty());
234 }
235 }
236}
Note: See TracBrowser for help on using the repository browser.