Skip to content

GSOC-E2E: Add unauth-to-auth sketch persistence test. - #4246

Merged
clairep94 merged 5 commits into
processing:developfrom
Geethegreat:auth-unauth-flow
Aug 17, 2026
Merged

GSOC-E2E: Add unauth-to-auth sketch persistence test.#4246
clairep94 merged 5 commits into
processing:developfrom
Geethegreat:auth-unauth-flow

Conversation

@Geethegreat

Copy link
Copy Markdown
Member

Issue:

Fixes #

Added a Playwright E2E test covering the unauthenticated-to-authenticated user workflow.

Demo:

e2e-ath-unath-flow.mp4

Changes:

  • Verifies that a guest user can write and execute a sketch before creating an account.
  • Confirms that attempting to save as an unauthenticated user prompts account creation.
  • Completes the signup flow through the UI.
  • Verifies that the authenticated session is established successfully after signup.
  • Ensures the sketch persists and can be executed successfully after the user becomes authenticated.
  • Opens the verification link in the mail and confirms redirection to the editor.

I have verified that this pull request:

  • has no linting errors (npm run lint)
  • has no test errors (npm run test)
  • has no typecheck errors (npm run typecheck)
  • is from a uniquely-named feature branch and is up to date with the develop branch.
  • is descriptively named and links to an issue number, i.e. Fixes #123
  • meets the standards outlined in the accessibility guidelines

clairep94
clairep94 previously approved these changes Aug 10, 2026
Comment thread e2e/specs/signup.spec.ts Outdated
Comment on lines +190 to +192
// Make sure link in email works
const verificationLink = await getVerificationLink(email);
await page.goto(verificationLink);

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

take this out as it's covered in the first test on this page

@Geethegreat Geethegreat changed the title GSOC-E2E: Add unauth-to-auth sketch persistence test.- #15 GSOC-E2E: Add unauth-to-auth sketch persistence test. Aug 11, 2026
@clairep94
clairep94 merged commit 8b6aea3 into processing:develop Aug 17, 2026
2 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.

2 participants