Skip to content

Skip test on Node.js versions where it does not apply - #2340

Merged
lpinca merged 1 commit into
masterfrom
skip/test
Oct 30, 2026
Merged

lpinca merged 1 commit into
masterfrom
skip/test

Conversation

@lpinca

@lpinca lpinca commented Oct 8, 2026 •

Copy link
Copy Markdown
Member

In Node.js versions >= 27.0.0-alpha.0.0.0 the HTTP client rejects the response before emitting the 'upgrade' event.

Refs: nodejs/node@e5778f78
Refs: nodejs/citgm#1190

Comment thread test/websocket.test.js Outdated
// response before emitting the `'upgrade'` event. See
// https://github.com/nodejs/node/commit/e5778f78.
//
if (process.versions.modules >= 152) return this.skip();

@lpinca lpinca Oct 8, 2026 •

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Waiting for the actual value of NODE_MODULE_VERSION for Node.js version 27.0.0-alpha.0.0.0.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was 150 when nodejs/node@e5778f78 landed.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm waiting for the first official release, not a nightly, not built on main.

In Node.js versions >= 27.0.0-alpha.0.0.0 the HTTP client rejects the
response before emitting the `'upgrade'` event.

Refs: nodejs/node@e5778f78
@MikeMcC399

Copy link
Copy Markdown

modules is 153 in Node.js 27.0.0-alpha.0.0.0

$ node -e "console.log(process.versions)"
{
  node: '27.0.0-alpha.0.0.0',
  acorn: '8.19.0',
  ada: '4.0.0',
  amaro: '1.1.11',
  ares: '1.34.9',
  brotli: '1.2.1',
  cldr: '48.0',
  icu: '78.3',
  libffi: '3.8.0',
  lief: '1.0.0',
  llhttp: '9.4.3',
  merve: '1.2.2',
  modules: '153',
  napi: '10',
  nbytes: '0.1.4',
  ncrypto: '0.0.1',
  nghttp2: '1.70.0',
  nghttp3: '',
  ngtcp2: '',
  openssl: '4.1.0',
  simdjson: '5.0.5',
  simdutf: '7.7.0',
  sqlite: '3.54.1',
  tz: '2026e',
  undici: '8.12.3',
  unicode: '17.0',
  uv: '1.52.1',
  uvwasi: '0.0.23',
  v8: '15.6.75.25-node.10',
  zlib: '1.3.2.1-motley-9f9f52c',
  zstd: '1.5.7'
}

I saw in nodejs/citgm#1190 that you're planning to cut a new release today. That would be great 👍🏻

@lpinca
lpinca merged commit aa91518 into master Oct 30, 2026
103 checks passed
@lpinca
lpinca deleted the skip/test branch October 30, 2026 15:10
@MikeMcC399

MikeMcC399 commented Oct 30, 2026 •

Copy link
Copy Markdown

@lpinca

If you want to test in GitHub Actions, you can use latest or the exact Node.js release 27.0.0-alpha.0.0.0

I submitted a feature request actions/setup-node#1657 earlier today about this.

@lpinca

lpinca commented Oct 30, 2026

Copy link
Copy Markdown
Member Author

Noticed, but there is another issue with macOS-latest and the x64 arch in the test matrix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants