borp 0.18.0 → 0.19.0

package/.github/workflows/ci.yml

@@ -16,7 +16,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [18.x, 20.x, 21.x, 22.x]
+        node-version: [18.x, 20.x, 21.x, 22.x, 23.x]
         os: [ubuntu-latest, windows-latest]
         exclude:
           - os: windows-latest

package/README.md

@@ -27,6 +27,9 @@ borp --reporter foo:stderr
 
 # with a local custom reporter
 borp --reporter ./lib/some-reporter.mjs
+
+# matching all test.js files except ones in nested node_modules directories
+borp 'test/**/*.test.js' '!test/**/node_modules/**/*.test.js'
 ```
 
 Borp will automatically run all tests files matching `*.test.{js|ts}`.
@@ -137,6 +140,9 @@ full path to some yaml file.
 The current supported options are:
 
 + `files` (string[]): An array of test files to include. Globs are supported.
+  Note: any glob that starts with a `!` (bang character) will be treated as
+  an ignore glob, e.g. `'!test/**/node_modules/**/*'` will ignore all files
+  in nested `node_modules` directories that would otherwise be matched.
 + `reporters` (string[]): An array of reporters to use. May be relative path
 strings, or module name strings.
 

package/lib/run.js

@@ -196,15 +196,27 @@ export default async function runWithTypeScript (config) {
     if (prefix) {
       files = files.map((file) => join(prefix, file.replace(/ts$/, 'js')))
     }
+
     const expandedFiles = []
+    const globs = []
     for (let i = 0; i < files.length; i += 1) {
       if (files[i].includes('*') === false) {
         expandedFiles.push(files[i])
         continue
       }
-      const parsed = await glob(files[i].replace(/^['"]/, '').replace(/['"]$/, ''), { ignore, cwd, windowsPathsNoEscape: true })
+      const pattern = files[i].replace(/^['"]/, '').replace(/['"]$/, '')
+      if (pattern[0] === '!') {
+        ignore.push(pattern.slice(1))
+        continue
+      }
+      globs.push(pattern)
+    }
+
+    if (globs.length > 0) {
+      const parsed = await glob(globs, { ignore, cwd, windowsPathsNoEscape: true })
       Array.prototype.push.apply(expandedFiles, parsed)
     }
+
     files = expandedFiles
   } else if (config.pattern) {
     let pattern = config.pattern
@@ -221,6 +233,7 @@ export default async function runWithTypeScript (config) {
 
   config.files = files
 
+  config.coverage = false
   stream = run(config)
 
   stream.on('close', () => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "borp",
-  "version": "0.18.0",
+  "version": "0.19.0",
   "type": "module",
   "description": "node:test wrapper with TypeScript support",
   "main": "borp.js",
@@ -14,7 +14,7 @@
   "scripts": {
     "clean": "rm -rf fixtures/*/dist .test-*",
     "lint": "standard | snazzy",
-    "unit": "node borp.js --ignore \"fixtures/**/*\" --coverage --coverage-exclude \"fixtures/**/*\" --coverage-exclude \"test*/**/*\"",
+    "unit": "node borp.js --ignore \"fixtures/**/*\" -C --coverage-exclude \"fixtures/**/*\" --coverage-exclude \"test*/**/*\"",
     "test": "npm run clean ; npm run lint && npm run unit"
   },
   "keywords": [],
@@ -26,6 +26,7 @@
     "@sinonjs/fake-timers": "^13.0.2",
     "@types/node": "^22.2.0",
     "desm": "^1.3.0",
+    "semver": "^7.6.3",
     "snazzy": "^9.0.0",
     "standard": "^17.1.0",
     "typescript": "^5.3.2"

package/test/cli.test.js

@@ -112,7 +112,24 @@ test('interprets globs for files', async () => {
     cwd
   })
 
-  strictEqual(stdout.indexOf('tests 2') >= 0, true)
+  strictEqual(stdout.indexOf('✔ add') >= 0, true)
+  strictEqual(stdout.indexOf('✔ add2') >= 0, true)
+  strictEqual(stdout.indexOf('✔ a thing'), -1)
+})
+
+test('interprets globs for files with an ignore rule', async () => {
+  const cwd = join(import.meta.url, '..', 'fixtures', 'files-glob')
+  const { stdout } = await execa('node', [
+    borp,
+    '\'**/*.test.js\'',
+    '\'!test1/**/node_modules/**/*\''
+  ], {
+    cwd
+  })
+
+  strictEqual(stdout.indexOf('✔ add') >= 0, true)
+  strictEqual(stdout.indexOf('✔ add2') >= 0, true)
+  strictEqual(stdout.indexOf('✔ a thing'), -1)
 })
 
 test('Post compile script should be executed when --post-compile  is sent with esm', async () => {

package/test/coverage.test.js

@@ -22,7 +22,7 @@ test('coverage', async () => {
 test('coverage excludes', async () => {
   const res = await execa('node', [
     borp,
-    '--coverage',
+    '-C',
     '--coverage-exclude=src'
   ], {
     cwd: join(import.meta.url, '..', 'fixtures', 'ts-esm')

package/test/watch.test.js

@@ -5,9 +5,10 @@ import { join } from 'desm'
 import { mkdtemp, cp, writeFile, rm } from 'node:fs/promises'
 import path from 'node:path'
 import { once } from 'node:events'
+import semver from 'semver'
 
-// These tests are currently broken on node v22
-const skip = process.platform === 'darwin' && process.version.startsWith('v22')
+// These tests are currently broken on some node versions
+const skip = process.platform === 'darwin' && semver.satisfies(process.version, '>=20.16.0 <22.10.0')
 
 test('watch', { skip }, async (t) => {
   const { strictEqual, completed, match } = tspl(t, { plan: 3 })