chore: remove .only from test (#6108)

* chore: remove `.only` from test

Looks like this has been here for 16 months 🤦🏼: #5386.

* test: update snapshot

* ci: add lint rules to prevent skip and only in test suite

* test: unskip test

* ci: lint warnings fail build, so commenting out the rules for now

* test: undo snapshot update

* test: replace snapshot test with assertions

* test: remove core plugin override test

* test: ensure the same test coverage as the snapshot had before

* test: pr feedback

* ci: regenerate lockfile

---------

Co-authored-by: Mateusz Bocian <[email protected]>

Author: serhalp

.eslintrc.cjs

@@ -13,7 +13,7 @@ const config = {
     ecmaVersion: 'latest',
     sourceType: 'module',
   },
-  plugins: ['import', '@typescript-eslint'],
+  plugins: ['import', '@typescript-eslint', 'ava', 'vitest'],
 
   ignorePatterns: [
     // TODO: remove when they are migrated to typescript
@@ -60,6 +60,14 @@ const config = {
         },
       },
     ],
+
+    // -----------------------------------------------------------
+    // Test rules
+    'ava/no-only-test': 'error',
+    // 'ava/no-skip-test': 'error', // Uncomment once we remove all skipped tests
+    // 'vitest/no-commented-out-tests': 'error', // Uncomment once we remove all commented out tests
+    'vitest/no-disabled-tests': 'error',
+    'vitest/no-focused-tests': 'error',
   },
   overrides: [
     {