test: enable 8 already-passing Node compat tests#33588
test: enable 8 already-passing Node compat tests#33588nathanwhitbot wants to merge 10 commits intodenoland:mainfrom
Conversation
All 8 tests pass against current main with no code changes; this just adds them to config.jsonc so they're tracked. Each was verified via `cargo test --test node_compat -- <name>` and re-run three times to rule out flakiness. parallel/test-cluster-send-deadlock.js parallel/test-cluster-worker-isdead.js parallel/test-http2-large-writes-session-memory-leak.js parallel/test-http2-serve-file.js parallel/test-http2-server-push-stream.js parallel/test-http2-server-unknown-protocol.js parallel/test-https-server-async-dispose.js parallel/test-watch-mode-kill-signal-default.mjs
nathanwhitbot
left a comment
There was a problem hiding this comment.
Config-only — 8 additive {} entries, all alphabetically placed correctly.
The failing test node_compat debug linux-x86_64 is an unrelated flake on
test-child-process-stdio-reuse-readable-stdio.js (an existing enabled
test, not one of the 8 added here). Failure mode: /usr/bin/head: error reading 'standard input': Resource temporarily unavailable followed by an
AssertionError — classic head -c N pipe-read race. cc @nathanwhit for a
rerun.
Otherwise LGTM.
96b6c29 to
a8c5359
Compare
fibibot
left a comment
There was a problem hiding this comment.
LGTM. 8 net config-only additions, body and diff match. Composition makes sense: 2 cluster tests presumably unblocked by #33493 (cluster.fork) landing on main, 4 http2 tests likely unblocked by the recent ext/node http2 churn (#33518/#33520/#33561/etc.), one https/server-async-dispose, and one watch-mode test.
Spot-checked alphabetical positions:
test-cluster-send-deadlock.js,test-cluster-worker-isdead.js— both fit cluster's alpha rangetest-http2-large-writes-session-memory-leak.js—large-writes-session-memory-leak<multiheaders(the next existing entry)test-http2-serve-file.js,test-http2-server-push-stream.js,test-http2-server-unknown-protocol.js— three sequential http2-server entriestest-https-server-async-dispose.js— fits https-server-* rangetest-watch-mode-kill-signal-default.mjs— late in alpha order
CI fully green: 134/134 checks pass, zero fail, zero pending. All test node_compat (3 shards × 6 platforms × debug+release = 36 jobs) clean, lint debug clean on all 3 platforms, all unit / unit_node / specs / integration green.
Author's "verified 3× for stability" claim is consistent with the CI pattern — none of the 8 enrollments tripped the flake retry loop.
|
|
Summary
Enables 8 Node.js compat tests that already pass on main without any code changes. No production code touched — just
tests/node_compat/config.jsoncentries.parallel/test-cluster-send-deadlock.jsparallel/test-cluster-worker-isdead.jsparallel/test-http2-large-writes-session-memory-leak.jsparallel/test-http2-serve-file.jsparallel/test-http2-server-push-stream.jsparallel/test-http2-server-unknown-protocol.jsparallel/test-https-server-async-dispose.jsparallel/test-watch-mode-kill-signal-default.mjsTest plan
cargo test --test node_compat -- <name>(8/8 pass)