mi/werkator
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

BuildExecutorIntegrationTest independent of the host's load (v1.30.4) #97

Merged
mi merged 4 commits from mihoe/robust-build-executor-test into main 2026-10-03 17:54:18 +02:00
Conversation 0 Commits 4 Files Changed 4
+153 -24
mi commented 2026-10-03 16:19:14 +02:00
Owner
Copy Link
Copy Source

Makes the load-dependent cases of BuildExecutorIntegrationTest robust (follow-up of PR#94). Test-only change, v1.30.4.

Under load (8 cores, 12 busy loops, parallel Gradle builds) the unchanged spec failed in 4 of 4 runs: Scenario#65.04 (drain flag read too early) three times, Scenario#67.03 and #73.01 (CPU-bound build exceeded the 30 s wait) once each. With this change: 16 of 16 runs green, 640 test cases, idle runtime unchanged at 37 s.

Changes: gates instead of fixed sleeps (#65.05, live log), waiting for the drain (#65.04), sleep 300 for builds the test cancels anyway, longer wait for CPU-bound builds, an already-ended command as the fixture for "too short to measure" (#67.04), a wider margin in #70.02, and #70.03/#70.04 read only their own branch's warning. No assertion became weaker.

The spec stays excluded from PIT; re-adding it is an open question in the PR-doc.

PR-doc: docs/prs/2026-10-03-PR#97-build-executor-test-independent-of-load.md

🤖 Generated with Claude Code

Makes the load-dependent cases of `BuildExecutorIntegrationTest` robust (follow-up of PR#94). Test-only change, v1.30.4. Under load (8 cores, 12 busy loops, parallel Gradle builds) the unchanged spec failed in 4 of 4 runs: Scenario#65.04 (drain flag read too early) three times, Scenario#67.03 and #73.01 (CPU-bound build exceeded the 30 s wait) once each. With this change: 16 of 16 runs green, 640 test cases, idle runtime unchanged at 37 s. Changes: gates instead of fixed sleeps (#65.05, live log), waiting for the drain (#65.04), `sleep 300` for builds the test cancels anyway, longer wait for CPU-bound builds, an already-ended command as the fixture for "too short to measure" (#67.04), a wider margin in #70.02, and #70.03/#70.04 read only their own branch's warning. No assertion became weaker. The spec stays excluded from PIT; re-adding it is an open question in the PR-doc. PR-doc: docs/prs/2026-10-03-PR#97-build-executor-test-independent-of-load.md 🤖 Generated with [Claude Code](https://claude.com/claude-code)
mi added 1 commit 2026-10-03 16:19:14 +02:00
BuildExecutorIntegrationTest independent of the host's load (v1.30.2)
werkator build succeeded after 01:48
Details
381f0cbed2 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
mi added 1 commit 2026-10-03 16:51:49 +02:00
PR-doc for PR#97
werkator build succeeded after 02:32
Details
c264b512c5 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
mi marked the pull request as ready for review 2026-10-03 16:51:57 +02:00
mi added 1 commit 2026-10-03 17:23:20 +02:00
PR-doc: BuildExecutorIntegrationTest stays out of PIT, BuildExecutor to be refactored
werkator build succeeded after 02:35
Details
5be27c4e73 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
mi changed title from BuildExecutorIntegrationTest independent of the host's load (v1.30.2) to BuildExecutorIntegrationTest independent of the host's load (v1.30.4) 2026-10-03 17:51:11 +02:00
mi added 1 commit 2026-10-03 17:51:12 +02:00
Merge origin/main into mihoe/robust-build-executor-test (v1.30.4)
werkator build succeeded after 02:29
Details
4a7c912f90 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
mi merged commit 69f07ad8e7 into main 2026-10-03 17:54:18 +02:00
mi deleted branch mihoe/robust-build-executor-test 2026-10-03 17:54:18 +02:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
ai-reviewer claude-code mi
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mi/werkator#97