department-of-veterans-affairs / va-mobile-app

"If VA were a company, it would have a flagship mobile app."
https://department-of-veterans-affairs.github.io/va-mobile-app/
16 stars 2 forks source link

QA prep work for remaining RQ migrations #9040

Closed TKDickson closed 2 months ago

TKDickson commented 3 months ago

Auth and notifications have not yet been react query migrated. - #8253 and #7743

We're going to be doing that soon, which means the normal QA prep work (understanding what's in the changes, writing a test plan and getting review) should also happen.

This ticket is for that.

TKDickson commented 2 months ago

High-level test plan from looking at the file changes in #7743.

For all of these case categories, will need to cover success & failure variations.

Auth Login

Refreshing access token PN with app in foreground and background Biometrics opt in/out

Cerner "first time login" and "first time in session" help popup in Health screen Caching (across sessions, within sessions) Logout (including multiple users on one device) Onboarding carousel pops up (or not) "reset to first time login" developer tool fetching and applying maintenance windows Native handoffs during login (permissions for biometric and PN)

Still need to investigate and then add to plan above:

TKDickson commented 2 months ago

High-level test plan from looking at file changes for #8253 (notifications) Changing PN preferences Registering for PN Unregistering for push (device settings) Receiving PN & they work PN info on the develop screen PN when app in foreground and background (with and without deep linking)

Still need to investigate & add to plan:

TKDickson commented 2 months ago

(bumping up estimate because I've already spent 2 on this, and I have a bunch of work left: test plan work listed above + test writing + TPR incorporation)

TKDickson commented 2 months ago

Relevant prior tickets for push notifications: #5036, #4587, #1656, #2368, #7926. I also reviewed our current case bank in testrail for push notifications and was.... unimpressed. I'm going to write a number of new cases:

Those, plus a quick cleanup of the existing cases (one of them is the "everything all in one box" format from MVP release) will cover everything in my high level plan, plus cover the behavior currently documented only in those prior tickets listed here.

TKDickson commented 2 months ago

Moved auth prep work (which is going to be a lot) to #9162, including copying over notes from this ticket.

Rachael during TPR pointed out the SM deeplinking PN cases (link) which I didn't know about and could have saved me some time writing out cases. Recommended looking at those cases & seeing if there was anything I wanted to add.

I will also be moving those cases over to the PN folder, after a quick conversation with Rachael, because they make more sense there/hopefully the next person will find them there!

TKDickson commented 2 months ago

Have incorporated TPR (added a couple of the SM cases) and moved them to the PN folder in TestRail. Closing ticket as complete; I'll be ready to test the notifications migration when it's moved in.