From 288246d9b5f94a2384bcfe51554b75b809749cc0 Mon Sep 17 00:00:00 2001 From: "Brent E. Edwards" Date: Mon, 16 Mar 2026 23:22:05 +0000 Subject: [PATCH] fix(test): remove eager tdd_test_helpers import from mocks __init__ The re-export of make_mock_scenario from features/mocks/__init__.py caused ASV benchmark discovery to fail because tdd_test_helpers imports behave.model.Status, which is unavailable in the ASV benchmark virtualenv. All callers already import via the full module path (features.mocks.tdd_test_helpers), so the re-export was unnecessary. ISSUES CLOSED: #628 --- features/mocks/__init__.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/features/mocks/__init__.py b/features/mocks/__init__.py index cde17c714..43b8f3660 100644 --- a/features/mocks/__init__.py +++ b/features/mocks/__init__.py @@ -9,9 +9,13 @@ from .fake_provider import FakeProviderInfo, FakeProviderRegistry from .lsp_transport_mock import MockLspTransport, parse_lsp_responses from .mock_ai_provider import MockAIProvider from .mock_mcp_transport import MockMCPTransport -from .tdd_test_helpers import make_mock_scenario from .transient_fail_audit_service import TransientFailAuditService +# NOTE: tdd_test_helpers is NOT re-exported here because it imports +# ``behave.model.Status`` which is unavailable in ASV benchmark +# environments. Import directly: ``from features.mocks.tdd_test_helpers +# import make_mock_scenario`` + __all__ = [ "FakeProviderInfo", "FakeProviderRegistry", @@ -19,6 +23,5 @@ __all__ = [ "MockLspTransport", "MockMCPTransport", "TransientFailAuditService", - "make_mock_scenario", "parse_lsp_responses", ] -- 2.52.0