Skip to content

[Coroutines] Convert braintreeClient.getConfiguration to suspend function#1532

Merged
noguier merged 23 commits intomainfrom
coroutines-convert-BTClient.getConfiguration
Feb 23, 2026
Merged

[Coroutines] Convert braintreeClient.getConfiguration to suspend function#1532
noguier merged 23 commits intomainfrom
coroutines-convert-BTClient.getConfiguration

Conversation

@noguier
Copy link
Contributor

@noguier noguier commented Feb 18, 2026

Summary of changes

  • converts getConfiguration() to a suspend function
  • adds coroutine calls to CardClient, DataCollector, GooglePayClient, LocalPaymentClient, PayPalClient, PayPalMessagingView, EligiblePaymentsApi, ThreeDSecureClient, VenmoClient, VisaCheckoutClient
  • updates all relevant tests for classes listed above

Checklist

  • Added a changelog entry
  • Relevant test coverage
  • Tested and confirmed payment flows affected by this change are functioning as expected

Authors

@noguier

# Conflicts:
#	BraintreeCore/src/main/java/com/braintreepayments/api/core/BraintreeClient.kt
#	ShopperInsights/src/main/java/com/braintreepayments/api/shopperinsights/EligiblePaymentsApi.kt
# Conflicts:
#	BraintreeCore/src/main/java/com/braintreepayments/api/core/BraintreeClient.kt
#	BraintreeCore/src/test/java/com/braintreepayments/api/core/BraintreeClientUnitTest.kt
#	TestUtils/src/main/java/com/braintreepayments/api/testutils/MockkBraintreeClientBuilder.kt
@noguier noguier requested a review from a team as a code owner February 18, 2026 17:05
@github-actions
Copy link

github-actions bot commented Feb 18, 2026

Dependency Review

The following issues were found:
  • ✅ 0 vulnerable package(s)
See the Details below.

Scanned Files

  • settings.gradle

@noguier noguier merged commit be7e411 into main Feb 23, 2026
6 of 12 checks passed
@noguier noguier deleted the coroutines-convert-BTClient.getConfiguration branch February 23, 2026 16:00
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