-
Notifications
You must be signed in to change notification settings - Fork 9
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
Cypress E2E #1507
Cypress E2E #1507
Conversation
@@ -240,7 +240,9 @@ | |||
SITE_ID = 1 | |||
ACCOUNT_AUTHENTICATION_METHOD = "email" | |||
ACCOUNT_EMAIL_REQUIRED = True | |||
ACCOUNT_EMAIL_VERIFICATION = "mandatory" | |||
ACCOUNT_EMAIL_VERIFICATION = str( | |||
config("ACCOUNT_EMAIL_VERIFICATION", default="mandatory") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not making use of this for this PR, and I thought I'd just remove it, but decided this would be handy for later.
cy.get('button[type="submit"]').should('be.disabled'); | ||
}); | ||
|
||
it('Attempts to create an account', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This actually creates a test account. It seems important to test, but without having a way to automatically delete these (say, via a script run under make
or something similar), I've got mixed feelings about leaving this in.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, Not sure if cypress should be a dependency or dev-dependency though
This just adds a basic framework for initial Cypress E2E tests.
Despite a bit of experimentation, the resulting PR isn't much different than the original Cypress PR.
I pulled in a few map tests from that, as well.
It seems like we have some decisions to make for when exactly we run this, but I left a manual workflow action in here for the moment. If we want to duplicate this and have it triggered elsewhere, that's easy enough to change.
Note: there are also some github repo secrets to set before we consider merging this.