From 2a682b685331282a68fdd8e8c9ab50e2107a6939 Mon Sep 17 00:00:00 2001 From: lluisemper <58423269+lluisemper@users.noreply.github.com> Date: Sun, 18 Oct 2026 21:06:53 +0200 Subject: [PATCH] test: fix zlib round-trip test parameters Skip gzip with windowBits set to 8 in the PUMMEL matrix, since gzip requires a value of at least 9. Rejection of 8 is tested separately. Pass chunkSize to the compression and decompression constructors so the buffer sizes selected by the test are actually exercised. Signed-off-by: lluisemper <58423269+lluisemper@users.noreply.github.com> --- test/parallel/test-zlib.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/test/parallel/test-zlib.js b/test/parallel/test-zlib.js index 35fd2c946030..d7eb6bbf68ae 100644 --- a/test/parallel/test-zlib.js +++ b/test/parallel/test-zlib.js @@ -201,7 +201,11 @@ testKeys.forEach(common.mustCall((file) => { zlibPairs.forEach(common.mustCall((pair) => { const Def = pair[0]; const Inf = pair[1]; - const opts = { level, windowBits, memLevel, strategy }; + // Gzip requires windowBits >= 9; rejection of 8 is tested above. + if (Def === zlib.Gzip && windowBits === 8) { + return; + } + const opts = { level, windowBits, memLevel, strategy, chunkSize }; const def = new Def(opts); const inf = new Inf(opts);