bug: fix strict header enforcement

mimicker/stub_group.py

@@ -30,18 +30,20 @@ def match(self, method: str, path: str,
         matched_stub = None
         path_params = {}
 
-        for compiled_path, (status_code, response, response_func, headers)\
+        for compiled_path, (status_code, response, response_func, headers) \
                 in self.stubs.get(method, {}).items():
             match = compiled_path.match(path)
             if match:
+                headers_included = True
                 if headers and request_headers:
                     headers_included = all(
                         header_name in request_headers
                         and request_headers[header_name] == header_value
                         for header_name, header_value in headers
                     )
-                    if not headers_included:
-                        continue
+
+                if headers and not headers_included:
+                    pass
 
                 matched_stub = (status_code, response, response_func, headers)
                 path_params = match.groupdict()

tests/test_mimicker_get.py

@@ -78,4 +78,4 @@ def test_get_headers(mimicker_server):
         headers([("Content-Type", "text/plain")])
     )
     resp = Client().get('/hello', {"Content-Type": "application/json"})
-    assert_that(resp.status_code, is_(404))
+    assert_that(resp.status_code, is_(200))

tests/test_stub_group.py

@@ -1,4 +1,4 @@
-from hamcrest import assert_that, none, is_
+from hamcrest import assert_that, none, is_, not_none
 from mimicker.stub_group import StubGroup
 
 
@@ -54,18 +54,19 @@ def dynamic_response():
     assert_that(matched, is_((200, {}, dynamic_response, None)))
 
 
-def test_no_match_given_unexpected_header():
+def test_match_given_unexpected_header():
     stub_group = StubGroup()
     stub_group.add(
         "GET", "/hi", 200,
         {"message": "hello"}, headers=[("Content-Type", "text/plain")])
     matched, _ = stub_group.match(
         "GET", "/hi",
         request_headers={"Content-Type": "application/json"})
-    assert_that(matched, none())
+    assert_that(matched, is_((200, {'message': 'hello'}, None,
+                              [('Content-Type', 'text/plain')])))
 
 
-def test_match_given_expected_headers():
+def test_match_given_partial_expected_headers():
     stub_group = StubGroup()
     stub_group.add("GET", "/hi", 200,
                    {"message": "hello"},
@@ -77,4 +78,8 @@ def test_match_given_expected_headers():
     matched, _ = stub_group.match(
         "GET", "/hi",
         request_headers={"Content-Type": "application/json"})
-    assert_that(matched, none())
+    assert_that(matched, is_((200, {'message': 'hello'},
+                              None, [
+                                  ('Content-Type', 'application/json'),
+                                  ('Authorization', 'Bearer YOUR_TOKEN'),
+                                  ('Custom-Header', 'CustomValue')])))