From 36795b0f092e9708bc7a3529ae4942a657fa5189 Mon Sep 17 00:00:00 2001 From: cristhianzl Date: Thu, 6 Jun 2024 12:06:17 -0300 Subject: [PATCH] =?UTF-8?q?=E2=9C=85=20(tests):=20increase=20waitForTimeou?= =?UTF-8?q?t=20to=203000ms=20in=20end-to-end=20tests?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ✅ (tests): increase waitForTimeout to 3000ms in end-to-end tests ✅ (tests): increase waitForTimeout to 3000ms in end-to-end tests for stability ✅ (tests): remove trailing commas in expect statements for consistency ✅ (tests): increase waitForTimeout to 3000ms in end-to-end tests for stability ✅ (tests): increase timeout in end-to-end tests to improve stability ✅ (tests): increase waitForTimeout to ensure element stability --- .../tests/end-to-end/codeAreaModalComponent.spec.ts | 2 +- src/frontend/tests/end-to-end/dropdownComponent.spec.ts | 2 +- src/frontend/tests/end-to-end/fileUploadComponent.spec.ts | 2 +- src/frontend/tests/end-to-end/filterEdge.spec.ts | 4 ++-- src/frontend/tests/end-to-end/floatComponent.spec.ts | 2 +- src/frontend/tests/end-to-end/flowPage.spec.ts | 2 +- src/frontend/tests/end-to-end/globalVariables.spec.ts | 2 +- src/frontend/tests/end-to-end/inputComponent.spec.ts | 2 +- src/frontend/tests/end-to-end/inputListComponent.spec.ts | 2 +- src/frontend/tests/end-to-end/intComponent.spec.ts | 2 +- .../tests/end-to-end/keyPairListComponent.spec.ts | 2 +- src/frontend/tests/end-to-end/langflowShortcuts.spec.ts | 2 +- src/frontend/tests/end-to-end/nestedComponent.spec.ts | 2 +- .../tests/end-to-end/promptModalComponent.spec.ts | 2 +- src/frontend/tests/end-to-end/saveComponents.spec.ts | 2 ++ .../tests/end-to-end/textAreaModalComponent.spec.ts | 2 +- src/frontend/tests/end-to-end/textInputOutput.spec.ts | 2 +- src/frontend/tests/end-to-end/toggleComponent.spec.ts | 2 +- src/frontend/tests/end-to-end/tweaks_test.spec.ts | 2 +- src/frontend/tests/end-to-end/userSettings.spec.ts | 8 ++++++++ 20 files changed, 29 insertions(+), 19 deletions(-) diff --git a/src/frontend/tests/end-to-end/codeAreaModalComponent.spec.ts b/src/frontend/tests/end-to-end/codeAreaModalComponent.spec.ts index 3b2913440..577c138a3 100644 --- a/src/frontend/tests/end-to-end/codeAreaModalComponent.spec.ts +++ b/src/frontend/tests/end-to-end/codeAreaModalComponent.spec.ts @@ -23,7 +23,7 @@ test("CodeAreaModalComponent", async ({ page }) => { await page.waitForTimeout(1000); await page.getByTestId("blank-flow").click(); - await page.waitForTimeout(1000); + await page.waitForTimeout(3000); await page.getByTestId("extended-disclosure").click(); await page.getByPlaceholder("Search").click(); await page.getByPlaceholder("Search").fill("python function"); diff --git a/src/frontend/tests/end-to-end/dropdownComponent.spec.ts b/src/frontend/tests/end-to-end/dropdownComponent.spec.ts index b5ff9ef00..0846e5575 100644 --- a/src/frontend/tests/end-to-end/dropdownComponent.spec.ts +++ b/src/frontend/tests/end-to-end/dropdownComponent.spec.ts @@ -22,7 +22,7 @@ test("dropDownComponent", async ({ page }) => { await page.waitForTimeout(1000); await page.getByTestId("blank-flow").click(); - await page.waitForTimeout(1000); + await page.waitForTimeout(3000); await page.getByTestId("extended-disclosure").click(); await page.getByPlaceholder("Search").click(); diff --git a/src/frontend/tests/end-to-end/fileUploadComponent.spec.ts b/src/frontend/tests/end-to-end/fileUploadComponent.spec.ts index 4cce05c83..00f761dea 100644 --- a/src/frontend/tests/end-to-end/fileUploadComponent.spec.ts +++ b/src/frontend/tests/end-to-end/fileUploadComponent.spec.ts @@ -23,7 +23,7 @@ test("dropDownComponent", async ({ page }) => { await page.waitForTimeout(1000); await page.getByTestId("blank-flow").click(); - await page.waitForTimeout(1000); + await page.waitForTimeout(3000); await page.getByTestId("extended-disclosure").click(); await page.getByPlaceholder("Search").click(); diff --git a/src/frontend/tests/end-to-end/filterEdge.spec.ts b/src/frontend/tests/end-to-end/filterEdge.spec.ts index b35b7c1da..7edbc0436 100644 --- a/src/frontend/tests/end-to-end/filterEdge.spec.ts +++ b/src/frontend/tests/end-to-end/filterEdge.spec.ts @@ -21,7 +21,7 @@ test("LLMChain - Tooltip", async ({ page }) => { } await page.getByTestId("blank-flow").click(); - await page.waitForTimeout(1000); + await page.waitForTimeout(3000); await page.getByTestId("extended-disclosure").click(); await page.getByPlaceholder("Search").click(); @@ -117,7 +117,7 @@ test("LLMChain - Filter", async ({ page }) => { ); await page.getByTestId("blank-flow").click(); - await page.waitForTimeout(1000); + await page.waitForTimeout(3000); await page.getByTestId("extended-disclosure").click(); await page.getByPlaceholder("Search").click(); await page.getByPlaceholder("Search").fill("llmchain"); diff --git a/src/frontend/tests/end-to-end/floatComponent.spec.ts b/src/frontend/tests/end-to-end/floatComponent.spec.ts index 0776355a6..e172a92e3 100644 --- a/src/frontend/tests/end-to-end/floatComponent.spec.ts +++ b/src/frontend/tests/end-to-end/floatComponent.spec.ts @@ -22,7 +22,7 @@ test("FloatComponent", async ({ page }) => { await page.waitForTimeout(1000); await page.getByTestId("blank-flow").click(); - await page.waitForTimeout(1000); + await page.waitForTimeout(3000); await page.getByTestId("extended-disclosure").click(); await page.getByPlaceholder("Search").click(); await page.getByPlaceholder("Search").fill("ollama"); diff --git a/src/frontend/tests/end-to-end/flowPage.spec.ts b/src/frontend/tests/end-to-end/flowPage.spec.ts index 28703bbc0..4cae45bf2 100644 --- a/src/frontend/tests/end-to-end/flowPage.spec.ts +++ b/src/frontend/tests/end-to-end/flowPage.spec.ts @@ -22,7 +22,7 @@ test.describe("Flow Page tests", () => { } await page.getByTestId("blank-flow").click(); - await page.waitForTimeout(1000); + await page.waitForTimeout(3000); await page.getByTestId("extended-disclosure").click(); await page.getByPlaceholder("Search").click(); await page.getByPlaceholder("Search").fill("custom"); diff --git a/src/frontend/tests/end-to-end/globalVariables.spec.ts b/src/frontend/tests/end-to-end/globalVariables.spec.ts index 546c332f6..64af7fe28 100644 --- a/src/frontend/tests/end-to-end/globalVariables.spec.ts +++ b/src/frontend/tests/end-to-end/globalVariables.spec.ts @@ -22,7 +22,7 @@ test("GlobalVariables", async ({ page }) => { await page.waitForTimeout(1000); await page.getByTestId("blank-flow").click(); - await page.waitForTimeout(1000); + await page.waitForTimeout(3000); await page.getByTestId("extended-disclosure").click(); await page.getByPlaceholder("Search").click(); await page.getByPlaceholder("Search").fill("openai"); diff --git a/src/frontend/tests/end-to-end/inputComponent.spec.ts b/src/frontend/tests/end-to-end/inputComponent.spec.ts index 317ab231e..3e6936b43 100644 --- a/src/frontend/tests/end-to-end/inputComponent.spec.ts +++ b/src/frontend/tests/end-to-end/inputComponent.spec.ts @@ -22,7 +22,7 @@ test("InputComponent", async ({ page }) => { await page.waitForTimeout(1000); await page.getByTestId("blank-flow").click(); - await page.waitForTimeout(1000); + await page.waitForTimeout(3000); await page.getByTestId("extended-disclosure").click(); await page.getByPlaceholder("Search").click(); await page.getByPlaceholder("Search").fill("Chroma"); diff --git a/src/frontend/tests/end-to-end/inputListComponent.spec.ts b/src/frontend/tests/end-to-end/inputListComponent.spec.ts index 463312839..74c527892 100644 --- a/src/frontend/tests/end-to-end/inputListComponent.spec.ts +++ b/src/frontend/tests/end-to-end/inputListComponent.spec.ts @@ -21,7 +21,7 @@ test("InputListComponent", async ({ page }) => { } await page.getByTestId("blank-flow").click(); - await page.waitForTimeout(1000); + await page.waitForTimeout(3000); await page.getByTestId("extended-disclosure").click(); await page.getByPlaceholder("Search").click(); await page.getByPlaceholder("Search").fill("astradb"); diff --git a/src/frontend/tests/end-to-end/intComponent.spec.ts b/src/frontend/tests/end-to-end/intComponent.spec.ts index be3918b77..6ec81819a 100644 --- a/src/frontend/tests/end-to-end/intComponent.spec.ts +++ b/src/frontend/tests/end-to-end/intComponent.spec.ts @@ -22,7 +22,7 @@ test("IntComponent", async ({ page }) => { await page.waitForTimeout(1000); await page.getByTestId("blank-flow").click(); - await page.waitForTimeout(1000); + await page.waitForTimeout(3000); await page.getByTestId("extended-disclosure").click(); await page.getByPlaceholder("Search").click(); await page.getByPlaceholder("Search").fill("openai"); diff --git a/src/frontend/tests/end-to-end/keyPairListComponent.spec.ts b/src/frontend/tests/end-to-end/keyPairListComponent.spec.ts index bf9336b3c..31a4521ae 100644 --- a/src/frontend/tests/end-to-end/keyPairListComponent.spec.ts +++ b/src/frontend/tests/end-to-end/keyPairListComponent.spec.ts @@ -22,7 +22,7 @@ test("KeypairListComponent", async ({ page }) => { await page.waitForTimeout(1000); await page.getByTestId("blank-flow").click(); - await page.waitForTimeout(1000); + await page.waitForTimeout(3000); await page.getByTestId("extended-disclosure").click(); await page.getByPlaceholder("Search").click(); await page.getByPlaceholder("Search").fill("amazon bedrock"); diff --git a/src/frontend/tests/end-to-end/langflowShortcuts.spec.ts b/src/frontend/tests/end-to-end/langflowShortcuts.spec.ts index a6848d856..42b8744d1 100644 --- a/src/frontend/tests/end-to-end/langflowShortcuts.spec.ts +++ b/src/frontend/tests/end-to-end/langflowShortcuts.spec.ts @@ -29,7 +29,7 @@ test("LangflowShortcuts", async ({ page }) => { } await page.getByTestId("blank-flow").click(); - await page.waitForTimeout(1000); + await page.waitForTimeout(3000); await page.getByTestId("extended-disclosure").click(); await page.getByPlaceholder("Search").click(); await page.getByPlaceholder("Search").fill("ollama"); diff --git a/src/frontend/tests/end-to-end/nestedComponent.spec.ts b/src/frontend/tests/end-to-end/nestedComponent.spec.ts index eaa37f2e2..2df9c1b7b 100644 --- a/src/frontend/tests/end-to-end/nestedComponent.spec.ts +++ b/src/frontend/tests/end-to-end/nestedComponent.spec.ts @@ -21,7 +21,7 @@ test("NestedComponent", async ({ page }) => { } await page.getByTestId("blank-flow").click(); - await page.waitForTimeout(1000); + await page.waitForTimeout(3000); await page.getByTestId("extended-disclosure").click(); await page.getByPlaceholder("Search").click(); await page.getByPlaceholder("Search").fill("pinecone"); diff --git a/src/frontend/tests/end-to-end/promptModalComponent.spec.ts b/src/frontend/tests/end-to-end/promptModalComponent.spec.ts index 201a4b624..8c6a5ab84 100644 --- a/src/frontend/tests/end-to-end/promptModalComponent.spec.ts +++ b/src/frontend/tests/end-to-end/promptModalComponent.spec.ts @@ -21,7 +21,7 @@ test("PromptTemplateComponent", async ({ page }) => { } await page.getByTestId("blank-flow").click(); - await page.waitForTimeout(1000); + await page.waitForTimeout(3000); await page.getByTestId("extended-disclosure").click(); await page.getByPlaceholder("Search").click(); await page.getByPlaceholder("Search").fill("prompt"); diff --git a/src/frontend/tests/end-to-end/saveComponents.spec.ts b/src/frontend/tests/end-to-end/saveComponents.spec.ts index 10b679a89..624bb5b2e 100644 --- a/src/frontend/tests/end-to-end/saveComponents.spec.ts +++ b/src/frontend/tests/end-to-end/saveComponents.spec.ts @@ -91,6 +91,8 @@ test.describe("save component tests", () => { if (replaceButton) { await page.getByTestId("replace-button").click(); } + await page.waitForTimeout(3000); + await page.getByTestId("extended-disclosure").click(); await page.getByPlaceholder("Search").click(); await page.getByPlaceholder("Search").fill("group"); diff --git a/src/frontend/tests/end-to-end/textAreaModalComponent.spec.ts b/src/frontend/tests/end-to-end/textAreaModalComponent.spec.ts index 40443cfa5..9443586e6 100644 --- a/src/frontend/tests/end-to-end/textAreaModalComponent.spec.ts +++ b/src/frontend/tests/end-to-end/textAreaModalComponent.spec.ts @@ -21,7 +21,7 @@ test("TextAreaModalComponent", async ({ page }) => { } await page.getByTestId("blank-flow").click(); - await page.waitForTimeout(1000); + await page.waitForTimeout(3000); await page.getByTestId("extended-disclosure").click(); await page.getByPlaceholder("Search").click(); await page.getByPlaceholder("Search").fill("prompt"); diff --git a/src/frontend/tests/end-to-end/textInputOutput.spec.ts b/src/frontend/tests/end-to-end/textInputOutput.spec.ts index c2012ce1b..3da77e867 100644 --- a/src/frontend/tests/end-to-end/textInputOutput.spec.ts +++ b/src/frontend/tests/end-to-end/textInputOutput.spec.ts @@ -30,7 +30,7 @@ test("TextInputOutputComponent", async ({ page }) => { await page.waitForTimeout(1000); await page.getByTestId("blank-flow").click(); - await page.waitForTimeout(1000); + await page.waitForTimeout(3000); await page.getByTestId("extended-disclosure").click(); await page.getByPlaceholder("Search").click(); await page.getByPlaceholder("Search").fill("text input"); diff --git a/src/frontend/tests/end-to-end/toggleComponent.spec.ts b/src/frontend/tests/end-to-end/toggleComponent.spec.ts index 051366354..58d909af4 100644 --- a/src/frontend/tests/end-to-end/toggleComponent.spec.ts +++ b/src/frontend/tests/end-to-end/toggleComponent.spec.ts @@ -21,7 +21,7 @@ test("ToggleComponent", async ({ page }) => { } await page.getByTestId("blank-flow").click(); - await page.waitForTimeout(1000); + await page.waitForTimeout(3000); await page.getByTestId("extended-disclosure").click(); await page.getByPlaceholder("Search").click(); await page.getByPlaceholder("Search").fill("directory"); diff --git a/src/frontend/tests/end-to-end/tweaks_test.spec.ts b/src/frontend/tests/end-to-end/tweaks_test.spec.ts index e2e4c7c11..586d0d9e1 100644 --- a/src/frontend/tests/end-to-end/tweaks_test.spec.ts +++ b/src/frontend/tests/end-to-end/tweaks_test.spec.ts @@ -82,7 +82,7 @@ test("check if tweaks are updating when someothing on the flow changes", async ( await page.waitForTimeout(1000); await page.getByTestId("blank-flow").click(); - await page.waitForTimeout(1000); + await page.waitForTimeout(3000); await page.getByTestId("extended-disclosure").click(); await page.getByPlaceholder("Search").click(); await page.getByPlaceholder("Search").fill("Chroma"); diff --git a/src/frontend/tests/end-to-end/userSettings.spec.ts b/src/frontend/tests/end-to-end/userSettings.spec.ts index af7c93824..4516d3898 100644 --- a/src/frontend/tests/end-to-end/userSettings.spec.ts +++ b/src/frontend/tests/end-to-end/userSettings.spec.ts @@ -4,7 +4,11 @@ test("should see general profile gradient", async ({ page }) => { await page.goto("/"); await page.waitForTimeout(2000); await page.getByTestId("user-profile-settings").click(); + await page.waitForTimeout(1000); + await page.getByText("Settings").click(); + await page.waitForTimeout(3000); + await page.getByText("General").nth(2).isVisible(); await page.getByText("Profile Gradient").isVisible(); }); @@ -70,7 +74,11 @@ test("should see shortcuts", async ({ page }) => { await page.goto("/"); await page.waitForTimeout(2000); await page.getByTestId("user-profile-settings").click(); + await page.waitForTimeout(1000); + await page.getByText("Settings").click(); + await page.waitForTimeout(3000); + await page.getByText("General").nth(2).isVisible(); await page.getByText("Shortcuts").nth(0).click(); await page.getByText("Shortcuts", { exact: true }).nth(1).isVisible();