Sync from GitHub · 2026-09-24 17:31 UTC

pytest.ini

@@ -0,0 +1,2 @@
+[pytest]
+testpaths = tests

requirements-dev.txt

@@ -0,0 +1,3 @@
+# Dev/test-only dependencies. Install with: pip install -r requirements-dev.txt
+-r requirements.txt
+pytest>=8.0.0

tests/conftest.py

@@ -0,0 +1,10 @@
+# Runs before any test module is imported (pytest loads conftest.py first).
+#
+# routes/bff.py imports utils/groq_client.py at module level, which raises
+# RuntimeError at import time if no GROQ_API_KEY* env var is set (see its own
+# docstring). Tests here only exercise pure functions (_stage_guidance,
+# BUSINESS_STAGES) that never make a real Groq call, so a fake key is enough
+# to get past import — never make a real network call using this value.
+import os
+
+os.environ.setdefault("GROQ_API_KEY", "test-key-not-a-real-credential")

tests/test_bff_stage_guidance.py

@@ -0,0 +1,41 @@
+"""
+Unit tests for the pure stage → prompt-guidance logic in routes/bff.py.
+
+Scope note (docs/qa/test-strategy.md): this deliberately stays at the level of
+"does the right guidance string get picked for the right stage", not "is the
+Groq call correct" — real Groq calls are explicitly out of scope for
+automation (cost, non-determinism, rate limits; see ADR-0009).
+"""
+from routes.bff import _stage_guidance, BUSINESS_STAGES, PRE_LAUNCH_STAGES
+
+
+def test_unknown_or_empty_stage_returns_no_guidance():
+    assert _stage_guidance("") == ""
+    assert _stage_guidance("not-a-real-stage-id") == ""
+
+
+def test_pre_launch_stages_tell_the_model_not_to_assume_customers():
+    for stage in PRE_LAUNCH_STAGES:
+        guidance = _stage_guidance(stage)
+        assert "has NOT launched" in guidance
+        assert BUSINESS_STAGES[stage] in guidance
+
+
+def test_restarting_gets_its_own_guidance_not_the_pre_launch_wording():
+    guidance = _stage_guidance("restarting")
+    assert "starting again or pivoting" in guidance.lower() or "changing direction" in guidance.lower()
+    assert "has NOT launched" not in guidance
+
+
+def test_operating_stages_ask_about_real_experience():
+    operating_stages = set(BUSINESS_STAGES) - PRE_LAUNCH_STAGES - {"restarting"}
+    assert operating_stages, "expected at least one operating stage to exist"
+    for stage in operating_stages:
+        guidance = _stage_guidance(stage)
+        assert "already operating" in guidance
+
+
+def test_pre_launch_stages_are_a_subset_of_known_stages():
+    # Guards against PRE_LAUNCH_STAGES and BUSINESS_STAGES drifting apart if a
+    # stage is ever renamed on one side and not the other.
+    assert PRE_LAUNCH_STAGES <= set(BUSINESS_STAGES.keys())