Tools/Notes
1
0
Fork 0
mirror of https://github.com/TriliumNext/Notes.git synced 2025-10-29 11:44:21 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(test/server): supertest-based tests no longer working

Browse Source
This commit is contained in:
Elian Doran 2025-04-28 16:41:41 +03:00
parent 4dafa56671
commit 59663ed696
No known key found for this signature in database
2 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
apps/server/src/routes/login.spec.ts
View File

@ -1,8 +1,9 @@
import { beforeAll, describe, expect, it } from "vitest";
import supertest from "supertest";
import { initializeTranslations } from "../services/i18n.js";
import type { Application, Request, Response, NextFunction } from "express";
import type { Application } from "express";
import dayjs from "dayjs";
import buildApp from "../app.js";
let app: Application;
@ -10,7 +11,7 @@ describe("Login Route test", () => {
beforeAll(async () => {
initializeTranslations();
app = (await import("../app.js")).default;
app = await buildApp();
});
it("should return the login page, when using a GET request", async () => {
@ -60,7 +61,7 @@ describe("Login Route test", () => {
expect(actualExpiresDate).to.not.eql("Invalid Date");
// ignore the seconds in the comparison, just to avoid flakiness in tests,
// ignore the seconds in the comparison, just to avoid flakiness in tests,
// if for some reason execution is slow between calculation of expected and actual
expect(actualExpiresDate.slice(0,23)).toBe(expectedExpiresDate.slice(0,23))
@ -90,4 +91,4 @@ describe("Login Route test", () => {
// TODO: actually identify what is causing this and fix the flakiness
});
});

3
apps/server/src/share/routes.spec.ts
View File

@ -3,6 +3,7 @@ import supertest from "supertest";
import { initializeTranslations } from "../services/i18n.js";
import type { Application, Request, Response, NextFunction } from "express";
import { safeExtractMessageAndStackFromError } from "../services/utils.js";
import buildApp from "../app.js";
let app: Application;
@ -11,7 +12,7 @@ describe("Share API test", () => {
beforeAll(async () => {
initializeTranslations();
app = (await import("../app.js")).default;
app = await buildApp();
app.use((err: unknown, req: Request, res: Response, next: NextFunction) => {
const [ errMessage ] = safeExtractMessageAndStackFromError(err);
if (errMessage.includes("Cannot set headers after they are sent to the client")) {