feat: npm script that runs cypress tests that don't require auth #2616
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
contolini
force-pushed
the
2604-non-auth-tests
branch
from
0c82c44 to
f5c1f41
Compare
billhimmelsbach
approved these changes
Oct 6, 2025
Contributor
billhimmelsbach
left a comment
billhimmelsbach
left a comment
There was a problem hiding this comment.
This runs great! Tried running it against dev/prod, things looks good, and things look good in the GitHub action logs.
If you wanted my take on "non-auth", I would probably just go with the longer but maybe clearer "unauthenticated"? Totally up to you though and it works great as is. 🚀
Member
Author
|
@billhimmelsbach I am very grateful that there are native English speakers such as yourself on my team |
…entication To run many of our tests, we have to pass credentials to log into keycloak. This prevents us from running our tests publicly. By tagging some of our tests as not requiring authentication, we can run those publicly on pull requests and reserve auth tests for private execution. See #2604
contolini
force-pushed
the
2604-non-auth-tests
branch
from
f5c1f41 to
8177324
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There's a lot of automatic linting fixes, view the diff with whitespace ignored.
Adds an
@auth-requiredtag to cypress tests that require keycloak authentication. Non-auth tests can be run utilizing cypress' grep package by passing--env grepTags='-@auth-required'which is saved as a script calledyarn test-non-auth.I also created a github workflow to run the tests on pull requests.
Fixes #2604
Changes
Testing
Notes
non-authis kind of a weird phrase.