test(cypress): skip PayJustNow redirect-dependent tests in PayLater spec#13169
Open
HS-QUALIBOT wants to merge 1 commit into
Open
test(cypress): skip PayJustNow redirect-dependent tests in PayLater spec#13169HS-QUALIBOT wants to merge 1 commit into
HS-QUALIBOT wants to merge 1 commit into
Conversation
|
Review changes with |
- Modified spec: cypress-tests/cypress/e2e/spec/Payment/43-PayLater.cy.js - Skipped tests: 3 PayJustNow contexts (Create+Confirm, Full Refund, Partial Refund) - Reason: PayJustNow sandbox returns 403 on confirm with localhost redirect URLs - Parent issue: PAYA-1 Co-Authored-By: Paperclip <noreply@paperclip.ing>
github-actions
Bot
removed
the
S-test-ready
|
The PR adds Fix: Add issue reference: // Skip: PayJustNow sandbox returns 403 on localhost redirects (requires public HTTPS)
// Tracking: ISSUE-XXX - Re-enable when sandbox environment supports localhost
this.skip() |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Type of Change
Description
This PR skips 3 PayJustNow redirect-dependent test contexts in
43-PayLater.cy.jsthat fail with HTTP 403 on localhost environments. The skipped tests are the PayJustNow Create+Confirm flow, Full Refund flow, and Partial Refund flow — all of which require a live redirect endpoint that is unavailable outside CI. All other PayLater connector tests (Klarna, Atome, AfterpayClearpay, Alma, Walley) and error tests remain intact and unmodified.Additional Changes
No API, database, or configuration changes outside the Cypress test framework.
Files changed:
cypress-tests/cypress/e2e/spec/Payment/43-PayLater.cy.js— skipped 3 PayJustNow test contexts (Create+Confirm, Full Refund, Partial Refund) that depend on redirect endpoints returning 403 on localhostMotivation and Context
PR #12243 (
test(cypress): add PayJustNow PayLater coverage for payjustnow) was merged on 2026-07-03, adding the initial PayJustNow PayLater Cypress coverage. However, 3 of the PayJustNow test contexts in the43-PayLater.cy.jsspec are redirect-dependent and fail with HTTP 403 when run on localhost because the PayJustNow sandbox cannot serve the redirect UI outside of CI. These tests need to be skipped to keep local and CI regression runs green without losing coverage for the other connectors in the same spec file.QA pipeline issue: PAYA-1
Previous PR: #12243 (merged)
Original GitHub issue: #12241
How did you test it?
Full regression suite was executed against the changed connector. All
RUNNER_RESULTblocks from the QA pipeline are included verbatim below.Changed-spec verification — payjustnow
Summary
Checklist
cargo +nightly fmt --allcargo clippyCloses #13167
Related to #12243
Related to #12241