fix: resolve remaining test timeouts in fieldToolbar and multipleElementAddButton

- Add microtask wait for Preact component rendering in variant icon test
- Add microtask wait for button generation loops (5 buttons) to complete
- Fixes 2 remaining timeout failures

All tests now passing locally with optimized performance

Author: karancs06

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

@@ -185,7 +185,7 @@ describe("FieldToolbarComponent", () => {
         const deleteButton = getByTestId(
             "visual-builder__focused-toolbar__multiple-field-toolbar__delete-button"
         );
-        
+
         fireEvent.click(deleteButton);
 
         expect(handleDeleteInstance).toHaveBeenCalledWith(
@@ -201,17 +201,18 @@ describe("FieldToolbarComponent", () => {
                 variant: "variant",
             },
         };
-        
+
         const { getByTestId } = await asyncRender(
             <FieldToolbarComponent
                 eventDetails={variantEventDetails}
                 hideOverlay={vi.fn()}
             />
         );
 
-        const variantIcon = getByTestId(
-            "visual-builder-canvas-variant-icon"
-        );
+        // Give component a tick to fully render variant icon
+        await new Promise((resolve) => setTimeout(resolve, 0));
+
+        const variantIcon = getByTestId("visual-builder-canvas-variant-icon");
         expect(variantIcon).toBeInTheDocument();
     });
 

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

@@ -673,7 +673,7 @@ describe("removeAddInstanceButtons", () => {
         expect(visualBuilderContainer.contains(nextButton)).toBeFalsy();
     });
 
-    test("should remove all buttons if forceRemoveAll is true", () => {
+    test("should remove all buttons if forceRemoveAll is true", async () => {
         for (let i = 0; i < 5; i++) {
             const button = generateAddInstanceButton({
                 fieldSchema: singleLineFieldSchema,
@@ -686,6 +686,9 @@ describe("removeAddInstanceButtons", () => {
             visualBuilderContainer.appendChild(button);
         }
 
+        // Give components a tick to render
+        await new Promise((resolve) => setTimeout(resolve, 0));
+
         let buttons = visualBuilderContainer.querySelectorAll(
             `[data-testid="visual-builder-add-instance-button"]`
         );
@@ -708,7 +711,7 @@ describe("removeAddInstanceButtons", () => {
         expect(buttons.length).toBe(0);
     });
 
-    test("should not remove all buttons if forceRemoveAll is false", () => {
+    test("should not remove all buttons if forceRemoveAll is false", async () => {
         for (let i = 0; i < 5; i++) {
             const button = generateAddInstanceButton({
                 fieldSchema: singleLineFieldSchema,
@@ -721,6 +724,9 @@ describe("removeAddInstanceButtons", () => {
             visualBuilderContainer.appendChild(button);
         }
 
+        // Give components a tick to render
+        await new Promise((resolve) => setTimeout(resolve, 0));
+
         let buttons = visualBuilderContainer.querySelectorAll(
             `[data-testid="visual-builder-add-instance-button"]`
         );
@@ -736,10 +742,6 @@ describe("removeAddInstanceButtons", () => {
             false
         );
 
-        buttons = visualBuilderContainer.querySelectorAll(
-            `[data-testid="visual-builder-add-instance-button"]`
-        );
-
         const addInstanceButtons = visualBuilderContainer.querySelectorAll(
             `[data-testid="visual-builder-add-instance-button"]`
         );