test: replace the force_flush test that passes without the override with a reasoned pragma (#262)

Author: lesnik512

lite_bootstrap/instruments/opentelemetry_instrument.py

@@ -127,7 +127,7 @@ def on_end(self, span: "ReadableSpan") -> None:
                 pyroscope.remove_thread_tag(_PYROSCOPE_SPAN_NAME_KEY, span.name)
 
         def force_flush(self, timeout_millis: int = 30000) -> bool:  # noqa: ARG002
-            return True
+            return True  # pragma: no cover - matters only at the sdk 1.28 floor, whose base returns None
 
 
 @dataclasses.dataclass(kw_only=True, slots=True)

tests/instruments/test_pyroscope_instrument.py

@@ -175,12 +175,6 @@ 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(