Skip to content

fix(e2e): increase waitForURL timeouts in vendor tests for slow CI#216

Merged
steilerDev merged 3 commits into
betafrom
docs/5-epic05-budget-docs
Feb 23, 2026
Merged

fix(e2e): increase waitForURL timeouts in vendor tests for slow CI#216
steilerDev merged 3 commits into
betafrom
docs/5-epic05-budget-docs

Conversation

@steilerDev
Copy link
Copy Markdown
Owner

Summary

The vendor detail test (vendors.spec.ts:341) consistently fails on CI because the navigation takes longer than 8000ms on slow GitHub Actions runners.

Test plan

  • Change is limited to E2E test timeouts — no production code affected
  • CI E2E Tests pass with the increased timeouts

🤖 Generated with Claude Code

Replace budget placeholder with comprehensive user guides covering
categories, financing sources, work item budgets, vendors & invoices,
subsidies, and the budget overview dashboard. Update roadmap and
README to reflect EPIC-05 completion.

Co-Authored-By: Claude docs-writer (Opus 4.6) <noreply@anthropic.com>
Wait for loading state to clear on vendor detail page and add
networkidle wait for login screenshot test. Both tests fail on slow
CI runners where 7s default isn't enough for API fetch + render.

Co-Authored-By: Claude qa-integration-tester (Opus 4.6) <noreply@anthropic.com>
The vendor detail navigation test fails on CI runners because
page.waitForURL uses an 8000ms timeout which is insufficient for
slow GitHub Actions runners. Increased all waitForURL and related
assertion timeouts to 15000ms for consistency with the other CI
timeout fixes in this file.

Co-Authored-By: Claude <qa-integration-tester> (opus-4) <noreply@anthropic.com>
@steilerDev steilerDev merged commit 39ee6c6 into beta Feb 23, 2026
7 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.9.0-beta.56 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@steilerDev steilerDev deleted the docs/5-epic05-budget-docs branch March 7, 2026 07:44
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