use new tap methods for testing

Author: lukekarrys

src/bin.mts

@@ -1,7 +1,6 @@
 #!/usr/bin/env node
 import type { RimrafAsyncOptions } from './index.js'
 import { rimraf } from './index.js'
-
 import { loadPackageJson } from 'package-json-from-dist'
 
 const { version } = loadPackageJson(import.meta.url, '../package.json')
@@ -253,18 +252,11 @@ const main = async (...args: string[]) => {
 
   return 0
 }
-main.help = help
-main.version = version
-
-export default main
 
-if (process.env.__TESTING_RIMRAF_BIN__ !== '1') {
-  const args = process.argv.slice(2)
-  main(...args).then(
-    code => process.exit(code),
-    er => {
-      console.error(er)
-      process.exit(1)
-    },
-  )
-}
+main(...process.argv.slice(2)).then(
+  code => process.exit(code),
+  er => {
+    console.error(er)
+    process.exit(1)
+  },
+)

src/index.ts

@@ -74,14 +74,14 @@ export const moveRemove = Object.assign(wrap(rimrafMoveRemove), {
 })
 
 export const rimrafSync = wrapSync((path, opt) =>
-  useNativeSync(opt) ?
+  useNativeSync?.(opt) ?
     rimrafNativeSync(path, opt)
   : rimrafManualSync(path, opt),
 )
 export const sync = rimrafSync
 
 const rimraf_ = wrap((path, opt) =>
-  useNative(opt) ? rimrafNative(path, opt) : rimrafManual(path, opt),
+  useNative?.(opt) ? rimrafNative(path, opt) : rimrafManual(path, opt),
 )
 export const rimraf = Object.assign(rimraf_, {
   rimraf: rimraf_,

src/use-native.ts

@@ -1,18 +1,21 @@
 import { RimrafAsyncOptions, RimrafOptions } from './index.js'
 
-const versArr = process.version.replace(/^v/, '').split('.')
-
-/* c8 ignore next */
-const [major = 0, minor = 0] = versArr.map(v => parseInt(v, 10))
+const [major = 0, minor = 0] = process.version
+  .replace(/^v/, '')
+  .split('.')
+  .map(v => parseInt(v, 10))
 const hasNative = major > 14 || (major === 14 && minor >= 14)
 
 // we do NOT use native by default on Windows, because Node's native
 // rm implementation is less advanced.  Change this code if that changes.
-export const useNative: (opt?: RimrafAsyncOptions) => boolean =
-  !hasNative || process.platform === 'win32' ?
-    () => false
-  : opt => !opt?.signal && !opt?.filter
-export const useNativeSync: (opt?: RimrafOptions) => boolean =
-  !hasNative || process.platform === 'win32' ?
-    () => false
-  : opt => !opt?.signal && !opt?.filter
+const doNotUseNative =
+  !hasNative || process.platform === 'win32' ? () => false : null
+
+// signal and filter options are not available on native methods
+const hasNativeOptions = (opt?: RimrafAsyncOptions | RimrafOptions) =>
+  !opt?.signal && !opt?.filter
+
+export const useNative =
+  doNotUseNative ?? ((o?: RimrafAsyncOptions) => hasNativeOptions(o))
+export const useNativeSync =
+  doNotUseNative ?? ((o?: RimrafOptions) => hasNativeOptions(o))