Skip to content

Commit

Permalink
[Tests] node v18.18 ships a broken Symbol.dispose as well
Browse files Browse the repository at this point in the history
  • Loading branch information
ljharb committed Oct 23, 2023
1 parent ae1f344 commit 000c7aa
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions test/tests.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,8 @@ var supportsDescriptors = require('define-properties').supportsDescriptors;
var v = require('es-value-fixtures');
var semver = require('semver');

var brokenNodePolyfill = semver.satisfies(process.version, '^18.18 || >= 20.4');

module.exports = {
DisposableStack: function testDisposableStack(t, DisposableStack, symbolDispose) {
t.equal(typeof DisposableStack, 'function', 'is a function');
Expand Down Expand Up @@ -617,7 +619,7 @@ module.exports = {
st.notOk(
isRegisteredSymbol(symbolDispose),
'is not a registered symbol',
{ skip: semver.satisfies(process.version, '>= 20.4') ? 'node ships a Symbol.dispose polyfill that is registered' : false }
{ skip: brokenNodePolyfill ? 'node ships a Symbol.dispose polyfill that is registered' : false }
);

st.end();
Expand All @@ -638,7 +640,7 @@ module.exports = {
st.notOk(
isRegisteredSymbol(symbolAsyncDispose),
'is not a registered symbol',
{ skip: semver.satisfies(process.version, '>= 20.4') ? 'node ships a Symbol.dispose polyfill that is registered' : false }
{ skip: brokenNodePolyfill ? 'node ships a Symbol.dispose polyfill that is registered' : false }
);

st.end();
Expand Down

0 comments on commit 000c7aa

Please sign in to comment.