chore: give every coverage pragma a reason (TS6, TS5) (#261)

Author: lesnik512

lite_bootstrap/instruments/logging_instrument.py

@@ -47,7 +47,7 @@ def tracer_injection(_: "WrappedLogger", __: str, event_dict: "EventDict") -> "E
         }
         return event_dict
 
-else:  # pragma: no cover
+else:  # pragma: no cover - only without opentelemetry, i.e. neither the [otl] nor the [otl-http] extra
 
     def tracer_injection(_: "WrappedLogger", __: str, event_dict: "EventDict") -> "EventDict":
         return event_dict

lite_bootstrap/instruments/opentelemetry_instrument.py

@@ -126,7 +126,7 @@ def on_end(self, span: "ReadableSpan") -> None:
                 pyroscope.remove_thread_tag(_PYROSCOPE_SPAN_ID_KEY, format_span_id(span.context.span_id))
                 pyroscope.remove_thread_tag(_PYROSCOPE_SPAN_NAME_KEY, span.name)
 
-        def force_flush(self, timeout_millis: int = 30000) -> bool:  # pragma: no cover  # noqa: ARG002
+        def force_flush(self, timeout_millis: int = 30000) -> bool:  # noqa: ARG002
             return True
 
 

pyproject.toml

@@ -267,5 +267,4 @@ asyncio_default_fixture_loop_scope = "function"
 fail_under = 100
 exclude_also = [
     "if typing.TYPE_CHECKING:",
-    "except ImportError:"
 ]

tests/instruments/test_pyroscope_instrument.py

@@ -175,6 +175,12 @@ def test_pyroscope_span_processor_on_start_remote_parent() -> None:
         mock_pyroscope.add_thread_tag.assert_any_call("span_name", "remote-root")
 
 
+def test_pyroscope_span_processor_does_not_fail_provider_force_flush() -> None:
+    provider = SDKTracerProvider()
+    provider.add_span_processor(otel_module.PyroscopeSpanProcessor())
+    assert provider.force_flush() is True
+
+
 def test_pyroscope_otel_adds_span_processor_when_configured() -> None:
     """OTel instrument adds PyroscopeSpanProcessor when pyroscope_endpoint is set."""
     config = FreeConfig(

tests/test_litestar_bootstrap.py

@@ -258,7 +258,7 @@ def test_litestar_otel_apps_cache_evicts_dead_refs() -> None:
     )
 
     async def transient_app(scope: dict, receive: object, send: object) -> None:  # noqa: ARG001
-        return None  # pragma: no cover
+        return None  # pragma: no cover - only a weakref target; the test never calls it
 
     weak_app = weakref.ref(transient_app)
     middleware._otel_apps[transient_app] = "marker"  # noqa: SLF001  # ty: ignore[invalid-assignment]