Browse Source

test: check zlib version for createDeflateRaw

We are currenly builing Node with --shared-zlib which happens to be
version 1.2.8. The test for zlib.createDeflateRaw is expected to fail
but does not when using version 1.2.8.

As far as I can tell the fix referred to in the comments was
introduced in version 1.2.9:
- Reject a window size of 256 bytes if not using the zlib wrapper

This commit suggests adding a check for the version and skipping this
assert if the version is less than 1.2.9.

Refs: http://zlib.net/ChangeLog.txt
Backport-PR-URL: https://github.com/nodejs/node/pull/15478
PR-URL: https://github.com/nodejs/node/pull/13697
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
v6.x
Daniel Bevenius 8 years ago
committed by Myles Borins
parent
commit
8e00315506
No known key found for this signature in database GPG Key ID: 933B01F40B5CA946
  1. 7
      test/parallel/test-zlib-failed-init.js

7
test/parallel/test-zlib-failed-init.js

@ -1,10 +1,15 @@
'use strict';
require('../common');
const common = require('../common');
const assert = require('assert');
const zlib = require('zlib');
if (process.config.variables.node_shared_zlib &&
/^1\.2\.[0-8]$/.test(process.versions.zlib)) {
common.skip("older versions of shared zlib don't throw on create");
}
// For raw deflate encoding, requests for 256-byte windows are rejected as
// invalid by zlib.
// (http://zlib.net/manual.html#Advanced)

Loading…
Cancel
Save