Skip to content
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

ci(cypressV2): update cypress v2 framework to accommodate hyperswitch v2 changes #6493

Merged
merged 16 commits into from
Nov 8, 2024

Conversation

pixincreate
Copy link
Member

@pixincreate pixincreate commented Nov 6, 2024

Type of Change

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

Description

This PR introduces multiple fixes for Cypress V2 at framework level. The fixes include:

Important

It is recommended that the PR is reviewed based on commits instead of going through the file at a time

Payment tests have not been added, but only template example just to serve as a reference for devs working on v2 to fill in.

Additional Changes

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

Motivation and Context

I'm the motivation.

How did you test it?

Ran Cypress V2 tests against Hyperswitch V2

image

Checklist

  • I formatted the code prettier . --write
  • I addressed lints thrown by cargo clippy
  • I reviewed the submitted code
  • I added unit tests for my changes where possible

@pixincreate pixincreate added A-CI-CD Area: Continuous Integration/Deployment S-waiting-on-review Status: This PR has been implemented and needs to be reviewed labels Nov 6, 2024
@pixincreate pixincreate added this to the October 2024 Release milestone Nov 6, 2024
@pixincreate pixincreate self-assigned this Nov 6, 2024
0001 contains an example for payment intent create call, confirm call
and retrieve call. and of course, these does not work at present and
hence, skipped intentionally from being run.

add more number contexts within describe to address all the cases
mentioned at the top of the file.

configs needs to be introduced. but before that, we need to do some
structural changes to change the behavior
@pixincreate pixincreate marked this pull request as ready for review November 6, 2024 12:31
@pixincreate pixincreate requested a review from a team as a code owner November 6, 2024 12:31
@pixincreate pixincreate changed the title ci(cypress): fix error messages being undefined ci(cypress v2): update cypress v2 framework to accommodate hyperswitch v2 changes Nov 6, 2024
@pixincreate pixincreate changed the title ci(cypress v2): update cypress v2 framework to accommodate hyperswitch v2 changes ci(cypressV2): update cypress v2 framework to accommodate hyperswitch v2 changes Nov 7, 2024
likhinbopanna
likhinbopanna previously approved these changes Nov 7, 2024
@Gnanasundari24 Gnanasundari24 added this pull request to the merge queue Nov 8, 2024
Merged via the queue into main with commit 28e3c36 Nov 8, 2024
17 checks passed
@Gnanasundari24 Gnanasundari24 deleted the cypress-v2-update branch November 8, 2024 13:06
@pixincreate pixincreate removed the S-waiting-on-review Status: This PR has been implemented and needs to be reviewed label Nov 8, 2024
bsayak03 pushed a commit that referenced this pull request Nov 26, 2024
bsayak03 pushed a commit that referenced this pull request Nov 26, 2024
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.

5 participants