Merge pull request #13 from BarnabasG/initial

1.2.2 Fixing flask testclient.open logic

Author: BarnabasG

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "pytest-api-cov"
-version = "1.2.1"
+version = "1.2.2"
 description = "Pytest Plugin to provide API Coverage statistics for Python Web Frameworks"
 readme = "README.md"
 authors = [{ name = "Barnaby Gill", email = "barnabasgill@gmail.com" }]

src/pytest_api_cov/plugin.py

@@ -246,7 +246,10 @@ def _extract_path_and_method(self, name: str, args: Any, kwargs: Any) -> Optiona
                 first = args[0]
                 if isinstance(first, str):
                     path = first.partition("?")[0]
-                    method = name.upper()
+                    method = kwargs.get("method", name).upper()
+                    if method == "OPEN":
+                        method = "GET"
+
                     return path, method
 
                 # For starlette/requests TestClient, args[0] may be a Request or PreparedRequest
@@ -259,19 +262,21 @@ def _extract_path_and_method(self, name: str, args: Any, kwargs: Any) -> Optiona
                     else:
                         return path, method
 
-            # Try kwargs-based FlaskClient open signature
             if kwargs:
                 path_kw = kwargs.get("path") or kwargs.get("url") or kwargs.get("uri")
                 if isinstance(path_kw, str):
                     path = path_kw.partition("?")[0]
                     method = kwargs.get("method", name).upper()
+                    if method == "OPEN":
+                        method = "GET"
+
                     return path, method
 
             return None
 
         def __getattr__(self, name: str) -> Any:
             attr = getattr(self._wrapped, name)
-            if name in ["get", "post", "put", "delete", "patch", "head", "options"]:
+            if name in {"get", "post", "put", "delete", "patch", "head", "options"}:
 
                 def tracked_method(*args: Any, **kwargs: Any) -> Any:
                     response = attr(*args, **kwargs)