Skip to content

test(issue-250): S1T2-S1T3 shared lane characterization coverage#252

Merged
graysurf merged 3 commits intomainfrom
issue/s1-t2-add-characterization-tests-for-auto-single-lane
Feb 25, 2026
Merged

test(issue-250): S1T2-S1T3 shared lane characterization coverage#252
graysurf merged 3 commits intomainfrom
issue/s1-t2-add-characterization-tests-for-auto-single-lane

Conversation

@graysurf
Copy link
Owner

@graysurf graysurf commented Feb 25, 2026

Summary

  • Shared lane s1-auto-g2 for Plan: plan-issue-cli Task Decomposition runtime-truth refactor #250 now includes S1T2 + S1T3 characterization coverage.
  • S1T2 locks the current auto single-lane metadata drift behavior in unit tests.
  • S1T3 adds end-to-end characterization for issue-table vs task-spec/prompt/comment drift and live start-sprint overwrite behavior.

Scope

  • S1T2 (execute.rs / task_spec.rs): strengthen auto single-lane drift assertions and add ignored future canonicalization targets.
  • S1T3 (tests/sprint3_delivery.rs): characterize local start-plan/start-sprint artifact mismatch for auto single-lane (Task Decomposition vs task-spec/prompt/comment semantics).
  • S1T3 (tests/sprint4_delivery.rs): characterize live start-sprint sync path overwriting issue rows from recomputed task-spec rows (using gh stub).
  • Keep PR in draft for follow-up review and later sprint work; no production behavior changes.

Testing

  • S1T2 prompt validation: cargo test -p nils-plan-issue-cli sync_issue_rows_from_task_spec_auto_single_group_uses_per_sprint_mode -- --exact (pass; package-level exact filter matched 0 tests in current target layout)
  • S1T2 prompt validation: cargo test -p nils-plan-issue-cli task_spec::tests::execution_mode_by_task_auto_single_lane_uses_per_sprint -- --exact (pass)
  • S1T2 targeted: cargo test -p nils-plan-issue-cli --lib execute::tests::sync_issue_rows_from_task_spec_auto_single_group_uses_per_sprint_mode -- --exact (pass)
  • S1T2 targeted: cargo test -p nils-plan-issue-cli --lib task_spec::tests:: -- --nocapture (pass; 4 passed, 1 ignored)
  • S1T3 prompt validation: cargo test -p nils-plan-issue-cli --test sprint3_delivery (pass)
  • S1T3 targeted: cargo test -p nils-plan-issue-cli --test sprint4_delivery (pass)
  • S1T3 targeted: cargo test -p nils-plan-issue-cli --test task_spec_flow (pass)
  • Formatting: cargo fmt -p nils-plan-issue-cli -- --check (pass)

Issue

- Extend execute sync characterization to assert per-sprint mode leaves lane metadata drift.
- Add task_spec unit tests for auto/group mode classification and anchor expectations.
- Add ignored target assertions for future single-lane canonicalization invariants.
@github-actions
Copy link

github-actions bot commented Feb 25, 2026

Coverage

Total line coverage: 86.35% (58130/67316 lines hit).

- Add sprint3 delivery characterization for auto single-lane artifact mismatches.
- Add sprint4 delivery characterization for live start-sprint issue-row overwrite drift.
- Lock task-spec, prompt, comment, and issue-table drift assertions for follow-on refactors.
- Allow dead_code on shared integration test helper used only by some test targets.
- Allow too_many_arguments on task_spec unit-test fixture helper to satisfy -D warnings.
@graysurf graysurf marked this pull request as ready for review February 25, 2026 15:53
@graysurf graysurf merged commit 554d4cd into main Feb 25, 2026
3 checks passed
@graysurf graysurf deleted the issue/s1-t2-add-characterization-tests-for-auto-single-lane branch February 25, 2026 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant