Skip to content

ci(cypress): add assertions for requires_cvv#7296

Merged
Gnanasundari24 merged 1 commit intomainfrom
requires_cvv
Feb 19, 2025
Merged

ci(cypress): add assertions for requires_cvv#7296
Gnanasundari24 merged 1 commit intomainfrom
requires_cvv

Conversation

@likhinbopanna
Copy link
Contributor

@likhinbopanna likhinbopanna commented Feb 18, 2025

Type of Change

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

Description

Add assertions to verify the requires_cvv field in the customer payment method list.

Additional Changes

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

Motivation and Context

Test cases enhancement

How did you test it?

image
  • Cybersource
image

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

@likhinbopanna likhinbopanna added A-CI-CD Area: Continuous Integration/Deployment S-waiting-on-review Status: This PR has been implemented and needs to be reviewed labels Feb 18, 2025
@likhinbopanna likhinbopanna added this to the February 2025 Release milestone Feb 18, 2025
@likhinbopanna likhinbopanna self-assigned this Feb 18, 2025
@likhinbopanna likhinbopanna requested a review from a team as a code owner February 18, 2025 10:32
@semanticdiff-com
Copy link

semanticdiff-com bot commented Feb 18, 2025

Review changes with  SemanticDiff

Changed Files
File Status
  cypress-tests/cypress/support/commands.js  0% smaller

@likhinbopanna likhinbopanna changed the title ci(cypress) : add assertions for requires_cvv ci(cypress): add assertions for requires_cvv Feb 19, 2025
@Gnanasundari24 Gnanasundari24 added this pull request to the merge queue Feb 19, 2025
Merged via the queue into main with commit 8457d3c Feb 19, 2025
26 of 32 checks passed
@Gnanasundari24 Gnanasundari24 deleted the requires_cvv branch February 19, 2025 15:34
pixincreate added a commit that referenced this pull request Feb 21, 2025
* 'main' of github.com:juspay/hyperswitch:
  fix(connector): [DATATRANS] Fix Force Sync Flow (#7331)
  chore(version): 2025.02.21.0
  fix(connector): [DATATRANS] Add new payment status (#7327)
  feat(router): [Xendit] add support for split payments (#7143)
  fix(payment_methods_v2): update fingerprint implementation in v2 (#7270)
  feat(router): Add Payments - List endpoint for v2 (#7191)
  feat(core): add support for confirmation flow for click to pay (#6982)
  chore(version): 2025.02.20.0
  fix(connector): [SCRIPT] Update template generating script and updated connector doc (#7301)
  ci(cypress): add assertions for `requires_cvv` (#7296)
  refactor(utils): match string for state with SDK's naming convention (#7300)
  feat(core): add hypersense integration api  (#7218)
  chore(version): 2025.02.19.0
  feat(connector): [Moneris] Add payments flow (#7249)
  feat(core): api ,domain and diesel model changes for extended authorization (#6607)
  feat(payments): [Payment links]  Add configs for payment link (#7288)
pixincreate added a commit that referenced this pull request Feb 22, 2025
…tusedPM

* 'main' of github.com:juspay/hyperswitch: (22 commits)
  feat(connector): add Samsung pay mandate support for Cybersource (#7298)
  chore(masking): add peek_mut to PeekInterface (#7281)
  feat(samsung_pay): collect customer address details form Samsung Pay based on business profile config and connector required fields (#7320)
  ci: update creds (#7345)
  feat(router): add `merchant_configuration_id` in netcetera metadata and make other merchant configurations optional (#7347)
  fix(routing): Fixed 5xx error logs in dynamic routing metrics (#7335)
  fix(connector): [DATATRANS] Fix Force Sync Flow (#7331)
  chore(version): 2025.02.21.0
  fix(connector): [DATATRANS] Add new payment status (#7327)
  feat(router): [Xendit] add support for split payments (#7143)
  fix(payment_methods_v2): update fingerprint implementation in v2 (#7270)
  feat(router): Add Payments - List endpoint for v2 (#7191)
  feat(core): add support for confirmation flow for click to pay (#6982)
  chore(version): 2025.02.20.0
  fix(connector): [SCRIPT] Update template generating script and updated connector doc (#7301)
  ci(cypress): add assertions for `requires_cvv` (#7296)
  refactor(utils): match string for state with SDK's naming convention (#7300)
  feat(core): add hypersense integration api  (#7218)
  chore(version): 2025.02.19.0
  feat(connector): [Moneris] Add payments flow (#7249)
  ...
@SanchithHegde SanchithHegde removed the S-waiting-on-review Status: This PR has been implemented and needs to be reviewed label Feb 23, 2025
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.

set requires_cvv to false when either connector_mandate_details or network_transaction_id is present during MITs

4 participants