From 31f8f6f7685672b7c41ad30752d1bc43d34f1b0e Mon Sep 17 00:00:00 2001
From: Gibson Fahnestock <gib@uk.ibm.com>
Date: Sun, 8 Jan 2017 15:23:00 +0000
Subject: [PATCH] tools, test: require const/let in test

PR-URL: https://github.com/nodejs/node/pull/10685
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
---
 test/.eslintrc.yaml | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/test/.eslintrc.yaml b/test/.eslintrc.yaml
index 5e73630174e75e..e76bd632e6927c 100644
--- a/test/.eslintrc.yaml
+++ b/test/.eslintrc.yaml
@@ -1,7 +1,13 @@
 ## Test-specific linter rules
 
 rules:
-  ## common module is mandatory in tests
-  required-modules: [2, common]
+  # ECMAScript 6
+  # http://eslint.org/docs/rules/#ecmascript-6
+  no-var: 2
+  prefer-const: 2
+
+  # Custom rules in tools/eslint-rules
   prefer-assert-iferror: 2
   prefer-assert-methods: 2
+  ## common module is mandatory in tests
+  required-modules: [2, common]