[Tests] temporarily skip SafeSet tests in envs where es-set has to po…

…lyfill
inspect-js · Jan 3, 2025 · d88acf0 · d88acf0
1 parent 89d84b5
commit d88acf0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/package.json b/package.json
@@ -48,6 +48,7 @@
 		"auto-changelog": "^2.5.0",
 		"core-js": "^2.6.12",
 		"encoding": "^0.1.13",
+		"es-set": "^2.0.1",
 		"es5-shim": "^4.6.7",
 		"es6-shim": "^0.35.8",
 		"eslint": "=8.8.0",

diff --git a/test/index.js b/test/index.js
@@ -4,6 +4,7 @@ var test = require('tape');
 var debug = require('object-inspect');
 var forEach = require('for-each');
 var SafeSet = require('safeset');
+var ESSet = require('es-set');
 
 var isSet = require('..');
 
@@ -59,9 +60,9 @@ test('WeakSets', { skip: typeof WeakSet !== 'function' }, function (t) {
 	t.end();
 });
 
-test('SafeSet', function (t) {
+test('SafeSet', { skip: ESSet !== Set && 'TODO: fix this test for envs where es-set has to polyfill' }, function (t) {
 	var ss = new SafeSet();
-	t.equal(isSet(ss), true, debug(ss) + ' is a Set');
+	t.equal(isSet(ss), true, debug(ss) + ' is a Set, when Set is present');
 
 	t.end();
 });