add tagging for dashed cycle lanes (#888)

- refactor the way how country specific resources are queried - take care of the special cycleway situation in NL,BE - add country-specific resources for cycleway icons
streetcomplete · Apr 5, 2018 · 46681a2 · 46681a2
1 parent fe290e2
commit 46681a2
Show file tree

Hide file tree

Showing 134 changed files with 1,965 additions and 6,292 deletions.
diff --git a/app/src/androidTest/java/de/westnordost/streetcomplete/data/meta/CountryInfosTest.java b/app/src/androidTest/java/de/westnordost/streetcomplete/data/meta/CountryInfosTest.java
@@ -39,16 +39,6 @@ private void checkRegularShoppingDaysIsBetween0And7(CountryInfo info)
 		assertTrue(info.getRegularShoppingDays() >= 0);
 	}
 
-	private void checkMaxSpeedLayoutExists(CountryInfo info)
-	{
-		if(info.getMaxspeedLayout() != null)
-		{
-			int resId = getContext().getResources().getIdentifier(
-					info.getMaxspeedLayout(), "layout", getContext().getPackageName());
-			assertTrue(resId != 0);
-		}
-	}
-
 	private static List<String> validWeekdays = Arrays.asList("Mo","Tu","We","Th","Fr","Sa","Su");
 	private void checkStartOfWorkweekValid(CountryInfo info)
 	{
@@ -60,7 +50,6 @@ private void checkForEach(CountryInfo info)
 		checkFirstDayOfWorkweekIsValid(info);
 		checkSpeedUnitIsEitherKmhOrMph(info);
 		checkRegularShoppingDaysIsBetween0And7(info);
-		checkMaxSpeedLayoutExists(info);
 		checkStartOfWorkweekValid(info);
 	}
 

diff --git a/app/src/androidTest/java/de/westnordost/streetcomplete/quests/AddCyclewayTest.java b/app/src/androidTest/java/de/westnordost/streetcomplete/quests/AddCyclewayTest.java
@@ -11,23 +11,39 @@ public class AddCyclewayTest extends AOsmElementQuestTypeTest
 {
 	public void testCyclewayLeftAndRightDontHaveToBeSpecified1()
 	{
-		bundle.putString(AddCyclewayForm.CYCLEWAY_LEFT, Cycleway.LANE.name());
+		bundle.putString(AddCyclewayForm.CYCLEWAY_LEFT, Cycleway.EXCLUSIVE_LANE.name());
 		StringMapChangesBuilder cb = new StringMapChangesBuilder(tags);
 		createQuestType().applyAnswerTo(bundle, cb);
 		// success if no exception thrown
 	}
 
 	public void testCyclewayLeftAndRightDontHaveToBeSpecified2()
 	{
-		bundle.putString(AddCyclewayForm.CYCLEWAY_RIGHT, Cycleway.LANE.name());
+		bundle.putString(AddCyclewayForm.CYCLEWAY_RIGHT, Cycleway.EXCLUSIVE_LANE.name());
 		StringMapChangesBuilder cb = new StringMapChangesBuilder(tags);
 		createQuestType().applyAnswerTo(bundle, cb);
 		// success if no exception thrown
 	}
 
 	public void testCyclewayLane()
 	{
-		putBothSides(Cycleway.LANE);
+		putBothSides(Cycleway.EXCLUSIVE_LANE);
+		verify(
+			new StringMapEntryAdd("cycleway:both", "lane"),
+			new StringMapEntryAdd("cycleway:both:lane", "exclusive"));
+	}
+
+	public void testCyclewayAdvisoryLane()
+	{
+		putBothSides(Cycleway.ADVISORY_LANE);
+		verify(
+			new StringMapEntryAdd("cycleway:both", "lane"),
+			new StringMapEntryAdd("cycleway:both:lane", "advisory"));
+	}
+
+	public void testCyclewayUnspecifiedLane()
+	{
+		putBothSides(Cycleway.LANE_UNSPECIFIED);
 		verify(new StringMapEntryAdd("cycleway:both", "lane"));
 	}
 
@@ -43,10 +59,22 @@ public void testCyclewayBusLane()
 		verify(new StringMapEntryAdd("cycleway:both", "share_busway"));
 	}
 
-	public void testCyclewaySharedLane()
+	public void testCyclewayPictogramLane()
 	{
-		putBothSides(Cycleway.SHARED);
-		verify(new StringMapEntryAdd("cycleway:both", "shared_lane"));
+		putBothSides(Cycleway.PICTOGRAMS);
+		verify(
+			new StringMapEntryAdd("cycleway:both", "shared_lane"),
+			new StringMapEntryAdd("cycleway:both:lane", "pictogram")
+		);
+	}
+
+	public void testCyclewaySuggestionLane()
+	{
+		putBothSides(Cycleway.SUGGESTION_LANE);
+		verify(
+			new StringMapEntryAdd("cycleway:both", "shared_lane"),
+			new StringMapEntryAdd("cycleway:both:lane", "advisory")
+		);
 	}
 
 	public void testCyclewayNone()
@@ -57,7 +85,7 @@ public void testCyclewayNone()
 
 	public void testCyclewayOnSidewalk()
 	{
-		putBothSides(Cycleway.SIDEWALK);
+		putBothSides(Cycleway.SIDEWALK_EXPLICIT);
 		verify(
 				new StringMapEntryAdd("cycleway:both", "track"),
 				new StringMapEntryAdd("sidewalk", "both"),
@@ -77,7 +105,7 @@ public void testCyclewaySidewalkOkay()
 
 	public void testCyclewaySidewalkAny()
 	{
-		bundle.putString(AddCyclewayForm.CYCLEWAY_RIGHT, Cycleway.SIDEWALK.name());
+		bundle.putString(AddCyclewayForm.CYCLEWAY_RIGHT, Cycleway.SIDEWALK_EXPLICIT.name());
 		bundle.putString(AddCyclewayForm.CYCLEWAY_LEFT, Cycleway.SIDEWALK_OK.name());
 		verify(
 				new StringMapEntryAdd("sidewalk", "both")
@@ -86,16 +114,17 @@ public void testCyclewaySidewalkAny()
 
 	public void testCyclewayDualTrack()
 	{
-		putBothSides(Cycleway.TRACK_DUAL);
+		putBothSides(Cycleway.DUAL_TRACK);
 		verify(
-				new StringMapEntryAdd("cycleway:both", "track"),
-				new StringMapEntryAdd("cycleway:both:oneway", "no")
+			new StringMapEntryAdd("cycleway:both", "track"),
+			new StringMapEntryAdd("cycleway:both:oneway", "no"),
+			new StringMapEntryAdd("cycleway:both:lane", "exclusive")
 		);
 	}
 
 	public void testCyclewayDualLane()
 	{
-		putBothSides(Cycleway.LANE_DUAL);
+		putBothSides(Cycleway.DUAL_LANE);
 		verify(
 				new StringMapEntryAdd("cycleway:both", "lane"),
 				new StringMapEntryAdd("cycleway:both:oneway", "no")
@@ -104,47 +133,53 @@ public void testCyclewayDualLane()
 
 	public void testLeftAndRightAreDifferent()
 	{
-		bundle.putString(AddCyclewayForm.CYCLEWAY_RIGHT, Cycleway.LANE.name());
+		bundle.putString(AddCyclewayForm.CYCLEWAY_RIGHT, Cycleway.EXCLUSIVE_LANE.name());
 		bundle.putString(AddCyclewayForm.CYCLEWAY_LEFT, Cycleway.TRACK.name());
 		verify(
 				new StringMapEntryAdd("cycleway:right", "lane"),
+				new StringMapEntryAdd("cycleway:right:lane","exclusive"),
 				new StringMapEntryAdd("cycleway:left", "track")
 		);
 	}
 
 	public void testCyclewayMakesStreetNotOnewayForBicycles()
 	{
-		putBothSides(Cycleway.LANE);
+		putBothSides(Cycleway.EXCLUSIVE_LANE);
 		bundle.putBoolean(AddCyclewayForm.IS_ONEWAY_NOT_FOR_CYCLISTS, true);
 		verify(
 				new StringMapEntryAdd("cycleway:both", "lane"),
-				new StringMapEntryAdd("oneway:bicycle", "no")
+				new StringMapEntryAdd("oneway:bicycle", "no"),
+				new StringMapEntryAdd("cycleway:both:lane", "exclusive")
 		);
 	}
 
 	public void testCyclewayLaneWithExplicitDirection()
 	{
 		// this would be a street that has lanes on both sides but is oneway=yes (in countries with
 		// right hand traffic)
-		putBothSides(Cycleway.LANE);
+		putBothSides(Cycleway.EXCLUSIVE_LANE);
 		bundle.putInt(AddCyclewayForm.CYCLEWAY_LEFT_DIR, -1);
 		verify(
 				new StringMapEntryAdd("cycleway:left", "lane"),
 				new StringMapEntryAdd("cycleway:left:oneway", "-1"),
-				new StringMapEntryAdd("cycleway:right", "lane")
+				new StringMapEntryAdd("cycleway:right", "lane"),
+				new StringMapEntryAdd("cycleway:left:lane","exclusive"),
+				new StringMapEntryAdd("cycleway:right:lane","exclusive")
 		);
 	}
 
 	public void testCyclewayLaneWithExplicitOtherDirection()
 	{
 		// this would be a street that has lanes on both sides but is oneway=-1 (in countries with
 		// right hand traffic)
-		putBothSides(Cycleway.LANE);
+		putBothSides(Cycleway.EXCLUSIVE_LANE);
 		bundle.putInt(AddCyclewayForm.CYCLEWAY_LEFT_DIR, +1);
 		verify(
 				new StringMapEntryAdd("cycleway:left", "lane"),
 				new StringMapEntryAdd("cycleway:left:oneway", "yes"),
-				new StringMapEntryAdd("cycleway:right", "lane")
+				new StringMapEntryAdd("cycleway:right", "lane"),
+				new StringMapEntryAdd("cycleway:left:lane","exclusive"),
+				new StringMapEntryAdd("cycleway:right:lane","exclusive")
 		);
 	}
 

diff --git a/app/src/main/assets/country_metadata/AM.yml b/app/src/main/assets/country_metadata/AM.yml
@@ -1,6 +1,5 @@
 # Do not edit. Source files are in /res/country_metadata
 additionalStreetsignLanguages: [en]
 isLivingStreetKnown: true
-livingStreetSign: ic_living_street_russian
 officialLanguages: [hy]
 orchardProduces: [grape, apple, apricot, tomatoe, peach, pear, plum, cherry, walnut, fig, hazelnut]
diff --git a/app/src/main/assets/country_metadata/AO.yml b/app/src/main/assets/country_metadata/AO.yml
@@ -1,5 +1,4 @@
 # Do not edit. Source files are in /res/country_metadata
 isLivingStreetKnown: true
-livingStreetSign: ic_living_street_sadc
 officialLanguages: [pt]
 orchardProduces: [banana, coffee, pineapple, palm_oil, tomatoe, cashew_nut, cacao, sisal]
diff --git a/app/src/main/assets/country_metadata/AS.yml b/app/src/main/assets/country_metadata/AS.yml
@@ -1,5 +1,4 @@
 # Do not edit. Source files are in /res/country_metadata
-maxspeedLayout: quest_maxspeed_us
 officialLanguages: [en, sm]
 orchardProduces: [coconut, banana, tomatoe, cacao]
 speedUnit: [mph, km/h]
diff --git a/app/src/main/assets/country_metadata/AU.yml b/app/src/main/assets/country_metadata/AU.yml
@@ -1,9 +1,7 @@
 # Do not edit. Source files are in /res/country_metadata
-advisorySpeedLimitLayout: quest_maxspeed_advisory_yellow
 isAdvisorySpeedLimitKnown: true
 isLeftHandTraffic: true
 isLivingStreetKnown: true
-livingStreetSign: ic_living_street_australia
 officialLanguages: [en]
 orchardProduces: [grape, olive, almond, apple, orange, banana, peach, avocado, guava, mango, mangosteen,
   walnut, apricot, pear, tomatoe, plum, cherry]

diff --git a/app/src/main/assets/country_metadata/BW.yml b/app/src/main/assets/country_metadata/BW.yml
@@ -1,6 +1,5 @@
 # Do not edit. Source files are in /res/country_metadata
 isLeftHandTraffic: true
 isLivingStreetKnown: true
-livingStreetSign: ic_living_street_sadc
 officialLanguages: [en, tn]
 orchardProduces: [orange]
diff --git a/app/src/main/assets/country_metadata/BY.yml b/app/src/main/assets/country_metadata/BY.yml
@@ -1,9 +1,7 @@
 # Do not edit. Source files are in /res/country_metadata
-advisorySpeedLimitLayout: quest_maxspeed_advisory_blue
 isAdvisorySpeedLimitKnown: true
 isLivingStreetKnown: true
 isSlowZoneKnown: true
-livingStreetSign: ic_living_street_russian
 officialLanguages: [be, ru]
 orchardProduces: [apple, cranberry, strawberry, plum, tomatoe, pear, walnut, grape, hazelnut, cherry]
 popularSports: [soccer, basketball, tennis, field_hockey, volleyball, ice_hockey]
diff --git a/app/src/main/assets/country_metadata/CA.yml b/app/src/main/assets/country_metadata/CA.yml
@@ -1,8 +1,6 @@
 # Do not edit. Source files are in /res/country_metadata
-advisorySpeedLimitLayout: quest_maxspeed_advisory_yellow
 isAdvisorySpeedLimitKnown: true
 isSlowZoneKnown: false
-maxspeedLayout: quest_maxspeed_ca
 officialLanguages: [en, fr]
 orchardProduces: [blueberry, apple, grape, cranberry, tomatoe, strawberry, peach, raspberry, chilli_pepper,
   sweet_pepper, cherry, pear, plum, apricot, kiwi]

diff --git a/app/src/main/assets/country_metadata/CN.yml b/app/src/main/assets/country_metadata/CN.yml
@@ -1,5 +1,4 @@
 # Do not edit. Source files are in /res/country_metadata
-advisorySpeedLimitLayout: quest_maxspeed_advisory_yellow
 isAdvisorySpeedLimitKnown: true
 officialLanguages: [zh]
 orchardProduces: [apple, tea, plum, orange, pear, tomatoe, persimmon, grape, peach, rubber, guava,

diff --git a/app/src/main/assets/country_metadata/CZ.yml b/app/src/main/assets/country_metadata/CZ.yml
@@ -1,5 +1,4 @@
 # Do not edit. Source files are in /res/country_metadata
-advisorySpeedLimitLayout: quest_maxspeed_advisory_blue
 isAdvisorySpeedLimitKnown: true
 isLivingStreetKnown: true
 isSlowZoneKnown: true

diff --git a/app/src/main/assets/country_metadata/DK.yml b/app/src/main/assets/country_metadata/DK.yml
@@ -1,5 +1,4 @@
 # Do not edit. Source files are in /res/country_metadata
-advisorySpeedLimitLayout: quest_maxspeed_advisory_blue
 isAdvisorySpeedLimitKnown: true
 isLivingStreetKnown: true
 isSlowZoneKnown: true

diff --git a/app/src/main/assets/country_metadata/EE.yml b/app/src/main/assets/country_metadata/EE.yml
@@ -1,6 +1,5 @@
 # Do not edit. Source files are in /res/country_metadata
 additionalStreetsignLanguages: [ru]
-advisorySpeedLimitLayout: quest_maxspeed_advisory_blue
 isAdvisorySpeedLimitKnown: true
 isLivingStreetKnown: true
 isSlowZoneKnown: true

diff --git a/app/src/main/assets/country_metadata/ES.yml b/app/src/main/assets/country_metadata/ES.yml
@@ -1,7 +1,6 @@
 # Do not edit. Source files are in /res/country_metadata
 additionalStreetsignLanguages: [ca, eu, gl]
 additionalValidHousenumberRegex: s/n
-advisorySpeedLimitLayout: quest_maxspeed_advisory_blue
 isAdvisorySpeedLimitKnown: true
 isLivingStreetKnown: true
 isSlowZoneKnown: true

diff --git a/app/src/main/assets/country_metadata/FI.yml b/app/src/main/assets/country_metadata/FI.yml
@@ -1,5 +1,4 @@
 # Do not edit. Source files are in /res/country_metadata
-advisorySpeedLimitLayout: quest_maxspeed_advisory_blue
 isAdvisorySpeedLimitKnown: true
 isLivingStreetKnown: true
 isSlowZoneKnown: true

diff --git a/app/src/main/assets/country_metadata/FR.yml b/app/src/main/assets/country_metadata/FR.yml
@@ -1,7 +1,6 @@
 # Do not edit. Source files are in /res/country_metadata
 additionalStreetsignLanguages: [oc, ca, br]
 additionalValidHousenumberRegex: \p{N}{1,4}\sbis
-advisorySpeedLimitLayout: quest_maxspeed_advisory_blue
 isAdvisorySpeedLimitKnown: true
 isLivingStreetKnown: true
 isSlowZoneKnown: true

diff --git a/app/src/main/assets/country_metadata/GB.yml b/app/src/main/assets/country_metadata/GB.yml
@@ -1,6 +1,5 @@
 # Do not edit. Source files are in /res/country_metadata
 additionalStreetsignLanguages: [cy, gd]
-advisorySpeedLimitLayout: quest_maxspeed_advisory_white
 isAdvisorySpeedLimitKnown: true
 isLeftHandTraffic: true
 isLivingStreetKnown: true

diff --git a/app/src/main/assets/country_metadata/GR.yml b/app/src/main/assets/country_metadata/GR.yml
@@ -1,6 +1,5 @@
 # Do not edit. Source files are in /res/country_metadata
 additionalStreetsignLanguages: [en]
-advisorySpeedLimitLayout: quest_maxspeed_advisory_blue
 isAdvisorySpeedLimitKnown: true
 isLivingStreetKnown: true
 isSlowZoneKnown: true

diff --git a/app/src/main/assets/country_metadata/GU.yml b/app/src/main/assets/country_metadata/GU.yml
@@ -1,5 +1,4 @@
 # Do not edit. Source files are in /res/country_metadata
-maxspeedLayout: quest_maxspeed_us
 officialLanguages: [en, ch]
 orchardProduces: [coconut, tomatoe, banana]
 speedUnit: [mph, km/h]
diff --git a/app/src/main/assets/country_metadata/HU.yml b/app/src/main/assets/country_metadata/HU.yml
@@ -1,5 +1,4 @@
 # Do not edit. Source files are in /res/country_metadata
-advisorySpeedLimitLayout: quest_maxspeed_advisory_blue
 isAdvisorySpeedLimitKnown: true
 isLivingStreetKnown: true
 isSlowZoneKnown: true

diff --git a/app/src/main/assets/country_metadata/IT.yml b/app/src/main/assets/country_metadata/IT.yml
@@ -1,5 +1,4 @@
 # Do not edit. Source files are in /res/country_metadata
-advisorySpeedLimitLayout: quest_maxspeed_advisory_blue
 isAdvisorySpeedLimitKnown: true
 isLivingStreetKnown: true
 isSlowZoneKnown: true

diff --git a/app/src/main/assets/country_metadata/JP.yml b/app/src/main/assets/country_metadata/JP.yml
@@ -1,6 +1,5 @@
 # Do not edit. Source files are in /res/country_metadata
 additionalStreetsignLanguages: [en]
-advisorySpeedLimitLayout: quest_maxspeed_advisory_white
 isAdvisorySpeedLimitKnown: true
 isLeftHandTraffic: true
 officialLanguages: [ja]

diff --git a/app/src/main/assets/country_metadata/KG.yml b/app/src/main/assets/country_metadata/KG.yml
@@ -1,6 +1,5 @@
 # Do not edit. Source files are in /res/country_metadata
 isLivingStreetKnown: true
-livingStreetSign: ic_living_street_russian
 officialLanguages: [ky, ru]
 orchardProduces: [apple, tomatoe, apricot, grape, hazelnut, pear, peach, plum, cherry, walnut, almond,
   raspberry, pistachio, chilli_pepper, sweet_pepper, strawberry]

diff --git a/app/src/main/assets/country_metadata/KR.yml b/app/src/main/assets/country_metadata/KR.yml
@@ -1,6 +1,5 @@
 # Do not edit. Source files are in /res/country_metadata
 additionalStreetsignLanguages: [en]
-advisorySpeedLimitLayout: quest_maxspeed_advisory_white
 isAdvisorySpeedLimitKnown: true
 officialLanguages: [ko]
 orchardProduces: [apple, chilli_pepper, sweet_pepper, chestnut, apple, persimmon, chilli_pepper, sweet_pepper,

diff --git a/app/src/main/assets/country_metadata/KZ.yml b/app/src/main/assets/country_metadata/KZ.yml
@@ -1,6 +1,5 @@
 # Do not edit. Source files are in /res/country_metadata
 isLivingStreetKnown: true
-livingStreetSign: ic_living_street_russian
 officialLanguages: [kk, ru]
 orchardProduces: [apple, tomatoe, grape, apricot, pear, cherry, strawberry, plum, walnut, peach, chilli_pepper,
   sweet_pepper, almond]

diff --git a/app/src/main/assets/country_metadata/LS.yml b/app/src/main/assets/country_metadata/LS.yml
@@ -1,5 +1,4 @@
 # Do not edit. Source files are in /res/country_metadata
 isLeftHandTraffic: true
 isLivingStreetKnown: true
-livingStreetSign: ic_living_street_sadc
 officialLanguages: [st, en]
diff --git a/app/src/main/assets/country_metadata/LU.yml b/app/src/main/assets/country_metadata/LU.yml
@@ -1,5 +1,4 @@
 # Do not edit. Source files are in /res/country_metadata
-advisorySpeedLimitLayout: quest_maxspeed_advisory_blue
 isAdvisorySpeedLimitKnown: true
 isLivingStreetKnown: true
 isSlowZoneKnown: true

diff --git a/app/src/main/assets/country_metadata/MG.yml b/app/src/main/assets/country_metadata/MG.yml
@@ -1,6 +1,5 @@
 # Do not edit. Source files are in /res/country_metadata
 isLivingStreetKnown: true
-livingStreetSign: ic_living_street_sadc
 officialLanguages: [mg, fr]
 orchardProduces: [coffee, vanilla, banana, guava, mango, mangosteen, coconut, cashew_nut, pineapple,
   sisal, orange, pepper, cacao, pistachio, tomatoe, apple, tung_nut]
diff --git a/app/src/main/assets/country_metadata/MN.yml b/app/src/main/assets/country_metadata/MN.yml
@@ -1,6 +1,5 @@
 # Do not edit. Source files are in /res/country_metadata
 isLivingStreetKnown: true
-livingStreetSign: ic_living_street_russian
 officialLanguages: [mn]
 orchardProduces: [hazelnut]
 popularReligions: [buddhist, chinese_folk, taoist]
diff --git a/app/src/main/assets/country_metadata/MU.yml b/app/src/main/assets/country_metadata/MU.yml
@@ -1,7 +1,6 @@
 # Do not edit. Source files are in /res/country_metadata
 isLeftHandTraffic: true
 isLivingStreetKnown: true
-livingStreetSign: ic_living_street_sadc
 officialLanguages: [en, fr, mfe]
 orchardProduces: [tomatoe, tea, pineapple, coconut, banana, chilli_pepper, sweet_pepper]
 popularReligions: [hindu, christian, muslim]
diff --git a/app/src/main/assets/country_metadata/MW.yml b/app/src/main/assets/country_metadata/MW.yml
@@ -1,7 +1,6 @@
 # Do not edit. Source files are in /res/country_metadata
 isLeftHandTraffic: true
 isLivingStreetKnown: true
-livingStreetSign: ic_living_street_sadc
 officialLanguages: [en, ny]
 orchardProduces: [tea, banana, guava, mango, mangosteen, tomatoe, coffee, tung_nut, pepper, chilli_pepper,
   sweet_pepper, sisal, nutmeg, vanilla]