Slow tests reporter and platform fixes#235
Merged
Merged
Conversation
…ma workers to nprocessors - Replace Minitest.after_run timing block in id58_test_base.rb with a dedicated SlowTestsReporter class, matching the pattern in differ/saver. Descriptions are trimmed and stripped of leading whitespace/pipes so each slow test prints as a single terminal line. - Add platform: linux/amd64 to docker-compose.yml services and all docker pull/run calls in bin/ to suppress arm64/amd64 mismatch warnings on Apple Silicon. - Set Puma workers(Etc.nprocessors) on both client and server configs so all cores are used, matching differ/saver. - Add $stdout.sync = true in test/lib/run.sh so dots appear in real time rather than all at the end. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add parallelize_me! and Minitest.parallel_executor to id58_test_base.rb, matching the pattern in differ/saver. - Remove ENV['ID58'] = id58, which is global shared state incompatible with parallel execution; remove its uses in bash_sheller_stub.rb and id58_test_test.rb. - Append OS initial to Docker container id for multi_os_test variants to prevent collisions when Alpine/Debian/Ubuntu run simultaneously; guarded by Id58TestBase.multi_os? so single-OS tests are unaffected. - Raise baseline speed limit 2s -> 4s: parallel OS variants now run their Docker containers concurrently, increasing system load. - Set SHOW_TEST_IDS=false for client container. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.
No description provided.