fix vitest (#1444)

* fix vitest

* Update patches/@[email protected]

* pnpm i

* try

* fix!

* prettier

* try

* pnpm i

* a

Author: dimaMachina

package.json

@@ -42,7 +42,7 @@
     "rimraf": "4.1.2",
     "tsx": "3.12.3",
     "typescript": "4.9.5",
-    "vitest": "0.26.3"
+    "vitest": "0.28.4"
   },
   "resolutions": {
     "graphql": "16.6.0"
@@ -52,7 +52,7 @@
       "[email protected]": "patches/[email protected]",
       "[email protected]": "patches/[email protected]",
       "[email protected]": "patches/[email protected]",
-      "vitest@0.26.3": "patches/[email protected].patch"
+      "@vitest/runner@0.28.4": "patches/@[email protected].patch"
     }
   }
 }

packages/plugin/src/testkit.ts

@@ -68,26 +68,7 @@ export class GraphQLRuleTester extends RuleTester {
       invalid: GraphQLInvalidTestCase<Options>[];
     },
   ): void {
-    const ruleTests = Linter.version.startsWith('8')
-      ? tests
-      : {
-          valid: tests.valid.map(test => {
-            if (typeof test === 'string') {
-              return test;
-            }
-            // eslint-disable-next-line @typescript-eslint/no-unused-vars
-            const { name, ...testCaseOptions } = test;
-            return testCaseOptions;
-          }),
-          invalid: tests.invalid.map(test => {
-            // ESLint 7 throws an error on CI - Unexpected top-level property "name"
-            // eslint-disable-next-line @typescript-eslint/no-unused-vars
-            const { name, ...testCaseOptions } = test;
-            return testCaseOptions;
-          }),
-        };
-    super.run(ruleId, rule as any, ruleTests);
-
+    super.run(ruleId, rule as any, tests);
     const linter = new Linter();
     linter.defineRule(ruleId, rule as any);
 

patches/@[email protected]

@@ -0,0 +1,17 @@
+# vitest's `--passWithNoTests` flag doesn't work when valid/invalid cases contain empty array 🤷‍
+
+diff --git a/dist/index.js b/dist/index.js
+index 14cbb2afd10bc64465281c25dc85a766fa3912ac..22dbf48554fcd2863871dfb003ee194af34532a9 100644
+--- a/dist/index.js
++++ b/dist/index.js
+@@ -545,9 +545,7 @@ async function runSuite(suite, runner) {
+     if (!hasTests(suite)) {
+       suite.result.state = "fail";
+       if (!suite.result.error) {
+-        const error = processError(new Error(`No test found in suite ${suite.name}`));
+-        suite.result.error = error;
+-        suite.result.errors = [error];
++        suite.result.state = "pass"
+       }
+     } else if (hasFailed(suite)) {
+       suite.result.state = "fail";