From 52586706c4549ded94e57ed8cfa79092dd2033fa Mon Sep 17 00:00:00 2001 From: Joe Previte Date: Tue, 13 Apr 2021 12:02:52 -0700 Subject: [PATCH] refactor: use playwright-test syntax for e2e tests --- test/e2e/browser.test.ts | 47 ++++++++++++---------------------- test/e2e/globalSetup.test.ts | 21 ++++++++------- test/e2e/login.test.ts | 10 ++++---- test/e2e/loginPage.test.ts | 16 ++++-------- test/e2e/logout.test.ts | 10 ++++---- test/e2e/openHelpAbout.test.ts | 12 ++++----- 6 files changed, 47 insertions(+), 69 deletions(-) diff --git a/test/e2e/browser.test.ts b/test/e2e/browser.test.ts index 642ff7a5..fe1ab0e6 100644 --- a/test/e2e/browser.test.ts +++ b/test/e2e/browser.test.ts @@ -1,35 +1,22 @@ -/// +import { test, expect } from "@playwright/test" -// This test is for nothing more than to make sure -// tests are running in multiple browsers -describe("Browser gutcheck", () => { - beforeEach(async () => { - await jestPlaywright.resetBrowser({ - logger: { - isEnabled: (name) => name === "browser", - log: (name, severity, message, args) => console.log(`${name} ${message}`), - }, - }) - }) +test("should display correct browser based on userAgent", async ({ page, browserName }) => { + const displayNames = { + chromium: "Chrome", + firefox: "Firefox", + webkit: "Safari", + } + const userAgent = await page.evaluate("navigator.userAgent") - test("should display correct browser based on userAgent", async () => { - const displayNames = { - chromium: "Chrome", - firefox: "Firefox", - webkit: "Safari", - } - const userAgent = await page.evaluate("navigator.userAgent") + if (browserName === "chromium") { + expect(userAgent).toContain(displayNames[browserName]) + } - if (browserName === "chromium") { - expect(userAgent).toContain(displayNames[browserName]) - } + if (browserName === "firefox") { + expect(userAgent).toContain(displayNames[browserName]) + } - if (browserName === "firefox") { - expect(userAgent).toContain(displayNames[browserName]) - } - - if (browserName === "webkit") { - expect(userAgent).toContain(displayNames[browserName]) - } - }) + if (browserName === "webkit") { + expect(userAgent).toContain(displayNames[browserName]) + } }) diff --git a/test/e2e/globalSetup.test.ts b/test/e2e/globalSetup.test.ts index 3608fff4..28bb59b5 100644 --- a/test/e2e/globalSetup.test.ts +++ b/test/e2e/globalSetup.test.ts @@ -1,20 +1,19 @@ -/// +import { test, expect } from "@playwright/test" import { CODE_SERVER_ADDRESS, STORAGE } from "../utils/constants" // This test is to make sure the globalSetup works as expected // meaning globalSetup ran and stored the storageState in STORAGE -describe("globalSetup", () => { - beforeEach(async () => { - // Create a new context with the saved storage state - // so we don't have to logged in - const storageState = JSON.parse(STORAGE) || {} - await jestPlaywright.resetContext({ +test.describe("globalSetup", () => { + // Create a new context with the saved storage state + // so we don't have to logged in + const storageState = JSON.parse(STORAGE) || {} + const options = { + contextOptions: { storageState, - }) + }, + } + test("should keep us logged in using the storageState", options, async ({ page }) => { await page.goto(CODE_SERVER_ADDRESS, { waitUntil: "networkidle" }) - }) - - it("should keep us logged in using the storageState", async () => { // Make sure the editor actually loaded expect(await page.isVisible("div.monaco-workbench")) }) diff --git a/test/e2e/login.test.ts b/test/e2e/login.test.ts index 41983855..14bec49d 100644 --- a/test/e2e/login.test.ts +++ b/test/e2e/login.test.ts @@ -1,13 +1,13 @@ -/// +import { test, expect } from "@playwright/test" import { CODE_SERVER_ADDRESS, PASSWORD } from "../utils/constants" -describe("login", () => { - beforeEach(async () => { - await jestPlaywright.resetBrowser() +test.describe("login", () => { + test.beforeEach(async ({ page }) => { + // TODO@jsjoeio reset the browser await page.goto(CODE_SERVER_ADDRESS, { waitUntil: "networkidle" }) }) - it("should be able to login", async () => { + test("should be able to login", async ({ page }) => { // Type in password await page.fill(".password", PASSWORD) // Click the submit button and login diff --git a/test/e2e/loginPage.test.ts b/test/e2e/loginPage.test.ts index 9016ded4..e6f03d3e 100644 --- a/test/e2e/loginPage.test.ts +++ b/test/e2e/loginPage.test.ts @@ -1,19 +1,13 @@ -/// - +import { test, expect } from "@playwright/test" import { CODE_SERVER_ADDRESS } from "../utils/constants" -describe("login page", () => { - beforeEach(async () => { - await jestPlaywright.resetContext({ - logger: { - isEnabled: (name, severity) => name === "browser", - log: (name, severity, message, args) => console.log(`${name} ${message}`), - }, - }) +test.describe("login page", () => { + test.beforeEach(async ({ page }) => { + // TODO@jsjoeio reset context somehow await page.goto(CODE_SERVER_ADDRESS, { waitUntil: "networkidle" }) }) - it("should see the login page", async () => { + test("should see the login page", async ({ page }) => { // It should send us to the login page expect(await page.title()).toBe("code-server login") }) diff --git a/test/e2e/logout.test.ts b/test/e2e/logout.test.ts index bb5c2003..f3acf08e 100644 --- a/test/e2e/logout.test.ts +++ b/test/e2e/logout.test.ts @@ -1,13 +1,13 @@ -/// +import { test, expect } from "@playwright/test" import { CODE_SERVER_ADDRESS, PASSWORD } from "../utils/constants" -describe("logout", () => { - beforeEach(async () => { - await jestPlaywright.resetBrowser() +test.describe("logout", () => { + test.beforeEach(async ({ page }) => { + // TODO@jsjoeio reset context await page.goto(CODE_SERVER_ADDRESS, { waitUntil: "networkidle" }) }) - it("should be able login and logout", async () => { + test("should be able login and logout", async ({ page }) => { // Type in password await page.fill(".password", PASSWORD) // Click the submit button and login diff --git a/test/e2e/openHelpAbout.test.ts b/test/e2e/openHelpAbout.test.ts index 18fe407b..62cf7519 100644 --- a/test/e2e/openHelpAbout.test.ts +++ b/test/e2e/openHelpAbout.test.ts @@ -1,18 +1,16 @@ -/// +import { test, expect } from "@playwright/test" import { CODE_SERVER_ADDRESS, STORAGE } from "../utils/constants" -describe("Open Help > About", () => { - beforeEach(async () => { +test.describe("Open Help > About", () => { + test.beforeEach(async ({ page }) => { // Create a new context with the saved storage state // so we don't have to logged in + // TODO@jsjoeio reset context and use storageState const storageState = JSON.parse(STORAGE) || {} - await jestPlaywright.resetContext({ - storageState, - }) await page.goto(CODE_SERVER_ADDRESS, { waitUntil: "networkidle" }) }) - it("should see a 'Help' then 'About' button in the Application Menu that opens a dialog", async () => { + test("should see a 'Help' then 'About' button in the Application Menu that opens a dialog", async ({ page }) => { // Make sure the editor actually loaded expect(await page.isVisible("div.monaco-workbench"))