Merge pull request #117 from getfiit/issues-109_tests

issue #109 tests & fix

Sources/URLMatcher/URLMatcher.swift

@@ -120,7 +120,7 @@ open class URLMatcher {
         return false
       }
     }
-    return hasSameNumberOfComponents || containsPathPlaceholderComponent
+    return hasSameNumberOfComponents || (containsPathPlaceholderComponent && stringPathComponents.count > candidatePathComponents.count)
   }
 
   func stringPathComponents(from url: URLConvertible) -> [String] {

Tests/URLMatcherTests/URLMatcherSpec.swift

@@ -137,5 +137,25 @@ final class URLMatcherSpec: QuickSpec {
       expect(result?.pattern) == "https://<path:url>"
       expect(result?.values["url"] as? String) == "google.com/search"
     }
+
+    it("returns nil when there's no candidates (issues-109)") {
+        let candidates = ["/anything/<path:url>"]
+        let result = matcher.match("", from: candidates)
+        expect(result).to(beNil())
+    }
+
+    it("returns nil when there's no candidates (issues-109)") {
+        let candidates = ["http://host/anything/<path:url>"]
+        let result = matcher.match("http://host/anything", from: candidates)
+        expect(result).to(beNil())
+    }
+
+    it("returns same candidate (issues-109)") {
+        let candidates1 = ["http://host/anything/<path:url>", "http://host/anything"]
+        let result1 = matcher.match("http://host/anything", from: candidates1)
+        let candidates2 = ["http://host/anything", "http://host/anything/<path:url>"]
+        let result2 = matcher.match("http://host/anything", from: candidates2)
+        expect(result1?.pattern).to(equal(result2?.pattern))
+    }
   }
 }