tests: New zlib version is 1.3, which only has two elements

[DimStar77]

Change the test code to accept one, two, or three elements past the major version

[DimStar77]

Background:

• openSUSE is in the progress of updating zlib from 1.2.13 to 1.3.
• nodejs is configure with --shared-zlib

With this constellation, the test suite failed with

[ 2208s] not ok 2407 parallel/test-process-versions [ 2208s] --- [ 2208s] duration_ms: 388.41800 [ 2208s] severity: fail [ 2208s] exitcode: 1 [ 2208s] stack: |- [ 2208s] node:assert:1026 [ 2208s] throw err; [ 2208s] ^ [ 2208s] [ 2208s] AssertionError [ERR_ASSERTION]: The input did not match the regular expression /^\d+(?:\.\d+){2,3}(?:-.*)?$/. Input: [ 2208s] [ 2208s] '1.3' [ 2208s] [ 2208s] at Object.<anonymous> (/home/abuild/rpmbuild/BUILD/node-v20.8.1/test/parallel/test-process-versions.js:64:8) [ 2208s] at Module._compile (node:internal/modules/cjs/loader:1241:14) [ 2208s] at Module._extensions..js (node:internal/modules/cjs/loader:1295:10) [ 2208s] at Module.load (node:internal/modules/cjs/loader:1091:32) [ 2208s] at Module._load (node:internal/modules/cjs/loader:938:12) [ 2208s] at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:83:12) [ 2208s] at node:internal/main/run_main_module:23:47{[ 2208s] generatedMessage: true, [ 2208s] code: 'ERR_ASSERTION', [ 2208s] actual: '1.3', [ 2208s] expected: /^\d+(?:\.\d+){2,3}(?:-.*)?$/, [ 2208s] operator: 'match' [ 2208s] } 

[lpinca]

I think a better approach would be to skip the assertion when Node.js is built with the --shared-zlib option as the version might not match even with the proposed regex.

[lpinca]

Refs: https://github.com/madler/zlib/blob/09155eaa2f9270dc4ed1fa13e2b4b2613e6e4851/zlib.h#L40

[lpinca]

Can you please fix the commit message in accordance with our guidelines? Thank you.

Neverming, commit message is ok. I was tricked by PR title.

[richardlau]

Can you please fix the commit message in accordance with our guidelines? Thank you.

Commit message looks okay (it was already changed from the PR title).

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/55387/

[DimStar77]

Can you please fix the commit message in accordance with our guidelines? Thank you.

Neverming, commit message is ok. I was tricked by PR title.

My bad - that was the 'original' commit message, saw the bot complaining about this and rewrote it - hopefully to comply

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/55409/

[nodejs-github-bot]

Landed in 5d01042