refactor(vue-script-setup-converter): change to inline snapshot (#55)

* test: change to inline snapshot * test: change to inline snapshot * test: change to inline snapshot * test: change to inline snapshot * test: change to inline snapshot
wattanx · May 10, 2024 · 1ab6a17 · 1ab6a17
1 parent 0537de9
commit 1ab6a17
Show file tree

Hide file tree

Showing 5 changed files with 91 additions and 162 deletions.
diff --git a/packages/vue-script-setup-converter/src/lib/converter/emitsConverter.test.ts b/packages/vue-script-setup-converter/src/lib/converter/emitsConverter.test.ts
@@ -1,8 +1,6 @@
 import { expect, test } from "vitest";
 import { CallExpression, ScriptTarget, SyntaxKind, Project } from "ts-morph";
 import { parse } from "@vue/compiler-sfc";
-import prettier from "prettier";
-import parserTypeScript from "prettier/parser-typescript";
 import { getNodeByKind } from "../helpers/node";
 import { convertEmits } from "./emitsConverter";
 
@@ -24,12 +22,7 @@ const parseScript = (input: string, lang: "js" | "ts" = "js") => {
 
   const emits = convertEmits(callexpression as CallExpression, lang);
 
-  const formatedText = prettier.format(emits, {
-    parser: "typescript",
-    plugins: [parserTypeScript],
-  });
-
-  return formatedText;
+  return emits;
 };
 
 const source = `<script>
@@ -52,10 +45,9 @@ export default defineComponent({
 test("defineEmits", () => {
   const output = parseScript(source, "js");
 
-  const expected = `const emit = defineEmits({ change: (value) => true });
-`;
-
-  expect(output).toBe(expected);
+  expect(output).toMatchInlineSnapshot(
+    `"const emit = defineEmits({change: (value) => true});"`
+  );
 });
 
 test("typed defineEmits", () => {
@@ -77,9 +69,9 @@ export default defineComponent({
 </script>`;
   const output = parseScript(source, "ts");
 
-  const expected = `const emit = defineEmits<{ (e: "change", value: number): void }>();
-`;
-  expect(output).toBe(expected);
+  expect(output).toMatchInlineSnapshot(
+    `"const emit = defineEmits<{(e: 'change', value: number): void}>();"`
+  );
 });
 
 test("string array", () => {
@@ -101,7 +93,9 @@ export default defineComponent({
 
   const expected = `const emit = defineEmits(["change"]);
 `;
-  expect(output).toBe(expected);
+  expect(output).toMatchInlineSnapshot(
+    `"const emit = defineEmits(['change']);"`
+  );
 });
 
 test("validation", () => {
@@ -125,11 +119,9 @@ export default defineComponent({
 </script>`;
   const output = parseScript(source, "ts");
 
-  const expected = `const emit = defineEmits({
-  change: (value: number) => {
-    return value !== 0;
-  },
-});
-`;
-  expect(output).toBe(expected);
+  expect(output).toMatchInlineSnapshot(`
+    "const emit = defineEmits({change: (value: number) => {
+          return value !== 0;
+        }});"
+  `);
 });
diff --git a/packages/vue-script-setup-converter/src/lib/converter/importDeclarationConverter.test.ts b/packages/vue-script-setup-converter/src/lib/converter/importDeclarationConverter.test.ts
@@ -1,8 +1,6 @@
 import { expect, describe, it } from "vitest";
 import { ScriptTarget, Project } from "ts-morph";
 import { parse } from "@vue/compiler-sfc";
-import prettier from "prettier";
-import parserTypeScript from "prettier/parser-typescript";
 import { convertImportDeclaration } from "./importDeclarationConverter";
 
 const parseScript = (input: string) => {
@@ -20,12 +18,7 @@ const parseScript = (input: string) => {
   const sourceFile = project.createSourceFile("s.tsx", script?.content ?? "");
   const convertedImportDeclarationText = convertImportDeclaration(sourceFile);
 
-  const formatedText = prettier.format(convertedImportDeclarationText, {
-    parser: "typescript",
-    plugins: [parserTypeScript],
-  });
-
-  return formatedText;
+  return convertedImportDeclarationText;
 };
 
 describe("convertImportDeclaration", () => {
@@ -40,9 +33,8 @@ describe("convertImportDeclaration", () => {
 
     it("returns import declaration text removed defineComponent", () => {
       const output = parseScript(source);
-      const expected = 'import { ref } from "vue";\n';
 
-      expect(output).toBe(expected);
+      expect(output).toMatchInlineSnapshot(`"import { ref } from 'vue';"`);
     });
   });
 
@@ -57,9 +49,8 @@ describe("convertImportDeclaration", () => {
 
     it("returns blank", () => {
       const output = parseScript(source);
-      const expected = "";
 
-      expect(output).toBe(expected);
+      expect(output).toBe("");
     });
   });
 
@@ -74,9 +65,8 @@ describe("convertImportDeclaration", () => {
 
     it("returns import declaration text removed defineNuxtComponent", () => {
       const output = parseScript(source);
-      const expected = 'import { ref } from "#imports";\n';
 
-      expect(output).toBe(expected);
+      expect(output).toMatchInlineSnapshot(`"import { ref } from '#imports';"`);
     });
   });
 
@@ -91,9 +81,8 @@ describe("convertImportDeclaration", () => {
 
     it("returns blank", () => {
       const output = parseScript(source);
-      const expected = "";
 
-      expect(output).toBe(expected);
+      expect(output).toBe("");
     });
   });
 });
diff --git a/packages/vue-script-setup-converter/src/lib/converter/pageMetaConverter.test.ts b/packages/vue-script-setup-converter/src/lib/converter/pageMetaConverter.test.ts
@@ -1,8 +1,6 @@
 import { expect, describe, it } from "vitest";
 import { CallExpression, ScriptTarget, SyntaxKind, Project } from "ts-morph";
 import { parse } from "@vue/compiler-sfc";
-import prettier from "prettier";
-import parserTypeScript from "prettier/parser-typescript";
 import { getNodeByKind } from "../helpers/node";
 import { convertPageMeta } from "./pageMetaConverter";
 
@@ -24,12 +22,7 @@ const parseScript = (input: string, lang: "js" | "ts" = "js") => {
 
   const pageMeta = convertPageMeta(callExpression as CallExpression, lang);
 
-  const formatedText = prettier.format(pageMeta, {
-    parser: "typescript",
-    plugins: [parserTypeScript],
-  });
-
-  return formatedText;
+  return pageMeta;
 };
 
 describe("convertPageMeta", () => {
@@ -47,14 +40,11 @@ describe("convertPageMeta", () => {
     it("returns text including definePageMeta", () => {
       const output = parseScript(source);
 
-      const expected = `definePageMeta({
-  name: "HelloWorld",
-  layout: "test-layout",
-  middleware: "test-middleware",
-});
-`;
-
-      expect(output).toBe(expected);
+      expect(output).toMatchInlineSnapshot(`
+        "definePageMeta({
+            name: 'HelloWorld',layout: 'test-layout',middleware: 'test-middleware'
+          });"
+      `);
     });
   });
 
@@ -72,14 +62,11 @@ describe("convertPageMeta", () => {
     it("returns text including definePageMeta", () => {
       const output = parseScript(source);
 
-      const expected = `definePageMeta({
-  name: "HelloWorld",
-  layout: "test-layout",
-  middleware: ["test-middleware-1", "test-middleware-2"],
-});
-`;
-
-      expect(output).toBe(expected);
+      expect(output).toMatchInlineSnapshot(`
+        "definePageMeta({
+            name: 'HelloWorld',layout: 'test-layout',middleware: ['test-middleware-1', 'test-middleware-2']
+          });"
+      `);
     });
   });
 });