Fix the requirement of 100% coverage...

isaacs · Nov 20, 2018 · 9e022a6 · 9e022a6
1 parent 072ac11
commit 9e022a6
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 3 deletions.
diff --git a/index.js b/index.js
@@ -11,7 +11,7 @@ var util = require('util')
 var Yallist = require('yallist')
 
 // use symbols if possible, otherwise just _props
-var hasSymbol = typeof Symbol === 'function'
+var hasSymbol = typeof Symbol === 'function' && !global._nodeLRUCacheForceNoSymbol
 var makeSymbol
 if (hasSymbol) {
   makeSymbol = function (key) {

diff --git a/package.json b/package.json
@@ -11,6 +11,7 @@
   "scripts": {
     "test": "tap test/*.js --100 -J",
     "posttest": "standard test/*.js index.js",
+    "coveragerport": "tap --coverage-report=html",
     "lintfix": "standard --fix test/*.js index.js",
     "preversion": "npm test",
     "postversion": "npm publish",

diff --git a/test/basic.js b/test/basic.js
@@ -1,5 +1,4 @@
 var test = require('tap').test
-if (global.NoSymbol) global.Symbol = false
 var LRU = require('../')
 
 test('basic', function (t) {

diff --git a/test/no-symbol.js b/test/no-symbol.js
@@ -1,3 +1,3 @@
 // hacky way to test that it still works when Symbol is not there
-global.NoSymbol = true
+global._nodeLRUCacheForceNoSymbol = true
 require('./basic.js')