chore: fix more of the tests

Author: twk3

basic/0-failing.spec.ts

@@ -2,7 +2,7 @@ import { expect } from "@playwright/test";
 import { test } from "./base.ts";
 
 test("basic test @basic", async ({ page }) => {
-  await page.goto("https://todomvc.com/examples/backbone/dist/");
+  await page.goto("https://demo.playwright.dev/todomvc");
 
   // Use locators to represent a selector and re-use them
   const inputBox = page.locator("input.new-todo");

basic/1-getting-started.spec.ts

@@ -2,7 +2,7 @@ import { expect } from "@playwright/test";
 import { test } from "./base.ts";
 
 test("basic test @basic", async ({ page }) => {
-  await page.goto("https://todomvc.com/examples/backbone/dist/");
+  await page.goto("https://demo.playwright.dev/todomvc");
 
   // Use locators to represent a selector and re-use them
   const inputBox = page.locator("input.new-todo");

basic/3-assertions.spec.ts

@@ -2,7 +2,7 @@ import { expect } from "@playwright/test";
 import { test } from "./base.ts";
 
 test.beforeEach(async ({ page }) => {
-  await page.goto("https://todomvc.com/examples/backbone/dist/");
+  await page.goto("https://demo.playwright.dev/todomvc");
 });
 
 /**
@@ -11,8 +11,8 @@ test.beforeEach(async ({ page }) => {
  */
 test("should be able to use assertions", async ({ page }) => {
   await test.step("toHaveTitle/toHaveURL", async () => {
-    await expect(page).toHaveTitle("TodoMVC: Backbone");
-    await expect(page).toHaveURL("https://todomvc.com/examples/backbone/dist/");
+    await expect(page).toHaveTitle("React • TodoMVC");
+    await expect(page).toHaveURL(/^https:\/\/demo.playwright.dev\/todomvc/);
   });
 
   await test.step("toBeEmpty/toHaveValue", async () => {
@@ -46,11 +46,9 @@ test("should be able to use assertions", async ({ page }) => {
     await expect(page.locator("text=Buy milk")).toBeHidden();
   });
 
-  await test.step("toHaveClass/toHaveCSS", async () => {
+  await test.step("toHaveClass", async () => {
     await expect(
       page.locator('[placeholder="What needs to be done?"]')
     ).toHaveClass("new-todo");
-    await page.click("text=Clear completed");
-    await expect(page.locator(".main")).toHaveCSS("display", "none");
   });
 });

basic/8-rules.spec.ts

@@ -11,7 +11,7 @@ import { test } from "./base.ts";
  * This is a failing test that will be quarantined by the file based rule
  */
 test("rules quarantine test", async ({ page }) => {
-  await page.goto("https://todomvc.com/examples/backbone/dist/");
+  await page.goto("https://demo.playwright.dev/todomvc");
 
   // Use locators to represent a selector and re-use them
   const inputBox = page.locator("input.new-todo");
@@ -26,7 +26,7 @@ test("rules quarantine test", async ({ page }) => {
  * This tests will be completely skipped by a rule that matches fileName + test name
  */
 test("rules skip test", async ({ page }) => {
-  await page.goto("https://todomvc.com/examples/backbone/dist/");
+  await page.goto("https://demo.playwright.dev/todomvc");
 
   // Use locators to represent a selector and re-use them
   const inputBox = page.locator("input.new-todo");

basic/base.ts

@@ -1,11 +1,11 @@
 import {
-    CurrentsFixtures,
-    CurrentsWorkerFixtures,
-    fixtures,
-  } from "@currents/playwright";
-  import { test as base } from "@playwright/test";
-  
-  export const test = base.extend<CurrentsFixtures, CurrentsWorkerFixtures>({
-    ...fixtures.baseFixtures,
-    ...fixtures.rulesFixtures,
-  });
+  CurrentsFixtures,
+  CurrentsWorkerFixtures,
+  fixtures,
+} from "@currents/playwright";
+import { test as base } from "@playwright/test";
+
+export const test = base.extend<CurrentsFixtures, CurrentsWorkerFixtures>({
+  ...fixtures.baseFixtures,
+  ...fixtures.rulesFixtures,
+});

basic/pom/todoPage.pom.ts

@@ -25,6 +25,6 @@ export class TodoPage {
   }
 
   async goto() {
-    await this.page.goto("https://todomvc.com/examples/backbone/dist/");
+    await this.page.goto("https://demo.playwright.dev/todomvc");
   }
 }

basic/pw.config.shared.ts

@@ -15,7 +15,6 @@ const config = defineConfig<CurrentsFixtures, CurrentsWorkerFixtures>({
   workers: process.env.CI ? 1 : undefined,
 
   use: {
-    baseURL: process.env.BASEURL || "http://localhost:4346",
     actionTimeout: 0,
     trace: "on",
     video: "on",
@@ -41,7 +40,7 @@ const config = defineConfig<CurrentsFixtures, CurrentsWorkerFixtures>({
 
   webServer: {
     command: 'node ./server/index.js',
-    url: 'http://127.0.0.1:4346',
+    port: 4346,
     reuseExistingServer: !process.env.CI,
     stdout: 'ignore',
     stderr: 'pipe',