#92 Fixed a bug that caused URLs containing regular expression reserv…

…ed characters to fail the URL matching. Added unit test for the fixed feature. (#93)
hibri · Jan 26, 2018 · a032634 · a032634
1 parent ea71da2
commit a032634
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/src/HttpMock.Unit.Tests/EndpointMatchingRuleTests.cs b/src/HttpMock.Unit.Tests/EndpointMatchingRuleTests.cs
@@ -226,8 +226,18 @@ public void should_match_when_the_query_string_has_a_trailing_ampersand()
 
 		}
 
+        [Test]
+        public void should_match_urls_containings_regex_reserved_characters()
+        {
+            var requestHandler = MockRepository.GenerateStub<IRequestHandler>();
+            requestHandler.Path = "/test()";
+            requestHandler.QueryParams = new Dictionary<string, string>();
 
-	}
+            var httpRequestHead = new HttpRequestHead { Uri = "/test()" };
+            var endpointMatchingRule = new EndpointMatchingRule();
+            Assert.That(endpointMatchingRule.IsEndpointMatch(requestHandler, httpRequestHead));
+        }
+    }
 
 
     [TestFixture]

diff --git a/src/HttpMock/EndpointMatchingRule.cs b/src/HttpMock/EndpointMatchingRule.cs
@@ -56,7 +56,7 @@ private static bool MatchPath(IRequestHandler requestHandler, HttpRequestHead re
 	        {
 	            pathToMatch = request.Uri.Substring(0, positionOfQueryStart);
 	        }
-            var pathMatch = new Regex(string.Format(@"^{0}\/*$", requestHandler.Path));
+            var pathMatch = new Regex(string.Format(@"^{0}\/*$", Regex.Escape(requestHandler.Path)));
 	        return pathMatch.IsMatch(pathToMatch);
 	    }