Skip to content

Fix Ruby 3.4 CI hang by adding test cleanup#11

Merged
takaokouji merged 1 commit intomasterfrom
fix-ruby34-hang-cleanup
Jan 29, 2026
Merged

Fix Ruby 3.4 CI hang by adding test cleanup#11
takaokouji merged 1 commit intomasterfrom
fix-ruby34-hang-cleanup

Conversation

@takaokouji
Copy link
Copy Markdown
Collaborator

Summary

  • Add teardown methods to all test files to properly close Node.js processes after each test
  • Fixes intermittent CI hangs on Ruby 3.4 caused by zombie processes accumulating

Root Cause

When schmoozer instances aren't properly closed after tests, Node.js processes remain running. Over multiple test runs, these zombie processes can cause fork() to block due to resource exhaustion.

Changes

  • schmooze_test.rb: Add teardown to close @Schmoozer
  • error_test.rb: Add setup/teardown and use @Schmoozer for process tracking
  • garbage_test.rb: Add setup/teardown and use @Schmoozer for process tracking
  • local_script_test.rb: Add teardown to close @Schmoozer

Test plan

  • Local tests pass with Ruby 3.4.2
  • CI passes for all Ruby versions (2.6 - 4.0)

🤖 Generated with Claude Code

This fixes intermittent CI hangs on Ruby 3.4 caused by zombie Node.js
processes accumulating across test runs. When processes aren't properly
closed, fork() can block due to resource exhaustion.

Each test file now has a teardown method that calls close() on the
schmoozer instance if it has a running process, ensuring Node.js
processes are properly terminated after each test.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@takaokouji takaokouji merged commit 459c023 into master Jan 29, 2026
16 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.

1 participant