Skip to content

fix: stabilize shipping preview and pre-filled pickup tests - #208

Merged
lucvysk merged 3 commits into
mainfrom
fix/test-failing
Jul 27, 2026
Merged

fix: stabilize shipping preview and pre-filled pickup tests#208
lucvysk merged 3 commits into
mainfrom
fix/test-failing

Conversation

@lucvysk

@lucvysk lucvysk commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes flaky/failing checkout UI e2e assertions that diverged by account lean-shipping config and by multi-country trade policy forms.

  • Shipping preview labels: assert account-specific delivery options instead of a single hardcoded carrier name
    • Delivery_Scheduled Delivery and Delivery: Expressa on NO_LEAN / CLEAN_NO_MAPS, Prazos variados on lean accounts
    • Delivery_Scheduled Delivery_Scheduled Pickup: Motoboy / PAC by name on non-lean, cheapest by data-testid on lean
  • Postal code fill: when #ship-country is present, select BRA before typing the CEP (skip if already selected, so the form is not reset)
  • Pickup pre-filled flow: send a complete pickup address payload (isDisposable, city/state/street/geo) with CEP 22011050, and drop the obsolete store-name assertion

Test scenarios

  • shipping-preview/Delivery_Scheduled Delivery and Delivery - *.test.js across lean and non-lean accounts
  • shipping-preview/Delivery_Scheduled Delivery_Scheduled Pickup - *.test.js (vtexgame1, vtexgame1clean, vtexgame1geo, vtexgame1invoice)
  • Postal-code shipping flows that hit fillPostalCodeOmnishipping (accounts with and without #ship-country)
  • shipping/Pickup - Pre-filled profile and shipping data - vtexgame1.test.js

@lucvysk
lucvysk requested a review from a team as a code owner July 27, 2026 18:32
@vtex-pr-sentinel

vtex-pr-sentinel Bot commented Jul 27, 2026

Copy link
Copy Markdown

🛡️ SDD Check — action required

I couldn't detect an SDD in this PR. Please check one option below (requires write access to the repo):

  • SDD lives in another PR — paste the SDD PR URL here:
  • This PR doesn't need an SDD
  • SDD applies, but I'm not adopting it in this PR

@jeffersontuc jeffersontuc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good ✅

@lucvysk
lucvysk merged commit fb7451a into main Jul 27, 2026
6 of 7 checks passed
@lucvysk
lucvysk deleted the fix/test-failing branch July 27, 2026 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants