Skip to content

Skip Github flow on local for easier testing of user creation flows #56

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

ChrisPenner
Copy link
Contributor

@ChrisPenner ChrisPenner commented Apr 11, 2025

Overview

When running in the local deployment, going through the "login" flow will short-circuit going to github and will instead complete the auth flow using dummy github user info, which helps make testing the user creation flow a lot easier.

Implementation notes

The only difference is that it skips redirecting through Github, and doesn't call the github APIs to get the user data and instead uses constant dummy user info.

Test coverage

I added a new user creation transcript to ensure we don't break the user creation flow.

Loose Ends

@hojberg you could optionally wire up the "Login", "Register" and "Logout" buttons in the local UI if you like :)

@ChrisPenner ChrisPenner marked this pull request as ready for review April 11, 2025 18:12
@ChrisPenner ChrisPenner requested a review from hojberg April 11, 2025 18:12
@ChrisPenner ChrisPenner changed the base branch from main to bump-unison April 25, 2025 18:43
@ChrisPenner ChrisPenner requested a review from a team as a code owner April 25, 2025 18:43
@ChrisPenner ChrisPenner changed the base branch from bump-unison to main April 25, 2025 18:43
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