test: test fixtures in parallel

Author: aminya

.github/workflows/CI.yml

@@ -35,15 +35,15 @@ jobs:
       - name: Install dependencies
         run: pnpm install
 
-      # - name: Format ✨
-      #   run: pnpm run test.format
+      - name: Format ✨
+        run: pnpm run test.format
 
       - name: Lint ✨
         run: pnpm run test.lint
 
-      # - name: Tests
-      #   run: |
-      #     pnpm test
+      - name: Tests
+        run: |
+          pnpm test
 
   # Release:
   #   needs: [Test]

.gitignore

@@ -16,3 +16,5 @@ spec/fixtures/atom-community
 spec/fixtures/atom-minimap
 spec/fixtures/aminya
 spec/fixtures/steelbrain
+
+*.tar.gz

package.json

@@ -14,10 +14,10 @@
     "*.js"
   ],
   "scripts": {
-    "dev": "tsc --pretty --watch",
     "build": "tsc --pretty",
     "bump": "ncu -u -x coffeescript,execa,read-pkg-up,make-synchronous && pnpm update && pnpm dedupe",
     "clean": "shx rm -rf ./spec/fixtures/ ./dist",
+    "dev": "tsc --pretty --watch",
     "format": "prettier --write .",
     "lint": "eslint ./ --ignore dist --fix",
     "prepare": "pnpm run build",

spec/test.cjs

@@ -45,42 +45,48 @@ async function testLint(packedPkg, testRepo, isWorkspace = false, isSilent = fal
     throw e
   }
 
-  const result = await execa.command("eslint .", { cwd: distFolder, stdout: !isSilent ? "inherit" : "pipe" })
+  const result = await execa.command("eslint . --cache", { cwd: distFolder, stdout: !isSilent ? "inherit" : "pipe" })
   if (result.failed) {
     throw new Error("An error happened")
   }
 }
 
 /** Main entry */
-;(async function main() {
+async function main() {
   const root = resolve(dirname(__dirname))
   const packedPkg = join(root, `${pkg.name}-${pkg.version}.tgz`)
   rm("-rf", packedPkg)
   await execa.command("pnpm pack", { cwd: root })
 
   const errs = []
-  for (const testRepo of testRepos) {
-    try {
-      // We want to observe the output in order, so we await inside loop
-      await testLint(packedPkg, testRepo, false) // eslint-disable-line no-await-in-loop
-    } catch (err) {
-      console.error(err)
-      errs.push(err)
-    }
-  }
+  await Promise.all([
+    ...testRepos.map(async (testRepo) => {
+      try {
+        await testLint(packedPkg, testRepo, false)
+      } catch (err) {
+        console.error(err)
+        errs.push(err)
+      }
+    }),
+    ...testWorkspaces.map(async (testWorkspace) => {
+      try {
+        await testLint(packedPkg, testWorkspace, true, true)
+      } catch (err) {
+        console.error(err)
+        errs.push(err)
+      }
+    }),
+  ])
 
-  for (const testWorkspace of testWorkspaces) {
-    try {
-      await testLint(packedPkg, testWorkspace, true, true) // eslint-disable-line no-await-in-loop
-    } catch (err) {
-      console.error(err)
-      errs.push(err)
-    }
-  }
   if (errs.length !== 0) {
     rm("-rf", packedPkg)
     throw new Error(`${errs.length} packages failed the tests. See the above.`)
   }
 
   rm("-rf", packedPkg)
-})()
+}
+
+main().catch((err) => {
+  console.error(err)
+  process.exit(1)
+})

src/index-react.cts

@@ -2,22 +2,22 @@ import type { Linter } from "eslint"
 import base from "./index.cjs"
 import { tsConfigs } from "./typescript.cjs"
 import reactPlugin from "eslint-plugin-react"
-import globals from "globals";
+import globals from "globals"
 
 const reactTypeScript: Linter.Config = {
   ...tsConfigs,
-  ...reactPlugin.configs.flat.recommended
-};
+  ...reactPlugin.configs.flat.recommended,
+}
 
 const nonStrictConfig: Linter.Config[] = [
   ...base,
   // JavaScript:
   {
-    files: ['**/*.{js,mjs,cjs,jsx,mjsx,ts,tsx,mtsx}'],
+    files: ["**/*.{js,mjs,cjs,jsx,mjsx,ts,tsx,mtsx}"],
     ...reactPlugin.configs.flat.recommended,
   },
   {
-    files: ['**/*.{js,mjs,cjs,jsx,mjsx,ts,tsx,mtsx}'],
+    files: ["**/*.{js,mjs,cjs,jsx,mjsx,ts,tsx,mtsx}"],
     languageOptions: {
       globals: {
         ...globals.serviceworker,

src/index-solid.cts

@@ -5,8 +5,8 @@ import solid from "eslint-plugin-solid"
 
 const solidTypeScript: Linter.Config = {
   ...tsConfig,
-  ...solid.configs["flat/typescript"]
-};
+  ...solid.configs["flat/typescript"],
+}
 
 const nonStrictConfig: Linter.Config[] = [
   ...base,

src/typescript.cts

@@ -148,7 +148,7 @@ export const tsConfig: Linter.Config = {
     ...pluginImportTypeScriptRulesExtra,
     ...importPlugin.configs.recommended.rules,
   },
-};
+}
 
 export const tsConfigs: Linter.Config[] = [
   // TypeScript files