Skip to content

ci(cypress): add payu connector #8567

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.

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

Merged
merged 4 commits into from
Jul 8, 2025
Merged

ci(cypress): add payu connector #8567

merged 4 commits into from
Jul 8, 2025

Conversation

likhinbopanna
Copy link
Contributor

@likhinbopanna likhinbopanna commented Jul 7, 2025

Type of Change

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

Description

Add Payu Connector to the cypress test suits

  • PayU doesn't support mandates so tests are skipped accordingly
  • PayU doesn't support three_ds

Additional Changes

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

Motivation and Context

Connector tests enhancement

How did you test it?

image

Failure issues:

1.3ds payments silently falling back to no-3ds(Should throw validation
2. Payment are going to succeeded state for manual capture payment
3. Connector doesn't support multiple refunds

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

Copy link

semanticdiff-com bot commented Jul 7, 2025

Review changes with  SemanticDiff

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

@likhinbopanna likhinbopanna marked this pull request as ready for review July 7, 2025 09:57
@likhinbopanna likhinbopanna requested a review from a team as a code owner July 7, 2025 09:57
@likhinbopanna likhinbopanna self-assigned this Jul 7, 2025
@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 Jul 7, 2025
Gnanasundari24
Gnanasundari24 previously approved these changes Jul 7, 2025
Copy link
Contributor

@arindam-sahoo arindam-sahoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@likhinbopanna Can you add payment_method_data assertions wherever applicable?

@Gnanasundari24 Gnanasundari24 enabled auto-merge July 8, 2025 11:24
@Gnanasundari24 Gnanasundari24 added this pull request to the merge queue Jul 8, 2025
Merged via the queue into main with commit 1cdabb6 Jul 8, 2025
16 of 20 checks passed
@Gnanasundari24 Gnanasundari24 deleted the add-payu branch July 8, 2025 12:28
pixincreate added a commit that referenced this pull request Jul 9, 2025
…ayload-webhooks

* 'main' of github.com:juspay/hyperswitch:
  refactor(connector): Move connector mappings and endpoints to dedicated modules (#8562)
  ci(cypress): fix `hipay` test cases (#8563)
  chore(version): 2025.07.09.0
  fix(payment_method): update entity id used for Vault to global customer id (#8380)
  refactor(routing): add conditional check for invoking DE routing flows (#8559)
  feat(connector): [AUTHIPAY] Integrate cards non 3ds payments (#8266)
  ci(cypress): add payu connector (#8567)
  feat(connector): [silverflow] template code (#8553)
  chore(version): 2025.07.08.0
  feat(cypress): [worldpayvantiv] add cypress test (#8234)
  feat(connectors): [worldpayvantiv] add connector mandate support  (#8546)
  feat(connector): [Celero] add Connector Template Code (#8489)
  feat(payment-methods): create payment_token in vault confirm / do payment-confirm with temp token from session (#8525)
  ci(cypress): Add Tsys,Square cypress test (#8543)
  chore(version): 2025.07.07.0
@hyperswitch-bot hyperswitch-bot bot removed the S-waiting-on-review Status: This PR has been implemented and needs to be reviewed label Jul 13, 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.

3 participants