diff options
| author | Jackson Tian <shyvo1987@gmail.com> | 2017-01-30 16:57:49 +0800 |
|---|---|---|
| committer | Italo A. Casas <me@italoacasas.com> | 2017-02-01 20:06:04 -0800 |
| commit | 58dc229d9aac29132cf038ed07b3b7e13b671dca (patch) | |
| tree | 2b0077284bf4e1f5fa348f799dd780af37a1b93a /test/pummel/test-tls-server-large-request.js | |
| parent | 773cdc31ef3e17327d1eb0a3b92828e25457f015 (diff) | |
| download | android-node-v8-58dc229d9aac29132cf038ed07b3b7e13b671dca.tar.gz android-node-v8-58dc229d9aac29132cf038ed07b3b7e13b671dca.tar.bz2 android-node-v8-58dc229d9aac29132cf038ed07b3b7e13b671dca.zip | |
test: use repeat() instead of new Array().join()
The usage of `new Array(length + 1).join(str)` is strange.
Change to `str.repeat(length)` is more clearly.
PR-URL: https://github.com/nodejs/node/pull/11071
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Italo A. Casas <me@italoacasas.com>
Diffstat (limited to 'test/pummel/test-tls-server-large-request.js')
| -rw-r--r-- | test/pummel/test-tls-server-large-request.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/pummel/test-tls-server-large-request.js b/test/pummel/test-tls-server-large-request.js index 41393120fc..36fb566fab 100644 --- a/test/pummel/test-tls-server-large-request.js +++ b/test/pummel/test-tls-server-large-request.js @@ -12,7 +12,7 @@ const fs = require('fs'); const stream = require('stream'); const util = require('util'); -const request = Buffer.from(new Array(1024 * 256).join('ABCD')); // 1mb +const request = Buffer.from('ABCD'.repeat(1024 * 256 - 1)); // 1mb const options = { key: fs.readFileSync(common.fixturesDir + '/keys/agent1-key.pem'), |