test: enable max-len for test-repl

Instead of disabling max-len (ESLint's line-length rule) for the entire
file, reformat to avoid exceeding length restrictions in most cases and
disable the rule only on specific lines where exceeding the length
restriction may be a better choice than conforming to it.
PR-URL: #11559
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>

Author: Trott

test/parallel/test-repl.js

@@ -1,4 +1,3 @@
-/* eslint-disable max-len */
 'use strict';
 
 constcommon=require('../common');
@@ -168,18 +167,32 @@ function error_test() {
 {client: client_unix,send: 'new RegExp("foo", "wrong modifier");',
 expect: /\bSyntaxError:InvalidflagssuppliedtoRegExpconstructor/},
 // strict mode syntax errors should be caught (GH-5178)
-{client: client_unix,send: '(function() { "use strict"; return 0755; })()',
+{client: client_unix,
+send: '(function() { "use strict"; return 0755; })()',
 expect: /\bSyntaxError:Octalliteralsarenotallowedinstrictmode/},
-{client: client_unix,send: '(function(a, a, b) { "use strict"; return a + b + c; })()',
-expect: /\bSyntaxError:Duplicateparameternamenotallowedinthiscontext/},
-{client: client_unix,send: '(function() { "use strict"; with (this) {} })()',
-expect: /\bSyntaxError:Strictmodecodemaynotincludeawithstatement/},
-{client: client_unix,send: '(function() { "use strict"; var x; delete x; })()',
-expect: /\bSyntaxError:Deleteofanunqualifiedidentifierinstrictmode/},
-{client: client_unix,send: '(function() { "use strict"; eval = 17; })()',
+{
+client: client_unix,
+send: '(function(a, a, b) { "use strict"; return a + b + c; })()',
+expect: /\bSyntaxError:Duplicateparameternamenotallowedinthiscontext/// eslint-disable-line max-len
+},
+{
+client: client_unix,
+send: '(function() { "use strict"; with (this) {} })()',
+expect: /\bSyntaxError:Strictmodecodemaynotincludeawithstatement/
+},
+{
+client: client_unix,
+send: '(function() { "use strict"; var x; delete x; })()',
+expect: /\bSyntaxError:Deleteofanunqualifiedidentifierinstrictmode/// eslint-disable-line max-len
+},
+{client: client_unix,
+send: '(function() { "use strict"; eval = 17; })()',
 expect: /\bSyntaxError:Unexpectedevalorargumentsinstrictmode/},
-{client: client_unix,send: '(function() { "use strict"; if (true) function f() { } })()',
-expect: /\bSyntaxError:Instrictmodecode,functionscanonlybedeclaredattoplevelorinsideablock./},
+{
+client: client_unix,
+send: '(function() { "use strict"; if (true) function f() { } })()',
+expect: /\bSyntaxError:Instrictmodecode,functionscanonlybedeclaredattoplevelorinsideablock./// eslint-disable-line max-len
+},
 // Named functions can be used:
 {client: client_unix,send: 'function blah() { return 1; }',
 expect: prompt_unix},
@@ -312,16 +325,20 @@ function error_test() {
 {client: client_unix,send: 'require("internal/repl")',
 expect: /^Error:Cannotfindmodule'internal\/repl'/},
 // REPL should handle quotes within regexp literal in multiline mode
-{client: client_unix,send: "function x(s) {\nreturn s.replace(/'/,'');\n}",
+{client: client_unix,
+send: "function x(s) {\nreturn s.replace(/'/,'');\n}",
 expect: prompt_multiline+prompt_multiline+
 'undefined\n'+prompt_unix},
-{client: client_unix,send: "function x(s) {\nreturn s.replace(/'/,'');\n}",
+{client: client_unix,
+send: "function x(s) {\nreturn s.replace(/'/,'');\n}",
 expect: prompt_multiline+prompt_multiline+
 'undefined\n'+prompt_unix},
-{client: client_unix,send: 'function x(s) {\nreturn s.replace(/"/,"");\n}',
+{client: client_unix,
+send: 'function x(s) {\nreturn s.replace(/"/,"");\n}',
 expect: prompt_multiline+prompt_multiline+
 'undefined\n'+prompt_unix},
-{client: client_unix,send: 'function x(s) {\nreturn s.replace(/.*/,"");\n}',
+{client: client_unix,
+send: 'function x(s) {\nreturn s.replace(/.*/,"");\n}',
 expect: prompt_multiline+prompt_multiline+
 'undefined\n'+prompt_unix},
 {client: client_unix,send: '{ var x = 4; }',
@@ -356,14 +373,20 @@ function error_test() {
 expect: '{ value: undefined, done: true }'},
 
 // https://github.com/nodejs/node/issues/9300
-{client: client_unix,send: 'function foo() {\nvar bar = 1 / 1; // "/"\n}',
-expect: prompt_multiline+prompt_multiline+'undefined\n'+prompt_unix},
-
-{client: client_unix,send: '(function() {\nreturn /foo/ / /bar/;\n}())',
-expect: prompt_multiline+prompt_multiline+'NaN\n'+prompt_unix},
-
-{client: client_unix,send: '(function() {\nif (false) {} /bar"/;\n}())',
-expect: prompt_multiline+prompt_multiline+'undefined\n'+prompt_unix}
+{
+client: client_unix,send: 'function foo() {\nvar bar = 1 / 1; // "/"\n}',
+expect: `${prompt_multiline}${prompt_multiline}undefined\n${prompt_unix}`
+},
+
+{
+client: client_unix,send: '(function() {\nreturn /foo/ / /bar/;\n}())',
+expect: prompt_multiline+prompt_multiline+'NaN\n'+prompt_unix
+},
+
+{
+client: client_unix,send: '(function() {\nif (false) {} /bar"/;\n}())',
+expect: prompt_multiline+prompt_multiline+'undefined\n'+prompt_unix
+}
 ]);
 }