Check coverage of (most) tests.

The end-to-end tests and their associated fixtures are deliberately not
included in normal test runs, so they also aren't included in coverage
reporting.

Author: ubernostrum

noxfile.py

@@ -80,6 +80,8 @@ def tests_with_coverage(session: nox.Session) -> None:
         "run",
         "--source",
         PACKAGE_NAME,
+        "--source",
+        "tests/",
         "-m",
         "pytest",
         "-m",

pyproject.toml

@@ -58,6 +58,10 @@ target-version = ["py39", "py310", "py311", "py312", "py313"]
 source = ["src", ".nox/tests_with_coverage*/**/site-packages"]
 
 [tool.coverage.report]
+omit = [
+    "tests/conftest.py",
+    "tests/end_to_end/*"
+]
 fail_under = 100
 
 [tool.coverage.run]

tests/test_legacy_client.py

@@ -11,7 +11,6 @@
 # SPDX-License-Identifier: BSD-3-Clause
 
 import os
-import typing
 import unittest
 from http import HTTPStatus
 
@@ -25,7 +24,6 @@
 def _make_fixed_response_transport(
     response_text: str = "true",
     status_code: HTTPStatus = HTTPStatus.OK,
-    response_json: typing.Optional[dict] = None,
 ) -> httpx.MockTransport:
     """
     Return an ``httpx`` transport that produces a fixed response, for use
@@ -34,7 +32,6 @@ def _make_fixed_response_transport(
     The transport will return a response consisting of:
 
     * ``status_code`` (default 200)
-    * ``response_json`` as the JSON content, if supplied
     * Otherwise ``response_text`` (default ``"true"``) as the response text
 
     """
@@ -46,29 +43,22 @@ def _handler(
         Mock transport handler which returns a controlled response.
 
         """
-        response_kwargs = {"status_code": status_code, "content": response_text}
-        if response_json is not None:
-            del response_kwargs["content"]
-            response_kwargs["json"] = response_json
-        return httpx.Response(**response_kwargs)  # type: ignore
+        return httpx.Response(status_code=status_code, content=response_text)
 
     return httpx.MockTransport(_handler)
 
 
 def _make_fixed_response_sync_client(
     response_text: str = "true",
     status_code: HTTPStatus = HTTPStatus.OK,
-    response_json: typing.Optional[dict] = None,
 ) -> httpx.Client:
     """
     Return a synchronous HTTP client that produces a fixed repsonse, for use in
     testing.
 
     """
     return httpx.Client(
-        transport=_make_fixed_response_transport(
-            response_text, status_code, response_json
-        )
+        transport=_make_fixed_response_transport(response_text, status_code)
     )