Skip to content

Add Trip Editor search-add persistence coverage#301

Merged
stef-k merged 2 commits into
mainfrom
fix/trip-editor-search-add-persistence-297-batch4
May 18, 2026
Merged

Add Trip Editor search-add persistence coverage#301
stef-k merged 2 commits into
mainfrom
fix/trip-editor-search-add-persistence-297-batch4

Conversation

@stef-k
Copy link
Copy Markdown
Owner

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

Summary

Implements #297 Batch 4 only: focused Playwright coverage for Trip Editor map search Add as place persistence after #295.

The tests mock only the geocode provider/search response. Save Place is not mocked: it posts to the real create-place endpoint, verifies success feedback and visible sidebar placement, rereads the editor API, reloads the page, and verifies persistence again.

Covered:

  • Unassigned Places default target and persistence.
  • One selected normal-region add and persistence.
  • Fresh search-add target reset back to Unassigned Places.
  • Pending marker and Add Place draft defaults (marker + bg-blue).

Out Of Scope

  • Batch 5 rich-notes sanitizer/persistence.
  • Batch 6 published production asset smoke.
  • Product behavior changes.

Validation

  • Focused spec: 2 passed.
  • npx playwright test --config=playwright.config.ts --list: passed, 116 tests listed.
  • dotnet build: passed.
  • dotnet test: passed, 1537 tests.
  • LOC check: passed under hard cap; warnings are existing large files, not the new 263-line spec.
  • git diff --check origin/main...HEAD: passed.
  • tracked artifact scan: clean.

@stef-k stef-k merged commit a496f2a into main May 18, 2026
1 check passed
@stef-k stef-k deleted the fix/trip-editor-search-add-persistence-297-batch4 branch May 18, 2026 09:38
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