Skip to content

Commit 0ffc01c

Browse files
committed
Use SyncPromise inside Buffer tests
1 parent 93dc899 commit 0ffc01c

File tree

1 file changed

+9
-8
lines changed

1 file changed

+9
-8
lines changed

packages/utils/test/promisebuffer.test.ts

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
import { PromiseBuffer } from '../src/promisebuffer';
2+
import { SyncPromise } from '../src/syncpromise';
23

34
// tslint:disable:no-floating-promises
45

@@ -10,23 +11,23 @@ describe('PromiseBuffer', () => {
1011
describe('add()', () => {
1112
test('no limit', () => {
1213
const q = new PromiseBuffer<void>();
13-
const p = new Promise<void>(resolve => setTimeout(resolve, 1));
14+
const p = new SyncPromise<void>(resolve => setTimeout(resolve, 1));
1415
q.add(p);
1516
expect(q.length()).toBe(1);
1617
});
1718
test('with limit', () => {
1819
const q = new PromiseBuffer<void>(1);
19-
const p = new Promise<void>(resolve => setTimeout(resolve, 1));
20+
const p = new SyncPromise<void>(resolve => setTimeout(resolve, 1));
2021
expect(q.add(p)).toEqual(p);
21-
expect(q.add(new Promise<void>(resolve => setTimeout(resolve, 1)))).rejects.toThrowError();
22+
expect(q.add(new SyncPromise<void>(resolve => setTimeout(resolve, 1)))).rejects.toThrowError();
2223
expect(q.length()).toBe(1);
2324
});
2425
});
2526

2627
test('resolved promises should not show up in buffer length', async () => {
2728
expect.assertions(2);
2829
const q = new PromiseBuffer<void>();
29-
const p = new Promise<void>(resolve => setTimeout(resolve, 1));
30+
const p = new SyncPromise<void>(resolve => setTimeout(resolve, 1));
3031
q.add(p).then(() => {
3132
expect(q.length()).toBe(0);
3233
});
@@ -37,7 +38,7 @@ describe('PromiseBuffer', () => {
3738
test('receive promise result outside and from buffer', async () => {
3839
expect.assertions(4);
3940
const q = new PromiseBuffer<string>();
40-
const p = new Promise<string>(resolve =>
41+
const p = new SyncPromise<string>(resolve =>
4142
setTimeout(() => {
4243
resolve('test');
4344
}, 1),
@@ -57,7 +58,7 @@ describe('PromiseBuffer', () => {
5758
expect.assertions(3);
5859
const q = new PromiseBuffer<void>();
5960
for (let i = 0; i < 5; i++) {
60-
const p = new Promise<void>(resolve => setTimeout(resolve, 1));
61+
const p = new SyncPromise<void>(resolve => setTimeout(resolve, 1));
6162
q.add(p);
6263
}
6364
expect(q.length()).toBe(5);
@@ -72,7 +73,7 @@ describe('PromiseBuffer', () => {
7273
expect.assertions(2);
7374
const q = new PromiseBuffer<void>();
7475
for (let i = 0; i < 5; i++) {
75-
const p = new Promise<void>(resolve => setTimeout(resolve, 100));
76+
const p = new SyncPromise<void>(resolve => setTimeout(resolve, 100));
7677
q.add(p);
7778
}
7879
expect(q.length()).toBe(5);
@@ -96,7 +97,7 @@ describe('PromiseBuffer', () => {
9697
test('rejecting', async () => {
9798
expect.assertions(1);
9899
const q = new PromiseBuffer<void>();
99-
const p = new Promise<void>((_, reject) => setTimeout(reject, 1));
100+
const p = new SyncPromise<void>((_, reject) => setTimeout(reject, 1));
100101
jest.runAllTimers();
101102
return q.add(p).then(null, () => {
102103
expect(true).toBe(true);

0 commit comments

Comments
 (0)