Skip to content

test(e2e): add critical flow sanity suite#94

Merged
cameroncuster merged 1 commit into
mainfrom
cameroncuster/add-critical-flow-sanity-tests
Jul 23, 2026
Merged

test(e2e): add critical flow sanity suite#94
cameroncuster merged 1 commit into
mainfrom
cameroncuster/add-critical-flow-sanity-tests

Conversation

@cameroncuster

Copy link
Copy Markdown
Owner

Summary

  • add a fast test:sanity selector for the most critical authentication and submission journeys
  • exercise the actual mocked GitHub OAuth redirect and callback, persisted admin session, admin submit access, logout revocation, and anonymous post-logout denial
  • include successful Codeforces and Kattis writes in the sanity set
  • extend the deterministic Supabase mock with OAuth authorize, user lookup, token refresh, logout, and admin-role behavior
  • retain the complete desktop/mobile Playwright suite as the required merge gate, including explicit member and anonymous authorization denials

Test isolation

All identities, tokens, role records, provider responses, and inserts are deterministic test fixtures. Writes land only in the mock server's resettable in-memory store; no real OAuth, provider, Supabase credential, or production data is used.

Verification

  • pnpm run test:sanity: 6/6 passed
  • formatting passed
  • ESLint passed
  • Svelte check passed
  • unit tests: 199/199 passed
  • production build passed
  • full desktop/mobile Playwright suite: 151 passed, 5 intentional live-only skips
  • git diff --check passed

Pull Request opened by Augment Code | View session

@vercel

vercel Bot commented Jul 23, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
gitgud Ready Ready Preview, Comment Jul 23, 2026 11:16pm

@cameroncuster
cameroncuster marked this pull request as ready for review July 23, 2026 23:16
@cameroncuster
cameroncuster merged commit 00a6066 into main Jul 23, 2026
6 checks passed
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