Add support for SOFORT PaymentMethod

ANDROID-510
stripe · Apr 15, 2020 · 81babf8 · 81babf8
1 parent a3434a0
commit 81babf8
Show file tree

Hide file tree

Showing 3 changed files with 44 additions and 6 deletions.
diff --git a/stripe/src/main/java/com/stripe/android/model/PaymentMethod.kt b/stripe/src/main/java/com/stripe/android/model/PaymentMethod.kt
@@ -109,7 +109,9 @@ data class PaymentMethod internal constructor(
 
     @JvmField val auBecsDebit: AuBecsDebit? = null,
 
-    @JvmField val bacsDebit: BacsDebit? = null
+    @JvmField val bacsDebit: BacsDebit? = null,
+
+    @JvmField val sofort: Sofort? = null
 ) : StripeModel {
 
     @Parcelize
@@ -119,11 +121,12 @@ data class PaymentMethod internal constructor(
     ) : Parcelable {
         Card("card"),
         CardPresent("card_present"),
-        Fpx("fpx", false),
+        Fpx("fpx", isReusable = false),
         Ideal("ideal"),
         SepaDebit("sepa_debit"),
         AuBecsDebit("au_becs_debit"),
-        BacsDebit("bacs_debit");
+        BacsDebit("bacs_debit"),
+        Sofort("sofort", isReusable = false);
 
         override fun toString(): String {
             return code
@@ -152,6 +155,7 @@ data class PaymentMethod internal constructor(
         private var sepaDebit: SepaDebit? = null
         private var auBecsDebit: AuBecsDebit? = null
         private var bacsDebit: BacsDebit? = null
+        private var sofort: Sofort? = null
 
         fun setId(id: String?): Builder = apply {
             this.id = id
@@ -209,6 +213,10 @@ data class PaymentMethod internal constructor(
             this.bacsDebit = bacsDebit
         }
 
+        fun setSofort(sofort: Sofort?): Builder = apply {
+            this.sofort = sofort
+        }
+
         override fun build(): PaymentMethod {
             return PaymentMethod(
                 id = id,
@@ -224,7 +232,8 @@ data class PaymentMethod internal constructor(
                 ideal = ideal,
                 sepaDebit = sepaDebit,
                 auBecsDebit = auBecsDebit,
-                bacsDebit = bacsDebit
+                bacsDebit = bacsDebit,
+                sofort = sofort
             )
         }
     }
@@ -644,6 +653,11 @@ data class PaymentMethod internal constructor(
         @JvmField val sortCode: String?
     ) : StripeModel
 
+    @Parcelize
+    data class Sofort internal constructor(
+        @JvmField val country: String?
+    ) : StripeModel
+
     companion object {
         @JvmStatic
         fun fromJson(paymentMethod: JSONObject?): PaymentMethod? {

diff --git a/stripe/src/main/java/com/stripe/android/model/parsers/PaymentMethodJsonParser.kt b/stripe/src/main/java/com/stripe/android/model/parsers/PaymentMethodJsonParser.kt
@@ -60,6 +60,12 @@ internal class PaymentMethodJsonParser : ModelJsonParser<PaymentMethod> {
                         BacsDebitJsonParser().parse(it)
                     }
                 )
+            PaymentMethod.Type.Sofort ->
+                builder.setSofort(
+                    json.optJSONObject(type.code)?.let {
+                        SofortJsonParser().parse(it)
+                    }
+                )
         }
 
         return builder.build()
@@ -256,6 +262,18 @@ internal class PaymentMethodJsonParser : ModelJsonParser<PaymentMethod> {
         }
     }
 
+    internal class SofortJsonParser : ModelJsonParser<PaymentMethod.Sofort> {
+        override fun parse(json: JSONObject): PaymentMethod.Sofort {
+            return PaymentMethod.Sofort(
+                country = StripeJsonUtils.optString(json, FIELD_COUNTRY)
+            )
+        }
+
+        private companion object {
+            private const val FIELD_COUNTRY = "country"
+        }
+    }
+
     private companion object {
         private const val FIELD_ID = "id"
         private const val FIELD_BILLING_DETAILS = "billing_details"

diff --git a/stripe/src/test/java/com/stripe/android/PaymentMethodEndToEndTest.kt b/stripe/src/test/java/com/stripe/android/PaymentMethodEndToEndTest.kt
@@ -38,7 +38,13 @@ class PaymentMethodEndToEndTest {
         val paymentMethod =
             Stripe(context, ApiKeyFixtures.SOFORT_PUBLISHABLE_KEY)
                 .createPaymentMethodSynchronous(params)
-        assertThat(paymentMethod)
-            .isNotNull()
+        assertThat(paymentMethod?.type)
+            .isEqualTo(PaymentMethod.Type.Sofort)
+        assertThat(paymentMethod?.sofort)
+            .isEqualTo(
+                PaymentMethod.Sofort(
+                    country = "DE"
+                )
+            )
     }
 }