Skip to content

test(jetbrains): isolate workspace concurrency test - #11629

Merged
markijbema merged 1 commit into
mainfrom
mark/fix-jetbrains-workspace-test-flake
Jun 24, 2026
Merged

test(jetbrains): isolate workspace concurrency test#11629
markijbema merged 1 commit into
mainfrom
mark/fix-jetbrains-workspace-test-flake

Conversation

@markijbema

Copy link
Copy Markdown
Contributor

What changed

The workspace manager concurrency test now starts the manager directly instead of waiting for the full application startup sequence.

Why

The test only verifies that concurrent lookups return one cached workspace instance. Depending on SSE connection and application data loading introduced an unrelated 10-second readiness timeout that flakes under CI contention.

Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
@kilo-code-bot

kilo-code-bot Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 files)
  • packages/kilo-jetbrains/backend/src/test/kotlin/ai/kilocode/backend/workspace/KiloBackendWorkspaceTest.kt

Reviewed by gpt-5.4-2026-03-05 · Input: 55.6K · Output: 6.6K · Cached: 270.3K

Review guidance: REVIEW.md from base branch main

@markijbema
markijbema enabled auto-merge June 24, 2026 11:51
@markijbema
markijbema merged commit f9b3579 into main Jun 24, 2026
23 checks passed
@markijbema
markijbema deleted the mark/fix-jetbrains-workspace-test-flake branch June 24, 2026 13:50
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
…rkspace-test-flake

test(jetbrains): isolate workspace concurrency test
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