Skip to content

Add Trip Editor real CRUD contract coverage#299

Merged
stef-k merged 2 commits into
mainfrom
fix/trip-editor-real-crud-contracts-297-batch2
May 18, 2026
Merged

Add Trip Editor real CRUD contract coverage#299
stef-k merged 2 commits into
mainfrom
fix/trip-editor-real-crud-contracts-297-batch2

Conversation

@stef-k
Copy link
Copy Markdown
Owner

@stef-k stef-k commented May 18, 2026

Summary

Implements #297 Batch 2 only: real endpoint CRUD contract coverage for the Trip Editor.

  • Adds focused real endpoint CRUD contract coverage for region, place, area, and segment persistence.
  • Region/place/area flows use UI creation/deletion where applicable and prove visible state plus reload/API reread.
  • Place coverage includes moving between a normal region and Unassigned Places.
  • Segment coverage is endpoint-focused: real segment create, route-save, reorder, and delete endpoints are exercised with reload/API reread plus visible row/order/delete assertions. This does not claim full route map-work UI drawing coverage.

Out Of Scope

Validation

  • Focused new spec: 4 passed.
  • Segment focused rerun: 1 passed.
  • npx playwright test --config=playwright.config.ts --list: passed, 110 tests listed.
  • dotnet build: passed.
  • dotnet test: passed, 1537 tests.
  • LOC check: passed with warnings only; new spec is 447 counted LOC, accepted as cohesive Batch 2 real CRUD contract coverage under the 600 hard cap.
  • git diff --check origin/main...HEAD: passed.
  • tracked artifact scan: clean.

Notes

Existing mocked Trip Editor specs remain in place for frontend rendering, request shape, Sortable/map-work draft behavior, rich notes UI, icon/marker visuals, and error/visual states. They are not used as CRUD persistence proof.

@stef-k stef-k merged commit e7cdab3 into main May 18, 2026
1 check passed
@stef-k stef-k deleted the fix/trip-editor-real-crud-contracts-297-batch2 branch May 18, 2026 08:52
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