Skip to content

Commit 16ea131

Browse files
committed
fixup! tools: add lint rule to ensure assertions are reached
1 parent bd14a06 commit 16ea131

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

test/pummel/test-net-many-clients.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ function runClient(port, callback) {
7676
} else {
7777
callback();
7878
}
79-
}));
79+
}, connections_per_client));
8080
}
8181

8282
server.listen(0, function() {

test/pummel/test-net-pingpong.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,14 +47,14 @@ const pingPongTest = common.mustCall((host, on_complete) => {
4747
socket.setNoDelay();
4848
socket.timeout = 0;
4949

50-
socket.on('data', common.mustCall((data) => {
50+
socket.on('data', common.mustCallAtLeast((data) => {
5151
console.log(`server got: ${JSON.stringify(data)}`);
5252
assert.strictEqual(socket.readyState, 'open');
5353
assert.strictEqual(count <= N, true);
5454
if (/PING/.test(data)) {
5555
socket.write('PONG');
5656
}
57-
}));
57+
}, N + 1));
5858

5959
socket.on('end', common.mustCall(() => {
6060
assert.strictEqual(socket.readyState, 'writeOnly');

0 commit comments

Comments
 (0)