Validation: #51 Request specs for properties CRUD + resolve

validation-51-2026-06-01 Doc

validation pass

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