Skip to content

test: add unified auth integration coverage - #1689

Draft
Zetazzz wants to merge 1 commit into
constructive-io:feat/oauth-sso-handofffrom
Zetazzz:feat/oauth-sso-integration-tests
Draft

test: add unified auth integration coverage#1689
Zetazzz wants to merge 1 commit into
constructive-io:feat/oauth-sso-handofffrom
Zetazzz:feat/oauth-sso-integration-tests

Conversation

@Zetazzz

@Zetazzz Zetazzz commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a real graphql-server-test seam over scoped routing, Express Context, GraphQL, cookies, and PostgreSQL
  • verify canonical-host login start, local sign-in, hashed one-time handoff, cookie and no-store behavior, browser redemption rejection, and OAuth-disabled behavior
  • fix the SSO surface loader to resolve the physical schema name rather than its logical metadata name

Stack

Test contract

  • the local SQL fixture implements only the frozen Constructive/Constructive DB call boundary
  • DB-owned transaction, RLS, and runtime semantics remain in the Constructive DB PR and are not duplicated here

Validation

  • Express Context: 5 suites, 36 tests
  • unified-auth integration: 4 tests
  • scoped routing: 7 tests
  • full graphql-server-test with local PostgreSQL and MinIO: 11 suites, 156 tests, 2 snapshots
  • affected builds and lints pass; existing unrelated lint warnings remain
  • git diff --check passes

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