Move CustomerState.Permissions to PaymentMethodsMetadata.CustomerMetadata

[tianzhao-stripe]

Summary

Moved permissions from customerState to PaymentMethodMetadata

Fixed tests that broke.
Moved tests from CustomerStateTests to CustomerMetadataTests

Motivation

Moving immutable parts of customerState to PaymentMethodMetadata.

During defaultPaymentMethods, to move immutable parts of customerState to PaymentMethodMetadata which is immutable.

That work was descoped for defaultPaymentMethods
https://jira.corp.stripe.com/browse/MOBILESDK-3324

Testing

• Added tests
• Modified tests
• Manually verified

Screenshots

N.A.

Changelog

N.A.

[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.9 KiB │ 302.9 KiB │  0 B │   457 KiB │   457 KiB │  0 B 
   native │   6.2 MiB │   6.2 MiB │  0 B │  15.8 MiB │  15.8 MiB │  0 B 
    asset │   7.3 KiB │   7.3 KiB │  0 B │   7.1 KiB │   7.1 KiB │  0 B 
    other │  95.1 KiB │  95.1 KiB │  0 B │ 182.4 KiB │ 182.4 KiB │  0 B 
──────────┼───────────┼───────────┼──────┼───────────┼───────────┼──────
    total │   9.6 MiB │   9.6 MiB │  0 B │  21.6 MiB │  21.6 MiB │  0 B 

 DEX     │ old   │ new   │ diff      
─────────┼───────┼───────┼───────────
   files │     1 │     1 │ 0         
 strings │ 20142 │ 20142 │ 0 (+0 -0) 
   types │  6235 │  6235 │ 0 (+0 -0) 
 classes │  5026 │  5026 │ 0 (+0 -0) 
 methods │ 30047 │ 30047 │ 0 (+0 -0) 
  fields │ 17385 │ 17385 │ 0 (+0 -0) 

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

APK

   compressed    │   uncompressed   │                        
──────────┬──────┼───────────┬──────┤                        
 size     │ diff │ size      │ diff │ path                   
──────────┼──────┼───────────┼──────┼────────────────────────
  1.2 KiB │ -4 B │   1.2 KiB │  0 B │ ∆ META-INF/CERT.RSA    
 25.7 KiB │ +3 B │    64 KiB │  0 B │ ∆ META-INF/MANIFEST.MF 
   29 KiB │ +1 B │    64 KiB │  0 B │ ∆ META-INF/CERT.SF     
──────────┼──────┼───────────┼──────┼────────────────────────
 55.8 KiB │  0 B │ 129.3 KiB │  0 B │ (total)

[amk-stripe]

should we have default values for these permissions defined somewhere instead of hardcoding false here?

[tianzhao-stripe]

For SavedPaymentMethodMutator, canRemoveDuplicates is true if using customerSessions and false if using legacy keys. I will check that here by checking if customerSessionClientSecret is null.

[tianzhao-stripe]

For this case, I'm going to set it based on whether currentCustomer.customerSessionClientSecret is null

For paymentSheet, canRemoveDuplicates is true for customer session and false for legacy ephemeral key.

When I port those over I'll add an exception like already exists on line 145.