test,benchmark,doc: enable dot-notation rule

This enables the eslint dot-notation rule for all code instead of
only in /lib.

PR-URL: #18749
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Matheus Marchini <[email protected]>

Author: BridgeAR

.eslintrc.yaml

@@ -44,6 +44,7 @@ rules:
   # http://eslint.org/docs/rules/#best-practices
   accessor-pairs: error
   dot-location: [error, property]
+  dot-notation: error
   eqeqeq: [error, smart]
   no-fallthrough: error
   no-global-assign: error

benchmark/misc/object-property-bench.js

@@ -1,5 +1,7 @@
 'use strict';
 
+/* eslint-disable dot-notation */
+
 const common = require('../common.js');
 
 const bench = common.createBenchmark(main, {

doc/api/http2.md

@@ -1236,7 +1236,7 @@ const server = http2.createServer();
 server.on('stream', (stream) => {
   stream.respond({ ':status': 200 }, {
     getTrailers(trailers) {
-      trailers['ABC'] = 'some value to send';
+      trailers.ABC = 'some value to send';
     }
   });
   stream.end('some data');
@@ -1326,7 +1326,7 @@ server.on('stream', (stream) => {
   };
   stream.respondWithFD(fd, headers, {
     getTrailers(trailers) {
-      trailers['ABC'] = 'some value to send';
+      trailers.ABC = 'some value to send';
     }
   });
 
@@ -1435,7 +1435,7 @@ const http2 = require('http2');
 const server = http2.createServer();
 server.on('stream', (stream) => {
   function getTrailers(trailers) {
-    trailers['ABC'] = 'some value to send';
+    trailers.ABC = 'some value to send';
   }
   stream.respondWithFile('/some/file',
                          { 'content-type': 'text/plain' },

lib/.eslintrc.yaml

@@ -1,6 +1,4 @@
 rules:
-  dot-notation: error
-
   # Custom rules in tools/eslint-rules
   require-buffer: error
   buffer-constructor: error

test/addons-napi/test_symbol/test1.js

@@ -8,11 +8,10 @@ const test_symbol = require(`./build/${common.buildType}/test_symbol`);
 const sym = test_symbol.New('test');
 assert.strictEqual(sym.toString(), 'Symbol(test)');
 
-
 const myObj = {};
 const fooSym = test_symbol.New('foo');
 const otherSym = test_symbol.New('bar');
-myObj['foo'] = 'bar';
+myObj.foo = 'bar';
 myObj[fooSym] = 'baz';
 myObj[otherSym] = 'bing';
 assert.strictEqual(myObj.foo, 'bar');

test/addons-napi/test_symbol/test2.js

@@ -7,7 +7,7 @@ const test_symbol = require(`./build/${common.buildType}/test_symbol`);
 
 const fooSym = test_symbol.New('foo');
 const myObj = {};
-myObj['foo'] = 'bar';
+myObj.foo = 'bar';
 myObj[fooSym] = 'baz';
 Object.keys(myObj); // -> [ 'foo' ]
 Object.getOwnPropertyNames(myObj); // -> [ 'foo' ]

test/common/index.js

@@ -512,7 +512,7 @@ exports.canCreateSymLink = function() {
     // whoami.exe needs to be the one from System32
     // If unix tools are in the path, they can shadow the one we want,
     // so use the full path while executing whoami
-    const whoamiPath = path.join(process.env['SystemRoot'],
+    const whoamiPath = path.join(process.env.SystemRoot,
                                  'System32', 'whoami.exe');
 
     let err = false;

test/common/inspector-helper.js

@@ -167,9 +167,7 @@ class InspectorSession {
         reject(message.error);
     } else {
       if (message.method === 'Debugger.scriptParsed') {
-        const script = message['params'];
-        const scriptId = script['scriptId'];
-        const url = script['url'];
+        const { scriptId, url } = message.params;
         this._scriptsIdsByUrl.set(scriptId, url);
         if (url === _MAINSCRIPT)
           this.mainScriptId = scriptId;
@@ -188,12 +186,12 @@ class InspectorSession {
 
   _sendMessage(message) {
     const msg = JSON.parse(JSON.stringify(message)); // Clone!
-    msg['id'] = this._nextId++;
+    msg.id = this._nextId++;
     if (DEBUG)
       console.log('[sent]', JSON.stringify(msg));
 
     const responsePromise = new Promise((resolve, reject) => {
-      this._commandResponsePromises.set(msg['id'], { resolve, reject });
+      this._commandResponsePromises.set(msg.id, { resolve, reject });
     });
 
     return new Promise(
@@ -238,12 +236,15 @@ class InspectorSession {
     return notification;
   }
 
-  _isBreakOnLineNotification(message, line, url) {
-    if ('Debugger.paused' === message['method']) {
-      const callFrame = message['params']['callFrames'][0];
-      const location = callFrame['location'];
-      assert.strictEqual(url, this._scriptsIdsByUrl.get(location['scriptId']));
-      assert.strictEqual(line, location['lineNumber']);
+  _isBreakOnLineNotification(message, line, expectedScriptPath) {
+    if ('Debugger.paused' === message.method) {
+      const callFrame = message.params.callFrames[0];
+      const location = callFrame.location;
+      const scriptPath = this._scriptsIdsByUrl.get(location.scriptId);
+      assert.strictEqual(scriptPath.toString(),
+                         expectedScriptPath.toString(),
+                         `${scriptPath} !== ${expectedScriptPath}`);
+      assert.strictEqual(line, location.lineNumber);
       return true;
     }
   }
@@ -259,12 +260,12 @@ class InspectorSession {
   _matchesConsoleOutputNotification(notification, type, values) {
     if (!Array.isArray(values))
       values = [ values ];
-    if ('Runtime.consoleAPICalled' === notification['method']) {
-      const params = notification['params'];
-      if (params['type'] === type) {
+    if ('Runtime.consoleAPICalled' === notification.method) {
+      const params = notification.params;
+      if (params.type === type) {
         let i = 0;
-        for (const value of params['args']) {
-          if (value['value'] !== values[i++])
+        for (const value of params.args) {
+          if (value.value !== values[i++])
             return false;
         }
         return i === values.length;
@@ -389,7 +390,7 @@ class NodeInstance {
   async connectInspectorSession() {
     console.log('[test]', 'Connecting to a child Node process');
     const response = await this.httpGet(null, '/json/list');
-    const url = response[0]['webSocketDebuggerUrl'];
+    const url = response[0].webSocketDebuggerUrl;
     return this.wsHandshake(url);
   }
 

test/parallel/test-cluster-fork-env.js

@@ -31,8 +31,8 @@ const cluster = require('cluster');
 
 if (cluster.isWorker) {
   const result = cluster.worker.send({
-    prop: process.env['cluster_test_prop'],
-    overwrite: process.env['cluster_test_overwrite']
+    prop: process.env.cluster_test_prop,
+    overwrite: process.env.cluster_test_overwrite
   });
 
   assert.strictEqual(result, true);
@@ -45,7 +45,7 @@ if (cluster.isWorker) {
 
   // To check that the cluster extend on the process.env we will overwrite a
   // property
-  process.env['cluster_test_overwrite'] = 'old';
+  process.env.cluster_test_overwrite = 'old';
 
   // Fork worker
   const worker = cluster.fork({

test/parallel/test-crypto-hmac.js

@@ -71,13 +71,13 @@ const wikipedia = [
 ];
 
 for (let i = 0, l = wikipedia.length; i < l; i++) {
-  for (const hash in wikipedia[i]['hmac']) {
+  for (const hash in wikipedia[i].hmac) {
     // FIPS does not support MD5.
     if (common.hasFipsCrypto && hash === 'md5')
       continue;
-    const expected = wikipedia[i]['hmac'][hash];
-    const actual = crypto.createHmac(hash, wikipedia[i]['key'])
-                         .update(wikipedia[i]['data'])
+    const expected = wikipedia[i].hmac[hash];
+    const actual = crypto.createHmac(hash, wikipedia[i].key)
+                         .update(wikipedia[i].data)
                          .digest('hex');
     assert.strictEqual(
       actual,
@@ -236,18 +236,18 @@ const rfc4231 = [
 ];
 
 for (let i = 0, l = rfc4231.length; i < l; i++) {
-  for (const hash in rfc4231[i]['hmac']) {
+  for (const hash in rfc4231[i].hmac) {
     const str = crypto.createHmac(hash, rfc4231[i].key);
     str.end(rfc4231[i].data);
     let strRes = str.read().toString('hex');
-    let actual = crypto.createHmac(hash, rfc4231[i]['key'])
-                       .update(rfc4231[i]['data'])
+    let actual = crypto.createHmac(hash, rfc4231[i].key)
+                       .update(rfc4231[i].data)
                        .digest('hex');
-    if (rfc4231[i]['truncate']) {
+    if (rfc4231[i].truncate) {
       actual = actual.substr(0, 32); // first 128 bits == 32 hex chars
       strRes = strRes.substr(0, 32);
     }
-    const expected = rfc4231[i]['hmac'][hash];
+    const expected = rfc4231[i].hmac[hash];
     assert.strictEqual(
       actual,
       expected,
@@ -368,10 +368,10 @@ const rfc2202_sha1 = [
 
 if (!common.hasFipsCrypto) {
   for (let i = 0, l = rfc2202_md5.length; i < l; i++) {
-    const actual = crypto.createHmac('md5', rfc2202_md5[i]['key'])
-      .update(rfc2202_md5[i]['data'])
+    const actual = crypto.createHmac('md5', rfc2202_md5[i].key)
+      .update(rfc2202_md5[i].data)
       .digest('hex');
-    const expected = rfc2202_md5[i]['hmac'];
+    const expected = rfc2202_md5[i].hmac;
     assert.strictEqual(
       actual,
       expected,
@@ -380,10 +380,10 @@ if (!common.hasFipsCrypto) {
   }
 }
 for (let i = 0, l = rfc2202_sha1.length; i < l; i++) {
-  const actual = crypto.createHmac('sha1', rfc2202_sha1[i]['key'])
-    .update(rfc2202_sha1[i]['data'])
+  const actual = crypto.createHmac('sha1', rfc2202_sha1[i].key)
+    .update(rfc2202_sha1[i].data)
     .digest('hex');
-  const expected = rfc2202_sha1[i]['hmac'];
+  const expected = rfc2202_sha1[i].hmac;
   assert.strictEqual(
     actual,
     expected,