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