1
0
Fork 0
mirror of https://github.com/immich-app/immich.git synced 2025-01-10 13:56:47 +01:00
immich/web/src/lib/utils/executor-queue.spec.ts
Ben McCann 907a95a746
chore(web): cleanup promise handling (#7382)
* no-misused-promises

* no-floating-promises

* format

* revert for now

* remove load function

* require-await

* revert a few no-floating-promises changes that would cause no-misused-promises failures

* format

* fix a few more

* fix most remaining errors

* executor-queue

* executor-queue.spec

* remove duplicate comments by grouping rules

* upgrade sveltekit and enforce rules

* oops. move await

* try this

* just ignore for now since it's only a test

* run in parallel

* Update web/src/routes/admin/jobs-status/+page.svelte

Co-authored-by: Michel Heusschen <59014050+michelheusschen@users.noreply.github.com>

* remove Promise.resolve call

* rename function

* remove unnecessary warning silencing

* make handleError sync

* fix new errors from recently merged PR to main

* extract method

* use handlePromiseError

---------

Co-authored-by: Michel Heusschen <59014050+michelheusschen@users.noreply.github.com>
Co-authored-by: Alex Tran <alex.tran1502@gmail.com>
2024-02-27 10:37:37 -06:00

56 lines
1.9 KiB
TypeScript

import { ExecutorQueue } from '$lib/utils/executor-queue';
describe('Executor Queue test', function () {
it('should run all promises', async function () {
const eq = new ExecutorQueue({ concurrency: 1 });
const n1 = await eq.addTask(() => Promise.resolve(10));
expect(n1).toBe(10);
const n2 = await eq.addTask(() => Promise.resolve(11));
expect(n2).toBe(11);
const n3 = await eq.addTask(() => Promise.resolve(12));
expect(n3).toBe(12);
});
it('should respect concurrency parameter', function () {
vi.useFakeTimers();
const eq = new ExecutorQueue({ concurrency: 3 });
const finished = vi.fn();
const started = vi.fn();
const timeoutPromiseBuilder = (delay: number, id: string) =>
new Promise((resolve) => {
started();
setTimeout(() => {
finished();
resolve(id);
}, delay);
});
// The first 3 should be finished within 200ms (concurrency 3)
// eslint-disable-next-line @typescript-eslint/no-floating-promises
eq.addTask(() => timeoutPromiseBuilder(100, 'T1'));
// eslint-disable-next-line @typescript-eslint/no-floating-promises
eq.addTask(() => timeoutPromiseBuilder(200, 'T2'));
// eslint-disable-next-line @typescript-eslint/no-floating-promises
eq.addTask(() => timeoutPromiseBuilder(150, 'T3'));
// The last task will be executed after 200ms and will finish at 400ms
// eslint-disable-next-line @typescript-eslint/no-floating-promises
eq.addTask(() => timeoutPromiseBuilder(200, 'T4'));
expect(finished).not.toBeCalled();
expect(started).toHaveBeenCalledTimes(3);
vi.advanceTimersByTime(100);
expect(finished).toHaveBeenCalledTimes(1);
vi.advanceTimersByTime(250);
expect(finished).toHaveBeenCalledTimes(3);
// expect(started).toHaveBeenCalledTimes(4)
//TODO : fix The test ...
vi.runAllTimers();
vi.useRealTimers();
});
});