chore: make the typescript linter a bit more indulgent in test files

.changeset/gorgeous-pots-heal.md

@@ -0,0 +1,9 @@
+---
+"@yungezeit/eslint-typescript": patch
+"@yungezeit/eslint-base": patch
+"@yungezeit/eslint-cypress": patch
+"@yungezeit/eslint-storybook": patch
+"@yungezeit/eslint-vue": patch
+---
+
+Made linter a bit more indulgent in test files

presets/base/features/es.js

@@ -8,11 +8,11 @@ import js from '@eslint/js';
 const config = {
   rules: {
     ...js.configs.recommended.rules,
-    // Only allow console.info, console.warn, and console.error.
+    /** Only allow console.info, console.warn, and console.error. */
     'no-console': ['error', { allow: ['info', 'warn', 'error'] }],
-    // Sparse arrays aren't that confusing and may simplify code.
+    /** Sparse arrays aren't that confusing and may simplify code. */
     'no-sparse-arrays': 'off',
-    // Allow unused variables that start with an underscore for later use.
+    /** Allow unused variables that start with an underscore for later use. */
     'no-unused-vars': ['error', { argsIgnorePattern: '^_', varsIgnorePattern: '^_' }],
   },
 };

presets/typescript/features/test-files.js

@@ -5,8 +5,17 @@
 export const testFiles = {
   files: ['**/*.test.ts'],
   rules: {
+    // Be a bit more indulgent in test files.
     '@typescript-eslint/no-unsafe-assignment': 'off',
-    '@typescript-eslint/no-unsafe-argument': 'off',
+    '@typescript-eslint/no-unsafe-call': 'off',
+    '@typescript-eslint/no-unsafe-declaration-merging': 'off',
+    '@typescript-eslint/no-unsafe-enum-comparison': 'off',
+    '@typescript-eslint/no-unsafe-function-type': 'off',
+    '@typescript-eslint/no-unsafe-member-access': 'off',
+    '@typescript-eslint/no-unsafe-return': 'off',
+    '@typescript-eslint/no-unsafe-unary-minus': 'off',
+
+    // The following rules may be annoying when testing.
     'max-classes-per-file': 'off',
     'no-new': 'off',
   },

presets/typescript/features/ts.js

@@ -8,11 +8,20 @@ import tseslint from 'typescript-eslint';
  */
 const config = {
   rules: {
+    /** Forcing `import type { }` makes imports clearer and my relieve parsers. */
     '@typescript-eslint/consistent-type-imports': ['error', { prefer: 'type-imports' }],
+    /**
+     * There are some legit cases where index access is necessary.
+     * But keep in mind that members visibilities are lost when using index access.
+     */
     '@typescript-eslint/dot-notation': 'off',
+    /** Sometimes we just have no choice. */
     '@typescript-eslint/no-explicit-any': 'off',
+    /** Sometimes we just have no choice. */
     '@typescript-eslint/no-non-null-assertion': 'off',
+    /** This rule may be a bit unstable. */
     '@typescript-eslint/no-unnecessary-type-parameters': 'off',
+    /** Allow unused variables that start with an underscore for later use. */
     '@typescript-eslint/no-unused-vars': [
       'error',
       { argsIgnorePattern: '^_', varsIgnorePattern: '^_' },