Browse Source
test: remove needless RegExp flags
* /m is needless if ^ and $ are not used
* /g is needless in split()
* /g is needless in test() with one-time RegExp/String
PR-URL: https://github.com/nodejs/node/pull/13690
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
v6.x
Vse Mozhet Byt
8 years ago
committed by
Myles Borins
No known key found for this signature in database
GPG Key ID: 933B01F40B5CA946
6 changed files with
12 additions and
12 deletions
test/parallel/test-http-full-response.js
test/parallel/test-http-outgoing-first-chunk-singlebyte-encoding.js
test/parallel/test-tls-client-verify.js
test/parallel/test-tls-server-verify.js
test/pummel/test-keep-alive.js
test/pummel/test-tls-securepair-client.js
@ -21,7 +21,7 @@ function runAb(opts, callback) {
const command = ` ab ${ opts } http://127.0.0.1: ${ server . address ( ) . port } / ` ;
exec ( command , function ( err , stdout , stderr ) {
if ( err ) {
if ( /ab|apr/m i . test ( stderr ) ) {
if ( /ab|apr/i . test ( stderr ) ) {
common . skip ( ` problem spawning \` ab \` . \n ${ stderr } ` ) ;
process . reallyExit ( 0 ) ;
}
@ -29,13 +29,13 @@ function runAb(opts, callback) {
return ;
}
let m = /Document Length:\s*(\d+) bytes/m i . exec ( stdout ) ;
let m = /Document Length:\s*(\d+) bytes/i . exec ( stdout ) ;
const documentLength = parseInt ( m [ 1 ] ) ;
m = /Complete requests:\s*(\d+)/m i . exec ( stdout ) ;
m = /Complete requests:\s*(\d+)/i . exec ( stdout ) ;
const completeRequests = parseInt ( m [ 1 ] ) ;
m = /HTML transferred:\s*(\d+) bytes/m i . exec ( stdout ) ;
m = /HTML transferred:\s*(\d+) bytes/i . exec ( stdout ) ;
const htmlTransfered = parseInt ( m [ 1 ] ) ;
assert . strictEqual ( bodyLength , documentLength ) ;
@ -24,7 +24,7 @@ for (const enc of ['utf8', 'utf16le', 'latin1', 'UTF-8']) {
const headerEnd = received . indexOf ( '\r\n\r\n' , 'utf8' ) ;
assert . notStrictEqual ( headerEnd , - 1 ) ;
const header = received . toString ( 'utf8' , 0 , headerEnd ) . split ( /\r\n/g ) ;
const header = received . toString ( 'utf8' , 0 , headerEnd ) . split ( /\r\n/ ) ;
const body = received . toString ( enc , headerEnd + 4 ) ;
assert . strictEqual ( header [ 0 ] , 'HTTP/1.1 200 OK' ) ;
@ -10,7 +10,7 @@ const tls = require('tls');
const fs = require ( 'fs' ) ;
const hosterr = /Hostname\/IP doesn't match certificate's altnames/g ;
const hosterr = /Hostname\/IP doesn't match certificate's altnames/ ;
const testCases =
[ { ca : [ 'ca1-cert' ] ,
key : 'agent2-key' ,
@ -196,7 +196,7 @@ function runClient(prefix, port, options, cb) {
client . stdout . on ( 'data' , function ( d ) {
out += d ;
if ( ! goodbye && /_unauthed/g . test ( out ) ) {
if ( ! goodbye && /_unauthed/ . test ( out ) ) {
console . error ( ` ${ prefix } * unauthed ` ) ;
goodbye = true ;
client . kill ( ) ;
@ -204,7 +204,7 @@ function runClient(prefix, port, options, cb) {
rejected = false ;
}
if ( ! goodbye && /_authed/g . test ( out ) ) {
if ( ! goodbye && /_authed/ . test ( out ) ) {
console . error ( ` ${ prefix } * authed ` ) ;
goodbye = true ;
client . kill ( ) ;
@ -58,10 +58,10 @@ function runAb(opts, callback) {
return ;
}
let matches = /Requests\/sec:\s*(\d+)\./m i . exec ( stdout ) ;
let matches = /Requests\/sec:\s*(\d+)\./i . exec ( stdout ) ;
const reqSec = parseInt ( matches [ 1 ] ) ;
matches = /Keep-Alive requests:\s*(\d+)/m i . exec ( stdout ) ;
matches = /Keep-Alive requests:\s*(\d+)/i . exec ( stdout ) ;
let keepAliveRequests ;
if ( matches ) {
keepAliveRequests = parseInt ( matches [ 1 ] ) ;
@ -63,7 +63,7 @@ function test(keyfn, certfn, check, next) {
console . error ( state ) ;
switch ( state ) {
case 'WAIT-ACCEPT' :
if ( /ACCEPT/g . test ( serverStdoutBuffer ) ) {
if ( /ACCEPT/ . test ( serverStdoutBuffer ) ) {
// Give s_server half a second to start up.
setTimeout ( startClient , 500 ) ;
state = 'WAIT-HELLO' ;
@ -71,7 +71,7 @@ function test(keyfn, certfn, check, next) {
break ;
case 'WAIT-HELLO' :
if ( /hello/g . test ( serverStdoutBuffer ) ) {
if ( /hello/ . test ( serverStdoutBuffer ) ) {
// End the current SSL connection and exit.
// See s_server(1ssl).