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

chore(connect-another-device): Add acceptence criteria/test plan #214

Merged
merged 2 commits into from Feb 2, 2017

Conversation

shane-tomlinson
Copy link
Contributor

@shane-tomlinson shane-tomlinson commented Jan 12, 2017

@rfk - here ya go. Were you thinking along these lines?

@rfk
Copy link
Contributor

rfk commented Jan 23, 2017

@shane-tomlinson this looks great! I think it makes the intended permutations very clear. @ryanfeeley it'd be great to get you to sign off on this as well, and to check whether there are any missing behaviours that should be called out.

@davismtl remind me, did we discuss getting softvision to step through the testcases for this feature?

#### Signup in Firefox Desktop

##### Verify in the same Firefox Desktop.
When verification is complete:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Remove When verification is complete:

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ryanfeeley suggests that I add the images here.


#### Signup in Firefox Desktop

##### Verify in the same Firefox Desktop.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ryanfeeley suggests I change the form of the sub-heading to "When the user clicks the email verificaiton link which opens in Firefox for iOS:"

@davismtl
Copy link
Contributor

I did not coordinate with them for testing but it is probably a good idea to check with them.

@shane-tomlinson shane-tomlinson force-pushed the test-plan-connect-another-device branch 5 times, most recently from 4bdef7b to 28d368e Compare February 2, 2017 14:29
@shane-tomlinson shane-tomlinson merged commit 300693d into master Feb 2, 2017
@shane-tomlinson shane-tomlinson deleted the test-plan-connect-another-device branch February 2, 2017 16:42
philbooth pushed a commit that referenced this pull request Oct 17, 2017
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

5 participants