fix: remove unnecessary package.json values (#6555)

lukekarrys · web-flow · commit 40d7e09aa9c0 · 2023-06-14T11:55:17.000-07:00
diff --git a/package.json b/package.json
@@ -34,7 +34,6 @@
   },
   "directories": {
     "doc": "./doc",
-    "lib": "./lib",
     "man": "./man"
   },
   "main": "./index.js",
@@ -213,14 +212,10 @@
     "test-all": "node . run test -ws -iwr --if-present",
     "snap": "tap",
     "prepack": "node . run build -w docs",
-    "test:nocleanup": "NO_TEST_CLEANUP=1 node . run test --",
-    "sudotest": "sudo node . run run test --",
-    "sudotest:nocleanup": "sudo NO_TEST_CLEANUP=1 node . run test --",
     "posttest": "node . run lint",
     "lint": "eslint \"**/*.js\"",
     "lintfix": "node . run lint -- --fix",
     "lint-all": "node . run lint -ws -iwr --if-present",
-    "prelint": "rimraf test/npm_cache*",
     "resetdeps": "node scripts/resetdeps.js",
     "rp-pull-request": "node scripts/update-authors.js",
     "postlint": "template-oss-check",
@@ -231,7 +226,6 @@
       "LC_ALL=sk"
     ],
     "color": 1,
-    "files": "test/{lib,bin,index.js}",
     "timeout": 600,
     "nyc-arg": [
       "--exclude",
diff --git a/tap-snapshots/test/lib/commands/publish.js.test.cjs b/tap-snapshots/test/lib/commands/publish.js.test.cjs
@@ -115,7 +115,6 @@ Object {
   "description": "a package manager for JavaScript",
   "directories": Object {
     "doc": "./doc",
-    "lib": "./lib",
     "man": "./man",
   },
   "exports": Object {
diff --git a/workspaces/arborist/package.json b/workspaces/arborist/package.json
@@ -74,18 +74,12 @@
   "bin": {
     "arborist": "bin/index.js"
   },
-  "//": "sk test-env locale to catch locale-specific sorting",
   "tap": {
-    "color": true,
     "after": "test/fixtures/cleanup.js",
     "test-env": [
-      "NODE_OPTIONS=--no-warnings",
       "LC_ALL=sk"
     ],
-    "node-arg": [
-      "--no-warnings",
-      "--no-deprecation"
-    ],
+    "color": 1,
     "timeout": "360",
     "nyc-arg": [
       "--exclude",
diff --git a/workspaces/arborist/test/isolated-mode.js b/workspaces/arborist/test/isolated-mode.js
@@ -902,7 +902,7 @@ tap.test('bundled dependencies of internal packages', async t => {
   rule7.apply(t, dir, resolved, asserted)
 
   const isexePath = path.join(dir, 'node_modules', 'isexe')
-  t.equals(isexePath, fs.realpathSync(isexePath))
+  t.equal(isexePath, fs.realpathSync(isexePath))
 })
 
 tap.test('nested bundled dependencies of internal packages', async t => {
@@ -954,7 +954,7 @@ tap.test('nested bundled dependencies of internal packages', async t => {
   rule7.apply(t, dir, resolved, asserted)
 
   const isexePath = path.join(dir, 'node_modules', 'isexe')
-  t.equals(isexePath, fs.realpathSync(isexePath))
+  t.equal(isexePath, fs.realpathSync(isexePath))
 })
 
 tap.test('nested bundled dependencies of workspaces', async t => {
@@ -998,9 +998,9 @@ tap.test('nested bundled dependencies of workspaces', async t => {
   rule7.apply(t, dir, resolved, asserted)
 
   const isexePath = path.join(dir, 'node_modules', 'isexe')
-  t.equals(isexePath, fs.realpathSync(isexePath))
+  t.equal(isexePath, fs.realpathSync(isexePath))
   const whichPath = path.join(dir, 'node_modules', 'which')
-  t.equals(whichPath, fs.realpathSync(whichPath))
+  t.equal(whichPath, fs.realpathSync(whichPath))
 })
 
 tap.test('nested bundled dependencies of workspaces with conflicting isolated dep', async t => {
@@ -1054,9 +1054,9 @@ tap.test('nested bundled dependencies of workspaces with conflicting isolated de
   rule7.apply(t, dir, resolved, asserted)
 
   const isexePath = path.join(dir, 'packages', 'bar', 'node_modules', 'isexe')
-  t.equals(isexePath, fs.realpathSync(isexePath))
+  t.equal(isexePath, fs.realpathSync(isexePath))
   const whichPath = path.join(dir, 'packages', 'bar', 'node_modules', 'which')
-  t.equals(whichPath, fs.realpathSync(whichPath))
+  t.equal(whichPath, fs.realpathSync(whichPath))
 })
 
 tap.test('adding a dependency', async t => {
