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

fix(cypress): fix cypress throwing error when connectorId is not passed and miscellaneous fixes #5746

Merged
merged 7 commits into from
Sep 2, 2024

Conversation

pixincreate
Copy link
Member

@pixincreate pixincreate commented Aug 29, 2024

Type of Change

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

Description

  • This PR aims to fix Cypress throwing error when the connectorId is undefined. This would occur when routing and payment method list tests were being run directly without passing the connectorId which is in fact redundant
    • As for reports, we now pass service as prefix without which it takes undefined as connector name which is not intended and for CI, reportName is considered from env
  • This PR also addresses one of the issue with routing test by removing should_continue flag where Cypress would throw an error stating it to be not defined
  • In addition to that, we now set the connectorId in globalState during connector create call to address occasional unintended issues where it would not get updated properly
  • Bumped Cypress version to v13.14.0 from v13.13.3
  • Lints

Note

Changes have been split into different commits for ease of reviewing.

Additional Changes

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

Motivation and Context

NIL

How did you test it?

PML ROUTING
image image
npm run cypress:payment-method-list npm run cypress:routing

Note

Routing tests now throw one less error.
PML and Routing tests now run without user needing to pass CYPRESS_CONNECTOR as env

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 Aug 29, 2024
@pixincreate pixincreate added this to the August 2024 Release milestone Aug 29, 2024
@pixincreate pixincreate self-assigned this Aug 29, 2024
@pixincreate pixincreate requested a review from a team as a code owner August 29, 2024 14:57
Copy link

semanticdiff-com bot commented Aug 29, 2024

Review changes with SemanticDiff.

Analyzed 6 of 6 files.

Overall, the semantic diff is 96% smaller than the GitHub diff.

1 files do not contain logic changes.

Filename Status
✔️ cypress-tests/cypress.config.js 7.69% smaller
✔️ cypress-tests/package-lock.json 56.61% smaller
✔️ cypress-tests/package.json Analyzed
✔️ cypress-tests/cypress/e2e/RoutingTest/00002-RuleBasedRouting.cy.js Analyzed
✔️ cypress-tests/cypress/e2e/PaymentUtils/Commons.js 0.92% smaller
✔️ cypress-tests/cypress/e2e/PaymentUtils/Fiservemea.js No logic changes found

likhinbopanna
likhinbopanna previously approved these changes Aug 30, 2024
likhinbopanna
likhinbopanna previously approved these changes Aug 30, 2024
@Gnanasundari24 Gnanasundari24 added this pull request to the merge queue Sep 2, 2024
Merged via the queue into main with commit 4eec869 Sep 2, 2024
14 checks passed
@Gnanasundari24 Gnanasundari24 deleted the cypress-fixes branch September 2, 2024 11:41
@pixincreate pixincreate removed the S-waiting-on-review Status: This PR has been implemented and needs to be reviewed label Sep 2, 2024
pixincreate added a commit that referenced this pull request Sep 3, 2024
…config-fix

* 'main' of github.com:juspay/hyperswitch:
  feat: add profile_id authentication for business profile update and list (#5673)
  chore(version): 2024.09.03.0
  feat(user): implement invitations api (#5769)
  feat(connector): [Adyenplatform] add webhooks for payout (#5749)
  refactor(v2_migrations): re-organize v2 migrations (#5760)
  chore: add wasm support for connector additional details (#5712)
  refactor(connector): Move globepay, powertranz, tsys, worldline to hyperswitch_connectors (#5758)
  fix(cypress): fix cypress throwing error when `connectorId` is not passed and miscellaneous fixes (#5746)
  chore: fix typos (#5766)
  refactor(business_profile): change id for business profile (#5748)
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.

4 participants