update tests

Author: pokornyd

tests/transfomers/portable-text-transformer/html-transformer.spec.ts

@@ -3,17 +3,15 @@ import { PortableTextTypeComponentOptions } from "@portabletext/to-html";
 
 import {
   ArbitraryTypedObject,
-  nodesToPortableText,
   PortableTextBlock,
   PortableTextComponentOrItem,
   PortableTextExternalLink,
   PortableTextImage,
   PortableTextItemLink,
   PortableTextMark,
   PortableTextTable,
+  transformToPortableText,
 } from "../../../src";
-import { browserParse } from "../../../src/parser/browser";
-import { nodeParse } from "../../../src/parser/node";
 import { PortableTextHtmlResolvers, resolveImage, toHTML } from "../../../src/utils/resolution/html";
 
 jest.mock("short-unique-id", () => {
@@ -125,21 +123,13 @@ describe("HTML transformer", () => {
   ) => {
     richTextInput.value = richTextValue;
 
-    const browserTree = browserParse(richTextInput.value);
-    const nodeTree = nodeParse(richTextInput.value);
-    const nodePortableText = nodesToPortableText(nodeTree);
-    const browserPortableText = nodesToPortableText(browserTree);
-    const nodeResult = toHTML(
-      nodePortableText,
-      getPortableTextComponents(richTextInput, customResolvers),
-    );
-    const browserResult = toHTML(
-      browserPortableText,
+    const portableText = transformToPortableText(richTextInput.value);
+    const result = toHTML(
+      portableText,
       getPortableTextComponents(richTextInput, customResolvers),
     );
 
-    expect(nodeResult).toMatchSnapshot();
-    expect(nodeResult).toEqual(browserResult);
+    expect(result).toMatchSnapshot();
   };
 
   it("builds basic portable text into HTML", () => {

tests/transfomers/portable-text-transformer/mapi-transformer.spec.ts

@@ -1,6 +1,4 @@
-import { ArbitraryTypedObject, nodesToPortableText, PortableTextSpan, traversePortableText } from "../../../src";
-import { browserParse } from "../../../src/parser/browser";
-import { nodeParse } from "../../../src/parser/node";
+import { ArbitraryTypedObject, PortableTextSpan, transformToPortableText, traversePortableText } from "../../../src";
 import { toManagementApiFormat } from "../../../src/utils/resolution/mapi";
 
 jest.mock("short-unique-id", () => {
@@ -17,29 +15,18 @@ const sortMarks = (obj: ArbitraryTypedObject) => isSpan(obj) ? { ...obj, marks:
 
 describe("portabletext to MAPI resolver", () => {
   const transformAndCompare = (richTextContent: string) => {
-    // Parse the rich text content into a tree
-    const browserTree = browserParse(richTextContent);
-    const nodeTree = nodeParse(richTextContent);
-
-    // Convert the tree to Portable Text
-    const nodePortableText = nodesToPortableText(nodeTree);
-    const browserPortableText = nodesToPortableText(browserTree);
+    const portableText = transformToPortableText(richTextContent);
 
     // Convert Portable Text to MAPI format
-    const nodeManagementApiFormat = toManagementApiFormat(nodePortableText);
-    const browserManagementApiFormat = toManagementApiFormat(browserPortableText);
+    const managementApiFormat = toManagementApiFormat(portableText);
 
     // Parse the MAPI format back into a tree and convert it to Portable Text
-    const secondParseTree = nodeParse(nodeManagementApiFormat);
-    const secondParsePortableText = nodesToPortableText(secondParseTree);
-
-    // Compare the MAPI formats to ensure consistency across platforms
-    expect(nodeManagementApiFormat).toEqual(browserManagementApiFormat);
+    const secondParsePortableText = transformToPortableText(managementApiFormat);
 
     // Compare the original Portable Text to the re-parsed Portable Text after MAPI conversion
     expect(
       traversePortableText(secondParsePortableText, sortMarks),
-    ).toStrictEqual(traversePortableText(nodePortableText, sortMarks));
+    ).toStrictEqual(traversePortableText(portableText, sortMarks));
   };
 
   it("handles nested style marks", () => {
@@ -99,12 +86,10 @@ describe("portabletext to MAPI resolver", () => {
      */
     const richTextContent =
       `<p><strong>strong text </strong><a href="https://example.com"><strong>example strong link text</strong>not strong link text</a></p>`;
-    const tree = nodeParse(richTextContent);
-    const portableText = nodesToPortableText(tree);
+    const portableText = transformToPortableText(richTextContent);
     const mapiFormat = toManagementApiFormat(portableText);
 
-    const secondParseTree = nodeParse(mapiFormat);
-    const secondParsePortableText = nodesToPortableText(secondParseTree);
+    const secondParsePortableText = transformToPortableText(mapiFormat);
     const secondParseMapiFormat = toManagementApiFormat(
       secondParsePortableText,
     );

tests/transfomers/portable-text-transformer/portable-text-transformer.spec.ts

@@ -1,4 +1,4 @@
-import { nodesToPortableText, PortableTextItem, traversePortableText } from "../../../src";
+import { DomNode, PortableTextItem, transformToPortableText, traversePortableText } from "../../../src";
 import { browserParse } from "../../../src/parser/browser";
 import { nodeParse } from "../../../src/parser/node";
 
@@ -14,21 +14,23 @@ describe("Portable Text Transformer", () => {
   const transformInput = (
     input: string,
   ): {
-    nodeResult: PortableTextItem[];
-    browserResult: PortableTextItem[];
+    nodeTree: DomNode[];
+    browserTree: DomNode[];
+    result: PortableTextItem[];
   } => {
     const browserTree = browserParse(input);
     const nodeTree = nodeParse(input);
     return {
-      nodeResult: nodesToPortableText(nodeTree),
-      browserResult: nodesToPortableText(browserTree),
+      nodeTree,
+      browserTree,
+      result: transformToPortableText(input),
     };
   };
 
   const transformAndCompare = (input: string) => {
-    const { nodeResult, browserResult } = transformInput(input);
-    expect(nodeResult).toMatchSnapshot();
-    expect(nodeResult).toMatchObject(browserResult);
+    const { nodeTree, browserTree, result } = transformInput(input);
+    expect(result).toMatchSnapshot();
+    expect(nodeTree).toMatchObject(browserTree);
   };
 
   it("transforms empty rich text", () => {
@@ -246,14 +248,10 @@ describe("Portable Text Transformer", () => {
     transformAndCompare(`<p>text<strong>bold</strong></p>`);
   });
 
-  it.each([nodeParse, browserParse])(
-    "throws error for non-supported tags for %s",
-    (parse) => {
-      const input = "<p>text in a paragraph</p><div>text in a div, which doesnt exist in kontent RTE</div>";
-      const tree = parse(input);
-      expect(() => nodesToPortableText(tree)).toThrow();
-    },
-  );
+  it("throws error for non-supported tags", () => {
+    const input = "<p>text in a paragraph</p><div>text in a div, which doesnt exist in kontent RTE</div>";
+    expect(() => transformToPortableText(input)).toThrow();
+  });
 
   it("doesn't extend link mark to adjacent spans", () => {
     transformAndCompare(
@@ -301,13 +299,14 @@ describe("Portable Text Transformer", () => {
       `<object type="application/kenticocloud" data-type="item" data-rel="link" data-codename="test_item"></object>`;
 
     const processBlock = (block: PortableTextItem) =>
-      block._type === "componentOrItem" ? { ...block, additionalData: "data" } : null;
+      block._type === "componentOrItem"
+        ? { ...block, additionalData: "data" }
+        : null;
 
-    const { nodeResult } = transformInput(input);
-    const modifiedResult = traversePortableText(nodeResult, processBlock);
+    const { result } = transformInput(input);
+    const modifiedResult = traversePortableText(result, processBlock);
 
     expect(modifiedResult).toMatchSnapshot();
-    expect(modifiedResult).toMatchObject(nodeResult);
   });
 
   it("extends link nested in a table with additional data", () => {
@@ -316,11 +315,10 @@ describe("Portable Text Transformer", () => {
     const processBlock = (block: PortableTextItem) =>
       block._type === "link" ? { ...block, additionalData: "data" } : null;
 
-    const { nodeResult } = transformInput(input);
-    const transformedResult = traversePortableText(nodeResult, processBlock);
+    const { result } = transformInput(input);
+    const transformedResult = traversePortableText(result, processBlock);
 
     expect(transformedResult).toMatchSnapshot();
-    expect(transformedResult).toMatchObject(nodeResult);
   });
 
   it("transforms a linked item and a component from MAPI with corresponding dataType", () => {

tests/transfomers/portable-text-transformer/vue-transformer.spec.ts

@@ -2,7 +2,7 @@ import { PortableText, PortableTextComponentProps, PortableTextComponents, toPla
 import { mount } from "@vue/test-utils";
 import { h } from "vue";
 
-import { nodesToPortableText, parseHtml, PortableTextImage, PortableTextTable } from "../../../src";
+import { PortableTextImage, PortableTextTable, transformToPortableText } from "../../../src";
 import { resolveImage, resolveTable } from "../../../src/utils/resolution/vue";
 
 const components: PortableTextComponents = {
@@ -17,8 +17,7 @@ describe("PortableText Vue Renderer", () => {
     richTextValue: string,
     customComponents = components,
   ) => {
-    const jsonTree = parseHtml(richTextValue);
-    const portableText = nodesToPortableText(jsonTree);
+    const portableText = transformToPortableText(richTextValue);
 
     return mount(PortableText, {
       props: {