unskipping

Author: karancs06

src/visualBuilder/__test__/index.test.ts

@@ -168,7 +168,7 @@ describe(
             }
         );
 
-        test.skip("should add overlay to DOM when clicked", async () => {
+        test("should add overlay to DOM when clicked", async () => {
             const h1Tag = document.createElement("h1");
             h1Tag.textContent = INLINE_EDITABLE_FIELD_VALUE;
             h1Tag.setAttribute(

src/visualBuilder/components/__test__/fieldLabelWrapper.test.tsx

@@ -155,7 +155,7 @@ const PARENT_PATHS = [
     `${pathPrefix}.parentPath3`,
 ];
 
-describe.skip("FieldLabelWrapperComponent", () => {
+describe("FieldLabelWrapperComponent", () => {
     beforeEach(() => {
         vi.mocked(isFieldDisabled).mockReturnValue({
             isDisabled: false,

src/visualBuilder/components/__test__/fieldToolbar.test.tsx

@@ -65,8 +65,11 @@ vi.mock("../../utils/isFieldDisabled", () => ({
 }));
 
 vi.mock("../FieldRevert/FieldRevertComponent", async (importOriginal) => {
-    const actual = await importOriginal<typeof import("../FieldRevert/FieldRevertComponent")>();
-    
+    const actual =
+        await importOriginal<
+            typeof import("../FieldRevert/FieldRevertComponent")
+        >();
+
     return {
         ...actual,
         getFieldVariantStatus: vi.fn().mockResolvedValue({
@@ -129,7 +132,7 @@ describe("FieldToolbarComponent", () => {
         vi.clearAllMocks();
     });
 
-    test.skip("renders toolbar buttons correctly", async () => {
+    test("renders toolbar buttons correctly", async () => {
         const { getByTestId } = await asyncRender(
             <FieldToolbarComponent
                 eventDetails={mockEventDetails}
@@ -153,7 +156,7 @@ describe("FieldToolbarComponent", () => {
         expect(deleteButton).toBeInTheDocument();
     });
 
-    test.skip("calls handleMoveInstance with 'previous' when move left button is clicked", async () => {
+    test("calls handleMoveInstance with 'previous' when move left button is clicked", async () => {
         const { getByTestId } = await asyncRender(
             <FieldToolbarComponent
                 eventDetails={mockEventDetails}
@@ -173,7 +176,7 @@ describe("FieldToolbarComponent", () => {
         );
     });
 
-    test.skip("calls handleMoveInstance with 'next' when move right button is clicked", async () => {
+    test("calls handleMoveInstance with 'next' when move right button is clicked", async () => {
         const { getByTestId } = await asyncRender(
             <FieldToolbarComponent
                 eventDetails={mockEventDetails}
@@ -193,7 +196,7 @@ describe("FieldToolbarComponent", () => {
         );
     });
 
-    test.skip("calls handleDeleteInstance when delete button is clicked", async () => {
+    test("calls handleDeleteInstance when delete button is clicked", async () => {
         const { getByTestId } = await asyncRender(
             <FieldToolbarComponent
                 eventDetails={mockEventDetails}
@@ -211,7 +214,7 @@ describe("FieldToolbarComponent", () => {
             mockMultipleFieldMetadata
         );
     });
-    test.skip("display variant icon instead of dropdown", async () => {
+    test("display variant icon instead of dropdown", async () => {
         // Create a fresh copy with variant set to avoid mutation issues
         const variantEventDetails = {
             ...mockEventDetails,
@@ -250,7 +253,7 @@ describe("FieldToolbarComponent", () => {
             );
         });
 
-        test.skip("'replace button' is hidden for parent wrapper of multiple file field", async () => {
+        test("'replace button' is hidden for parent wrapper of multiple file field", async () => {
             const parentWrapperMetadata: CslpData = {
                 ...mockMultipleFieldMetadata,
                 fieldPathWithIndex: "files",
@@ -277,7 +280,7 @@ describe("FieldToolbarComponent", () => {
             expect(replaceButton).not.toBeInTheDocument();
         });
 
-        test.skip("'replace button' is visible for individual field in multiple file field", async () => {
+        test("'replace button' is visible for individual field in multiple file field", async () => {
             const individualFieldMetadata: CslpData = {
                 ...mockMultipleFieldMetadata,
                 fieldPathWithIndex: "files",
@@ -305,7 +308,7 @@ describe("FieldToolbarComponent", () => {
         });
     });
 
-    test.skip("passes disabled state correctly to child components when field is disabled", async () => {
+    test("passes disabled state correctly to child components when field is disabled", async () => {
         // Mock isFieldDisabled to return disabled state
         vi.mocked(isFieldDisabled).mockReturnValue({
             isDisabled: true,

src/visualBuilder/utils/__test__/multipleElementAddButton.test.ts

@@ -563,7 +563,7 @@ describe("removeAddInstanceButtons", () => {
         visualBuilderContainer = document.createElement("div");
         visualBuilderContainer.classList.add("visual-builder__container");
         document.body.appendChild(visualBuilderContainer);
-        
+
         // Set longer timeout for CI environment where Preact rendering is slower
         if (process.env.CI) {
             vi.setConfig({ testTimeout: 60000, hookTimeout: 60000 });
@@ -712,7 +712,7 @@ describe("removeAddInstanceButtons", () => {
         expect(buttons.length).toBe(0);
     });
 
-    test.skip("should not remove all buttons if forceRemoveAll is false", () => {
+    test("should not remove all buttons if forceRemoveAll is false", () => {
         for (let i = 0; i < 5; i++) {
             const button = generateAddInstanceButton({
                 fieldSchema: singleLineFieldSchema,