From a99612af3bc7622d499d5ba26878eed95d9fc451 Mon Sep 17 00:00:00 2001 From: cristhianzl Date: Wed, 27 Mar 2024 17:04:19 -0300 Subject: [PATCH] =?UTF-8?q?=E2=9C=A8=20(playwright.config.ts):=20set=20the?= =?UTF-8?q?=20number=20of=20workers=20to=202=20to=20improve=20test=20paral?= =?UTF-8?q?lelization=20=F0=9F=93=9D=20(API/index.ts):=20remove=20conditio?= =?UTF-8?q?nal=20return=20of=20100=20for=20getRepoStars=20function=20to=20?= =?UTF-8?q?ensure=20accurate=20retrieval=20of=20stargazers=20count=20from?= =?UTF-8?q?=20GitHub=20API=20=F0=9F=93=9D=20(API/index.ts):=20remove=20con?= =?UTF-8?q?ditional=20return=20of=20example=20data=20for=20getExamples=20f?= =?UTF-8?q?unction=20to=20ensure=20accurate=20retrieval=20of=20examples=20?= =?UTF-8?q?from=20GitHub=20API=20=F0=9F=93=9D=20(auto=5Flogin.spec.ts):=20?= =?UTF-8?q?update=20page.goto=20URLs=20to=20include=20the=20protocol=20to?= =?UTF-8?q?=20fix=20navigation=20issues=20=F0=9F=93=9D=20(codeAreaModalCom?= =?UTF-8?q?ponent.spec.ts):=20update=20page.goto=20URLs=20to=20include=20t?= =?UTF-8?q?he=20protocol=20to=20fix=20navigation=20issues=20=F0=9F=93=9D?= =?UTF-8?q?=20(dragAndDrop.spec.ts):=20update=20page.goto=20URLs=20to=20in?= =?UTF-8?q?clude=20the=20protocol=20to=20fix=20navigation=20issues=20?= =?UTF-8?q?=F0=9F=93=9D=20(dropdownComponent.spec.ts):=20update=20page.got?= =?UTF-8?q?o=20URLs=20to=20include=20the=20protocol=20to=20fix=20navigatio?= =?UTF-8?q?n=20issues=20=F0=9F=93=9D=20(floatComponent.spec.ts):=20update?= =?UTF-8?q?=20page.goto=20URLs=20to=20include=20the=20protocol=20to=20fix?= =?UTF-8?q?=20navigation=20issues=20=F0=9F=93=9D=20(flowPage.spec.ts):=20u?= =?UTF-8?q?pdate=20page.goto=20URLs=20to=20include=20the=20protocol=20to?= =?UTF-8?q?=20fix=20navigation=20issues=20=F0=9F=93=9D=20(group.spec.ts):?= =?UTF-8?q?=20update=20page.goto=20URLs=20to=20include=20the=20protocol=20?= =?UTF-8?q?to=20fix=20navigation=20issues=20=F0=9F=93=9D=20(inputComponent?= =?UTF-8?q?.spec.ts):=20update=20page.goto=20URLs=20to=20include=20the=20p?= =?UTF-8?q?rotocol=20to=20fix=20navigation=20issues=20=F0=9F=93=9D=20(intC?= =?UTF-8?q?omponent.spec.ts):=20update=20page.goto=20URLs=20to=20include?= =?UTF-8?q?=20the=20protocol=20to=20fix=20navigation=20issues=20?= =?UTF-8?q?=F0=9F=93=9D=20(keyPairListComponent.spec.ts):=20update=20page.?= =?UTF-8?q?goto=20URLs=20to=20include=20the=20protocol=20to=20fix=20naviga?= =?UTF-8?q?tion=20issues=20=F0=9F=93=9D=20(langflowShortcuts.spec.ts):=20u?= =?UTF-8?q?pdate=20page.goto=20URLs=20to=20include=20the=20protocol=20to?= =?UTF-8?q?=20fix=20navigation=20issues=20=F0=9F=93=9D=20(nestedComponent.?= =?UTF-8?q?spec.ts):=20update=20page.goto=20URLs=20to=20include=20the=20pr?= =?UTF-8?q?otocol=20to=20fix=20navigation=20issues=20=F0=9F=93=9D=20(promp?= =?UTF-8?q?tModalComponent.spec.ts):=20update=20page.goto=20URLs=20to=20in?= =?UTF-8?q?clude=20the=20protocol=20to=20fix=20navigation=20issues?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ✅ (saveComponents.spec.ts): update page.goto() URL to include the correct protocol and port for the frontend app ✅ (toggleComponent.spec.ts): update page.goto() URL to include the correct protocol and port for the frontend app --- src/frontend/playwright.config.ts | 2 +- src/frontend/src/controllers/API/index.ts | 16 ---------------- src/frontend/tests/end-to-end/auto_login.spec.ts | 4 ++-- .../end-to-end/codeAreaModalComponent.spec.ts | 2 +- .../tests/end-to-end/dragAndDrop.spec.ts | 2 +- .../tests/end-to-end/dropdownComponent.spec.ts | 2 +- .../tests/end-to-end/floatComponent.spec.ts | 2 +- src/frontend/tests/end-to-end/flowPage.spec.ts | 4 ++-- src/frontend/tests/end-to-end/group.spec.ts | 2 +- .../tests/end-to-end/inputComponent.spec.ts | 2 +- .../tests/end-to-end/intComponent.spec.ts | 2 +- .../end-to-end/keyPairListComponent.spec.ts | 2 +- .../tests/end-to-end/langflowShortcuts.spec.ts | 2 +- .../tests/end-to-end/nestedComponent.spec.ts | 2 +- .../end-to-end/promptModalComponent.spec.ts | 2 +- .../tests/end-to-end/saveComponents.spec.ts | 2 +- .../tests/end-to-end/toggleComponent.spec.ts | 2 +- 17 files changed, 18 insertions(+), 34 deletions(-) diff --git a/src/frontend/playwright.config.ts b/src/frontend/playwright.config.ts index 59f30c4ba..97f3e2c9e 100644 --- a/src/frontend/playwright.config.ts +++ b/src/frontend/playwright.config.ts @@ -18,7 +18,7 @@ export default defineConfig({ /* Retry on CI only */ retries: process.env.CI ? 2 : 0, /* Opt out of parallel tests on CI. */ - workers: process.env.CI ? 2 : undefined, + workers: 2, /* Reporter to use. See https://playwright.dev/docs/test-reporters */ timeout: 120 * 1000, // reporter: [ diff --git a/src/frontend/src/controllers/API/index.ts b/src/frontend/src/controllers/API/index.ts index 6d3ac028d..694428a22 100644 --- a/src/frontend/src/controllers/API/index.ts +++ b/src/frontend/src/controllers/API/index.ts @@ -40,9 +40,6 @@ const GITHUB_API_URL = "https://api.github.com"; export async function getRepoStars(owner: string, repo: string) { try { - if (process.env.CI === "True") { - return 100; - } const response = await api.get(`${GITHUB_API_URL}/repos/${owner}/${repo}`); return response.data.stargazers_count; } catch (error) { @@ -92,19 +89,6 @@ export async function postValidatePrompt( * @returns {Promise} A promise that resolves to an array of FlowType objects. */ export async function getExamples(): Promise { - debugger; - if (process.env.CI === "True") { - return new Promise((resolve, reject) => { - resolve([ - { - name: "Example", - id: "example", - data: null, - description: "Example description", - }, - ]); - }); - } const url = "https://api.github.com/repos/logspace-ai/langflow_examples/contents/examples?ref=main"; const response = await api.get(url); diff --git a/src/frontend/tests/end-to-end/auto_login.spec.ts b/src/frontend/tests/end-to-end/auto_login.spec.ts index 8eaed40c7..929fb6a0d 100644 --- a/src/frontend/tests/end-to-end/auto_login.spec.ts +++ b/src/frontend/tests/end-to-end/auto_login.spec.ts @@ -2,12 +2,12 @@ import { test } from "@playwright/test"; test.describe("Auto_login tests", () => { test("auto_login sign in", async ({ page }) => { - await page.goto("/"); + await page.goto("http:localhost:3000/"); await page.locator('//*[@id="new-project-btn"]').click(); }); test("auto_login block_admin", async ({ page }) => { - await page.goto("/"); + await page.goto("http:localhost:3000/"); await page.locator('//*[@id="new-project-btn"]').click(); await page.waitForTimeout(5000); diff --git a/src/frontend/tests/end-to-end/codeAreaModalComponent.spec.ts b/src/frontend/tests/end-to-end/codeAreaModalComponent.spec.ts index f812cbbfd..94de56e74 100644 --- a/src/frontend/tests/end-to-end/codeAreaModalComponent.spec.ts +++ b/src/frontend/tests/end-to-end/codeAreaModalComponent.spec.ts @@ -1,7 +1,7 @@ import { expect, test } from "@playwright/test"; test("CodeAreaModalComponent", async ({ page }) => { - await page.goto("/"); + await page.goto("http:localhost:3000/"); await page.waitForTimeout(2000); await page.locator('//*[@id="new-project-btn"]').click(); diff --git a/src/frontend/tests/end-to-end/dragAndDrop.spec.ts b/src/frontend/tests/end-to-end/dragAndDrop.spec.ts index 3850709d7..72cccaa9a 100644 --- a/src/frontend/tests/end-to-end/dragAndDrop.spec.ts +++ b/src/frontend/tests/end-to-end/dragAndDrop.spec.ts @@ -4,7 +4,7 @@ import { readFileSync } from "fs"; test.describe("drag and drop test", () => { /// test("drop collection", async ({ page }) => { - await page.goto("/"); + await page.goto("http:localhost:3000/"); await page.locator("span").filter({ hasText: "My Collection" }).isVisible(); // Read your file into a buffer. const jsonContent = readFileSync( diff --git a/src/frontend/tests/end-to-end/dropdownComponent.spec.ts b/src/frontend/tests/end-to-end/dropdownComponent.spec.ts index 3352e21bb..7d0f52d36 100644 --- a/src/frontend/tests/end-to-end/dropdownComponent.spec.ts +++ b/src/frontend/tests/end-to-end/dropdownComponent.spec.ts @@ -1,7 +1,7 @@ import { expect, test } from "@playwright/test"; test("dropDownComponent", async ({ page }) => { - await page.goto("/"); + await page.goto("http:localhost:3000/"); await page.waitForTimeout(2000); await page.locator('//*[@id="new-project-btn"]').click(); diff --git a/src/frontend/tests/end-to-end/floatComponent.spec.ts b/src/frontend/tests/end-to-end/floatComponent.spec.ts index c148f8c05..95e3478e8 100644 --- a/src/frontend/tests/end-to-end/floatComponent.spec.ts +++ b/src/frontend/tests/end-to-end/floatComponent.spec.ts @@ -1,7 +1,7 @@ import { expect, test } from "@playwright/test"; test("FloatComponent", async ({ page }) => { - await page.goto("/"); + await page.goto("http:localhost:3000/"); await page.waitForTimeout(2000); await page.locator('//*[@id="new-project-btn"]').click(); diff --git a/src/frontend/tests/end-to-end/flowPage.spec.ts b/src/frontend/tests/end-to-end/flowPage.spec.ts index 4bba77ab0..ca0bc076a 100644 --- a/src/frontend/tests/end-to-end/flowPage.spec.ts +++ b/src/frontend/tests/end-to-end/flowPage.spec.ts @@ -2,12 +2,12 @@ import { Page, test } from "@playwright/test"; test.describe("Flow Page tests", () => { async function goToFlowPage(page: Page) { - await page.goto("/"); + await page.goto("http:localhost:3000/"); await page.getByRole("button", { name: "New Project" }).click(); } test("save", async ({ page }) => { - await page.goto("/"); + await page.goto("http:localhost:3000/"); await page.waitForTimeout(2000); await page.locator('//*[@id="new-project-btn"]').click(); diff --git a/src/frontend/tests/end-to-end/group.spec.ts b/src/frontend/tests/end-to-end/group.spec.ts index 50dce0850..e937a3308 100644 --- a/src/frontend/tests/end-to-end/group.spec.ts +++ b/src/frontend/tests/end-to-end/group.spec.ts @@ -4,7 +4,7 @@ import { readFileSync } from "fs"; test.describe("group node test", () => { /// test("group and ungroup updating values", async ({ page }) => { - await page.goto("/"); + await page.goto("http:localhost:3000/"); await page.locator('//*[@id="new-project-btn"]').click(); await page.getByTestId("blank-flow").click(); diff --git a/src/frontend/tests/end-to-end/inputComponent.spec.ts b/src/frontend/tests/end-to-end/inputComponent.spec.ts index 43f108af7..8756c1914 100644 --- a/src/frontend/tests/end-to-end/inputComponent.spec.ts +++ b/src/frontend/tests/end-to-end/inputComponent.spec.ts @@ -1,7 +1,7 @@ import { expect, test } from "@playwright/test"; test("InputComponent", async ({ page }) => { - await page.goto("/"); + await page.goto("http:localhost:3000/"); await page.waitForTimeout(2000); await page.locator('//*[@id="new-project-btn"]').click(); diff --git a/src/frontend/tests/end-to-end/intComponent.spec.ts b/src/frontend/tests/end-to-end/intComponent.spec.ts index 3e5e6cabf..bf2c1ea5e 100644 --- a/src/frontend/tests/end-to-end/intComponent.spec.ts +++ b/src/frontend/tests/end-to-end/intComponent.spec.ts @@ -1,7 +1,7 @@ import { expect, test } from "@playwright/test"; test("IntComponent", async ({ page }) => { - await page.goto("/"); + await page.goto("http:localhost:3000/"); await page.waitForTimeout(2000); await page.locator('//*[@id="new-project-btn"]').click(); diff --git a/src/frontend/tests/end-to-end/keyPairListComponent.spec.ts b/src/frontend/tests/end-to-end/keyPairListComponent.spec.ts index 4245fffdc..88b9a2e25 100644 --- a/src/frontend/tests/end-to-end/keyPairListComponent.spec.ts +++ b/src/frontend/tests/end-to-end/keyPairListComponent.spec.ts @@ -1,7 +1,7 @@ import { expect, test } from "@playwright/test"; test("KeypairListComponent", async ({ page }) => { - await page.goto("/"); + await page.goto("http:localhost:3000/"); await page.waitForTimeout(2000); await page.locator('//*[@id="new-project-btn"]').click(); diff --git a/src/frontend/tests/end-to-end/langflowShortcuts.spec.ts b/src/frontend/tests/end-to-end/langflowShortcuts.spec.ts index 67af2dd4b..173ec850e 100644 --- a/src/frontend/tests/end-to-end/langflowShortcuts.spec.ts +++ b/src/frontend/tests/end-to-end/langflowShortcuts.spec.ts @@ -10,7 +10,7 @@ test("LangflowShortcuts", async ({ page }) => { control = "Meta"; } - await page.goto("/"); + await page.goto("http:localhost:3000/"); await page.waitForTimeout(1000); await page.locator('//*[@id="new-project-btn"]').click(); diff --git a/src/frontend/tests/end-to-end/nestedComponent.spec.ts b/src/frontend/tests/end-to-end/nestedComponent.spec.ts index ee36b6f20..994395ad6 100644 --- a/src/frontend/tests/end-to-end/nestedComponent.spec.ts +++ b/src/frontend/tests/end-to-end/nestedComponent.spec.ts @@ -1,7 +1,7 @@ import { expect, test } from "@playwright/test"; test("NestedComponent", async ({ page }) => { - await page.goto("/"); + await page.goto("http:localhost:3000/"); await page.waitForTimeout(2000); await page.locator('//*[@id="new-project-btn"]').click(); diff --git a/src/frontend/tests/end-to-end/promptModalComponent.spec.ts b/src/frontend/tests/end-to-end/promptModalComponent.spec.ts index d945f2436..e90deef46 100644 --- a/src/frontend/tests/end-to-end/promptModalComponent.spec.ts +++ b/src/frontend/tests/end-to-end/promptModalComponent.spec.ts @@ -1,7 +1,7 @@ import { expect, test } from "@playwright/test"; test("PromptTemplateComponent", async ({ page }) => { - await page.goto("/"); + await page.goto("http:localhost:3000/"); await page.waitForTimeout(2000); await page.locator('//*[@id="new-project-btn"]').click(); diff --git a/src/frontend/tests/end-to-end/saveComponents.spec.ts b/src/frontend/tests/end-to-end/saveComponents.spec.ts index ac5ee69db..13d8e20f5 100644 --- a/src/frontend/tests/end-to-end/saveComponents.spec.ts +++ b/src/frontend/tests/end-to-end/saveComponents.spec.ts @@ -11,7 +11,7 @@ test.describe("save component tests", () => { /// test("save group component tests", async ({ page }) => { - await page.goto("/"); + await page.goto("http:localhost:3000/"); await page.locator('//*[@id="new-project-btn"]').click(); await page.getByTestId("blank-flow").click(); diff --git a/src/frontend/tests/end-to-end/toggleComponent.spec.ts b/src/frontend/tests/end-to-end/toggleComponent.spec.ts index 9d4eb9ed8..aa28dbc45 100644 --- a/src/frontend/tests/end-to-end/toggleComponent.spec.ts +++ b/src/frontend/tests/end-to-end/toggleComponent.spec.ts @@ -1,7 +1,7 @@ import { expect, test } from "@playwright/test"; test("ToggleComponent", async ({ page }) => { - await page.goto("/"); + await page.goto("http:localhost:3000/"); await page.waitForTimeout(2000); await page.locator('//*[@id="new-project-btn"]').click();