Skip to content

fix(cypress): use no_three_ds for Calida Bluecode wallet flow#13138

Open
venkatakarthikm-maker wants to merge 2 commits into
mainfrom
calida-fix
Open

fix(cypress): use no_three_ds for Calida Bluecode wallet flow#13138
venkatakarthikm-maker wants to merge 2 commits into
mainfrom
calida-fix

Conversation

@venkatakarthikm-maker

@venkatakarthikm-maker venkatakarthikm-maker commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

The Bluecode wallet test hardcoded 'three_ds' for payment intent creation but Calida's confirm config specifies 'no_three_ds'. This auth type mismatch caused the confirm call to fail with 400, cascading to undefined nextActionUrl and 500 on sync.

Type of Change

  • Bugfix

Description

Changed the hardcoded "three_ds" to "no_three_ds" in the Bluecode test context in 19-Wallet.cy.js to match the auth type declared in Calida.js. Calida's backend connector does not support 3DS — it's a pure wallet redirect flow (Bluecode QR scan).

Additional Changes

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

Motivation and Context

Fixes #13139

How did you test it?

Ran Cypress specs 01, 02, 03, and 19 locally with CYPRESS_CONNECTOR=calida. All 5 Bluecode wallet tests pass:

  1. create-payment-call-test ✅
  2. payment_methods-call-test ✅
  3. Confirm wallet redirect ✅
  4. Handle wallet redirection ✅
  5. Sync payment status ✅

Checklist

  • I formatted the code
  • I reviewed the submitted code

The Bluecode wallet test hardcoded 'three_ds' for payment intent creation
but Calida's confirm config specifies 'no_three_ds'. This auth type
mismatch caused the confirm call to fail with 400, cascading to
undefined nextActionUrl and 500 on sync.
@venkatakarthikm-maker venkatakarthikm-maker self-assigned this Jul 2, 2026
@venkatakarthikm-maker venkatakarthikm-maker requested a review from a team as a code owner July 2, 2026 11:19
@semanticdiff-com

semanticdiff-com Bot commented Jul 2, 2026

Copy link
Copy Markdown

Review changes with  SemanticDiff

Changed Files
File Status
  cypress-tests/cypress/e2e/spec/Payment/19-Wallet.cy.js  0% smaller

@venkatakarthikm-maker venkatakarthikm-maker added the S-test-ready Status: This PR is ready for cypress-tests label Jul 2, 2026
@XyneSpaces

Copy link
Copy Markdown

Code Review: Cypress Test Fix

Findings

[nit] This is a straightforward test configuration fix. Changing "three_ds" to "no_three_ds" for the Calida Bluecode wallet flow is correct if the connector doesn't support 3DS for this payment method.

[should-fix] Add a brief code comment explaining why no_three_ds is required for Calida Bluecode specifically. This helps future maintainers understand if this is:

  1. A connector limitation
  2. A test environment constraint
  3. A specific flow requirement

Example:

// Calida Bluecode doesn't support 3DS authentication
"no_three_ds",

No blocking issues found.

@github-actions github-actions Bot removed the S-test-ready Status: This PR is ready for cypress-tests label Jul 2, 2026
@venkatakarthikm-maker venkatakarthikm-maker added the S-test-ready Status: This PR is ready for cypress-tests label Jul 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-test-ready Status: This PR is ready for cypress-tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Calida Bluecode wallet Cypress test fails with 400 on confirm due to auth type mismatch

3 participants