Merge pull request #9125 from emberjs/test-linting/all-the-eslints

Author: gitKrystan

config/eslint/diagnostic.cjs

@@ -0,0 +1,53 @@
+const isolation = require('./isolation.cjs');
+
+function defaults(config = {}) {
+  return {
+    files: config.files || ['tests/**/*-test.{js,ts}'],
+    // HACK: diagnostic API significantly overlaps with the qunit API, so we're
+    // using the qunit plugin to lint diagnostic files.
+    extends: ['plugin:qunit/recommended'],
+    rules: Object.assign(
+      isolation.rules({
+        allowedImports: ['@ember/debug', '@ember/test-helpers', ...(config.allowedImports ?? [])],
+      }),
+      config?.rules,
+      {
+        'qunit/no-assert-equal': 'off',
+        'qunit/no-assert-logical-expression': 'off',
+        'qunit/no-conditional-assertions': 'off',
+        'qunit/no-early-return': 'off',
+        'qunit/no-ok-equality': 'off',
+        'qunit/require-expect': 'off',
+      }
+    ),
+  };
+}
+
+function config() {
+  return {
+    files: ['./diagnostic.js'],
+    parserOptions: {
+      sourceType: 'module',
+      ecmaVersion: 2022,
+    },
+    env: {
+      browser: false,
+      node: true,
+      es6: true,
+    },
+    plugins: ['n'],
+    extends: 'plugin:n/recommended',
+    rules: {
+      // It's ok to use unpublished files here since we don't ship these
+      'n/no-unpublished-require': 'off',
+      // TODO: '@warp-drive/diagnostic/server/default-setup.js' keeps setting these off
+      'n/no-missing-import': 'off',
+      'n/no-unpublished-import': 'off',
+    },
+  };
+}
+
+module.exports = {
+  config,
+  defaults,
+};

config/eslint/node.cjs

@@ -1,9 +1,12 @@
 function defaults(config) {
   const result = {
-    files: !config?.useModules ? ['./index.js', './addon-main.cjs', './addon-main.js'] : [],
+    files: !config?.useModules
+      ? ['./index.js', './addon-main.cjs', './addon-main.js', './config/environment.js', './config/targets.js']
+      : [],
     parserOptions: {
       sourceType: config?.useModules ? 'module' : 'script',
       ecmaVersion: 2022,
+      ...(config?.parserOptions ?? {}),
     },
     env: {
       browser: false,
@@ -24,7 +27,14 @@ function defaults(config) {
 
 function config(config) {
   const result = {
-    files: ['./babel.config.js', './.eslintrc.cjs', './rollup.config.mjs'],
+    files: [
+      './.eslintrc.cjs',
+      './babel.config.js',
+      './ember-cli-build.js',
+      './rollup.config.mjs',
+      './testem.js',
+      './config/ember-try.js',
+    ],
     parserOptions: {
       sourceType: config?.useModules ? 'module' : 'script',
       ecmaVersion: 2022,

config/eslint/qunit.cjs

@@ -9,14 +9,19 @@ function defaults(config = {}) {
         allowedImports: [
           '@ember/debug',
           '@ember/test-helpers',
+          '@ember/test-waiters',
           'ember-qunit',
           'qunit',
           ...(config.allowedImports ?? []),
         ],
       }),
       config?.rules,
       {
+        'qunit/no-assert-logical-expression': 'off',
+        'qunit/no-conditional-assertions': 'off',
+        'qunit/no-early-return': 'off',
         'qunit/no-ok-equality': 'off',
+        'qunit/require-expect': 'off',
       }
     ),
   };

packages/holodeck/package.json

@@ -41,7 +41,7 @@
     "holodeck": "./bin/holodeck.js"
   },
   "scripts": {
-    "build:types": "tsc --build",
+    "build:types": "tsc --build --force",
     "build:client": "rollup --config",
     "_build": "bun run build:client && bun run build:types",
     "start": "rollup --config --watch",