✅ fix tests

dudeofawesome · May 3, 2024 · 443fc9b · 443fc9b
1 parent 9751306
commit 443fc9b
Show file tree

Hide file tree

Showing 21 changed files with 299 additions and 295 deletions.
diff --git a/.eslintrc.yaml b/.eslintrc.yaml
@@ -7,6 +7,7 @@ extends:
   - '@code-style/eslint-config-typescript'
   - '@code-style/eslint-config-esmodule'
 ignorePatterns:
+  - '.prettierrc.mjs'
   - '**/test/*/**'
   - 'packages/code-style'
   - 'packages/create-configs'

diff --git a/package-lock.json b/package-lock.json
diff --git a/packages/eslint-config-browser/test/index.ts b/packages/eslint-config-browser/test/index.ts
@@ -13,22 +13,18 @@ void describe('eslint-config-browser', () => {
   void describe('passes', () => {});
 
   void describe('fails', () => {
-    void it(
-      `should fail no-console`,
+    void it(`should fail no-console`, () =>
       testRuleFail({
         linter,
         ruleId: 'no-console',
         files: [{ code: `console.log('foo');\n` }],
-      }),
-    );
+      }));
 
-    void it(
-      `should fail radix`,
+    void it(`should fail radix`, () =>
       testRuleFail({
         linter,
         ruleId: 'radix',
         files: [{ code: `parseInt('10');\n` }],
-      }),
-    );
+      }));
   });
 });
diff --git a/packages/eslint-config-cli/test/index.ts b/packages/eslint-config-cli/test/index.ts
@@ -15,15 +15,11 @@ void describe('eslint-config-cli base', () => {
   defaultTestSet(linter);
 
   void describe('passes', () => {
-    void it(
-      `should pass radix`,
-      testNoFail({ linter, files: [{ code: `parseInt('10');\n` }] }),
-    );
+    void it(`should pass radix`, () =>
+      testNoFail({ linter, files: [{ code: `parseInt('10');\n` }] }));
 
-    void it(
-      `should pass no-console`,
-      testNoFail({ linter, files: [{ code: `console.log('foo');\n` }] }),
-    );
+    void it(`should pass no-console`, () =>
+      testNoFail({ linter, files: [{ code: `console.log('foo');\n` }] }));
   });
 
   void describe('fails', () => {});

diff --git a/packages/eslint-config-esmodule/test/index.ts b/packages/eslint-config-esmodule/test/index.ts
@@ -15,24 +15,20 @@ void describe('eslint-config-esmodule base', () => {
   defaultTestSet(linter);
 
   void describe('passes', () => {
-    void it(
-      `should pass esmodule import`,
+    void it(`should pass esmodule import`, () =>
       testNoFail({
         linter,
         files: [{ code: `import { foo } from 'console';\n\nfoo();\n` }],
-      }),
-    );
+      }));
   });
 
   void describe('fails', () => {
-    void it(
-      `should fail commonjs`,
+    void it(`should fail commonjs`, () =>
       testRuleFail({
         linter,
         ruleId: 'import/no-commonjs',
         files: [{ code: `const foo = require('foo');\n\nfoo();\n` }],
-      }),
-    );
+      }));
 
     void it(`should only log single duplicate-import error`, async () =>
       linter

diff --git a/packages/eslint-config-jest/test/index.ts b/packages/eslint-config-jest/test/index.ts
@@ -14,8 +14,7 @@ void describe('eslint-config-jest', () => {
   defaultTestSet(linter);
 
   void describe('passes', () => {
-    void it(
-      `should have jest globals in test file`,
+    void it(`should have jest globals in test file`, () =>
       testNoFail({
         linter,
         files: [
@@ -24,18 +23,15 @@ void describe('eslint-config-jest', () => {
             test: true,
           },
         ],
-      }),
-    );
+      }));
   });
 
   void describe('fails', () => {
-    void it(
-      `should not have jest globals in non-test file`,
+    void it(`should not have jest globals in non-test file`, () =>
       testRuleFail({
         linter,
         ruleId: 'no-undef',
         files: [{ code: `describe('test', () => {});\n` }],
-      }),
-    );
+      }));
   });
 });
diff --git a/packages/eslint-config-nest/test/index.ts b/packages/eslint-config-nest/test/index.ts
@@ -24,24 +24,19 @@ void describe('eslint-config-nest strict', () => {
   defaultTestSet(linter);
 
   void describe('passes', () => {
-    void it(
-      `should pass radix`,
-      testNoFail({ linter, files: [{ code: `parseInt('10');\n`, ts: true }] }),
-    );
+    void it(`should pass radix`, () =>
+      testNoFail({ linter, files: [{ code: `parseInt('10');\n`, ts: true }] }));
   });
 
   void describe('fails', () => {
-    void it(
-      `should fail no-console`,
+    void it(`should fail no-console`, () =>
       testRuleFail({
         linter,
         ruleId: 'no-console',
         files: [{ code: `console.log('foo');\n`, ts: true }],
-      }),
-    );
+      }));
 
-    void it(
-      `should fail no-restricted-imports`,
+    void it(`should fail no-restricted-imports`, () =>
       testRuleFail({
         linter,
         ruleId: 'no-restricted-imports',
@@ -52,7 +47,6 @@ void describe('eslint-config-nest strict', () => {
             ts: true,
           },
         ],
-      }),
-    );
+      }));
   });
 });
diff --git a/packages/eslint-config-nextjs/test/index.ts b/packages/eslint-config-nextjs/test/index.ts
@@ -20,8 +20,7 @@ void describe('eslint-config-nextjs', () => {
   defaultTestSet(linter);
 
   void describe('passes', () => {
-    void it(
-      `should parse jsx`,
+    void it(`should parse jsx`, () =>
       testNoFail({
         linter,
         files: [
@@ -30,31 +29,25 @@ void describe('eslint-config-nextjs', () => {
             react: true,
           },
         ],
-      }),
-    );
+      }));
   });
 
   void describe('fails', () => {
-    void it(
-      `should fail radix`,
+    void it(`should fail radix`, () =>
       testRuleFail({
         linter,
         ruleId: 'radix',
         files: [{ code: `parseInt('10');\n`, react: true }],
-      }),
-    );
+      }));
 
-    void it(
-      `should fail no-console`,
+    void it(`should fail no-console`, () =>
       testRuleFail({
         linter,
         ruleId: 'no-console',
         files: [{ code: `console.log('foo');\n`, react: true }],
-      }),
-    );
+      }));
 
-    void it(
-      `should fail react/jsx-key`,
+    void it(`should fail react/jsx-key`, () =>
       testRuleFail({
         linter,
         ruleId: 'react/jsx-key',
@@ -68,7 +61,6 @@ export const Bar = (props) => props.list.map((l) => <Foo text={l}></Foo>);
             react: true,
           },
         ],
-      }),
-    );
+      }));
   });
 });
diff --git a/packages/eslint-config-node/test/lenient.ts b/packages/eslint-config-node/test/lenient.ts
@@ -18,23 +18,19 @@ void describe('eslint-config-node lenient', () => {
   defaultTestSet(linter);
 
   void describe('passes', () => {
-    void it(
-      `should pass n/no-sync`,
+    void it(`should pass n/no-sync`, () =>
       testNoFail({
         linter,
         files: [{ code: `const { readFileSync } = require('fs');` }],
-      }),
-    );
+      }));
   });
 
   void describe('fails', () => {
-    void it(
-      `should fail no-console`,
+    void it(`should fail no-console`, () =>
       testRuleFail({
         linter,
         ruleId: 'no-console',
         files: [{ code: `console.log('foo');\n` }],
-      }),
-    );
+      }));
   });
 });
diff --git a/packages/eslint-config-node/test/strict.ts b/packages/eslint-config-node/test/strict.ts
@@ -14,24 +14,19 @@ void describe('eslint-config-node strict', () => {
   defaultTestSet(linter);
 
   void describe('passes', () => {
-    void it(
-      `should pass radix`,
-      testNoFail({ linter, files: [{ code: `parseInt('10');\n` }] }),
-    );
+    void it(`should pass radix`, () =>
+      testNoFail({ linter, files: [{ code: `parseInt('10');\n` }] }));
   });
 
   void describe('fails', () => {
-    void it(
-      `should fail no-console`,
+    void it(`should fail no-console`, () =>
       testRuleFail({
         linter,
         ruleId: 'no-console',
         files: [{ code: `console.log('foo');\n` }],
-      }),
-    );
+      }));
 
-    void it(
-      `should fail n/no-sync`,
+    void it(`should fail n/no-sync`, () =>
       testRuleFail({
         linter,
         ruleId: 'n/no-sync',
@@ -40,7 +35,6 @@ void describe('eslint-config-node strict', () => {
             code: `const { readFileSync } = require('fs');\n\nreadFileSync('test');\n`,
           },
         ],
-      }),
-    );
+      }));
   });
 });
diff --git a/packages/eslint-config-react/test/index.ts b/packages/eslint-config-react/test/index.ts
@@ -19,8 +19,7 @@ void describe('eslint-config-react', () => {
   defaultTestSet(linter);
 
   void describe('passes', () => {
-    void it(
-      `should parse jsx`,
+    void it(`should parse jsx`, () =>
       testNoFail({
         linter,
         files: [
@@ -29,31 +28,25 @@ void describe('eslint-config-react', () => {
             react: true,
           },
         ],
-      }),
-    );
+      }));
   });
 
   void describe('fails', () => {
-    void it(
-      `should fail radix`,
+    void it(`should fail radix`, () =>
       testRuleFail({
         linter,
         ruleId: 'radix',
         files: [{ code: `parseInt('10');\n`, react: true }],
-      }),
-    );
+      }));
 
-    void it(
-      `should fail no-console`,
+    void it(`should fail no-console`, () =>
       testRuleFail({
         linter,
         ruleId: 'no-console',
         files: [{ code: `console.log('foo');\n`, react: true }],
-      }),
-    );
+      }));
 
-    void it(
-      `should fail react/jsx-key`,
+    void it(`should fail react/jsx-key`, () =>
       testRuleFail({
         linter,
         ruleId: 'react/jsx-key',
@@ -67,7 +60,6 @@ export const Bar = (props) => props.list.map((l) => <Foo text={l}></Foo>);
             react: true,
           },
         ],
-      }),
-    );
+      }));
   });
 });