Browse Source

test: use String.prototype.repeat() for clarity

There are a few places where tests repeatedly concatenate strings to
themselves in order to make them very long. Using `.repeat()` makes the
code clearer.

For example, before:

    for (var i = 0; i < 8; ++i) lots_of_headers += lots_of_headers;

After:

    lots_of_headers = lots_of_headers.repeat(256);

Using `.repeat()` makes it clear that the string will be repeated 256
times rather than 8 times. ("What?! That first one doesn't repeat 256
times! It only repeats 8... Oh, wait. Yes, I see your point now.")

PR-URL: https://github.com/nodejs/node/pull/5311
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michaël Zasso <mic.besace@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
v5.x
Rich Trott 9 years ago
committed by Rod Vagg
parent
commit
0e85530d8c
  1. 2
      test/parallel/test-crypto-from-binary.js
  2. 7
      test/parallel/test-http-dns-error.js
  3. 2
      test/parallel/test-http-parser.js
  4. 7
      test/parallel/test-net-dns-error.js

2
test/parallel/test-crypto-from-binary.js

@ -19,7 +19,7 @@ var ucs2_control = 'a\u0000';
// grow the strings to proper length // grow the strings to proper length
while (ucs2_control.length <= EXTERN_APEX) { while (ucs2_control.length <= EXTERN_APEX) {
ucs2_control += ucs2_control; ucs2_control = ucs2_control.repeat(2);
} }

7
test/parallel/test-http-dns-error.js

@ -10,12 +10,7 @@ if (common.hasCrypto) {
console.log('1..0 # Skipped: missing crypto'); console.log('1..0 # Skipped: missing crypto');
} }
var host = '********'; var host = '*'.repeat(256);
host += host;
host += host;
host += host;
host += host;
host += host;
function do_not_call() { function do_not_call() {
throw new Error('This function should not have been called.'); throw new Error('This function should not have been called.');

2
test/parallel/test-http-parser.js

@ -245,7 +245,7 @@ function expectBody(expected) {
(function() { (function() {
// 256 X-Filler headers // 256 X-Filler headers
var lots_of_headers = 'X-Filler: 42' + CRLF; var lots_of_headers = 'X-Filler: 42' + CRLF;
for (var i = 0; i < 8; ++i) lots_of_headers += lots_of_headers; lots_of_headers = lots_of_headers.repeat(256);
var request = Buffer( var request = Buffer(
'GET /foo/bar/baz?quux=42#1337 HTTP/1.0' + CRLF + 'GET /foo/bar/baz?quux=42#1337 HTTP/1.0' + CRLF +

7
test/parallel/test-net-dns-error.js

@ -7,12 +7,7 @@ var net = require('net');
var expected_bad_connections = 1; var expected_bad_connections = 1;
var actual_bad_connections = 0; var actual_bad_connections = 0;
var host = '********'; var host = '*'.repeat(256);
host += host;
host += host;
host += host;
host += host;
host += host;
function do_not_call() { function do_not_call() {
throw new Error('This function should not have been called.'); throw new Error('This function should not have been called.');

Loading…
Cancel
Save