Skip to content
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

[CORL-3221] Part 1: Introduce full integration testing around auth and signup flows #4747

Draft
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

nick-funk
Copy link
Contributor

@nick-funk nick-funk commented Feb 19, 2025

What does this PR do?

  • Introduces in-memory Redis and Mongo support for the testing environments
  • Creates a bunch of mocks and helpers to support testing full request handlers
  • Adds tests for the signup and auth flows (passport)
    • TODO: working on sso profiles tests now that signup flow has been mocked and tested

These changes will impact:

  • commenters
  • moderators
  • admins
  • developers

What changes to the GraphQL/Database Schema does this PR introduce?

None

Does this PR introduce any new environment variables or feature flags?

No

If any indexes were added, were they added to INDEXES.md?

No

How do I test this PR?

  • Run the server tests

Were any tests migrated to React Testing Library?

No

How do we deploy this PR?

  • Merge into develop

@nick-funk nick-funk requested review from kabeaty and tessalt February 19, 2025 23:57
Copy link

netlify bot commented Feb 19, 2025

Deploy Preview for gallant-galileo-14878c canceled.

Name Link
🔨 Latest commit b28ed82
🔍 Latest deploy log https://app.netlify.com/sites/gallant-galileo-14878c/deploys/67b66fdd926fed0008a4e25c

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