fix: run tests on windows (#127)

.github/workflows/tests.yml

@@ -1,6 +1,8 @@
 name: Node CI
 
-on: [push]
+on:
+  - push
+  - pull_request
 
 jobs:
   test:
@@ -9,7 +11,7 @@ jobs:
     strategy:
       matrix:
         node_version: [10, 12, 14, 15]
-        os: [ubuntu-latest]
+        os: [ubuntu-latest, windows-latest]
     steps:
     - uses: actions/checkout@v1
     - name: Use Node.js ${{ matrix.node_version }}

cli.js

@@ -52,6 +52,10 @@ if (args.help || args._.length === 0) {
   process.exit(1)
 }
 
+// windows leaves leading/trailing quotes on strings needed on unix to
+// stop shells from doing path expansion, so strip them if present
+args._ = args._.map(stripQuotes)
+
 function extensions (arg) {
   if (!arg) return undefined
   const extensions = {}
@@ -73,6 +77,18 @@ function extensions (arg) {
   return extensions
 }
 
+function stripQuotes (string) {
+  if (string.startsWith("'") || string.startsWith('"')) {
+    string = string.slice(1)
+  }
+
+  if (string.endsWith("'") || string.endsWith('"')) {
+    string = string.slice(0, -1)
+  }
+
+  return string
+}
+
 check({
   path: args._.shift(),
   entries: args._,

index.js

@@ -26,6 +26,9 @@ const promisedResolveModule = (file, options) => new Promise((resolve, reject) =
 async function resolveGlobbedPath (entries, cwd) {
   if (typeof entries === 'string') entries = [entries]
 
+  // replace backslashes for forward slashes for windows
+  entries = entries.map(entry => entry.replace(/\\/g, '/'))
+
   debug('globby resolving', entries)
 
   const resolvedEntries = await globby(entries, {

package.json

@@ -13,6 +13,7 @@
     "lint": "standard",
     "test-cli:custom-detective": "node cli.js test/ -e js:detective-cjs",
     "test-cli:glob": "node cli.js 'test/**/*.js' --no-default-entries",
+    "test-cli:multi-glob": "node cli.js test/foo.js 'test/*.js' \"test/donkey/*.js\" --no-default-entries",
     "test-cli:main-as-file": "node cli.js test/index.js",
     "test-cli:simple": "node cli.js test/",
     "test-cli": "run-p test-cli:*",