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

Write e2e tests for onboarding - Closes #589 #629

Merged
merged 1 commit into from
Mar 29, 2018

Conversation

ginacontrino
Copy link
Contributor

@ginacontrino ginacontrino commented Mar 28, 2018

What was the problem?

We had no e2e tests for the onboarding

How did I fix it?

Added the tests

How to test it?

Read through the tests & make sure every scenario is covered + run them

Review checklist

  • The PR solves Write tests for onboarding #589
  • All new code is covered with unit tests
  • All new features are covered with e2e tests
  • All new code follows best practices

@ginacontrino ginacontrino self-assigned this Mar 28, 2018
@ginacontrino ginacontrino force-pushed the 589-onboarding-tests branch from b353c65 to bf1b496 Compare March 28, 2018 12:10
@ginacontrino ginacontrino force-pushed the 589-onboarding-tests branch from bf1b496 to 65ab13f Compare March 28, 2018 13:37
@ginacontrino ginacontrino changed the title Write tests for onboarding - Closes #589 Write e2e tests for onboarding - Closes #589 Mar 28, 2018
@ginacontrino ginacontrino requested a review from slaweet March 28, 2018 13:37
@ginacontrino ginacontrino force-pushed the 589-onboarding-tests branch from 65ab13f to 047453e Compare March 28, 2018 15:57
@ginacontrino ginacontrino force-pushed the 589-onboarding-tests branch from 047453e to 6b5700e Compare March 28, 2018 16:22
Copy link
Contributor

@slaweet slaweet left a comment

Choose a reason for hiding this comment

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

Good job 👍

@ginacontrino ginacontrino merged commit da83bac into 0.4.0 Mar 29, 2018
@ginacontrino ginacontrino deleted the 589-onboarding-tests branch March 29, 2018 08:44
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