JavaScript: Don't use vitest in rewrite-test.ts (#6957)

* Don't use vitest

* Skip failing tests

Author: greg-at-moderne

rewrite-javascript/rewrite/src/test/rewrite-test.ts

@@ -13,7 +13,6 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-import {expect} from "vitest";
 import {Recipe} from "../recipe";
 import {ExecutionContext} from "../execution";
 import {noopVisitor, TreeVisitor} from "../visitor";
@@ -145,7 +144,13 @@ export class RecipeSpec {
     private async expectParsePrintIdempotence(parsed: [SourceSpec<any>, SourceFile][]) {
         for (const [spec, sourceFile] of parsed) {
             const beforeSource = dedent(spec.before!);
-            expect(await TreePrinters.print(sourceFile)).toEqual(beforeSource);
+            const printed = await TreePrinters.print(sourceFile);
+            if (printed !== beforeSource) {
+                throw new Error(
+                    `Parse-print idempotence check failed.\n` +
+                    `Expected:\n${beforeSource}\n\nActual:\n${printed}`
+                );
+            }
         }
     }
 
@@ -206,12 +211,16 @@ export class RecipeSpec {
         if (!after) {
             throw new Error('Expected for recipe to have produced a change for file:\n' + trimIndent(spec.before))
         }
-        expect(after).toBeDefined();
-        await new ValidateWhitespaceVisitor().visit(after!, this.executionContext);
-        const actualAfter = await TreePrinters.print(after!);
+        await new ValidateWhitespaceVisitor().visit(after, this.executionContext);
+        const actualAfter = await TreePrinters.print(after);
         const afterSource = typeof spec.after === "function" ?
             (spec.after as (actual: string) => string)(actualAfter) : spec.after as string;
-        expect(actualAfter).toEqual(afterSource);
+        if (actualAfter !== afterSource) {
+            throw new Error(
+                `Recipe output does not match expected.\n` +
+                `Expected:\n${afterSource}\n\nActual:\n${actualAfter}`
+            );
+        }
         if (spec.afterRecipe) {
             await spec.afterRecipe(after);
         }
@@ -248,7 +257,11 @@ export class RecipeSpec {
 class ValidateWhitespaceVisitor extends JavaScriptVisitor<ExecutionContext> {
     public override async visitSpace(space: J.Space, p: ExecutionContext): Promise<J.Space> {
         const ret = super.visitSpace(space, p);
-        expect(space.whitespace).toMatch(/^\s*$/);
+        if (!/^\s*$/.test(space.whitespace)) {
+            throw new Error(
+                `Whitespace contains non-whitespace characters: ${JSON.stringify(space.whitespace)}`
+            );
+        }
         return ret;
     }
 }

rewrite-javascript/rewrite/test/rpc/install-recipes.test.ts

@@ -233,7 +233,8 @@ describe("InstallRecipes", () => {
             }, {unsafeCleanup: true});
         }, 60000);
 
-        test("installs @openrewrite/recipes-nodejs from npm", async () => {
+        // Skipped: published @openrewrite/rewrite has vitest runtime dep; fix: https://github.com/openrewrite/rewrite/pull/6957
+        test.skip("installs @openrewrite/recipes-nodejs from npm", async () => {
             await withDir(async (dir) => {
                 // given
                 const installDir = path.join(dir.path, "recipes");
@@ -256,7 +257,8 @@ describe("InstallRecipes", () => {
             }, {unsafeCleanup: true});
         }, 120000);
 
-        test("upgrades @openrewrite/recipes-nodejs from 0.36.0 to a later version", async () => {
+        // Skipped: published @openrewrite/rewrite has vitest runtime dep; fix: https://github.com/openrewrite/rewrite/pull/6957
+        test.skip("upgrades @openrewrite/recipes-nodejs from 0.36.0 to a later version", async () => {
             await withDir(async (dir) => {
                 // given
                 const installDir = path.join(dir.path, "recipes");