Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 9244d6d

Browse filesBrowse files
daeyeonruyadorno
authored andcommitted
test: fix test-cluster-concurrent-disconnect
The error code, `ECONNRESET`, is observed on linux. This commit adds it as an expected error code. Signed-off-by: Daeyeon Jeong daeyeon.dev@gmail.com PR-URL: #43961 Refs: https://ci.nodejs.org/job/node-test-commit-linux-containered/nodes=ubuntu1804_sharedlibs_withoutintl_x64/32901/testReport/junit/(root)/test/parallel_test_cluster_concurrent_disconnect/ Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com> Reviewed-By: Feng Yu <F3n67u@outlook.com> Reviewed-By: Darshan Sen <raisinten@gmail.com>
1 parent 1b32830 commit 9244d6d
Copy full SHA for 9244d6d

File tree

Expand file treeCollapse file tree

1 file changed

+1
-1
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

1 file changed

+1
-1
lines changed
Open diff view settings
Collapse file

‎test/parallel/test-cluster-concurrent-disconnect.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-cluster-concurrent-disconnect.js
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ if (cluster.isPrimary) {
2929
if (common.isOSX) {
3030
assert(['EPIPE', 'ENOTCONN'].includes(err.code), err);
3131
} else {
32-
assert.strictEqual(err.code, 'EPIPE');
32+
assert(['EPIPE', 'ECONNRESET'].includes(err.code), err);
3333
}
3434
});
3535

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.