split test runs into groups

[digitaltom]

No description provided.

[Copilot]

🟡 Changes recommended

CI jobs use an unsupported flag, and failed coverage checks can leave runtime timing data stale.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

This pull request splits the RSpec suite across CI matrix jobs and centralizes coverage and runtime data.

Changes:

• Distributes test execution across parallel groups.
• Defers coverage enforcement until results are merged.
• Collects and caches coverage and runtime artifacts.

File summaries

File	Summary
spec/rails_helper.rb	Defers per-job coverage enforcement.
.github/workflows/ci.yml	Adds matrix testing and artifact aggregation. Critical: parallel_tests 5.7.0 does not support --allowed-missing, causing jobs to fail. Moderate: timing data may remain stale when coverage checks fail.

Review details

Suppressed comments (1)

.github/workflows/ci.yml:195

• These steps are skipped whenever Check merged coverage fails. A suite can pass while coverage is below 100%, so the runtime data from that run is still valid; skipping the merge/save leaves the cache stale and makes later runs use old distribution timings. Run both timing steps with if: always() (or place them before the coverage check).

      - name: Merge spec runtimes
        run: |
          mkdir -p tmp
          cat parts/*/tmp/parallel_runtime_rspec.log > tmp/parallel_runtime_rspec.log
      - name: Save spec runtimes
        uses: actions/cache/save@v4

• Files reviewed: 2/2 changed files
• Comments generated: 1
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.