-
Notifications
You must be signed in to change notification settings - Fork 20
Jetpack: Add tests for critical Jetpack Connect flows #1000
Comments
I would prioritize automating these flows:
I'm not sure what flows exist currently that send users through flow 3 (adding a monthly plan) but that'd probably be next on my list. What do you think, @alisterscott? How many of these flows do you think we should automate tests for? |
Thanks for organizing this @rachelmcr
The ones you mentioned are good initial candidates.
I'm trying to work out whether this is actually any different than just via the pricing page, it doesn't appear to be, so I think we could not worry about these specific ones that have different origins but same entry points - what do you think? |
Agreed. I'm just double checking with the Jetpack team to make sure the logged-out pricing page behavior is expected, since the I'll get started on the first flow (from the "Try it free" button) for now. |
The logged-out behavior on the pricing page was unintentional and is fixed now, so let's start with these flows:
|
This one is implemented here #1007 |
I'm going to start "Install Jetpack" flow while logged out UPD: Implemented here: #1037 |
Implemented here: #1057 |
I'm going to close this placeholder and if there are any missing specific flows we can create individual items to work on |
In addition to tests for the specific flows in #777, we should make sure the critical Jetpack Connect flows are covered. We currently have tests for:
Jetpack Connect flows to consider (internal ref: PCYsg-dek-p2):
jetpack/connect/store/[monthly]
jetpack/connect/personal|premium|pro
jetpack/connect/personal|premium|pro/monthly
jetpack/connect/install
jetpack/connect/es
&jetpack/connect/install/es
jetpack/connect/?url=https://yourgroovydomain.com
(covered by existing tests) or variants of other flows with Site URL added e.g.jetpack/connect/pro/?url=https://yourgroovydomain.com
We should also consider how users enter these flows, e.g. from Jetpack.com:
The text was updated successfully, but these errors were encountered: