Skip to content

Commit 2afd54d

Browse files
committed
test
1 parent 1c48a23 commit 2afd54d

File tree

4 files changed

+14
-15
lines changed

4 files changed

+14
-15
lines changed

lib/docker-event-stream.ts

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
import * as es from 'event-stream';
21
import JSONStream from 'JSONStream';
32
import type Docker from 'dockerode';
43
import { Stream } from 'node:stream';
@@ -64,8 +63,8 @@ export const parseEventStream = async (docker: Docker) => {
6463
layerMtimes[image.Id] = 0;
6564
}
6665

67-
return es.pipeline(
68-
JSONStream.parse(undefined) as any as es.MapStream,
66+
return [
67+
JSONStream.parse(undefined),
6968
new Stream.Transform({
7069
objectMode: true,
7170
transform(evt: DockerEvent, _encoding, cb) {
@@ -86,17 +85,15 @@ export const parseEventStream = async (docker: Docker) => {
8685
}
8786
},
8887
}),
89-
);
90-
};
91-
92-
const noop = () => {
93-
// noop
88+
] as const;
9489
};
9590

9691
export async function dockerMtimeStream(docker: Docker) {
9792
const [stream, streamParser] = await Promise.all([
9893
docker.getEvents(),
9994
parseEventStream(docker),
10095
]);
101-
return Stream.pipeline(stream, streamParser, noop);
96+
return Stream.pipeline(stream, ...streamParser, () => {
97+
// noop
98+
});
10299
}

package.json

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,12 +15,10 @@
1515
"description": "Automatically cleanup unused images based on various cache replacement algorithms",
1616
"dependencies": {
1717
"@types/dockerode": "^3.3.44",
18-
"@types/event-stream": "^4.0.5",
1918
"@types/JSONStream": "npm:@types/jsonstream@^0.8.33",
2019
"@types/node": "^16.18.126",
2120
"docker-progress": "^5.3.1",
2221
"dockerode": "^4.0.9",
23-
"event-stream": "^4.0.1",
2422
"eventemitter3": "^5.0.1",
2523
"JSONStream": "^1.3.5"
2624
},

test/docker-event-stream.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,13 +13,15 @@ describe('parseEventStream', function () {
1313

1414
it('should return updated mtimes', async () => {
1515
const docker = getDocker({});
16-
const streamParser = await parseEventStream(docker);
16+
const streamParsers = await parseEventStream(docker);
1717
const data = await new Promise<LayerMtimes>(function (resolve, reject) {
1818
let mtimes: LayerMtimes;
1919

2020
return fs
2121
.createReadStream(__dirname + '/fixtures/docker-events.json')
22-
.pipe(streamParser)
22+
.pipe(streamParsers[0])
23+
.on('error', reject)
24+
.pipe(streamParsers[1])
2325
.on('error', reject)
2426
.pipe(
2527
new Stream.Transform({

test/docker-image-tree.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,12 +10,14 @@ import { getDocker } from '../build/docker';
1010

1111
const getLayerMtimes = async () => {
1212
const docker = getDocker({});
13-
const streamParser = await parseEventStream(docker);
13+
const streamParsers = await parseEventStream(docker);
1414
return await new Promise<LayerMtimes>(function (resolve, reject) {
1515
let mtimes: LayerMtimes;
1616
return fs
1717
.createReadStream(__dirname + '/fixtures/docker-events.json')
18-
.pipe(streamParser)
18+
.pipe(streamParsers[0])
19+
.on('error', reject)
20+
.pipe(streamParsers[1])
1921
.on('error', reject)
2022
.pipe(
2123
new Stream.Transform({

0 commit comments

Comments
 (0)