899cdb0e1d
* Skip unwritten tests Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Tidy jest fake timers Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Remove unnecessary sessionStorage mock Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Improve types Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Improve async assertions Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Improve error assertions Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Improve object assertions Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Remove assertion testing unclear mock This test failed when ran individually, same as after the clearAllMocks call Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Avoid awaiting non-thenables Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Pass nop function when stubbing out console, vitest won't accept it any other way Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Remove unnecessary mock which causes tests to fail after updating fetch-mock & fix typo Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Fix mistaken assertions not testing all values in array Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Fix hidden non-running tests in room.spec.ts Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Update fetch-mock-jest to @fetch-mock/jest Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Delint Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Make knip happier Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Make knip happier 2.0 Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Delint Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Switch from Jest to Vitest Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Iterate Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Delint Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Fix CI Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Remove unnecessary fake timers Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Update vite Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Revert irrelevant changes Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Fix coverage spec paths Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Fix slow test reporter Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Fix bad merge conflict resolution Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> * Fix babel config Signed-off-by: Michael Telatynski <7t3chguy@gmail.com> --------- Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
389 lines
12 KiB
TypeScript
389 lines
12 KiB
TypeScript
/*
|
|
Copyright 2022 - 2024 The Matrix.org Foundation C.I.C.
|
|
|
|
Licensed under the Apache License, Version 2.0 (the "License");
|
|
you may not use this file except in compliance with the License.
|
|
You may obtain a copy of the License at
|
|
|
|
http://www.apache.org/licenses/LICENSE-2.0
|
|
|
|
Unless required by applicable law or agreed to in writing, software
|
|
distributed under the License is distributed on an "AS IS" BASIS,
|
|
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
See the License for the specific language governing permissions and
|
|
limitations under the License.
|
|
*/
|
|
|
|
import {
|
|
anySignal,
|
|
ConnectionError,
|
|
HTTPError,
|
|
MatrixError,
|
|
MatrixSafetyError,
|
|
MatrixSafetyErrorCode,
|
|
parseErrorResponse,
|
|
retryNetworkOperation,
|
|
timeoutSignal,
|
|
} from "../../../src";
|
|
import { sleep } from "../../../src/utils";
|
|
|
|
vi.mock("../../../src/utils");
|
|
// setupTests mocks `timeoutSignal` due to hanging timers
|
|
vi.unmock("../../../src/http-api/utils");
|
|
|
|
describe("timeoutSignal", () => {
|
|
vi.useFakeTimers();
|
|
|
|
it("should fire abort signal after specified timeout", () => {
|
|
const signal = timeoutSignal(3000);
|
|
const onabort = vi.fn();
|
|
signal.onabort = onabort;
|
|
expect(signal.aborted).toBeFalsy();
|
|
expect(onabort).not.toHaveBeenCalled();
|
|
|
|
vi.advanceTimersByTime(3000);
|
|
expect(signal.aborted).toBeTruthy();
|
|
expect(onabort).toHaveBeenCalled();
|
|
});
|
|
});
|
|
|
|
describe("anySignal", () => {
|
|
vi.useFakeTimers();
|
|
|
|
it("should fire when any signal fires", () => {
|
|
const { signal } = anySignal([timeoutSignal(3000), timeoutSignal(2000)]);
|
|
|
|
const onabort = vi.fn();
|
|
signal.onabort = onabort;
|
|
expect(signal.aborted).toBeFalsy();
|
|
expect(onabort).not.toHaveBeenCalled();
|
|
|
|
vi.advanceTimersByTime(2000);
|
|
expect(signal.aborted).toBeTruthy();
|
|
expect(onabort).toHaveBeenCalled();
|
|
});
|
|
|
|
it("should cleanup when instructed", () => {
|
|
const { signal, cleanup } = anySignal([timeoutSignal(3000), timeoutSignal(2000)]);
|
|
|
|
const onabort = vi.fn();
|
|
signal.onabort = onabort;
|
|
expect(signal.aborted).toBeFalsy();
|
|
expect(onabort).not.toHaveBeenCalled();
|
|
|
|
cleanup();
|
|
vi.advanceTimersByTime(2000);
|
|
expect(signal.aborted).toBeFalsy();
|
|
expect(onabort).not.toHaveBeenCalled();
|
|
});
|
|
|
|
it("should abort immediately if passed an aborted signal", () => {
|
|
const controller = new AbortController();
|
|
controller.abort();
|
|
const { signal } = anySignal([controller.signal]);
|
|
expect(signal.aborted).toBeTruthy();
|
|
});
|
|
});
|
|
|
|
describe("parseErrorResponse", () => {
|
|
const url = "https://example.org";
|
|
|
|
let headers: Headers;
|
|
const xhrHeaderMethods = {
|
|
responseURL: url,
|
|
getResponseHeader: (name: string) => {
|
|
headers.get(name);
|
|
},
|
|
getAllResponseHeaders: () => {
|
|
let allHeaders = "";
|
|
headers.forEach((value, key) => {
|
|
allHeaders += `${key.toLowerCase()}: ${value}\r\n`;
|
|
});
|
|
return allHeaders;
|
|
},
|
|
};
|
|
|
|
beforeEach(() => {
|
|
headers = new Headers();
|
|
});
|
|
|
|
it("should resolve Matrix Errors from XHR", () => {
|
|
headers.set("Content-Type", "application/json");
|
|
expect(
|
|
parseErrorResponse(
|
|
{
|
|
...xhrHeaderMethods,
|
|
status: 500,
|
|
} as XMLHttpRequest,
|
|
'{"errcode": "TEST"}',
|
|
),
|
|
).toStrictEqual(
|
|
new MatrixError(
|
|
{
|
|
errcode: "TEST",
|
|
},
|
|
500,
|
|
url,
|
|
undefined,
|
|
expect.any(Headers),
|
|
),
|
|
);
|
|
});
|
|
|
|
it("should resolve Matrix Errors from fetch", () => {
|
|
headers.set("Content-Type", "application/json");
|
|
expect(
|
|
parseErrorResponse(
|
|
{
|
|
url,
|
|
headers,
|
|
status: 500,
|
|
} as Response,
|
|
'{"errcode": "TEST"}',
|
|
),
|
|
).toStrictEqual(
|
|
new MatrixError(
|
|
{
|
|
errcode: "TEST",
|
|
},
|
|
500,
|
|
url,
|
|
undefined,
|
|
expect.any(Headers),
|
|
),
|
|
);
|
|
});
|
|
|
|
it("should resolve Matrix Errors from XHR with urls", () => {
|
|
headers.set("Content-Type", "application/json");
|
|
expect(
|
|
parseErrorResponse(
|
|
{
|
|
...xhrHeaderMethods,
|
|
responseURL: "https://example.com",
|
|
status: 500,
|
|
} as XMLHttpRequest,
|
|
'{"errcode": "TEST"}',
|
|
),
|
|
).toStrictEqual(
|
|
new MatrixError(
|
|
{
|
|
errcode: "TEST",
|
|
},
|
|
500,
|
|
"https://example.com",
|
|
undefined,
|
|
expect.any(Headers),
|
|
),
|
|
);
|
|
});
|
|
|
|
it("should resolve Matrix Errors from fetch with urls", () => {
|
|
headers.set("Content-Type", "application/json");
|
|
expect(
|
|
parseErrorResponse(
|
|
{
|
|
url: "https://example.com",
|
|
headers,
|
|
status: 500,
|
|
} as Response,
|
|
'{"errcode": "TEST"}',
|
|
),
|
|
).toStrictEqual(
|
|
new MatrixError(
|
|
{
|
|
errcode: "TEST",
|
|
},
|
|
500,
|
|
"https://example.com",
|
|
undefined,
|
|
expect.any(Headers),
|
|
),
|
|
);
|
|
});
|
|
it.each([
|
|
{
|
|
errcode: MatrixSafetyErrorCode.name,
|
|
error: "Spammy",
|
|
},
|
|
{
|
|
errcode: MatrixSafetyErrorCode.name,
|
|
error: "Spammy",
|
|
expiry: 5000,
|
|
},
|
|
{
|
|
errcode: MatrixSafetyErrorCode.name,
|
|
error: "Spammy",
|
|
harms: ["m.spam", "org.example.additional-harm"],
|
|
expiry: 5000,
|
|
},
|
|
])("should resolve MatrixSafetyErrors from fetch", (errContent) => {
|
|
headers.set("Content-Type", "application/json");
|
|
const value = parseErrorResponse(
|
|
{
|
|
headers,
|
|
status: 400,
|
|
} as Response,
|
|
JSON.stringify(errContent),
|
|
) as MatrixSafetyError;
|
|
expect(value).toBeInstanceOf(MatrixSafetyError);
|
|
expect(value.harms.size).toEqual(errContent.harms?.length ?? 0);
|
|
expect(value.expiry?.getTime()).toEqual(errContent.expiry);
|
|
});
|
|
|
|
describe("with HTTP headers", () => {
|
|
function addHeaders(headers: Headers) {
|
|
headers.set("Age", "0");
|
|
headers.set("Date", "Thu, 01 Jan 1970 00:00:00 GMT"); // value contains colons
|
|
headers.set("x-empty", "");
|
|
headers.set("x-multi", "1");
|
|
headers.append("x-multi", "2");
|
|
}
|
|
|
|
function compareHeaders(expectedHeaders: Headers, otherHeaders: Headers | undefined) {
|
|
expect(new Map(otherHeaders as any)).toEqual(new Map(expectedHeaders as any));
|
|
}
|
|
|
|
it("should resolve HTTP Errors from XHR with headers", () => {
|
|
headers.set("Content-Type", "text/plain");
|
|
addHeaders(headers);
|
|
const err = parseErrorResponse({
|
|
...xhrHeaderMethods,
|
|
status: 500,
|
|
} as XMLHttpRequest) as HTTPError;
|
|
compareHeaders(headers, err.httpHeaders);
|
|
});
|
|
|
|
it("should resolve HTTP Errors from fetch with headers", () => {
|
|
headers.set("Content-Type", "text/plain");
|
|
addHeaders(headers);
|
|
const err = parseErrorResponse({
|
|
headers,
|
|
status: 500,
|
|
} as Response) as HTTPError;
|
|
compareHeaders(headers, err.httpHeaders);
|
|
});
|
|
|
|
it("should resolve Matrix Errors from XHR with headers", () => {
|
|
headers.set("Content-Type", "application/json");
|
|
addHeaders(headers);
|
|
const err = parseErrorResponse(
|
|
{
|
|
...xhrHeaderMethods,
|
|
status: 500,
|
|
} as XMLHttpRequest,
|
|
'{"errcode": "TEST"}',
|
|
) as MatrixError;
|
|
compareHeaders(headers, err.httpHeaders);
|
|
});
|
|
|
|
it("should resolve Matrix Errors from fetch with headers", () => {
|
|
headers.set("Content-Type", "application/json");
|
|
addHeaders(headers);
|
|
const err = parseErrorResponse(
|
|
{
|
|
headers,
|
|
status: 500,
|
|
} as Response,
|
|
'{"errcode": "TEST"}',
|
|
) as MatrixError;
|
|
compareHeaders(headers, err.httpHeaders);
|
|
});
|
|
});
|
|
|
|
it("should set a sensible default error message on MatrixError", () => {
|
|
let err = new MatrixError();
|
|
expect(err.message).toEqual("MatrixError: Unknown message");
|
|
err = new MatrixError({
|
|
error: "Oh no",
|
|
});
|
|
expect(err.message).toEqual("MatrixError: Oh no");
|
|
});
|
|
|
|
it("should handle no type gracefully", () => {
|
|
// No Content-Type header
|
|
expect(
|
|
parseErrorResponse(
|
|
{
|
|
headers,
|
|
status: 500,
|
|
} as Response,
|
|
'{"errcode": "TEST"}',
|
|
),
|
|
).toStrictEqual(new HTTPError("Server returned 500 error", 500, expect.any(Headers)));
|
|
});
|
|
|
|
it("should handle empty type gracefully", () => {
|
|
headers.set("Content-Type", " ");
|
|
expect(
|
|
parseErrorResponse(
|
|
{
|
|
headers,
|
|
status: 500,
|
|
} as Response,
|
|
'{"errcode": "TEST"}',
|
|
),
|
|
).toStrictEqual(new Error("Error parsing Content-Type '': TypeError: argument string is required"));
|
|
});
|
|
|
|
it("should handle invalid type gracefully", () => {
|
|
headers.set("Content-Type", "unknown");
|
|
expect(
|
|
parseErrorResponse(
|
|
{
|
|
headers,
|
|
status: 500,
|
|
} as Response,
|
|
'{"errcode": "TEST"}',
|
|
),
|
|
).toStrictEqual(new Error("Error parsing Content-Type 'unknown': TypeError: invalid media type"));
|
|
});
|
|
|
|
it("should handle plaintext errors", () => {
|
|
headers.set("Content-Type", "text/plain");
|
|
expect(
|
|
parseErrorResponse(
|
|
{
|
|
headers,
|
|
status: 418,
|
|
} as Response,
|
|
"I'm a teapot",
|
|
),
|
|
).toStrictEqual(new HTTPError("Server returned 418 error: I'm a teapot", 418, expect.any(Headers)));
|
|
});
|
|
});
|
|
|
|
describe("retryNetworkOperation", () => {
|
|
it("should retry given number of times with exponential sleeps", async () => {
|
|
const err = new ConnectionError("test");
|
|
const fn = vi.fn().mockRejectedValue(err);
|
|
vi.mocked(sleep).mockResolvedValue(undefined);
|
|
await expect(retryNetworkOperation(4, fn)).rejects.toThrow(err);
|
|
expect(fn).toHaveBeenCalledTimes(4);
|
|
expect(vi.mocked(sleep)).toHaveBeenCalledTimes(3);
|
|
expect(vi.mocked(sleep).mock.calls[0][0]).toBe(2000);
|
|
expect(vi.mocked(sleep).mock.calls[1][0]).toBe(4000);
|
|
expect(vi.mocked(sleep).mock.calls[2][0]).toBe(8000);
|
|
});
|
|
|
|
it("should bail out on errors other than ConnectionError", async () => {
|
|
const err = new TypeError("invalid JSON");
|
|
const fn = vi.fn().mockRejectedValue(err);
|
|
vi.mocked(sleep).mockResolvedValue(undefined);
|
|
await expect(retryNetworkOperation(3, fn)).rejects.toThrow(err);
|
|
expect(fn).toHaveBeenCalledTimes(1);
|
|
});
|
|
|
|
it("should return newest ConnectionError when giving up", async () => {
|
|
const err1 = new ConnectionError("test1");
|
|
const err2 = new ConnectionError("test2");
|
|
const err3 = new ConnectionError("test3");
|
|
const errors = [err1, err2, err3];
|
|
const fn = vi.fn().mockImplementation(() => {
|
|
throw errors.shift();
|
|
});
|
|
vi.mocked(sleep).mockResolvedValue(undefined);
|
|
await expect(retryNetworkOperation(3, fn)).rejects.toThrow(err3);
|
|
});
|
|
});
|