desktop: migrate tests to vitest 3

apps/desktop/__tests__/auto-updates.test.ts

@@ -17,19 +17,21 @@ You should have received a copy of the GNU General Public License
 along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
 
-import { test } from "vitest";
-import { harness } from "./utils.js";
+import { test, testCleanup } from "./utils.js";
 import { writeFile } from "fs/promises";
 import { Page } from "playwright";
 import { gt, lt } from "semver";
+import { describe } from "vitest";
+
+test("update starts downloading if version is outdated", async ({
+  ctx: { page },
+  expect,
+  onTestFinished
+}) => {
+  onTestFinished(testCleanup);
 
-test("update starts downloading if version is outdated", async (t) => {
-  await harness(
-    t,
-    async ({ page }) => {
   await page.waitForSelector("#authForm");
-
-      t.expect(
+  expect(
     await page.getByRole("button", { name: "Create account" }).isVisible()
   ).toBe(true);
 
@@ -45,15 +47,15 @@ test("update starts downloading if version is outdated", async (t) => {
     .locator(".theme-scope-statusBar")
     .getByRole("button", { name: /updating/i })
     .waitFor({ state: "attached" });
-    },
-    { version: "3.0.0" }
-  );
 });
 
-test("update is only shown if version is outdated and auto updates are disabled", async (t) => {
-  await harness(
-    t,
-    async (ctx) => {
+test("update is only shown if version is outdated and auto updates are disabled", async ({
+  ctx,
+  expect,
+  onTestFinished
+}) => {
+  onTestFinished(testCleanup);
+
   await ctx.app.close();
   await writeFile(
     ctx.configPath,
@@ -68,7 +70,7 @@ test("update is only shown if version is outdated and auto updates are disabled"
 
   await page.waitForSelector("#authForm");
 
-      t.expect(
+  expect(
     await page.getByRole("button", { name: "Create account" }).isVisible()
   ).toBe(true);
 
@@ -84,15 +86,13 @@ test("update is only shown if version is outdated and auto updates are disabled"
     .locator(".theme-scope-statusBar")
     .getByRole("button", { name: /available/i })
     .waitFor({ state: "attached" });
-    },
-    { version: "3.0.0" }
-  );
 });
 
-test("update to stable if it is newer", async (t) => {
-  await harness(
-    t,
-    async (ctx) => {
+describe("update to stable if it is newer", () => {
+  test.scoped({ options: { version: "3.0.0-beta.0" } });
+  test("test", async ({ ctx, expect, onTestFinished }) => {
+    onTestFinished(testCleanup);
+
     await ctx.app.close();
     await writeFile(
       ctx.configPath,
@@ -108,7 +108,7 @@ test("update to stable if it is newer", async (t) => {
 
     await page.waitForSelector("#authForm");
 
-      t.expect(
+    expect(
       await page.getByRole("button", { name: "Create account" }).isVisible()
     ).toBe(true);
 
@@ -126,16 +126,15 @@ test("update to stable if it is newer", async (t) => {
     await updateButton.waitFor({ state: "visible" });
     const content = await updateButton.textContent();
     const version = content?.split(" ")?.[0] || "";
-      t.expect(gt(version, "3.0.0-beta.0")).toBe(true);
-    },
-    { version: "3.0.0-beta.0" }
-  );
+    expect(gt(version, "3.0.0-beta.0")).toBe(true);
+  });
 });
 
-test("update is not available if it latest stable version is older", async (t) => {
-  await harness(
-    t,
-    async (ctx) => {
+describe("update is not available if it latest stable version is older", () => {
+  test.scoped({ options: { version: "99.0.0-beta.0" } });
+  test("test", async ({ ctx, expect, onTestFinished }) => {
+    onTestFinished(testCleanup);
+
     await ctx.app.close();
     await writeFile(
       ctx.configPath,
@@ -151,7 +150,7 @@ test("update is not available if it latest stable version is older", async (t) =
 
     await page.waitForSelector("#authForm");
 
-      t.expect(
+    expect(
       await page.getByRole("button", { name: "Create account" }).isVisible()
     ).toBe(true);
 
@@ -168,21 +167,20 @@ test("update is not available if it latest stable version is older", async (t) =
       .getByRole("button", { name: /checking for updates/i })
       .waitFor({ state: "hidden" });
 
-      t.expect(
+    expect(
       await page
         .locator(".theme-scope-statusBar")
         .getByRole("button", { name: /available/i })
         .isHidden()
     ).toBe(true);
-    },
-    { version: "99.0.0-beta.0" }
-  );
+  });
 });
 
-test("downgrade to stable on switching to stable release track", async (t) => {
-  await harness(
-    t,
-    async (ctx) => {
+describe("downgrade to stable on switching to stable release track", () => {
+  test.scoped({ options: { version: "99.0.0-beta.0" } });
+  test("test", async ({ ctx, expect, onTestFinished }) => {
+    onTestFinished(testCleanup);
+
     await ctx.app.close();
     await writeFile(
       ctx.configPath,
@@ -198,7 +196,7 @@ test("downgrade to stable on switching to stable release track", async (t) => {
 
     await page.waitForSelector("#authForm");
 
-      t.expect(
+    expect(
       await page.getByRole("button", { name: "Create account" }).isVisible()
     ).toBe(true);
 
@@ -221,10 +219,8 @@ test("downgrade to stable on switching to stable release track", async (t) => {
     await updateButton.waitFor({ state: "visible" });
     const content = await updateButton.textContent();
     const version = content?.split(" ")?.[0] || "";
-      t.expect(lt(version, "99.0.0-beta.0")).toBe(true);
-    },
-    { version: "99.0.0-beta.0" }
-  );
+    expect(lt(version, "99.0.0-beta.0")).toBe(true);
+  });
 });
 
 async function skipDialog(page: Page) {
@@ -237,9 +233,9 @@ async function skipDialog(page: Page) {
       "button[data-role='negative-button']"
     );
     if (await positiveButton.isVisible())
-      await positiveButton.click({ timeout: 2000 });
+      await positiveButton.click({ timeout: 1000 });
     else if (await negativeButton.isVisible())
-      await negativeButton.click({ timeout: 2000 });
+      await negativeButton.click({ timeout: 1000 });
   } catch (e) {
     // ignore error
   }

apps/desktop/__tests__/launch.test.ts

@@ -17,17 +17,20 @@ You should have received a copy of the GNU General Public License
 along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
 
-import { test } from "vitest";
-import { harness } from "./utils.js";
-import assert from "assert";
+import { testCleanup, test } from "./utils.js";
+
+test("make sure app loads", async ({
+  ctx: { page },
+  expect,
+  onTestFinished
+}) => {
+  onTestFinished(testCleanup);
 
-test("make sure app loads", async (t) => {
-  await harness(t, async ({ page }) => {
   await page.waitForSelector("#authForm");
 
-    assert.ok(
+  expect(
     await page.getByRole("button", { name: "Create account" }).isVisible()
-    );
+  ).toBe(true);
 
   await page
     .getByRole("button", { name: "Skip & go directly to the app" })
@@ -35,4 +38,3 @@ test("make sure app loads", async (t) => {
 
   await page.waitForSelector(".ProseMirror");
 });
-});

apps/desktop/__tests__/utils.ts

@@ -23,7 +23,7 @@ import { fileURLToPath } from "node:url";
 import path from "path";
 import { _electron as electron } from "playwright";
 import slugify from "slugify";
-import { TaskContext } from "vitest";
+import { test as vitestTest, TestContext } from "vitest";
 
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
@@ -42,31 +42,34 @@ interface TestOptions {
   version: string;
 }
 
-export async function harness(
-  t: TaskContext,
-  cb: (ctx: AppContext) => Promise<void>,
-  options?: TestOptions
-) {
-  const ctx = await buildAndLaunchApp(options);
+interface Fixtures {
+  options: TestOptions;
+  ctx: AppContext;
+}
 
-  t.onTestFinished(async (result) => {
-    if (result.state === "fail") {
+export const test = vitestTest.extend<Fixtures>({
+  options: { version: "3.0.0" } as TestOptions,
+  ctx: async ({ options }, use) => {
+    const ctx = await buildAndLaunchApp(options);
+    await use(ctx);
+    await ctx.app.close();
+    await rm(ctx.userDataDir, { recursive: true, force: true });
+    await rm(ctx.outputDir, { recursive: true, force: true });
+  }
+});
+
+export async function testCleanup(context: TestContext) {
+  if (context.task.result?.state === "fail") {
     await mkdir("test-results", { recursive: true });
-      await ctx.page.screenshot({
+    await (context.task.context as unknown as Fixtures).ctx.page.screenshot({
       path: path.join(
         "test-results",
-          `${slugify(t.task.name)}-${process.platform}-${
+        `${slugify(context.task.name)}-${process.platform}-${
           process.arch
         }-error.png`
       )
     });
   }
-    await ctx.app.close();
-    await rm(ctx.userDataDir, { recursive: true, force: true });
-    await rm(ctx.outputDir, { recursive: true, force: true });
-  });
-
-  await cb(ctx);
 }
 
 async function buildAndLaunchApp(options?: TestOptions): Promise<AppContext> {