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

Remove jest and add starter login tests #692

Merged
merged 2 commits into from
Jan 10, 2024

Conversation

NiallJoeMaher
Copy link
Contributor

✨ Codu Pull Request 💻

Codu Logo

Pull Request details:

Working on improving testing

  • Removes Jest in favour of Playwright because mocking things breaks my heart 😂
  • Add start of login test
  • Update ESLint to not cry because of the conflicts between jest/playwright

Any Breaking changes:

  • Unit tests with Jest no longer work (it's only two) and need to be updated and caught with e2e. Commented so I can use them as a reference and not forget them.

Copy link

vercel bot commented Jan 10, 2024

@NiallJoeMaher is attempting to deploy a commit to the Codú Team on Vercel.

A member of the Team first needs to authorize it.

@NiallJoeMaher NiallJoeMaher merged commit 0ea2e74 into codu-code:develop Jan 10, 2024
3 of 4 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