mirror of
https://github.com/matrix-org/mjolnir.git
synced 2024-10-01 01:36:06 -04:00
ac2e736e96
So the test before sometimes sent the report *before* the protection (that is used to check whether we have received the report) was registered. This of course meant that we didn't ever receive the report from the perspectivee of the test. This PR should now mean we always send the report after registering the protection.
48 lines
2.4 KiB
TypeScript
48 lines
2.4 KiB
TypeScript
import { Mjolnir } from "../../src/Mjolnir";
|
|
import { IProtection } from "../../src/protections/IProtection";
|
|
import { newTestUser } from "./clientHelper";
|
|
|
|
describe("Test: Report polling", function() {
|
|
let client;
|
|
this.beforeEach(async function () {
|
|
client = await newTestUser({ name: { contains: "protection-settings" }});
|
|
})
|
|
it("Mjolnir correctly retrieves a report from synapse", async function() {
|
|
this.timeout(40000);
|
|
|
|
let protectedRoomId = await this.mjolnir.client.createRoom({ invite: [await client.getUserId()] });
|
|
await client.joinRoom(protectedRoomId);
|
|
await this.mjolnir.addProtectedRoom(protectedRoomId);
|
|
|
|
const eventId = await client.sendMessage(protectedRoomId, {msgtype: "m.text", body: "uwNd3q"});
|
|
await new Promise(async resolve => {
|
|
await this.mjolnir.registerProtection(new class implements IProtection {
|
|
name = "jYvufI";
|
|
description = "A test protection";
|
|
settings = { };
|
|
handleEvent = async (mjolnir: Mjolnir, roomId: string, event: any) => { };
|
|
handleReport = (mjolnir: Mjolnir, roomId: string, reporterId: string, event: any, reason?: string) => {
|
|
if (reason === "x5h1Je") {
|
|
resolve(null);
|
|
}
|
|
};
|
|
});
|
|
await this.mjolnir.enableProtection("jYvufI");
|
|
await client.doRequest(
|
|
"POST",
|
|
`/_matrix/client/r0/rooms/${encodeURIComponent(protectedRoomId)}/report/${encodeURIComponent(eventId)}`, "", {
|
|
reason: "x5h1Je"
|
|
}
|
|
);
|
|
});
|
|
// So I kid you not, it seems like we can quit before the webserver for reports sends a respond to the client (via L#26)
|
|
// because the promise above gets resolved before we finish awaiting the report sending request on L#31,
|
|
// then mocha's cleanup code runs (and shuts down the webserver) before the webserver can respond.
|
|
// Wait a minute 😲😲🤯 it's not even supposed to be using the webserver if this is testing report polling.
|
|
// Ok, well apparently that needs a big refactor to change, but if you change the config before running this test,
|
|
// then you can ensure that report polling works. https://github.com/matrix-org/mjolnir/issues/326.
|
|
await new Promise(resolve => setTimeout(resolve, 1000));
|
|
});
|
|
});
|
|
|