chore: remove code coverage bypass (#215)

We're at 100% now.

Author: wraithgar

lib/npa.js

@@ -66,8 +66,6 @@ function isFileSpec (spec) {
   if (isWindows) {
     return isWindowsFile.test(spec)
   }
-  // We never hit this in windows tests, obviously
-  /* istanbul ignore next */
   return isPosixFile.test(spec)
 }
 

package.json

@@ -47,7 +47,6 @@
     "node": "^20.17.0 || >=22.9.0"
   },
   "tap": {
-    "branches": 97,
     "nyc-arg": [
       "--exclude",
       "tap-snapshots/**"

test/posix.js

@@ -0,0 +1,26 @@
+// redefine process.platform before any requires so that we don't cache a require that got the non-redefined value
+const { platform } = process
+Object.defineProperty(process, 'platform', { value: 'linux' })
+
+const t = require('tap')
+const npa = require('..')
+
+t.teardown(() => {
+  Object.defineProperty(process, 'platform', { value: platform })
+})
+
+// These are purely for coverage
+// We need tests that act like linux so that code paths are covered both ways when testing in windows itself.
+//
+t.test('file spec', t => {
+  const actual = npa('./local')
+  t.equal(actual.type, 'directory')
+  t.equal(actual.raw, './local')
+  t.end()
+})
+
+t.test('encoded path', t => {
+  const actual = npa('./path\\backslash')
+  t.equal(actual.rawSpec, './path\\backslash')
+  t.end()
+})