Skip to content

fix: align wallet E2E tests to current UI structure#347

Merged
d-beezee merged 1 commit intodevelopfrom
fix/wallet-e2e-tests
Feb 19, 2026
Merged

fix: align wallet E2E tests to current UI structure#347
d-beezee merged 1 commit intodevelopfrom
fix/wallet-e2e-tests

Conversation

@sinatragianpaolo
Copy link
Collaborator

Sommario

Fix di due test E2E del wallet che stavano fallendo in CI a causa di modifiche recenti all'UI.

1. table_net_gross.spec.ts — selettore ambiguo

È stata aggiunta di recente una seconda tabella nella pagina wallet (expired attribution table, 5 colonne). Il selettore .wallet-table trovava entrambe le tabelle, restituendo 12 header invece dei 7 attesi.

Fix: aggiunto .first() su tutti i test del file per scoped alla tabella dei pagamenti.

2. payment_requests/manual.spec.ts — counter non resettato

La variabile count (che traccia le GET a /users/me/payments dopo il POST) era dichiarata a livello describe ma non resettata nel beforeEach, causando accumulo tra i test dello stesso describe block (0→2→4 invece di 0→2 per ogni test).

Fix: aggiunto count = 0 all'inizio del beforeEach interno.

Test plan

  • table_net_gross.spec.ts: tutti e 4 i test passano
  • payment_requests/manual.spec.ts: tutti e 92 i test passano

- table_net_gross: scope to first .wallet-table to avoid matching the new expired attribution table (7+5=12 headers)
- manual payment: reset count in beforeEach to avoid cross-test accumulation
@currents-bot
Copy link

currents-bot bot commented Feb 19, 2026

Currents Tests: Tryber.me tests passed after 3m 16.1s

🟢 235 · 🔴 0 · ⚪️ 0 · 🟣 1

View Run Details

Run Details

  • Project: Tryber.me

  • Groups: 1

  • Framework: Playwright

  • Run Status: Passed

  • Commit: d1731e5

  • Spec files: 12

  • Overall tests: 235

  • Duration: 3m 16.1s

  • Parallelization: 0


This message was posted automatically by currents.dev | Integration Settings

@d-beezee d-beezee merged commit e4019f6 into develop Feb 19, 2026
4 checks passed
@d-beezee d-beezee deleted the fix/wallet-e2e-tests branch February 19, 2026 09:18
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

Comments