timeout

Author: karancs06

src/__test__/utils.ts

@@ -43,47 +43,67 @@ export const waitForHoverOutline = async (options?: {
     await waitFor(
         () => {
             const hoverOutline = document.querySelector(
-                "[data-testid='visual-builder__hover-outline'][style]"
-            );
-            expect(hoverOutline).not.toBeNull();
+                "[data-testid='visual-builder__hover-outline']"
+            ) as HTMLElement | null;
+            if (!hoverOutline) {
+                throw new Error("Hover outline element not found");
+            }
+            // Check that style is set (either via attribute or style object)
+            // CI environments can be slower, so check both attribute and style properties
+            const styleAttr = hoverOutline.getAttribute("style");
+            const hasStyleAttr = styleAttr && styleAttr.trim() !== "";
+            const hasStyleProps =
+                hoverOutline.style.top &&
+                hoverOutline.style.top !== "" &&
+                hoverOutline.style.left &&
+                hoverOutline.style.left !== "";
+            if (!hasStyleAttr && !hasStyleProps) {
+                throw new Error("Hover outline style not set");
+            }
         },
         {
-            timeout: options?.timeout ?? 2000, // Reduced from 5s to 2s - mocks resolve immediately
-            interval: options?.interval ?? 10, // Faster polling: 10ms default
+            timeout: options?.timeout ?? 3000, // Increased to 3s for CI - accounts for debounce (50ms) + async operations
+            interval: options?.interval ?? 20, // Slightly slower polling for CI stability
         }
     );
 };
-  
-export const waitForBuilderSDKToBeInitialized = async (visualBuilderPostMessage: EventManager | undefined) => {
+
+export const waitForBuilderSDKToBeInitialized = async (
+    visualBuilderPostMessage: EventManager | undefined
+) => {
     await waitFor(() => {
         expect(visualBuilderPostMessage?.send).toBeCalledWith(
             VisualBuilderPostMessageEvents.INIT,
             expect.any(Object)
         );
     });
-}
+};
 interface WaitForClickActionOptions {
     skipWaitForFieldType?: boolean;
 }
-export const triggerAndWaitForClickAction = async (visualBuilderPostMessage: EventManager | undefined, element: HTMLElement, {skipWaitForFieldType}: WaitForClickActionOptions = {}) => {
+export const triggerAndWaitForClickAction = async (
+    visualBuilderPostMessage: EventManager | undefined,
+    element: HTMLElement,
+    { skipWaitForFieldType }: WaitForClickActionOptions = {}
+) => {
     await waitForBuilderSDKToBeInitialized(visualBuilderPostMessage);
     await act(async () => {
         await fireEvent.click(element);
-    })
-    if(!skipWaitForFieldType) {
+    });
+    if (!skipWaitForFieldType) {
         await waitFor(() => {
-            expect(element).toHaveAttribute("data-cslp-field-type")
-        })
+            expect(element).toHaveAttribute("data-cslp-field-type");
+        });
     }
-}
+};
 export const waitForToolbaxToBeVisible = async () => {
     await waitFor(() => {
         const toolbar = document.querySelector(
             ".visual-builder__focused-toolbar__field-label-container"
         );
         expect(toolbar).not.toBeNull();
     });
-}
+};
 
 export const waitForCursorToBeVisible = async (options?: {
     timeout?: number;
@@ -117,8 +137,8 @@ export const waitForCursorIcon = async (
             expect(customCursor).toHaveAttribute("data-icon", icon);
         },
         {
-            timeout: options?.timeout ?? 1000, // Reduced from 2s to 1s - mocks resolve immediately
-            interval: options?.interval ?? 10, // Faster polling: 10ms default
+            timeout: options?.timeout ?? 2000, // Increased to 2s for CI - async operations can be slower
+            interval: options?.interval ?? 20, // Slightly slower polling for CI stability
         }
     );
 };
@@ -129,17 +149,24 @@ const defaultRect = {
     bottom: 20,
     width: 10,
     height: 5,
-}
-export const mockGetBoundingClientRect = (element: HTMLElement, rect = defaultRect) => {
-    vi.spyOn(element, "getBoundingClientRect").mockImplementation(() => rect as DOMRect);
-}
+};
+export const mockGetBoundingClientRect = (
+    element: HTMLElement,
+    rect = defaultRect
+) => {
+    vi.spyOn(element, "getBoundingClientRect").mockImplementation(
+        () => rect as DOMRect
+    );
+};
 export const getElementBytestId = (testId: string) => {
     return document.querySelector(`[data-testid="${testId}"]`);
-}
-export const asyncRender: (componentChild: ComponentChild) => ReturnType<typeof render> = async (...args) => {
+};
+export const asyncRender: (
+    componentChild: ComponentChild
+) => ReturnType<typeof render> = async (...args) => {
     let returnValue: ReturnType<typeof render>;
     await act(async () => {
         returnValue = render(...args);
     });
     return returnValue;
-}
+};