Skip to content

refactor(cypress): update BOA configs for manual payments' refunds and connector agnostic#7690

Merged
likhinbopanna merged 7 commits intomainfrom
boa-no3ds-refunds-cypress
Apr 8, 2025
Merged

refactor(cypress): update BOA configs for manual payments' refunds and connector agnostic#7690
likhinbopanna merged 7 commits intomainfrom
boa-no3ds-refunds-cypress

Conversation

@arindam-sahoo
Copy link
Contributor

@arindam-sahoo arindam-sahoo commented Apr 2, 2025

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

Refunds for Manually Captured No3DS Payments and Connector Agnostic NTID Test Cases in Cypress were failing for Bank of America Payment Processor.

  • The Refunds for Manually Captured No3DS Payments were staying in pending status instead of succeeded with error_message as null and on RSync, it yields errorInformation.message as "Request was processed successfully." with status still as pending.
  • The 00024-ConnectorAgnosticNTID was failing with the error message in the response as "No eligible connector was found for the current payment method configuration" because it did not support Connector Agnostic.

So, made the following changes:-

  • Mapped the status accordingly. Changed succeeded to pending for No3DS Manual Payment Refunds.
  • Excluded bankofamerica from the Connectors which are allowing Connector Agnostic.

Additional Changes

  • This PR modifies the API contract
  • This PR modifies the database schema
  • This PR modifies application configuration/environment variables

Motivation and Context

Bank of America Cypress Test Cases were failing.

How did you test it?

image

(Tested on Integ Environment)

Checklist

  • I formatted the code cargo +nightly fmt --all
  • I addressed lints thrown by cargo clippy
  • I reviewed the submitted code
  • I added unit tests for my changes where possible

@semanticdiff-com
Copy link

semanticdiff-com bot commented Apr 2, 2025

Review changes with  SemanticDiff

Changed Files
File Status
  cypress-tests/cypress/e2e/configs/Payment/Utils.js  6% smaller
  cypress-tests/cypress/e2e/configs/Payment/BankOfAmerica.js  0% smaller

@arindam-sahoo arindam-sahoo self-assigned this Apr 2, 2025
@arindam-sahoo arindam-sahoo added the A-CI-CD Area: Continuous Integration/Deployment label Apr 2, 2025
@arindam-sahoo arindam-sahoo added this to the April 2025 Release milestone Apr 2, 2025
@arindam-sahoo arindam-sahoo marked this pull request as ready for review April 2, 2025 09:58
@arindam-sahoo arindam-sahoo requested a review from a team as a code owner April 2, 2025 09:58
@Gnanasundari24 Gnanasundari24 enabled auto-merge April 7, 2025 11:29
@Gnanasundari24 Gnanasundari24 disabled auto-merge April 7, 2025 11:30
@likhinbopanna likhinbopanna enabled auto-merge April 8, 2025 08:43
@likhinbopanna likhinbopanna added this pull request to the merge queue Apr 8, 2025
Merged via the queue into main with commit 4a89567 Apr 8, 2025
17 of 23 checks passed
@likhinbopanna likhinbopanna deleted the boa-no3ds-refunds-cypress branch April 8, 2025 10:27
pixincreate added a commit that referenced this pull request Apr 10, 2025
…acilitapay-pix-pmt

* 'main' of github.com:juspay/hyperswitch: (21 commits)
  chore(version): 2025.04.10.0
  refactor(connector): [STRIPE] Remove sofort bank redirect from stripe (#7733)
  feat(connector): Add invoice number and email in AuthorizeDotNet connector (#7726)
  fix(router): fix retry_count and add validation for process_tracker (#7614)
  feat(payment_link): expose configurations for payment links (#7742)
  chore(version): 2025.04.09.0
  chore(postman): update Postman collection files
  feat(connector): [AIRWALLEX, ELAVON, NOVALNET, XENDIT] add in feature API (#7163)
  refactor: move merchant_key_store table to accounts schema (#7746)
  chore(postman): update `Stripe` response `status`, `error_code`, and `error_message` for deprecated `Sofort` (#7730)
  feat(connector): Add recovery support for recurly [v2] (#7497)
  refactor(cypress): update BOA configs for manual payments' refunds and connector agnostic (#7690)
  feat(router): Support `card` in `payment_method_subtype` [V2] (#7662)
  feat: Add open API reference for Intelligent router (#7727)
  ci(cypress-ci): remove wise payout from running in github ci (#7756)
  feat(authentication): create authentications to fallback to ApiKeyAuth if AdminApiAuth fails (#7744)
  chore(version): 2025.04.08.0
  feat(core): added force_3ds_challenge for decoupled txns (#7484)
  chore(version): 2025.04.07.0
  chore(postman): update Postman collection files
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-CI-CD Area: Continuous Integration/Deployment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor(cypress): BOA testcases are failing due to failed assertions of Refunds and Connector Agnostic

3 participants