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): uncaught exceptions thrown by hyperswitch.io #7092

Merged
merged 2 commits into from
Jan 24, 2025

Conversation

pixincreate
Copy link
Member

@pixincreate pixincreate commented Jan 23, 2025

Type of Change

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

Description

the return url i.e., hyperswitch.io is throwing uncaught exceptions. the issue has been escalated to the team managing the website. and for now, we're ignoring any uncaught exceptions thrown by it.

additionally, have bumped the package versions except for cypress since it is a major upgrade and breaks redirection tests left right and center.

this pr also removes redundant code from redirection handler

closes #7093

Additional Changes

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

Motivation and Context

ci

How did you test it?

ci checks should pass, but works in local:

payouts: wise
image

payments: adyen
image

payments: bankofamerica
image

payments: bluesnap
image

payments: cybersource
image

payments: nmi
image

payments: paypal
image

payments: stripe
image

payments: trustpay
image

payment method list
image

routing
image

Checklist

  • I formatted the code npm run format
  • I addressed lints thrown by npm run lint
  • I reviewed the submitted code
  • I added unit tests for my changes where possible

@pixincreate pixincreate added the A-CI-CD Area: Continuous Integration/Deployment label Jan 23, 2025
@pixincreate pixincreate self-assigned this Jan 23, 2025
Copy link

semanticdiff-com bot commented Jan 23, 2025

Review changes with  SemanticDiff

Changed Files
File Status
  cypress-tests/package-lock.json  63% smaller
  cypress-tests/cypress/support/redirectionHandler.js  58% smaller
  cypress-tests/cypress/support/e2e.js  34% smaller
  cypress-tests/package.json  21% smaller

@pixincreate pixincreate added this to the January 2025 Release milestone Jan 23, 2025
@pixincreate pixincreate marked this pull request as ready for review January 23, 2025 10:21
@pixincreate pixincreate requested a review from a team as a code owner January 23, 2025 10:21
@Gnanasundari24 Gnanasundari24 added this pull request to the merge queue Jan 24, 2025
Merged via the queue into main with commit cf82861 Jan 24, 2025
23 of 24 checks passed
@Gnanasundari24 Gnanasundari24 deleted the fix-exceptions branch January 24, 2025 07:22
pixincreate added a commit that referenced this pull request Jan 24, 2025
…d-memory-cache

* 'main' of github.com:juspay/hyperswitch: (38 commits)
  fix(cypress): uncaught exceptions thrown by `hyperswitch.io` (#7092)
  ci(workflows): Add code coverage reporting for v2 (#6884)
  chore(version): 2025.01.23.0
  refactor: [CYBERSOURCE, BANKOFAMERICA, WELLSFARGO] Move code to crate hyperswitch_connectors (#6908)
  Documentation edits made through Mintlify web editor
  chore: enable 128-column-tables feature for diesel crate (#6857)
  feat(connector): [ADYEN ] Consume transaction id for PaymentsPreProcessing error (#7061)
  chore(version): 2025.01.22.0
  feat(routing): Integrate global success rates (#6950)
  feat(email): Add mailhog by default in docker-compose for local smtp server (#6869)
  refactor: check allowed payment method types in enabled options (#7019)
  feat(router): add payment method-specific features to connector feature list (#6963)
  Refactor: Customer email and browser Information (#7034)
  feat(connectors): fiuu,novalnet,worldpay - extend NTI flows (#6946)
  fix(cypress): address cybersource redirection inconsistency (#7057)
  chore(version): 2025.01.21.0
  refactor(payment-link): use shouldRemoveBeforeUnloadEvents flag for handling removal of beforeunload events through SDK (#7072)
  refactor(router): refactor ctp flow to fetch mca_id and get the connector creds instead of connector_name (#6859)
  chore(version): 2025.01.20.0
  fix(relay): populate connector metadata in the refunds flow (#7045)
  ...
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.

[BUG] hyperswitch.io website has some uncaught exceptions resulting in cypress ci checks to fail
3 participants