Navigate back to AddFirstPaymentMethod immediately after removing last PM

[amk-stripe]

Summary

Navigate back to AddFirstPaymentMethod immediately after removing last PM

Motivation

Walk the store PQ bug: https://docs.google.com/document/d/1jAte_o-0Ifl6242HB-okkpeoOn_te-dmepN3VGgn9ls/edit?tab=t.0#bookmark=id.dj6xrfb3vqpr

Testing

• Added tests
• Modified tests
• Manually verified

We already have a test that verifies that we navigate back to the AddFirstPaymentMethod screen: here. It didn't previously account for the fact that we were on SelectSavedPaymentMethods for a short time before navigating back to that screen, so there's no test updating to do for that logic.

Screenshots

Before:

removing.pms.-.before.mp4

After:

removing.pms.-.after.mp4

[github-actions]

Diffuse output:

OLD: identity-example-release-base.apk (signature: V1, V2)
NEW: identity-example-release-pr.apk (signature: V1, V2)

          │          compressed          │         uncompressed         
          ├───────────┬───────────┬──────┼───────────┬───────────┬──────
 APK      │ old       │ new       │ diff │ old       │ new       │ diff 
──────────┼───────────┼───────────┼──────┼───────────┼───────────┼──────
      dex │     2 MiB │     2 MiB │  0 B │   4.1 MiB │   4.1 MiB │  0 B 
     arsc │     1 MiB │     1 MiB │  0 B │     1 MiB │     1 MiB │  0 B 
 manifest │   2.3 KiB │   2.3 KiB │  0 B │     8 KiB │     8 KiB │  0 B 
      res │ 302.6 KiB │ 302.6 KiB │  0 B │ 456.7 KiB │ 456.7 KiB │  0 B 
   native │   6.2 MiB │   6.2 MiB │  0 B │  15.8 MiB │  15.8 MiB │  0 B 
    asset │   7.1 KiB │   7.1 KiB │  0 B │   6.9 KiB │   6.9 KiB │  0 B 
    other │  90.8 KiB │  90.8 KiB │ +5 B │   171 KiB │   171 KiB │  0 B 
──────────┼───────────┼───────────┼──────┼───────────┼───────────┼──────
    total │   9.6 MiB │   9.6 MiB │ +5 B │  21.5 MiB │  21.5 MiB │  0 B 

 DEX     │ old   │ new   │ diff      
─────────┼───────┼───────┼───────────
   files │     1 │     1 │ 0         
 strings │ 19975 │ 19975 │ 0 (+0 -0) 
   types │  6193 │  6193 │ 0 (+0 -0) 
 classes │  4985 │  4985 │ 0 (+0 -0) 
 methods │ 29820 │ 29820 │ 0 (+0 -0) 
  fields │ 17538 │ 17538 │ 0 (+0 -0) 

 ARSC    │ old  │ new  │ diff 
─────────┼──────┼──────┼──────
 configs │  164 │  164 │  0   
 entries │ 3624 │ 3624 │  0

APK

   compressed    │   uncompressed   │                        
──────────┬──────┼───────────┬──────┤                        
 size     │ diff │ size      │ diff │ path                   
──────────┼──────┼───────────┼──────┼────────────────────────
 28.6 KiB │ +8 B │  63.2 KiB │  0 B │ ∆ META-INF/CERT.SF     
 25.4 KiB │ -3 B │  63.1 KiB │  0 B │ ∆ META-INF/MANIFEST.MF 
──────────┼──────┼───────────┼──────┼────────────────────────
   54 KiB │ +5 B │ 126.3 KiB │  0 B │ (total)

[amk-stripe]

we no longer have embedded-specific logic in SavedPaymentMethodMutator, so we no longer need embedded specific tests either.

[jaynewstrom-stripe]

Does this work for vertical mode too?

[amk-stripe]

Does this work for vertical mode too?

No, I wasn't sure if we wanted to make this update for vertical mode or not. It would be really simple to make that change though. Happy to follow up with that!