Skip to content

Commit

Permalink
Remove flaky test
Browse files Browse the repository at this point in the history
Fixes #4331
  • Loading branch information
t3chguy authored Jul 29, 2024
1 parent d32f398 commit 01d2774
Showing 1 changed file with 0 additions and 16 deletions.
16 changes: 0 additions & 16 deletions spec/unit/room.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1394,22 +1394,6 @@ describe("Room", function () {
expect(name).toEqual(userB);
});
});

it("recalculates in acceptable time without heroes", function () {
for (let i = 0; i < 5000; i++) {
addMember(`@person${i}:bar`, KnownMembership.Join, { name: `Person ${i % 20} ${i % 10} ${i % 3}` });
}

// This isn't a real performance test and has plenty of headroom because github
// runners don't offer any kind of speed consistency guarantee, but this should at
// least assert that the perf doesn't suddenly become n^2.
const start = performance.now();
for (let i = 0; i < 50; i++) {
room.recalculate();
}
const duration = performance.now() - start;
expect(duration).toBeLessThan(200);
});
});

describe("receipts", function () {
Expand Down

0 comments on commit 01d2774

Please sign in to comment.