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

Add playwright testing library + intial integration tests for the Join Form #58

Merged
merged 14 commits into from
Aug 1, 2024

Conversation

WillNilges
Copy link
Collaborator

@WillNilges WillNilges commented Jul 27, 2024

  • Set up playwright
  • Add exhaustive Join Form tests
  • Configure GitHub action to point at dev instance of MeshDB
  • (optional) Nuke dev instance daily to manage cruft
  • Write docs

@WillNilges WillNilges force-pushed the wdn/testing branch 5 times, most recently from c243721 to d8a391b Compare July 29, 2024 03:05
@WillNilges WillNilges changed the title Add playwright testing library + integration tests for the Join Form Add playwright testing library + intial integration tests for the Join Form Jul 31, 2024
@WillNilges
Copy link
Collaborator Author

One day, when I grow up, I want to make these tests run on meshdb repo as well <3

Oh and also k8s-infra because fuck you :)

@Andrew-Dickinson
Copy link
Member

These are a great start on testing, but having PRs on this repo connected to the beta endpoint is not ideal. In theory we'd really like to have unit tests that run on PRs here and just confirm that the generated HTTP request is correct. Then the tests from this PR would be used after PRs are merged, at deploy time to validate that beta is still working in order to gate prod deployments

@WillNilges WillNilges force-pushed the wdn/testing branch 2 times, most recently from ee38881 to 7550f9e Compare August 1, 2024 00:24
@WillNilges
Copy link
Collaborator Author

TODO: Check S3 for submission.

@WillNilges WillNilges mentioned this pull request Aug 1, 2024
3 tasks
@WillNilges WillNilges merged commit 0a5d662 into main Aug 1, 2024
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.

2 participants