Add set as default feature enabled and disabled state to CustomerState

[amk-stripe]

Summary

Add set as default feature enabled and disabled state to CustomerState

Motivation

I'm working on replacing the feature flag for the set as default feature with the customer session feature. jira

This is pre-work for that. This consolidates where we are reading the feature flag to be only within CustomerState and then as a next step we can replace the feature flag with the feature enabled/disabled value from the customer session.

Testing

• Added tests
• Modified tests
• Manually verified

No behavior changes -- all covered by existing tests

[amk-stripe]

I re-worked this function and also withDefaultPaymentMethodOrLastUsedPaymentMethodFirst.

Accessing the defaultPaymentMethodId is a bit more complicated now and the code became hard to read unless I adjusted a bit. The behavior is the same though

[jaynewstrom-stripe]

Is it worth breaking this code out into another object, testing the defaulting logic in isolation, and injecting that object into the loader for use?

[amk-stripe]

I updated a few of these tests to only assert on the fields relevant to the test, so that I wouldn't need to update the defaultPaymentMethodId field in as many places

[tianzhao-stripe]

I realized that we can remove

            customerId = "cus_1",
            ephemeralKeySecret = "ek_1",

from the calls of createElementsSessionCustomer b/c createElementsSessionCustomer has

        customerId: String = "cus_1",
        ephemeralKeySecret: String = "ek_1",

As the default values already

[amk-stripe]

I updated a this test and another one below this to only assert on the field relevant to the test, so that I wouldn't need to update the defaultPaymentMethodId field in as many places

[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.4 KiB │  90.4 KiB │ +5 B │ 170.7 KiB │ 170.7 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.5 KiB │ +6 B │ 63.1 KiB │  0 B │ ∆ META-INF/CERT.SF  
  1.2 KiB │ -1 B │  1.2 KiB │  0 B │ ∆ META-INF/CERT.RSA 
──────────┼──────┼──────────┼──────┼─────────────────────
 29.7 KiB │ +5 B │ 64.3 KiB │  0 B │ (total)

[tianzhao-stripe]

Is there a difference between customerDeferred.await() inside retrieveInitialPaymentSelection

and

retrieveInitialPaymentSelection(
                savedSelection = savedSelection,
                customer = customer.await(),
                isGooglePayReady = isGooglePayReady,
)

[tianzhao-stripe]

This is better than what I wrote