Skip to content

Update PaymentSheet to use 'TEST' instead of 'TESTMODE' for test flows#10761

Merged
lemuel-stripe merged 8 commits intomasterfrom
lemuel-update-payment-sheet-testmode-callout
May 5, 2025
Merged

Update PaymentSheet to use 'TEST' instead of 'TESTMODE' for test flows#10761
lemuel-stripe merged 8 commits intomasterfrom
lemuel-update-payment-sheet-testmode-callout

Conversation

@lemuel-stripe
Copy link
Contributor

@lemuel-stripe lemuel-stripe commented May 1, 2025

Summary

Changing PaymentSheets to use the badge label TEST instead of TESTMODE when in a test flow.

Motivation

As part of the upcoming sandboxes GA at sessions, we want to eliminate the usage of the term "test mode" to ensure developers are aware of sandboxes and moving their testing to sandboxes.

Testing

Screenshots

Before After
Screenshot 2025-05-01 at 4 29 01 PM Screenshot 2025-05-01 at 4 16 55 PM

Changelog

PaymentSheet

  • [FIXED]10761 Updated the "TEST MODE" badge to say "TEST", due to the upcoming release of Sandboxes.

@lemuel-stripe lemuel-stripe changed the title updated PaymentSheet to use 'TEST' instead of 'TESTMODE' for test flows Update PaymentSheet to use 'TEST' instead of 'TESTMODE' for test flows May 1, 2025
@github-actions
Copy link
Contributor

github-actions bot commented May 1, 2025

Diffuse output:

OLD: paymentsheet-example-release-master.apk (signature: V1, V2)
NEW: paymentsheet-example-release-pr.apk (signature: V1, V2)

          │           compressed           │         uncompressed          
          ├───────────┬───────────┬────────┼───────────┬───────────┬───────
 APK      │ old       │ new       │ diff   │ old       │ new       │ diff  
──────────┼───────────┼───────────┼────────┼───────────┼───────────┼───────
      dex │   4.3 MiB │   4.3 MiB │ -105 B │   9.5 MiB │   9.5 MiB │ -16 B 
     arsc │   2.5 MiB │   2.5 MiB │   -4 B │   2.5 MiB │   2.5 MiB │  -4 B 
 manifest │   5.6 KiB │   5.6 KiB │    0 B │  28.7 KiB │  28.7 KiB │   0 B 
      res │ 911.4 KiB │ 911.4 KiB │    0 B │   1.4 MiB │   1.4 MiB │   0 B 
   native │   3.5 MiB │   3.5 MiB │    0 B │   8.5 MiB │   8.5 MiB │   0 B 
    asset │   1.6 MiB │   1.6 MiB │   +1 B │   1.6 MiB │   1.6 MiB │  +1 B 
    other │ 197.1 KiB │ 197.1 KiB │   -9 B │ 372.2 KiB │ 372.2 KiB │   0 B 
──────────┼───────────┼───────────┼────────┼───────────┼───────────┼───────
    total │    13 MiB │    13 MiB │ -117 B │  23.9 MiB │  23.9 MiB │ -19 B 

         │         raw          │           unique           
         ├───────┬───────┬──────┼───────┬───────┬────────────
 DEX     │ old   │ new   │ diff │ old   │ new   │ diff       
─────────┼───────┼───────┼──────┼───────┼───────┼────────────
   files │     2 │     2 │    0 │       │       │            
 strings │ 46058 │ 46057 │   -1 │ 45472 │ 45471 │ -1 (+0 -1) 
   types │ 16185 │ 16185 │    0 │ 15951 │ 15951 │  0 (+0 -0) 
 classes │ 13469 │ 13469 │    0 │ 13469 │ 13469 │  0 (+0 -0) 
 methods │ 66218 │ 66218 │    0 │ 65901 │ 65901 │  0 (+0 -0) 
  fields │ 43846 │ 43846 │    0 │ 43725 │ 43725 │  0 (+0 -0) 

 ARSC    │ old  │ new  │ diff 
─────────┼──────┼──────┼──────
 configs │  242 │  242 │  0   
 entries │ 6308 │ 6308 │  0
APK
    compressed     │   uncompressed    │                               
──────────┬────────┼───────────┬───────┤                               
 size     │ diff   │ size      │ diff  │ path                          
──────────┼────────┼───────────┼───────┼───────────────────────────────
  4.3 MiB │ -105 B │   9.4 MiB │ -16 B │ ∆ classes.dex                 
   51 KiB │   -7 B │ 120.8 KiB │   0 B │ ∆ META-INF/MANIFEST.MF        
  2.5 MiB │   -4 B │   2.5 MiB │  -4 B │ ∆ resources.arsc              
 54.4 KiB │   -2 B │ 120.9 KiB │   0 B │ ∆ META-INF/CERT.SF            
  8.2 KiB │   +1 B │   8.1 KiB │  +1 B │ ∆ assets/dexopt/baseline.prof 
──────────┼────────┼───────────┼───────┼───────────────────────────────
  6.9 MiB │ -117 B │  12.2 MiB │ -19 B │ (total)
DEX
STRINGS:

   old   │ new   │ diff       
  ───────┼───────┼────────────
   45472 │ 45471 │ -1 (+0 -1) 
  
  - TEST MODE

@lemuel-stripe lemuel-stripe marked this pull request as ready for review May 2, 2025 17:41
@lemuel-stripe lemuel-stripe requested review from a team as code owners May 2, 2025 17:41
@lemuel-stripe
Copy link
Contributor Author

r? @stripe/android-sdk-reviewers

@lemuel-stripe lemuel-stripe requested a review from toluo-stripe May 2, 2025 22:14
@lemuel-stripe lemuel-stripe force-pushed the lemuel-update-payment-sheet-testmode-callout branch from 7a58ff5 to 20c91aa Compare May 5, 2025 14:22
@lemuel-stripe lemuel-stripe merged commit 0a1f400 into master May 5, 2025
16 checks passed
@lemuel-stripe lemuel-stripe deleted the lemuel-update-payment-sheet-testmode-callout branch May 5, 2025 15:01
jaynewstrom-stripe pushed a commit that referenced this pull request May 5, 2025
#10761)

* updated PaymentSheet to use 'TEST' instead of 'TESTMODE' for test flows

* re-recorded screenshot tests

* updated CHANGELOG because this is a large visual change

* updated payment sheet example screenshots

* Revert "re-recorded screenshot tests"

This reverts commit 91ba17e.

* updated screenshot tests for only relevant changes

* updated screenshots to latest build from build binary
jaynewstrom-stripe pushed a commit that referenced this pull request May 5, 2025
#10761)

* updated PaymentSheet to use 'TEST' instead of 'TESTMODE' for test flows

* re-recorded screenshot tests

* updated CHANGELOG because this is a large visual change

* updated payment sheet example screenshots

* Revert "re-recorded screenshot tests"

This reverts commit 91ba17e.

* updated screenshot tests for only relevant changes

* updated screenshots to latest build from build binary
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants