From 84350ea71c88ca7e871be718d89670036a8f61bb Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bel=C3=A9n=20Albeza?= Date: Thu, 20 Jun 2024 09:25:16 +0200 Subject: [PATCH] :paperclip: Remove dashboard visual tests that depend on specific config flags that are not on by default --- .../ui/visual-specs/visual-dashboard.spec.js | 111 +----------------- 1 file changed, 1 insertion(+), 110 deletions(-) diff --git a/frontend/playwright/ui/visual-specs/visual-dashboard.spec.js b/frontend/playwright/ui/visual-specs/visual-dashboard.spec.js index 88f4992e3..f7eeeb01a 100644 --- a/frontend/playwright/ui/visual-specs/visual-dashboard.spec.js +++ b/frontend/playwright/ui/visual-specs/visual-dashboard.spec.js @@ -3,11 +3,7 @@ import DashboardPage from "../pages/DashboardPage"; test.beforeEach(async ({ page }) => { await DashboardPage.init(page); - await DashboardPage.mockRPC( - page, - "get-profile", - "logged-in-user/get-profile-logged-in-no-onboarding.json", - ); + await DashboardPage.mockRPC(page, "get-profile", "logged-in-user/get-profile-logged-in-no-onboarding.json"); }); test("User goes to an empty dashboard", async ({ page }) => { @@ -123,15 +119,12 @@ test("User goes to an full search page", async ({ page }) => { await dashboardPage.setupDashboardFull(); await dashboardPage.goToSearch(); - await expect(dashboardPage.searchInput).toBeVisible(); await dashboardPage.searchInput.fill("New"); await expect(dashboardPage.searchTitle).toBeVisible(); - await expect(dashboardPage.newFileName).toBeVisible(); - await expect(dashboardPage.page).toHaveScreenshot(); }); @@ -141,9 +134,7 @@ test("User opens user account", async ({ page }) => { const dashboardPage = new DashboardPage(page); await dashboardPage.goToDashboard(); - await expect(dashboardPage.userAccount).toBeVisible(); - await dashboardPage.goToAccount(); await expect(dashboardPage.page).toHaveScreenshot(); @@ -153,11 +144,9 @@ test("User goes to user profile", async ({ page }) => { const dashboardPage = new DashboardPage(page); await dashboardPage.goToDashboard(); - await dashboardPage.goToAccount(); await expect(dashboardPage.userAccountTitle).toBeVisible(); - await expect(dashboardPage.page).toHaveScreenshot(); }); @@ -165,13 +154,11 @@ test("User goes to password management section", async ({ page }) => { const dashboardPage = new DashboardPage(page); await dashboardPage.goToDashboard(); - await dashboardPage.goToAccount(); await page.getByText("Password").click(); await expect(page.getByRole("heading", { name: "Change Password" })).toBeVisible(); - await expect(dashboardPage.page).toHaveScreenshot(); }); @@ -179,91 +166,11 @@ test("User goes to settings section", async ({ page }) => { const dashboardPage = new DashboardPage(page); await dashboardPage.goToDashboard(); - await dashboardPage.goToAccount(); await page.getByTestId("settings-profile").click(); await expect(page.getByRole("heading", { name: "Settings" })).toBeVisible(); - - await expect(dashboardPage.page).toHaveScreenshot(); -}); - -test("User goes to an empty access tokens secction", async ({ page }) => { - const dashboardPage = new DashboardPage(page); - - await dashboardPage.goToDashboard(); - - await dashboardPage.setupAccessTokensEmpty(); - - await dashboardPage.goToAccount(); - - await page.getByText("Access tokens").click(); - - await expect(page.getByRole("heading", { name: "Personal access tokens" })).toBeVisible(); - - await expect(dashboardPage.page).toHaveScreenshot(); -}); - -test("User can create an access token", async ({ page }) => { - const dashboardPage = new DashboardPage(page); - - await dashboardPage.goToDashboard(); - - await dashboardPage.setupAccessTokensEmpty(); - - await dashboardPage.goToAccount(); - - await page.getByText("Access tokens").click(); - - await expect(page.getByRole("heading", { name: "Personal access tokens" })).toBeVisible(); - - await page.getByRole("button", { name: "Generate New Token" }).click(); - - await dashboardPage.createAccessToken(); - - await expect(page.getByPlaceholder("The name can help to know")).toBeVisible(); - - await page.getByPlaceholder("The name can help to know").fill("New token"); - - await expect(page.getByRole("button", { name: "Create token" })).not.toBeDisabled(); - - await page.getByRole("button", { name: "Create token" }).click(); - - await expect(page.getByRole("button", { name: "Create token" })).not.toBeVisible(); - - await expect(dashboardPage.page).toHaveScreenshot(); -}); - -test("User goes to a full access tokens secction", async ({ page }) => { - const dashboardPage = new DashboardPage(page); - - await dashboardPage.goToDashboard(); - - await dashboardPage.setupAccessTokens(); - - await dashboardPage.goToAccount(); - - await page.getByText("Access tokens").click(); - - await expect(page.getByRole("heading", { name: "Personal access tokens" })).toBeVisible(); - - await expect(page.getByText("new token", { exact: true })).toBeVisible(); - - await expect(dashboardPage.page).toHaveScreenshot(); -}); - -test("User goes to the feedback secction", async ({ page }) => { - const dashboardPage = new DashboardPage(page); - - await dashboardPage.goToDashboard(); - - await dashboardPage.goToAccount(); - - await page.getByText("Give feedback").click(); - - await expect(page.getByRole("heading", { name: "Email" })).toBeVisible(); - await expect(dashboardPage.page).toHaveScreenshot(); }); @@ -273,13 +180,11 @@ test("User opens teams selector with only one team", async ({ page }) => { const dashboardPage = new DashboardPage(page); await dashboardPage.goToDashboard(); - await expect(dashboardPage.titleLabel).toBeVisible(); await dashboardPage.teamDropdown.click(); await expect(page.getByText("Create new team")).toBeVisible(); - await expect(dashboardPage.page).toHaveScreenshot(); }); @@ -288,30 +193,25 @@ test("User opens teams selector with more than one team", async ({ page }) => { await dashboardPage.setupDashboardFull(); await dashboardPage.goToDashboard(); - await expect(dashboardPage.titleLabel).toBeVisible(); await dashboardPage.teamDropdown.click(); await expect(page.getByText("Second Team")).toBeVisible(); - await expect(dashboardPage.page).toHaveScreenshot(); }); test("User goes to second team", async ({ page }) => { const dashboardPage = new DashboardPage(page); await dashboardPage.setupDashboardFull(); - await dashboardPage.goToDashboard(); await dashboardPage.teamDropdown.click(); - await expect(page.getByText("Second Team")).toBeVisible(); await page.getByText("Second Team").click(); await expect(page.getByText("Team Up")).toBeVisible(); - await expect(dashboardPage.page).toHaveScreenshot(); }); @@ -320,13 +220,11 @@ test("User opens team management dropdown", async ({ page }) => { await dashboardPage.setupDashboardFull(); await dashboardPage.goToSecondTeamDashboard(); - await expect(page.getByText("Team Up")).toBeVisible(); await page.getByRole("button", { name: "team-management" }).click(); await expect(page.getByTestId("team-members")).toBeVisible(); - await expect(dashboardPage.page).toHaveScreenshot(); }); @@ -365,24 +263,20 @@ test("User goes to a complete invitations section", async ({ page }) => { await expect(dashboardPage.page).toHaveScreenshot(); }); - test("User invite people to the team", async ({ page }) => { const dashboardPage = new DashboardPage(page); await dashboardPage.setupDashboardFull(); await dashboardPage.setupTeamInvitationsEmpty(); await dashboardPage.goToSecondTeamInvitationsSection(); - await expect(page.getByTestId("invite-member")).toBeVisible(); await page.getByTestId("invite-member").click(); - await expect(page.getByText("Invite with the role")).toBeVisible(); await page.getByPlaceholder("Emails, comma separated").fill("test5@mail.com"); await expect(page.getByText("Send invitation")).not.toBeDisabled(); - await expect(dashboardPage.page).toHaveScreenshot(); }); @@ -394,7 +288,6 @@ test("User goes to an empty webhook section", async ({ page }) => { await dashboardPage.goToSecondTeamWebhooksSection(); await expect(page.getByText("No webhooks created so far.")).toBeVisible(); - await expect(dashboardPage.page).toHaveScreenshot(); }); @@ -406,7 +299,6 @@ test("User goes to a complete webhook section", async ({ page }) => { await dashboardPage.goToSecondTeamWebhooksSection(); await expect(page.getByText("https://www.google.com")).toBeVisible(); - await expect(dashboardPage.page).toHaveScreenshot(); }); @@ -418,6 +310,5 @@ test("User goes to the team settings section", async ({ page }) => { await dashboardPage.goToSecondTeamSettingsSection(); await expect(page.getByText("TEAM INFO")).toBeVisible(); - await expect(dashboardPage.page).toHaveScreenshot(); });