Skip to content

Commit

Permalink
s/applyAttestationParameters/assertAndApplyAttestationParameters
Browse files Browse the repository at this point in the history
  • Loading branch information
mats-stripe committed Jan 22, 2025
1 parent ef2ec32 commit 89bc90c
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ final class FinancialConnectionsAPIClient {

/// Applies attestation-related parameters to the given base parameters
/// In case of an assertion error, returns the unmodified base parameters
func applyAttestationParameters(
func assertAndApplyAttestationParameters(
to baseParameters: [String: Any]
) -> Future<[String: Any]> {
let promise = Promise<[String: Any]>()
Expand Down Expand Up @@ -930,7 +930,7 @@ extension FinancialConnectionsAPIClient: FinancialConnectionsAPI {
parameters["request_surface"] = requestSurface
parameters["session_id"] = sessionId
parameters["email_source"] = emailSource.rawValue
return applyAttestationParameters(to: parameters)
return assertAndApplyAttestationParameters(to: parameters)
.chained { [weak self] updatedParameters in
guard let self else {
return Promise(error: FinancialConnectionsSheetError.unknown(debugDescription: "FinancialConnectionsAPIClient was deallocated."))
Expand Down Expand Up @@ -1040,7 +1040,7 @@ extension FinancialConnectionsAPIClient: FinancialConnectionsAPI {
}

if useMobileEndpoints {
return applyAttestationParameters(to: parameters)
return assertAndApplyAttestationParameters(to: parameters)
.chained { [weak self] updatedParameters in
guard let self else {
return Promise(error: FinancialConnectionsSheetError.unknown(debugDescription: "FinancialConnectionsAPIClient was deallocated."))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ final class FinancialConnectionsAsyncAPIClient {

/// Applies attestation-related parameters to the given base parameters
/// In case of an assertion error, returns the unmodified base parameters
func applyAttestationParameters(to baseParameters: [String: Any]) async -> [String: Any] {
func assertAndApplyAttestationParameters(to baseParameters: [String: Any]) async -> [String: Any] {
do {
let attest = backingAPIClient.stripeAttest
let handle = try await attest.assert()
Expand Down Expand Up @@ -829,7 +829,7 @@ extension FinancialConnectionsAsyncAPIClient: FinancialConnectionsAsyncAPI {
parameters["request_surface"] = requestSurface
parameters["session_id"] = sessionId
parameters["email_source"] = emailSource.rawValue
let updatedParameters = await applyAttestationParameters(to: parameters)
let updatedParameters = await assertAndApplyAttestationParameters(to: parameters)
return try await post(endpoint: .mobileConsumerSessionLookup, parameters: updatedParameters)
} else {
parameters["client_secret"] = clientSecret
Expand Down Expand Up @@ -927,7 +927,7 @@ extension FinancialConnectionsAsyncAPIClient: FinancialConnectionsAsyncAPI {
}
}
if useMobileEndpoints {
let updatedParameters = await applyAttestationParameters(to: parameters)
let updatedParameters = await assertAndApplyAttestationParameters(to: parameters)
return try await post(endpoint: .mobileLinkAccountSignup, parameters: updatedParameters)
} else {
return try await post(endpoint: .linkAccountsSignUp, parameters: parameters)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,7 @@ class FinancialConnectionsAPIClientTests: XCTestCase {
]
let apiClient = FinancialConnectionsAPIClient(apiClient: mockApiClient)
apiClient
.applyAttestationParameters(to: baseParameters)
.assertAndApplyAttestationParameters(to: baseParameters)
.observe { result in
switch result {
case .success(let updatedParameters):
Expand Down

0 comments on commit 89bc90c

Please sign in to comment.