Run mypy on all files, test for explicit re-export

Teran McKinney · csernazs · commit ef24cbd1e2a7 · 2021-11-09T09:53:19.000+01:00
Includes some mypy fixes and some type ignores.
diff --git a/Makefile b/Makefile
@@ -18,7 +18,7 @@ cs: venv
 
 .PHONY: mypy
 mypy: venv
-	.venv/bin/mypy pytest_httpserver
+	.venv/bin/mypy
 
 .PHONY: autoformat
 autoformat: dev
diff --git a/doc/conf.py b/doc/conf.py
@@ -20,6 +20,7 @@
 # import os
 # import sys
 # sys.path.insert(0, os.path.abspath('.'))
+from typing import Dict
 
 
 # -- General configuration ------------------------------------------------
@@ -135,7 +136,7 @@
 
 # -- Options for LaTeX output ---------------------------------------------
 
-latex_elements = {
+latex_elements: Dict[str, str] = {
     # The paper size ('letterpaper' or 'a4paper').
     #
     # 'papersize': 'letterpaper',
diff --git a/mypy.ini b/mypy.ini
@@ -0,0 +1,6 @@
+[mypy]
+files = .
+implicit_reexport = False
+
+[mypy-setuptools.*]
+ignore_missing_imports = True
diff --git a/setup.py b/setup.py
@@ -35,6 +35,7 @@
             "reno",
             "autopep8",
             "mypy",
+            "types-requests",
         ],
         "test": [
             "pytest",
diff --git a/tests/test_headers.py b/tests/test_headers.py
@@ -25,7 +25,7 @@ def custom_header_value_matcher(actual: str, expected: str) -> bool:
         expected_dict = parse_dict_header(expected_dict_str)
         return actual_scheme == expected_scheme and actual_dict == expected_dict
 
-    matchers = HeaderValueMatcher.DEFAULT_MATCHERS.copy()
+    matchers = HeaderValueMatcher.DEFAULT_MATCHERS.copy()  # type: ignore
     matchers['Custom'] = custom_header_value_matcher
     header_value_matcher = HeaderValueMatcher(matchers)
 
diff --git a/tests/test_permanent.py b/tests/test_permanent.py
@@ -19,7 +19,7 @@ def test_expected_request_data(httpserver: HTTPServer):
 
 
 def test_expected_request_handler(httpserver: HTTPServer):
-    httpserver.expect_request("/foobar").respond_with_handler(lambda request: JSON_STRING)
+    httpserver.expect_request("/foobar").respond_with_handler(lambda request: JSON_STRING)  # type: ignore
     assert requests.get(httpserver.url_for("/foobar")).json() == {'foo': 'bar'}
 
 
@@ -29,7 +29,7 @@ def test_expected_request_response(httpserver: HTTPServer):
 
 
 def test_expected_request_response_as_string(httpserver: HTTPServer):
-    httpserver.expect_request("/foobar").respond_with_response(JSON_STRING)
+    httpserver.expect_request("/foobar").respond_with_response(JSON_STRING)  # type: ignore
     assert requests.get(httpserver.url_for("/foobar")).json() == {'foo': 'bar'}
 
 
diff --git a/tests/test_urimatch.py b/tests/test_urimatch.py
@@ -44,7 +44,7 @@ def test_regexp(httpserver: HTTPServer):
 
 
 def test_object_with_eq(httpserver: HTTPServer):
-    httpserver.expect_request(PrefixMatchEq("/foo")).respond_with_json({"foo": "bar"})
+    httpserver.expect_request(PrefixMatchEq("/foo")).respond_with_json({"foo": "bar"})  # type: ignore
     assert requests.get(httpserver.url_for("/foo")).json() == {"foo": "bar"}
     assert requests.get(httpserver.url_for("/foobar")).json() == {"foo": "bar"}
     assert requests.get(httpserver.url_for("/foobaz")).json() == {"foo": "bar"}
