Skip to content

added timeouts + dropped prev vscode version due to build issues#620

Merged
siddhusaladi merged 4 commits into
OpenLiberty:mainfrom
siddhusaladi:drop-prev-vscode-version
Jul 8, 2026
Merged

added timeouts + dropped prev vscode version due to build issues#620
siddhusaladi merged 4 commits into
OpenLiberty:mainfrom
siddhusaladi:drop-prev-vscode-version

Conversation

@siddhusaladi

@siddhusaladi siddhusaladi commented Jul 7, 2026

Copy link
Copy Markdown
Contributor
  • Drop previous from the VS Code matrix for CI because cold previous version runners can cause Liberty LS JVM startup times of 5–7 minutes per workspace transition, exceeding any reasonable test timeout
  • Bump before(src/test/shared/devModeTestSuite.ts), afterEach(src/test/shared/devModeTestSuite.ts), and after hook(in devModeActions tests) timeouts
  • waitForDashboardToLoad (src/test/utils/testUtils.ts) timeout increased from 120s to 180s
  • Fresh SideBarView created on each poll in getDashboardSection to prevent stale DOM errors during workspace transitions between tests

Comment thread src/test/utils/testUtils.ts Outdated
@dshimo dshimo self-requested a review July 7, 2026 21:49
@siddhusaladi siddhusaladi merged commit e1b0a11 into OpenLiberty:main Jul 8, 2026
4 checks passed
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