fix: modernize test suite for Vitest 4.x and stabilize Docker integration#62
Merged
ttbombadil merged 17 commits intodevfrom Mar 13, 2026
Merged
fix: modernize test suite for Vitest 4.x and stabilize Docker integration#62ttbombadil merged 17 commits intodevfrom
ttbombadil merged 17 commits intodevfrom
Conversation
… worker count in CI
…s management and stability
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR stabilizes the integration test suite and modernizes the testing infrastructure to resolve execution hangs and compatibility issues with Vitest 4.x and macOS 26 (Darwin 25).
Key Changes:
Vitest 4.0 Migration: Refactored all legacy done() callbacks to modern async/await patterns to resolve FixtureParseError and align with the latest Vitest standards.
macOS Beta Stability (Docker Bypass): Implemented a forced Docker execution mode for integration tests. This bypasses a known _dyld_start hang on macOS 26.3 Beta where unsigned binaries from temporary directories were blocked or delayed by system security policies.
Sandbox Runner Optimization:
Increased Docker memory limits from 128MB to 256MB to prevent OOM kills during complex Arduino-cli compilations.
Fixed a race condition in the isCompilePhase detection where successful, silent compilations would block the serial output buffer.
Added the --rm flag to Docker commands to ensure automatic container cleanup and prevent resource leaking.
Environment Consistency:
Improved path resolution using realpathSync to handle macOS /tmp to /private/tmp symlink mapping within Docker volumes.
Cleaned up unused TypeScript variables and ESLint warnings to ensure a green CI pass on GitHub Actions.
Result:
The full test suite (990+ tests) now passes consistently both locally (macOS/Docker) and in the GitHub CI environment. No leaked processes or race conditions were detected in final stability checks.