test: fix order of assertion arguments in test-event-emitter-num-args · nodejs/node@4973f21 · GitHub

nodejs / node Public

• Notifications You must be signed in to change notification settings
• Fork 36.6k
• Star 120k

• Code
• Issues 638
• Pull requests 757
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit 4973f21

Browse filesBrowse the repository at this point in the historyBrowse files

luisgallon

authored and

targos

committed

test: fix order of assertion arguments in test-event-emitter-num-args

PR-URL: #28368 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>

1 parent e3dd4d5 commit 4973f21
Copy full SHA for 4973f21

1 file changed

+1-1Lines changed: 1 addition & 1 deletion

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test/parallel
  • test-event-emitter-num-args.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/parallel/test-event-emitter-num-args.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-event-emitter-num-args.js

+1-1Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -50,5 +50,5 @@ e.emit('numArgs', null, null, null, null, null);
50	50	e.emit('foo', null, null, null, null);
51	51
52	52	process.on('exit', function() {
53		- assert.deepStrictEqual([0, 1, 2, 3, 4, 5, 4, 4], num_args_emitted);
	53	+ assert.deepStrictEqual(num_args_emitted, [0, 1, 2, 3, 4, 5, 4, 4]);
54	54	});

0 commit comments

Comments

0 (0)