Validation: #51 Request specs for properties CRUD + resolve
Validation: #51 Request specs for properties CRUD + resolve
Verdict: PASS
PR
PR #53 — merged 2026-05-29 via squash
Checks
- CI pipeline #108 (PR check): SUCCESS — lint + RSpec passed
- CI pipeline #109 (main push): running — build+push to Harbor (test step already passed)
- QA review: APPROVED — 22 new specs verified line-by-line against controller logic, no blockers
- No app code changed: test-only PR, zero risk to production behavior
Coverage Added
GET /properties/:id— 6 examples (content rendering, services, notes)GET /properties/:id/edit— 3 examples (form, services)PATCH /properties/:id— 5 examples (HTML redirect, JSON, validation failure)POST /properties/resolve— 8 examples (create new, find existing, TBD default, service assignment, special notes)
Deferred
- Two nit-level edge cases from QA (number-without-street, same-address-different-city) — non-blocking, can be added later