Title: repl: fix flaky test-repl-programmatic-history by mcollina · Pull Request #61614 · nodejs/node · GitHub
Open Graph Title: repl: fix flaky test-repl-programmatic-history by mcollina · Pull Request #61614 · nodejs/node
X Title: repl: fix flaky test-repl-programmatic-history by mcollina · Pull Request #61614 · nodejs/node
Description: Summary Fixes flaky test test/parallel/test-repl-programmatic-history.js. Problem The test was flaky because the FileHandle for the history file could be garbage collected before being explicitly closed, causing ERR_INVALID_STATE errors when the handle was already closed by the GC finalizer. Solution Added an explicit closeHandle() method to ReplHistory class Modified REPLServer.close() to wait for the history file handle to be properly closed before completing the close operation This ensures deterministic cleanup regardless of GC timing Verification Verified with python3 tools/test.py --repeat 1000 test/parallel/test-repl-programmatic-history.js - all 1000 runs passed with 0 failures. Refs: nodejs/reliability#1450
Open Graph Description: Summary Fixes flaky test test/parallel/test-repl-programmatic-history.js. Problem The test was flaky because the FileHandle for the history file could be garbage collected before being explicitly c...
X Description: Summary Fixes flaky test test/parallel/test-repl-programmatic-history.js. Problem The test was flaky because the FileHandle for the history file could be garbage collected before being explicitly c...
Opengraph URL: https://github.com/nodejs/node/pull/61614
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:04572322-7a2d-4303-e173-8a6042baf540 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | AF90:3A75DA:BA157F:FB84B8:6A4C52CD |
| html-safe-nonce | 5d36360437615b479466322cb5a8267d82521b8fb11941770734ad0e29248d93 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJBRjkwOjNBNzVEQTpCQTE1N0Y6RkI4NEI4OjZBNEM1MkNEIiwidmlzaXRvcl9pZCI6IjIxMDM1ODY3MzQ2NzExNTU5MTciLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 1e6c996986fe9f0c42e30dc5c534867fa26d4ac76227e78dc0b94b859fd54f47 |
| hovercard-subject-tag | pull_request:3233272714 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/nodejs/node/pull/61614/files |
| twitter:image | https://avatars.githubusercontent.com/u/52195?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/52195?s=400&v=4 |
| og:image:alt | Summary Fixes flaky test test/parallel/test-repl-programmatic-history.js. Problem The test was flaky because the FileHandle for the history file could be garbage collected before being explicitly c... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 3d11bb817438277de2a940854450e83a7d32b6aeb5014e9e6b00a6423900251c |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/nodejs/node git https://github.com/nodejs/node.git |
| octolytics-dimension-user_id | 9950313 |
| octolytics-dimension-user_login | nodejs |
| octolytics-dimension-repository_id | 27193779 |
| octolytics-dimension-repository_nwo | nodejs/node |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 27193779 |
| octolytics-dimension-repository_network_root_nwo | nodejs/node |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 14099438da5379150f15a2892474c7c7e6c0e55e |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width