tests/test_review.py

• _Hub · class · L20-L33 — class _Hub
• init · method · L21-L22 — def init(self) -> None
• emitter · method · L24-L29 — def emitter(self, root_frame_id: str)
• emit · function · L25-L27 — def emit(event: dict) -> None
• broadcast · method · L31-L33 — def broadcast(self, root_frame_id: str, event: dict) -> None
• _cfg · function · L36-L41 — def _cfg(tmp_path: Path) -> Config
• _review_context · function · L44-L52 — def _review_context(tmp_path: Path)
• test_json_object_accepts_plain_fenced_and_prose_wrapped_json · function · L69-L70 — def test_json_object_accepts_plain_fenced_and_prose_wrapped_json(raw, expected)
• test_json_object_rejects_responses_without_an_object · function · L74-L76 — def test_json_object_rejects_responses_without_an_object(raw)
• test_normalize_review_cleans_caps_and_forces_issue_verdict · function · L79-L114 — def test_normalize_review_cleans_caps_and_forces_issue_verdict()
• test_normalize_review_pass_is_canonical_and_invalid_schema_is_unavailable · function · L117-L131 — def test_normalize_review_pass_is_canonical_and_invalid_schema_is_unavailable()
• test_bounded_packet_preserves_valid_json_artifacts_tools_and_execution · function · L134-L185 — def test_bounded_packet_preserves_valid_json_artifacts_tools_and_execution()
• test_review_evidence_refuses_omitted_artifacts_without_calling_model · function · L188-L207 — def test_review_evidence_refuses_omitted_artifacts_without_calling_model(monkeypatch)
• test_review_evidence_is_one_bounded_call_with_usage_and_model · function · L210-L256 — def test_review_evidence_is_one_bounded_call_with_usage_and_model(monkeypatch)
• fake_chat · function · L213-L221 — def fake_chat(messages, cfg, **kwargs)
• test_run_reviewer_pass_persists_evidence_step_and_usage · function · L259-L381 — def test_run_reviewer_pass_persists_evidence_step_and_usage(monkeypatch, tmp_path)
• fake_review_evidence · function · L311-L320 — def fake_review_evidence(evidence, cfg)
• test_run_reviewer_issues_are_persisted_and_streamed · function · L384-L422 — def test_run_reviewer_issues_are_persisted_and_streamed(monkeypatch, tmp_path)
• test_run_reviewer_error_is_nonfatal_and_marks_step_unavailable · function · L425-L456 — def test_run_reviewer_error_is_nonfatal_and_marks_step_unavailable( monkeypatch, tmp_path )
• unavailable · function · L430-L431 — def unavailable(_evidence, _cfg)
• test_reviewer_keeps_metadata_beyond_twelve_changed_artifacts · function · L459-L492 — def test_reviewer_keeps_metadata_beyond_twelve_changed_artifacts(monkeypatch, tmp_path)
• fake_review · function · L468-L476 — def fake_review(evidence, cfg)
• test_manual_review_reserves_operation_and_honors_pre_state_cancel · function · L495-L540 — def test_manual_review_reserves_operation_and_honors_pre_state_cancel( monkeypatch, tmp_path )
• delayed_state · function · L506-L509 — def delayed_state(root_frame_id, project_id)
• test_run_reviewer_cancel_stops_waiting_and_marks_step_cancelled · function · L543-L620 — def test_run_reviewer_cancel_stops_waiting_and_marks_step_cancelled( monkeypatch, tmp_path )
• slow_review · function · L551-L560 — def slow_review(_evidence, _cfg)
• test_submit_manual_review_preserves_frame_status_and_joins_assistant_blocks · function · L623-L655 — def test_submit_manual_review_preserves_frame_status_and_joins_assistant_blocks( monkeypatch, tmp_path )
• fake_reviewer · function · L639-L641 — def fake_reviewer(_st, _emit, **kwargs)
• test_submit_manual_review_repairs_stale_processing_status · function · L658-L672 — def test_submit_manual_review_repairs_stale_processing_status(monkeypatch, tmp_path)
• test_submit_manual_review_reads_tail_of_long_conversation · function · L675-L702 — def test_submit_manual_review_reads_tail_of_long_conversation(monkeypatch, tmp_path)
• tail_messages · function · L679-L688 — def tail_messages(_fid, *, branch_id, limit)
• test_ws_resume_buffer_reopens_for_manual_review_and_closes_on_ready · function · L705-L723 — def test_ws_resume_buffer_reopens_for_manual_review_and_closes_on_ready()
• test_review_model_agent_sentinel_ignores_global_override · function · L726-L755 — def test_review_model_agent_sentinel_ignores_global_override(tmp_path)
• test_run_message_auto_review_hook_runs_only_for_completed_non_plan_turns · function · L762-L803 — def test_run_message_auto_review_hook_runs_only_for_completed_non_plan_turns( monkeypatch, tmp_path, enabled, plan, expected_calls )
• ensure_kernel · function · L769-L770 — def ensure_kernel(st)
• finish_loop · function · L775-L777 — def finish_loop(_st, _emit, visible)
• fake_reviewer · function · L782-L785 — def fake_reviewer(_st, emit, **kwargs)
• test_review_settings_route_inherits_then_persists_frame_overrides · function · L806-L852 — def test_review_settings_route_inherits_then_persists_frame_overrides(tmp_path)
• test_manual_review_route_rejects_duplicate_provider_call · function · L855-L869 — def test_manual_review_route_rejects_duplicate_provider_call(monkeypatch, tmp_path)
• test_project_artifact_zip_contains_current_files_and_deduplicates_names · function · L872-L910 — def test_project_artifact_zip_contains_current_files_and_deduplicates_names(tmp_path)