Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for EPS PaymentMethod #2395

Merged
merged 5 commits into from
Apr 17, 2020
Merged

Add support for EPS PaymentMethod #2395

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
93ccb38
Add support for Giropay PaymentMethod
smaskell-stripe Apr 16, 2020
0ab3b8d
Fix test
smaskell-stripe Apr 16, 2020
6bada24
Add support for EPS PaymentMethod
smaskell-stripe Apr 16, 2020
03e2ab2
EPS -> Eps
smaskell-stripe Apr 17, 2020
623d68f
Merge branch 'master' into smaskell/ANDROID-516
smaskell-stripe Apr 17, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion stripe/src/main/java/com/stripe/android/model/PaymentMethod.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,8 @@ data class PaymentMethod internal constructor(
Sofort("sofort", isReusable = false),
P24("p24", isReusable = false),
Bancontact("bancontact", isReusable = false),
Giropay("giropay", isReusable = false);
Giropay("giropay", isReusable = false),
Eps("eps", isReusable = false);

override fun toString(): String {
return code
Expand Down
18 changes: 16 additions & 2 deletions stripe/src/main/java/com/stripe/android/model/PaymentMethodCreateParams.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -148,7 +148,7 @@ data class PaymentMethodCreateParams internal constructor(
Type.AuBecsDebit -> auBecsDebit?.toParamMap()
Type.BacsDebit -> bacsDebit?.toParamMap()
Type.Sofort -> sofort?.toParamMap()
Type.P24, Type.Bancontact, Type.Giropay -> null
Type.P24, Type.Bancontact, Type.Giropay, Type.Eps -> null
}.takeUnless { it.isNullOrEmpty() }?.let {
mapOf(type.code to it)
}.orEmpty()
Expand All @@ -164,7 +164,8 @@ data class PaymentMethodCreateParams internal constructor(
Sofort("sofort"),
P24("p24"),
Bancontact("bancontact"),
Giropay("giropay")
Giropay("giropay"),
Eps("eps")
}

@Parcelize
Expand Down Expand Up @@ -525,6 +526,19 @@ data class PaymentMethodCreateParams internal constructor(
)
}

@JvmStatic
@JvmOverloads
internal fun createEps(
billingDetails: PaymentMethod.BillingDetails,
metadata: Map<String, String>? = null
): PaymentMethodCreateParams {
return PaymentMethodCreateParams(
type = Type.Eps,
billingDetails = billingDetails,
metadata = metadata
)
}

/**
* @param googlePayPaymentData a [JSONObject] derived from Google Pay's
* [PaymentData#toJson()](https://developers.google.com/pay/api/android/reference/client#tojson)
Expand Down
5 changes: 4 additions & 1 deletion stripe/src/main/java/com/stripe/android/model/parsers/PaymentMethodJsonParser.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,10 @@ internal class PaymentMethodJsonParser : ModelJsonParser<PaymentMethod> {
SofortJsonParser().parse(it)
}
)
PaymentMethod.Type.P24, PaymentMethod.Type.Bancontact, PaymentMethod.Type.Giropay -> {
PaymentMethod.Type.P24,
PaymentMethod.Type.Bancontact,
PaymentMethod.Type.Giropay,
PaymentMethod.Type.Eps -> {
// no-op
}
}
Expand Down
1 change: 1 addition & 0 deletions stripe/src/test/java/com/stripe/android/ApiKeyFixtures.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,4 +14,5 @@ internal object ApiKeyFixtures {
const val P24_PUBLISHABLE_KEY = "pk_test_vOo1umqsYxSrP5UXfOeL3ecm"
const val BANCONTACT_PUBLISHABLE_KEY = "pk_test_vOo1umqsYxSrP5UXfOeL3ecm"
const val GIROPAY_PUBLISHABLE_KEY = "pk_test_vOo1umqsYxSrP5UXfOeL3ecm"
const val EPS_PUBLISHABLE_KEY = "pk_test_vOo1umqsYxSrP5UXfOeL3ecm"
}
23 changes: 23 additions & 0 deletions stripe/src/test/java/com/stripe/android/PaymentMethodEndToEndTest.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -106,4 +106,27 @@ class PaymentMethodEndToEndTest {
.createPaymentMethodSynchronous(params)
}
}

@Test
fun createPaymentMethod_withEps_shouldCreateObject() {
val params = PaymentMethodCreateParamsFixtures.EPS
val paymentMethod =
Stripe(context, ApiKeyFixtures.EPS_PUBLISHABLE_KEY)
.createPaymentMethodSynchronous(params)
assertThat(paymentMethod?.type)
.isEqualTo(PaymentMethod.Type.Eps)
}

@Test
fun createPaymentMethod_withEps_missingName_shouldFail() {
val params = PaymentMethodCreateParams.createEps(
billingDetails = PaymentMethodCreateParamsFixtures.BILLING_DETAILS.copy(name = null)
)
assertFailsWith<InvalidRequestException>(
"A name is required to create a EPS payment method"
) {
Stripe(context, ApiKeyFixtures.EPS_PUBLISHABLE_KEY)
.createPaymentMethodSynchronous(params)
}
}
}
4 changes: 4 additions & 0 deletions stripe/src/test/java/com/stripe/android/model/PaymentMethodCreateParamsFixtures.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,10 @@ internal object PaymentMethodCreateParamsFixtures {
billingDetails = BILLING_DETAILS
)

internal val EPS = PaymentMethodCreateParams.createEps(
billingDetails = BILLING_DETAILS
)

@JvmStatic
fun createWith(metadata: Map<String, String>): PaymentMethodCreateParams {
return PaymentMethodCreateParams.create(
Expand Down