test(rate-of-closure): recover ten stranded test modules covering source already on main (remainder of #4466)

[dieterolson]

Extracted remainder of #4466. Test-only — no source file is touched.

Why

#4466 is being absorbed into main as reviewable slices (#4517–#4523). Those slices took source modules but left their suites behind on the branch. The clearest case: the five ground_playback modules that #4522 landed arrived on main with zero test coverage — grepping the whole repo for ground_playback matches SPEC.md and the source files themselves, and nothing under tests/.

This recovers the suites that already pass against main unchanged, so the coverage stops depending on a branch that cannot be merged.

What

Ten files, all under tests/. Because no source changes, every assertion runs against main's implementation exactly as it stands.

File	Covers
tests/rate_of_closure/test_ground_playback.py	simulation/ground_playback* (#4522) — previously untested
tests/rate_of_closure/test_capability_observation_adapter.py	capability observation adapter
tests/rate_of_closure/test_chip_forgiveness_analysis.py	chip forgiveness analysis
tests/rate_of_closure/test_ground_study_scalar_adapter.py	ground-study scalar adapter
tests/rate_of_closure/test_regional_ground_study_adapter.py	regional ground study adapter
tests/rate_of_closure/test_regional_ground_target_projection.py	regional ground target projection
tests/rate_of_closure/test_scalar_ensemble_contract.py	scalar ensemble contract
tests/rate_of_closure/test_wind_strategy_plot_adapter.py	wind-strategy plot adapter
tests/rate_of_closure/regional_ground_target_support.py	shared helper the projection suite imports
tests/shared/python/test_canonical_numeric_json.py	swing_sim/canonical_numeric_json

Verification

Repo venv (Python 3.11, PyQt6, QT_QPA_PLATFORM=offscreen): 93 passed, 0 failed.
Pinned ruff 0.14.10: check and format --check clean on all ten files.

Deliberately excluded

Eleven further stranded suites were run against main and left out rather than widening this PR to chase their dependencies:

Suite	Blocked on
test_chip_forgiveness_runner.py	SimulationEnsembleResult.runs absent from main
test_club_assembly_simulation_adapter.py	SimulationRun.club_assembly_usage absent from main
test_regional_ground_result_golden.py	__fixtures__ golden absent from main
test_club_assembly_binding.py	__fixtures__ golden absent from main
test_clubhead_engineering_sidecar.py	__fixtures__ golden absent from main
test_ground_playback_workspace_v2.py	__fixtures__ golden absent from main
test_regional_ground_variation_request_io.py	needs a ui.pyqt6 module of the same name
test_campaign_release_manifest.py	needs scripts/rate_campaign_manifest
test_browser_companion_harness.py	needs sibling browser_companion_harness helper
tests/ops/test_maturin_swing_core_workflow.py	asserts on a workflow file main lacks
tests/ops/test_rate_of_closure_windows_state_workflow.py	asserts on a workflow file main lacks

test_regional_ground_variation.py is excluded for a genuine behavioural divergence that deserves its own fix: one bounds case expects the module's own validation error, but on main a DbC pre-condition (lower must be) raises first. test_regional_ground_variation_execution.py imports from it, so it is excluded too.

Note on unrelated CI

ruff format --check is currently red on main for four files under src/data_processing/data_processor/python/ that this PR does not touch. That is being fixed separately.

Part of #4103

🤖 Generated with Claude Code

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.