✅ add eslint-config-typescript tests

Author: dudeofawesome

eslint-config-typescript/package.json

@@ -25,6 +25,11 @@
   },
   "peerDependencies": {
     "@dudeofawesome/eslint-config": "^1.0.0",
-    "typescript": "^4.7.4"
+    "typescript": "^5.0.0, <5.2.0"
+  },
+  "devDependencies": {
+    "@types/eslint": "^8.44.2",
+    "@types/node": "^20.6.3",
+    "ts-node": "^10.9.1"
   }
 }

eslint-config-typescript/test/index.ts

@@ -6,7 +6,11 @@ import {
   testRuleFail,
 } from '../../utils/testing/eslint';
 
-const linter = new ESLint({ cwd: __dirname });
+// const linter = new ESLint({ cwd: __dirname });
+const linter = new ESLint({
+  cwd: __dirname,
+  overrideConfigFile: '.eslintrc.yaml',
+});
 
 describe('eslint-config-typescript', () => {
   defaultTestSet(linter);
@@ -15,16 +19,8 @@ describe('eslint-config-typescript', () => {
     it(`should parse typescript`, () =>
       testNoFail(linter, `((a: string): string[] => a.split(''))()\n`, true));
 
-    it(`should parse javascript`, () =>
-      testNoFail(linter, `((a) => a.split(''))()\n`));
-
     it(`should not give eslint error on use before define`, () =>
-      testNoFail(
-        linter,
-        `Number(a);
-const a = 10;\n`,
-        true,
-      ));
+      testNoFail(linter, `Number(a);\nconst a = 10;\n`, true));
   });
 
   describe('fails', () => {

eslint-config-typescript/test/tsconfig.json

@@ -0,0 +1,4 @@
+{
+  "extends": "@dudeofawesome/typescript-configs/node.json",
+  "files": ["index.ts", "index.js"]
+}

eslint-config/test/index.ts

@@ -2,6 +2,7 @@ import { describe, it } from 'node:test';
 import { ESLint } from 'eslint';
 import {
   defaultTestSet,
+  filePath,
   testNoFail,
   testRuleFail,
 } from '../../utils/testing/eslint';
@@ -21,7 +22,7 @@ describe('eslint-config', () => {
     it(`should not parse typescript`, () =>
       linter
         .lintText(`((a: string): string[] => a.split(''))()\n`, {
-          filePath: 'input.ts',
+          filePath: filePath(true),
         })
         .then((res) => {
           equal(res[0].messages[0].ruleId, null);
@@ -31,6 +32,12 @@ describe('eslint-config', () => {
           );
         }));
 
+    it(`should fail no-console`, async () => {
+      const res = await linter.lintText(`console.log('foo');\n`);
+      // this gets 2 errors due to `console` not being defined
+      strictEqual(res[0].messages[0].ruleId, 'no-console');
+    });
+
     // TODO: test for shopify rule
 
     // TODO: test for prettier rule