mirrors/signal-desktop
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 2 Wiki Activity Actions

Fix CI test flake for AttachmentDownloadManager

Browse source
This commit is contained in:
automated-signal 2024-08-07 08:36:58 -05:00 • committed by GitHub
parent f3f721f1b5
commit 16876d4025
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 6 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

8
ts/test-electron/services/AttachmentDownloadManager_test.ts
View file

@ -157,18 +157,22 @@ describe('AttachmentDownloadManager/JobManager', () => {
);
}
async function flushSQLReads() {
await DataWriter.getNextAttachmentDownloadJobs({ limit: 10 });
}
async function advanceTime(ms: number) {
// When advancing the timers, we want to make sure any DB operations are completed
// first. In cases like maybeStartJobs where we prevent re-entrancy, without this,
// prior (unfinished) invocations can prevent subsequent calls after the clock is
// ticked forward and make tests unreliable
await DataReader.getAllItems();
await flushSQLReads();
const now = Date.now();
while (Date.now() < now + ms) {
// eslint-disable-next-line no-await-in-loop
await clock.tickAsync(downloadManager?.tickInterval ?? 1000);
// eslint-disable-next-line no-await-in-loop
await DataReader.getAllItems();
await flushSQLReads();
}
}