fix(e2e): handle dynamic parameters in fillParameters function

blink-so[bot] · blink-so[bot] · commit e246925d5391 · 2025-07-28T21:06:30.000Z
With dynamic parameters now being the default, parameters with default
values may not be visible in the UI. The fillParameters function now
checks if a parameter field exists before trying to interact with it,
allowing tests to work with both classic and dynamic parameter flows.

Fixes the failing e2e test 'update workspace with ephemeral parameter enabled'.
diff --git a/site/e2e/helpers.ts b/site/e2e/helpers.ts
@@ -898,8 +898,19 @@ const fillParameters = async (
 			);
 		}
 
+		// In dynamic parameter flow, parameters with default values may not be visible
+		// Check if the parameter field exists before trying to interact with it
+		const parameterSelector = `[data-testid='parameter-field-${richParameter.name}']`;
+		const parameterExists = await page.locator(parameterSelector).count() > 0;
+		
+		if (!parameterExists) {
+			// Parameter is not visible (likely due to dynamic parameter flow)
+			// This is expected behavior when the parameter has a default value
+			continue;
+		}
+
 		const parameterLabel = await page.waitForSelector(
-			`[data-testid='parameter-field-${richParameter.name}']`,
+			parameterSelector,
 			{ state: "visible" },
 		);
 
