test: simplify forEach() usage

Trott · targos · commit d69b9b753a90 · 2019-12-02T11:11:26.000+01:00
Use an array literal instead of using split() on a very short string. PR-URL: #30712 Reviewed-By: David Carlier <devnexen@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
diff --git a/test/sequential/test-http-max-http-headers.js b/test/sequential/test-http-max-http-headers.js
@@ -24,7 +24,7 @@ function once(cb) {
 }
 
 function finished(client, callback) {
-  'abort error end'.split(' ').forEach((e) => {
+  ['abort', 'error', 'end'].forEach((e) => {
     client.on(e, once(() => setImmediate(callback)));
   });
 }

Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,7 @@ function once(cb) {`
`24`	`24`	`}`
`25`	`25`
`26`	`26`	`function finished(client, callback) {`
`27`		`- 'abort error end'.split(' ').forEach((e) => {`
	`27`	`+ ['abort', 'error', 'end'].forEach((e) => {`
`28`	`28`	`client.on(e, once(() => setImmediate(callback)));`
`29`	`29`	`});`
`30`	`30`	`}`