Skip to content

test(tenancy): await split-host projections (#157) - #375

Merged
smiggleworth merged 2 commits into
mainfrom
fix/split-tenant-projection-poll-deadlines
Sep 23, 2026
Merged

smiggleworth merged 2 commits into
mainfrom
fix/split-tenant-projection-poll-deadlines

Conversation

@smiggleworth

@smiggleworth smiggleworth commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

Change

Give the administrator-access and staff-membership projection polls their own 45-second windows after the corresponding invitation acceptance succeeds. The earlier invitation-token poll consumed part of the shared deadline, so the later polls could skip directly to an assertion after otherwise successful acceptance.

Poll the tenant projection until it reports active after the administrator team becomes readable. The team and tenant projections are independent; their visibility can arrive in either order.

Evidence

  • PR #371 CI run 35916466821 failed at the administrator-access assertion (Forbidden instead of OK).
  • PR #370 CI run 35917447426 observed provisioning immediately after the team became readable.
  • Focused split-host broker tests for both affected cases: 1/1 each passed after the changes.
  • Targeted dotnet format and git diff --check: passed.
  • Current-head CI run 35919416926: Validate, dependency review, and native AMD64/ARM64 checks passed on 1155859b1ec1c5c6e3fab77ee4a7efd31afc6805.

This changes only test polling. Keep #157 open for the remaining tenancy proof.

@smiggleworth smiggleworth changed the title test(tenancy): give split-host projection polls fresh deadlines (#157) test(tenancy): await split-host projections (#157) Sep 23, 2026
@smiggleworth
smiggleworth merged commit c5d5249 into main Sep 23, 2026
4 checks passed
@smiggleworth
smiggleworth deleted the fix/split-tenant-projection-poll-deadlines branch September 23, 2026 21:10
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