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 UI tests for users who sign up through /join/:section_code #38225

Merged
merged 8 commits into from
Jan 5, 2021

Conversation

drew-beckmen
Copy link
Contributor

@drew-beckmen drew-beckmen commented Dec 14, 2020

This PR includes two UI tests to validate the behavior of the sign up form users hit at /join/:section_code when they are not yet signed in/registered users. Our lack of test coverage in this area was highlighted by a bug I introduced last week (details in the Slack thread). These tests allow us to verify the sign up workflow for users who submit both valid and invalid inputs into the sign up form.

Testing story

Ran UI tests locally.

Links

Reviewer Checklist:

  • Tests provide adequate coverage
  • Privacy and Security impacts have been assessed
  • Code is well-commented
  • New features are translatable or updates will not break translations
  • Relevant documentation has been added or updated
  • User impact is well-understood and desirable
  • Pull Request is labeled appropriately
  • Follow-up work items (including potential tech debt) are tracked and linked

@drew-beckmen drew-beckmen marked this pull request as draft December 14, 2020 03:16
@drew-beckmen drew-beckmen requested a review from a team December 14, 2020 03:16
@drew-beckmen drew-beckmen marked this pull request as ready for review December 14, 2020 03:16
And I type "#{password}" into "#user_password"
And I type "#{password}" into "#user_password_confirmation"
And I select the "#{age}" option in dropdown "user_age"
And I click selector ".btn.btn-primary" once I see it
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@drew-beckmen I would suggest trying one of the steps with the "to load a new page" clause Otherwise, it can lead to problems if the next step runs before the navigation happens.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @davidsbailey!

Copy link
Contributor

@jmkulwik jmkulwik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for adding these tests! This looks good to me (assuming Dave's comment gets addressed).

Copy link
Contributor

@jamescodeorg jamescodeorg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for filling in this gap in the testing!

Copy link
Contributor

@jamescodeorg jamescodeorg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add a link to the Jira ticket in the PR description? Thanks!

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.

None yet

4 participants