ci(cypress): Add Checkout Connector #6559
Conversation
|
Review changes with Changed Files
|
likhinbopanna
added
A-CI-CD
| ZeroAuthMandate: { | ||
| Response: { | ||
| status: 501, | ||
| body: { | ||
| error: { | ||
| type: "invalid_request", | ||
| message: "Setup Mandate flow for Checkout is not implemented", | ||
| code: "IR_00", | ||
| }, | ||
| }, | ||
| }, |
There was a problem hiding this comment.
should we move this to commons and remove the same from all configs?
we can directly use connectorName within commons to dynamically modify the error message.
There was a problem hiding this comment.
@pixincreate Let's add this in the to-dos, and we can handle changes for all connectors in a separate PR. Updating all connectors at once would require testing each one individually, so we'll treat this as a separate task
There was a problem hiding this comment.
yes, that's what even i want to do. lets have it as a separate pr so that only that area will be focused.
| // Don't continue if payment status is processing during auto capture | ||
| // Payment data is tokenized only after payment is successful | ||
| let notProcessing = res_data?.body?.status != "processing"; | ||
| should_continue = | ||
| notProcessing && utils.should_continue_further(res_data); | ||
| should_continue = utils.should_continue_further(res_data); |
There was a problem hiding this comment.
why was this reverted? this was added by @kashif-m to add support for WorldPay. Removing this would result in WorldPay tests failing.
There was a problem hiding this comment.
If status is processing after the payment confirm we cannot skip the next test cases because Psync call would make payment success and save card flow to work , we need to handle worldpay in a different way.
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
SanchithHegde
removed
the
S-waiting-on-review
Type of Change
Description
Add Checkout Connector for Cypress Automation
Additional Changes
Motivation and Context
Checkout connector was not available in the cypress automation
How did you test it?
Cypress Tests
Checklist
cargo +nightly fmt --allcargo clippy