fix: Duplicate formatConversionTest files (#1798)

* Removed duplicate files

* Small fix

* Small fix

* Removed file

* Removed old comment

Author: matthewlipski

packages/xl-multi-column/src/test/conversions/formatConversionTestUtil.ts

@@ -9,12 +9,12 @@ import {
   createExternalHTMLExporter,
   createInternalHTMLSerializer,
 } from "@blocknote/core";
-import { afterEach, beforeEach, describe, expect, it } from "vitest";
-
 import {
   addIdsToBlocks,
   partialBlocksToBlocksForTesting,
-} from "./formatConversionTestUtil.js";
+} from "@shared/formatConversionTestUtil.js";
+import { afterEach, beforeEach, describe, expect, it } from "vitest";
+
 import { multiColumnSchemaTestCases } from "./testCases.js";
 
 // TODO: code same from @blocknote/core, maybe create separate test util package

packages/xl-multi-column/src/test/conversions/htmlConversion.test.ts

@@ -7,8 +7,8 @@ import {
   blockToNode,
   nodeToBlock,
 } from "@blocknote/core";
+import { partialBlockToBlockForTesting } from "@shared/formatConversionTestUtil.js";
 
-import { partialBlockToBlockForTesting } from "./formatConversionTestUtil.js";
 import { multiColumnSchemaTestCases } from "./testCases.js";
 
 function addIdsToBlock(block: PartialBlock<any, any, any>) {

packages/xl-multi-column/src/test/conversions/nodeConversion.test.ts

@@ -19,7 +19,21 @@
     "declarationDir": "types",
     "composite": true,
     "skipLibCheck": true,
-    "emitDeclarationOnly": true
+    "emitDeclarationOnly": true,
+    "paths": {
+      "@shared/*": ["../../shared/*"]
+    }
   },
-  "include": ["src"]
+  "include": ["src"],
+  "references": [
+    {
+      "path": "../core"
+    },
+    {
+      "path": "../react"
+    },
+    {
+      "path": "../../shared"
+    }
+  ]
 }

packages/xl-multi-column/tsconfig.json

@@ -20,8 +20,11 @@ export default defineConfig((conf) => ({
   resolve: {
     alias:
       conf.command === "build"
-        ? ({} as Record<string, string>)
+        ? ({
+            "@shared": path.resolve(__dirname, "../../shared/"),
+          } as Record<string, string>)
         : ({
+            "@shared": path.resolve(__dirname, "../../shared/"),
             // load live from sources with live reload working
             "@blocknote/core": path.resolve(__dirname, "../core/src/"),
           } as Record<string, string>),

packages/xl-multi-column/vite.config.ts

@@ -24,7 +24,7 @@ export default defineConfig((conf) => ({
         nesting,
         tailwindcss("../packages/shadcn/tailwind.config.js"), // Adjust the path as necessary
         auto,
-      ],
+      ] as any,
     },
     // postcss: "../packages/shadcn/postcss.config.js",
   },

playground/vite.config.ts

@@ -5,10 +5,10 @@ import {
   InlineContentSchema,
   StyleSchema,
 } from "@blocknote/core";
+import { addIdsToBlocks } from "@shared/formatConversionTestUtil.js";
 import { prettify } from "htmlfy";
 import { expect } from "vitest";
 
-import { addIdsToBlocks } from "../formatConversionTestUtil.js";
 import { ExportTestCase } from "./exportTestCase.js";
 
 export const testExportBlockNoteHTML = async <

tests/src/unit/shared/formatConversion/export/exportTestExecutors.ts

@@ -6,13 +6,12 @@ import {
   nodeToBlock,
   StyleSchema,
 } from "@blocknote/core";
-import { expect } from "vitest";
-
-// TODO: fix import and add lint rule, or allow?
 import {
   addIdsToBlocks,
   partialBlocksToBlocksForTesting,
-} from "../formatConversionTestUtil.js";
+} from "@shared/formatConversionTestUtil.js";
+import { expect } from "vitest";
+
 import { ExportParseEqualityTestCase } from "./exportParseEqualityTestCase.js";
 
 export const testExportParseEqualityBlockNoteHTML = async <