90bd05aa98
Some checks failed
Architecture Governance / architecture (push) Has been cancelled
Closes out F63 - every domain named in the delivery plan's Q7 gap now has facade-level coverage. admin-users.facade.spec.ts (14 tests): sessions/audit dialog targeting (a revokeSession must reload the CURRENTLY open user's sessions, not whichever user happened to be open first), invite() trims and rejects whitespace-only email, roleName() falls back to the raw id rather than rendering blank for an unmapped role. cart.service.spec.ts (23 tests) - this had zero coverage despite feeding totalWithDelivery and allRequiredDeliveriesSelected directly into cart.component.ts's checkout gate and the offers/qty payload sent to POST /api/v2/storefront/checkout, the exact contract this session's F14-F16 rewired. A regression here would silently let checkout proceed with a missing delivery selection, or compute the wrong total. Covers the delivery- requirement matrix explicitly (digital items never require selection, deliverySelectionRequired: false overrides having options present, a selection satisfies the requirement once made) and the quantity-to-zero -> line-removal behavior in updateQuantity. Verified: 237/237 unit tests (37 new since the last commit), arch:check clean, 5/5 E2E, production build succeeds. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>