Skip to content

Commit

Permalink
Merge pull request #32 from mapbox/1ec5-escape-from-space-30
Browse files Browse the repository at this point in the history
Escape queries
  • Loading branch information
1ec5 committed Apr 6, 2016
2 parents 9aad731 + c4422c8 commit 3c38c63
Show file tree
Hide file tree
Showing 5 changed files with 93 additions and 2 deletions.
12 changes: 12 additions & 0 deletions MapboxGeocoder.xcodeproj/project.pbxproj
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,9 @@
/* Begin PBXBuildFile section */
8DF260A67CA4C55F8FA7CAA0 /* Pods_MapboxGeocoderTests.framework in Frameworks */ = {isa = PBXBuildFile; fileRef = 9CD6EA86BBE0D7DE16654CB4 /* Pods_MapboxGeocoderTests.framework */; };
9FB47CA319B32DD7142B5830 /* Pods.framework in Frameworks */ = {isa = PBXBuildFile; fileRef = 6953F60F050AE274E006CB85 /* Pods.framework */; };
DA210BAB1CB4BE73008088FD /* ForwardGeocodingTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = DA210BAA1CB4BE73008088FD /* ForwardGeocodingTests.swift */; };
DA210BAD1CB4BFF7008088FD /* forward_valid.json in Resources */ = {isa = PBXBuildFile; fileRef = DA210BAC1CB4BFF7008088FD /* forward_valid.json */; };
DA210BAF1CB4C5A7008088FD /* forward_invalid.json in Resources */ = {isa = PBXBuildFile; fileRef = DA210BAE1CB4C5A7008088FD /* forward_invalid.json */; };
DA2E03F01CB0FDB400D1269A /* MBPlacemark.swift in Sources */ = {isa = PBXBuildFile; fileRef = DA2E03EF1CB0FDB400D1269A /* MBPlacemark.swift */; };
DA2E03F21CB0FE0200D1269A /* MBRectangularRegion.swift in Sources */ = {isa = PBXBuildFile; fileRef = DA2E03F11CB0FE0200D1269A /* MBRectangularRegion.swift */; };
DA2E03F71CB0FFD800D1269A /* MBGeocoderConfiguration.swift in Sources */ = {isa = PBXBuildFile; fileRef = DA2E03F61CB0FFD800D1269A /* MBGeocoderConfiguration.swift */; };
Expand Down Expand Up @@ -97,6 +100,9 @@
A7FEE3C3AC9D553F9763CB06 /* Pods.debug.xcconfig */ = {isa = PBXFileReference; includeInIndex = 1; lastKnownFileType = text.xcconfig; name = Pods.debug.xcconfig; path = "Pods/Target Support Files/Pods/Pods.debug.xcconfig"; sourceTree = "<group>"; };
CC09570CD1A6D11014400590 /* Pods.release.xcconfig */ = {isa = PBXFileReference; includeInIndex = 1; lastKnownFileType = text.xcconfig; name = Pods.release.xcconfig; path = "Pods/Target Support Files/Pods/Pods.release.xcconfig"; sourceTree = "<group>"; };
D9281D748078BEB61AACF246 /* Pods_Unit_Tests.framework */ = {isa = PBXFileReference; explicitFileType = wrapper.framework; includeInIndex = 0; path = Pods_Unit_Tests.framework; sourceTree = BUILT_PRODUCTS_DIR; };
DA210BAA1CB4BE73008088FD /* ForwardGeocodingTests.swift */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.swift; path = ForwardGeocodingTests.swift; sourceTree = "<group>"; };
DA210BAC1CB4BFF7008088FD /* forward_valid.json */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text.json; path = forward_valid.json; sourceTree = "<group>"; };
DA210BAE1CB4C5A7008088FD /* forward_invalid.json */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text.json; path = forward_invalid.json; sourceTree = "<group>"; };
DA2E03EF1CB0FDB400D1269A /* MBPlacemark.swift */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.swift; path = MBPlacemark.swift; sourceTree = "<group>"; };
DA2E03F11CB0FE0200D1269A /* MBRectangularRegion.swift */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.swift; path = MBRectangularRegion.swift; sourceTree = "<group>"; };
DA2E03F61CB0FFD800D1269A /* MBGeocoderConfiguration.swift */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.swift; path = MBGeocoderConfiguration.swift; sourceTree = "<group>"; };
Expand Down Expand Up @@ -283,6 +289,7 @@
isa = PBXGroup;
children = (
DA701C001CB1292C00B0E520 /* MBGeocoderConfigurationTests.swift */,
DA210BAA1CB4BE73008088FD /* ForwardGeocodingTests.swift */,
DDF1E84C1BD6F7BA00C40C78 /* ReverseGeocodingTests.swift */,
DDF1E84E1BD6F7BA00C40C78 /* Info.plist */,
DDF1E8571BD700EB00C40C78 /* Fixtures */,
Expand All @@ -294,6 +301,8 @@
DDF1E8571BD700EB00C40C78 /* Fixtures */ = {
isa = PBXGroup;
children = (
DA210BAC1CB4BFF7008088FD /* forward_valid.json */,
DA210BAE1CB4C5A7008088FD /* forward_invalid.json */,
DDF1E85B1BD70E4C00C40C78 /* reverse_valid.json */,
DDF1E85A1BD70E4C00C40C78 /* reverse_invalid.json */,
);
Expand Down Expand Up @@ -468,8 +477,10 @@
isa = PBXResourcesBuildPhase;
buildActionMask = 2147483647;
files = (
DA210BAD1CB4BFF7008088FD /* forward_valid.json in Resources */,
DDF1E85C1BD70E4C00C40C78 /* reverse_invalid.json in Resources */,
DDF1E85D1BD70E4C00C40C78 /* reverse_valid.json in Resources */,
DA210BAF1CB4C5A7008088FD /* forward_invalid.json in Resources */,
);
runOnlyForDeploymentPostprocessing = 0;
};
Expand Down Expand Up @@ -605,6 +616,7 @@
buildActionMask = 2147483647;
files = (
DDF1E84D1BD6F7BA00C40C78 /* ReverseGeocodingTests.swift in Sources */,
DA210BAB1CB4BE73008088FD /* ForwardGeocodingTests.swift in Sources */,
DA701C011CB1292C00B0E520 /* MBGeocoderConfigurationTests.swift in Sources */,
DA701C031CB12A1800B0E520 /* Fixture.swift in Sources */,
);
Expand Down
5 changes: 3 additions & 2 deletions MapboxGeocoder/MBGeocoderRouter.swift
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ internal enum MBGeocoderRouter: Router {
case .V5(_, _, _, let ISOCountryCodes, let focusCoordinate, let scopes, let autocomplete):
var params: [String: String] = [:]
if let ISOCountryCodes = ISOCountryCodes {
params["country"] = ISOCountryCodes.joinWithSeparator(",")
params["country"] = ISOCountryCodes.joinWithSeparator(",").lowercaseString
}
if let focusCoordinate = focusCoordinate {
params["proximity"] = String(format: "%.3f,%.3f", focusCoordinate.longitude, focusCoordinate.latitude)
Expand All @@ -43,7 +43,8 @@ internal enum MBGeocoderRouter: Router {
var path: String {
switch self {
case .V5(_, let isPermanent, let query, _, _, _, _):
return "geocoding/v5/mapbox.places\(isPermanent ? "-permanent" : "")/\(query).json"
let encodedQuery = query.stringByReplacingOccurrencesOfString(" ", withString: "+").stringByAddingPercentEncodingWithAllowedCharacters(NSCharacterSet.URLPathAllowedCharacterSet())!
return "geocoding/v5/mapbox.places\(isPermanent ? "-permanent" : "")/\(encodedQuery).json"
}
}
}
76 changes: 76 additions & 0 deletions MapboxGeocoderTests/ForwardGeocodingTests.swift
Original file line number Diff line number Diff line change
@@ -0,0 +1,76 @@
import XCTest
import Nocilla
import CoreLocation
@testable import MapboxGeocoder

class ForwardGeocodingTests: XCTestCase {
override func setUp() {
super.setUp()
LSNocilla.sharedInstance().start()
}

override func tearDown() {
LSNocilla.sharedInstance().clearStubs()
LSNocilla.sharedInstance().stop()
super.setUp()
}

func testValidForwardGeocode() {
let expectation = expectationWithDescription("forward geocode should return results")

let json = Fixture.stringFromFileNamed("forward_valid")
stubRequest("GET", "https://api.mapbox.com/geocoding/v5/mapbox.places/1600+pennsylvania+ave+nw.json?access_token=\(BogusToken)&country=ca").andReturn(200).withHeaders(["Content-Type": "application/json"]).withBody(json)

let geocoder = MBGeocoder(accessToken: BogusToken)
var addressPlacemark: MBPlacemark! = nil
geocoder.geocodeAddressString("1600 pennsylvania ave nw", inCountries: ["CA"]) { (placemarks, error) in
XCTAssertEqual(placemarks?.count, 5, "forward geocode should have 5 results")
addressPlacemark = placemarks![0]

expectation.fulfill()
}

waitForExpectationsWithTimeout(1) { (error) in
XCTAssertNil(error, "Error: \(error)")
XCTAssertFalse(geocoder.geocoding)
}

XCTAssertEqual(addressPlacemark.description, "Pennsylvania Ave, Stellarton, Nova Scotia B0K 1S0, Canada", "forward geocode should populate description")
XCTAssertEqual(addressPlacemark.name, "Pennsylvania Ave", "forward geocode should populate name")
XCTAssertEqual(addressPlacemark.location?.coordinate, CLLocationCoordinate2D(latitude: 45.5562851, longitude: -62.661944), "forward geocode should populate location")
XCTAssertEqual(addressPlacemark.scope, .Address, "forward geocode should populate scope")
XCTAssertEqual(addressPlacemark.ISOcountryCode, "CA", "forward geocode should populate ISO country code")
XCTAssertEqual(addressPlacemark.country, "Canada", "forward geocode should populate country")
XCTAssertEqual(addressPlacemark.postalCode, "B0K 1S0", "forward geocode should populate postal code")
XCTAssertEqual(addressPlacemark.administrativeArea, "Nova Scotia", "forward geocode should populate administrative area")
XCTAssertEqual(addressPlacemark.subAdministrativeArea, "Stellarton", "forward geocode should populate sub-administrative area")
XCTAssertEqual(addressPlacemark.locality, "Stellarton", "forward geocode should populate locality")
XCTAssertEqual(addressPlacemark.thoroughfare, "Pennsylvania Ave", "forward geocode should populate thoroughfare")
XCTAssertNil(addressPlacemark.subThoroughfare, "forward geocode should not populate sub-thoroughfare for street-only result")

XCTAssertNotNil(addressPlacemark.addressDictionary)
let addressDictionary = addressPlacemark.addressDictionary!
XCTAssertEqual(addressDictionary[MBPostalAddressStreetKey] as? String, "Pennsylvania Ave", "forward geocode should populate street in address dictionary")
XCTAssertEqual(addressDictionary[MBPostalAddressCityKey] as? String, "Stellarton", "forward geocode should populate city in address dictionary")
XCTAssertEqual(addressDictionary[MBPostalAddressStateKey] as? String, "Nova Scotia", "forward geocode should populate state in address dictionary")
XCTAssertEqual(addressDictionary[MBPostalAddressCountryKey] as? String, "Canada", "forward geocode should populate country in address dictionary")
XCTAssertEqual(addressDictionary[MBPostalAddressISOCountryCodeKey] as? String, "CA", "forward geocode should populate ISO country code in address dictionary")
}

func testInvalidForwardGeocode() {
let json = Fixture.stringFromFileNamed("forward_invalid")
stubRequest("GET", "https://api.mapbox.com/geocoding/v5/mapbox.places/Sandy+Island,+New+Caledonia.json?access_token=\(BogusToken)&country=fr&types=region%2Cplace%2Clocality%2Cpoi").andReturn(200).withHeaders(["Content-Type": "application/json"]).withBody(json)

let expection = expectationWithDescription("forward geocode execute completion handler for invalid query")
let geocoder = MBGeocoder(accessToken: BogusToken)
geocoder.geocodeAddressString("Sandy Island, New Caledonia", withAllowedScopes: [.AdministrativeArea, .Place, .Locality, .PointOfInterest], inCountries: ["FR"]) { (placemarks, error) in
XCTAssertEqual(placemarks?.count, 0, "forward geocode should return no results for invalid query")
expection.fulfill()
}

waitForExpectationsWithTimeout(1) { (error) in
XCTAssertNil(error, "Error: \(error)")
XCTAssertFalse(geocoder.geocoding)
}
}
}
1 change: 1 addition & 0 deletions MapboxGeocoderTests/fixtures/forward_invalid.json
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
{"type":"FeatureCollection","query":["sandy","island","new","caledonia"],"features":[],"attribution":"NOTICE: © 2016 Mapbox and its suppliers. All rights reserved. Use of this data is subject to the Mapbox Terms of Service (https://www.mapbox.com/about/maps/). This response and the information it contains may not be retained."}
1 change: 1 addition & 0 deletions MapboxGeocoderTests/fixtures/forward_valid.json
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
{"type":"FeatureCollection","query":["1600","pennsylvania","ave","nw"],"features":[{"id":"address.12284612417573100","type":"Feature","text":"Pennsylvania Ave","place_name":"Pennsylvania Ave, Stellarton, Nova Scotia B0K 1S0, Canada","relevance":0.645,"properties":{},"center":[-62.661944,45.5562851],"geometry":{"type":"Point","coordinates":[-62.661944,45.5562851]},"context":[{"id":"place.55130","text":"Stellarton"},{"id":"postcode.13310912212063190","text":"B0K 1S0"},{"id":"region.10539239752558240","text":"Nova Scotia"},{"id":"country.15589894856372040","text":"Canada","short_code":"ca"}]},{"id":"address.18050667373573100","type":"Feature","text":"Pennsylvania Ave","place_name":"Pennsylvania Ave, Wasaga Beach, Ontario L9Z 3A8, Canada","relevance":0.39,"properties":{},"center":[-79.989025,44.503369],"geometry":{"type":"Point","coordinates":[-79.989025,44.503369]},"context":[{"id":"place.61967","text":"Wasaga Beach"},{"id":"postcode.17609609551414490","text":"L9Z 3A8"},{"id":"region.13373639426376420","text":"Ontario"},{"id":"country.15589894856372040","text":"Canada","short_code":"ca"}]},{"id":"address.6485281316573100","type":"Feature","text":"Pennsylvania Ave","place_name":"Pennsylvania Ave, Vaughan, Ontario L4K 3X6, Canada","relevance":0.39,"properties":{},"center":[-79.532497,43.802232],"geometry":{"type":"Point","coordinates":[-79.532497,43.802232]},"context":[{"id":"neighborhood.5043724752221980","text":"Vellore Woods"},{"id":"place.60547","text":"Vaughan"},{"id":"postcode.6385804433309570","text":"L4K 3X6"},{"id":"region.13373639426376420","text":"Ontario"},{"id":"country.15589894856372040","text":"Canada","short_code":"ca"}]},{"id":"address.6456604709573100","type":"Feature","text":"Pennsylvania Ave","place_name":"Pennsylvania Ave, Vaughan, Ontario L4K 3X8, Canada","relevance":0.39,"properties":{},"center":[-79.531684,43.802706],"geometry":{"type":"Point","coordinates":[-79.531684,43.802706]},"context":[{"id":"neighborhood.5043724752221980","text":"Vellore Woods"},{"id":"place.60547","text":"Vaughan"},{"id":"postcode.6367605302663760","text":"L4K 3X8"},{"id":"region.13373639426376420","text":"Ontario"},{"id":"country.15589894856372040","text":"Canada","short_code":"ca"}]},{"id":"region.14011193683716580","type":"Feature","text":"Northwest Territories","place_name":"Northwest Territories, Canada","relevance":0.24,"properties":{},"bbox":[-141.102750026,59.9981700000001,-102.0004990085,79.229293683],"center":[-128.403942,69.564626],"geometry":{"type":"Point","coordinates":[-128.403942,69.564626]},"context":[{"id":"country.15589894856372040","text":"Canada","short_code":"ca"}]}],"attribution":"NOTICE: © 2016 Mapbox and its suppliers. All rights reserved. Use of this data is subject to the Mapbox Terms of Service (https://www.mapbox.com/about/maps/). This response and the information it contains may not be retained."}

0 comments on commit 3c38c63

Please sign in to comment.