Skip to content

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

Open
lpinca wants to merge 1 commit into
masterfrom
skip/test
Open

lpinca wants to merge 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
// 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

This branch has not been deployed

No deployments
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