tools: python: update flake8 rules

* Tree-factor location of some *.py files for easy demarcation of
  areas to exclude.

PR-URL: https://github.com/nodejs/node/pull/25614
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>

57 files changed, 9 insertions, 13 deletions

diff --git a/.flake8 b/.flake8
index 859c4d3575..07baf0a8a8 100644
--- a/.flake8
+++ b/.flake8
@@ -1,3 +1,3 @@
 [flake8]
-exclude=.git,deps,lib,src,test/fixtures,tools/*_macros.py,tools/gyp,tools/inspector_protocol,tools/jinja2,tools/markupsafe,tools/pip,tools/v8_gypfiles/broken
-select=E901,E999,F821,F822,F823
+exclude=.git,deps,lib,src,tools/gyp,tools/inspector_protocol,tools/pip,tools/v8_gypfiles/broken
+select=E9,F82
diff --git a/node.gyp b/node.gyp
index f23fd42908..6e792c907b 100644
--- a/node.gyp
+++ b/node.gyp
@@ -805,20 +805,20 @@
           'inputs': [
             '<@(library_files)',
             'config.gypi',
-            'tools/check_macros.py'
+            'tools/js2c_macros/check_macros.py'
           ],
           'outputs': [
             '<(SHARED_INTERMEDIATE_DIR)/node_javascript.cc',
           ],
           'conditions': [
             [ 'node_use_dtrace=="false" and node_use_etw=="false"', {
-              'inputs': [ 'src/notrace_macros.py' ]
+              'inputs': [ 'tools/js2c_macros/notrace_macros.py' ]
             }],
             [ 'node_debug_lib=="false"', {
-              'inputs': [ 'tools/nodcheck_macros.py' ]
+              'inputs': [ 'tools/js2c_macros/nodcheck_macros.py' ]
             }],
             [ 'node_debug_lib=="true"', {
-              'inputs': [ 'tools/dcheck_macros.py' ]
+              'inputs': [ 'tools/js2c_macros/dcheck_macros.py' ]
             }]
           ],
           'action': [
diff --git a/src/inspector/node_inspector.gypi b/src/inspector/node_inspector.gypi
index 2eabd9dd31..6ec85461dd 100644
--- a/src/inspector/node_inspector.gypi
+++ b/src/inspector/node_inspector.gypi
@@ -97,7 +97,7 @@
       'action': [
         'python',
         'tools/inspector_protocol/code_generator.py',
-        '--jinja_dir', '<@(protocol_tool_path)/..',
+        '--jinja_dir', '<@(protocol_tool_path)',
         '--output_base', '<(SHARED_INTERMEDIATE_DIR)/src/',
         '--config', 'src/inspector/node_protocol_config.json',
       ],
diff --git a/test/fixtures/wpt/.gitignore b/test/fixtures/wpt/.gitignore
new file mode 100644
index 0000000000..f104652b68
--- /dev/null
+++ b/test/fixtures/wpt/.gitignore
@@ -0,0 +1 @@
+*.py
diff --git a/test/fixtures/wpt/encoding/resources/single-byte-raw.py b/test/fixtures/wpt/encoding/resources/single-byte-raw.py
deleted file mode 100644
index b4a6c90405..0000000000
--- a/test/fixtures/wpt/encoding/resources/single-byte-raw.py
+++ /dev/null
@@ -1,3 +0,0 @@
-def main(request, response):
-  response.headers.set("Content-Type", "text/plain;charset=" + request.GET.first("label"))
-  response.content = "".join(chr(byte) for byte in xrange(255))
diff --git a/test/fixtures/wpt/encoding/resources/text-plain-charset.py b/test/fixtures/wpt/encoding/resources/text-plain-charset.py
deleted file mode 100644
index a1c07e7018..0000000000
--- a/test/fixtures/wpt/encoding/resources/text-plain-charset.py
+++ /dev/null
@@ -1,3 +0,0 @@
-def main(request, response):
-  response.headers.set("Content-Type", "text/plain;charset=" + request.GET.first("label"))
-  response.content = "hello encoding"
diff --git a/tools/jinja2/AUTHORS b/tools/inspector_protocol/jinja2/AUTHORS
index fd6dbfcbfd..fd6dbfcbfd 100644
--- a/tools/jinja2/AUTHORS
+++ b/tools/inspector_protocol/jinja2/AUTHORS
diff --git a/tools/jinja2/Jinja2-2.10.tar.gz.md5 b/tools/inspector_protocol/jinja2/Jinja2-2.10.tar.gz.md5
index 9137ee129a..9137ee129a 100644
--- a/tools/jinja2/Jinja2-2.10.tar.gz.md5
+++ b/tools/inspector_protocol/jinja2/Jinja2-2.10.tar.gz.md5
diff --git a/tools/jinja2/Jinja2-2.10.tar.gz.sha512 b/tools/inspector_protocol/jinja2/Jinja2-2.10.tar.gz.sha512
index 087d24c18e..087d24c18e 100644
--- a/tools/jinja2/Jinja2-2.10.tar.gz.sha512
+++ b/tools/inspector_protocol/jinja2/Jinja2-2.10.tar.gz.sha512
diff --git a/tools/jinja2/LICENSE b/tools/inspector_protocol/jinja2/LICENSE
index 31bf900e58..31bf900e58 100644
--- a/tools/jinja2/LICENSE
+++ b/tools/inspector_protocol/jinja2/LICENSE
diff --git a/tools/jinja2/OWNERS b/tools/inspector_protocol/jinja2/OWNERS
index ee2bec9ba3..ee2bec9ba3 100644
--- a/tools/jinja2/OWNERS
+++ b/tools/inspector_protocol/jinja2/OWNERS
diff --git a/tools/jinja2/README.chromium b/tools/inspector_protocol/jinja2/README.chromium
index 5246c2f84b..5246c2f84b 100644
--- a/tools/jinja2/README.chromium
+++ b/tools/inspector_protocol/jinja2/README.chromium
diff --git a/tools/jinja2/__init__.py b/tools/inspector_protocol/jinja2/__init__.py
index 42aa763d57..42aa763d57 100644
--- a/tools/jinja2/__init__.py
+++ b/tools/inspector_protocol/jinja2/__init__.py
diff --git a/tools/jinja2/_compat.py b/tools/inspector_protocol/jinja2/_compat.py
index 61d85301a4..61d85301a4 100644
--- a/tools/jinja2/_compat.py
+++ b/tools/inspector_protocol/jinja2/_compat.py
diff --git a/tools/jinja2/_identifier.py b/tools/inspector_protocol/jinja2/_identifier.py
index 2eac35d5c3..2eac35d5c3 100644
--- a/tools/jinja2/_identifier.py
+++ b/tools/inspector_protocol/jinja2/_identifier.py
diff --git a/tools/jinja2/asyncfilters.py b/tools/inspector_protocol/jinja2/asyncfilters.py
index 5c1f46d7fa..5c1f46d7fa 100644
--- a/tools/jinja2/asyncfilters.py
+++ b/tools/inspector_protocol/jinja2/asyncfilters.py
diff --git a/tools/jinja2/asyncsupport.py b/tools/inspector_protocol/jinja2/asyncsupport.py
index b1e7b5ce9a..b1e7b5ce9a 100644
--- a/tools/jinja2/asyncsupport.py
+++ b/tools/inspector_protocol/jinja2/asyncsupport.py
diff --git a/tools/jinja2/bccache.py b/tools/inspector_protocol/jinja2/bccache.py
index 080e527cab..080e527cab 100644
--- a/tools/jinja2/bccache.py
+++ b/tools/inspector_protocol/jinja2/bccache.py
diff --git a/tools/jinja2/compiler.py b/tools/inspector_protocol/jinja2/compiler.py
index d534a82739..d534a82739 100644
--- a/tools/jinja2/compiler.py
+++ b/tools/inspector_protocol/jinja2/compiler.py
diff --git a/tools/jinja2/constants.py b/tools/inspector_protocol/jinja2/constants.py
index 11efd1ed15..11efd1ed15 100644
--- a/tools/jinja2/constants.py
+++ b/tools/inspector_protocol/jinja2/constants.py
diff --git a/tools/jinja2/debug.py b/tools/inspector_protocol/jinja2/debug.py
index b61139f0cd..b61139f0cd 100644
--- a/tools/jinja2/debug.py
+++ b/tools/inspector_protocol/jinja2/debug.py
diff --git a/tools/jinja2/defaults.py b/tools/inspector_protocol/jinja2/defaults.py
index 7c93dec0ae..7c93dec0ae 100644
--- a/tools/jinja2/defaults.py
+++ b/tools/inspector_protocol/jinja2/defaults.py
diff --git a/tools/jinja2/environment.py b/tools/inspector_protocol/jinja2/environment.py
index 549d9afab4..549d9afab4 100644
--- a/tools/jinja2/environment.py
+++ b/tools/inspector_protocol/jinja2/environment.py
diff --git a/tools/jinja2/exceptions.py b/tools/inspector_protocol/jinja2/exceptions.py
index c018a33e32..c018a33e32 100644
--- a/tools/jinja2/exceptions.py
+++ b/tools/inspector_protocol/jinja2/exceptions.py
diff --git a/tools/jinja2/ext.py b/tools/inspector_protocol/jinja2/ext.py
index 0734a84f73..0734a84f73 100644
--- a/tools/jinja2/ext.py
+++ b/tools/inspector_protocol/jinja2/ext.py
diff --git a/tools/jinja2/filters.py b/tools/inspector_protocol/jinja2/filters.py
index 267ddddaa0..267ddddaa0 100644
--- a/tools/jinja2/filters.py
+++ b/tools/inspector_protocol/jinja2/filters.py
diff --git a/tools/jinja2/get_jinja2.sh b/tools/inspector_protocol/jinja2/get_jinja2.sh
index bc6c4c3068..bc6c4c3068 100755
--- a/tools/jinja2/get_jinja2.sh
+++ b/tools/inspector_protocol/jinja2/get_jinja2.sh
diff --git a/tools/jinja2/idtracking.py b/tools/inspector_protocol/jinja2/idtracking.py
index 491bfe0836..491bfe0836 100644
--- a/tools/jinja2/idtracking.py
+++ b/tools/inspector_protocol/jinja2/idtracking.py
diff --git a/tools/jinja2/jinja2.gni b/tools/inspector_protocol/jinja2/jinja2.gni
index cef0d48a88..cef0d48a88 100644
--- a/tools/jinja2/jinja2.gni
+++ b/tools/inspector_protocol/jinja2/jinja2.gni
diff --git a/tools/jinja2/lexer.py b/tools/inspector_protocol/jinja2/lexer.py
index 6fd135dd5b..6fd135dd5b 100644
--- a/tools/jinja2/lexer.py
+++ b/tools/inspector_protocol/jinja2/lexer.py
diff --git a/tools/jinja2/loaders.py b/tools/inspector_protocol/jinja2/loaders.py
index 4c79793760..4c79793760 100644
--- a/tools/jinja2/loaders.py
+++ b/tools/inspector_protocol/jinja2/loaders.py
diff --git a/tools/jinja2/meta.py b/tools/inspector_protocol/jinja2/meta.py
index 7421914f77..7421914f77 100644
--- a/tools/jinja2/meta.py
+++ b/tools/inspector_protocol/jinja2/meta.py
diff --git a/tools/jinja2/nativetypes.py b/tools/inspector_protocol/jinja2/nativetypes.py
index fe17e4138d..fe17e4138d 100644
--- a/tools/jinja2/nativetypes.py
+++ b/tools/inspector_protocol/jinja2/nativetypes.py
diff --git a/tools/jinja2/nodes.py b/tools/inspector_protocol/jinja2/nodes.py
index 4d9a01ad8b..4d9a01ad8b 100644
--- a/tools/jinja2/nodes.py
+++ b/tools/inspector_protocol/jinja2/nodes.py
diff --git a/tools/jinja2/optimizer.py b/tools/inspector_protocol/jinja2/optimizer.py
index 65ab3ceb71..65ab3ceb71 100644
--- a/tools/jinja2/optimizer.py
+++ b/tools/inspector_protocol/jinja2/optimizer.py
diff --git a/tools/jinja2/parser.py b/tools/inspector_protocol/jinja2/parser.py
index ed00d9708e..ed00d9708e 100644
--- a/tools/jinja2/parser.py
+++ b/tools/inspector_protocol/jinja2/parser.py
diff --git a/tools/jinja2/runtime.py b/tools/inspector_protocol/jinja2/runtime.py
index f9d7a6806c..f9d7a6806c 100644
--- a/tools/jinja2/runtime.py
+++ b/tools/inspector_protocol/jinja2/runtime.py
diff --git a/tools/jinja2/sandbox.py b/tools/inspector_protocol/jinja2/sandbox.py
index 93fb9d45f3..93fb9d45f3 100644
--- a/tools/jinja2/sandbox.py
+++ b/tools/inspector_protocol/jinja2/sandbox.py
diff --git a/tools/jinja2/tests.py b/tools/inspector_protocol/jinja2/tests.py
index 0adc3d4dbc..0adc3d4dbc 100644
--- a/tools/jinja2/tests.py
+++ b/tools/inspector_protocol/jinja2/tests.py
diff --git a/tools/jinja2/utils.py b/tools/inspector_protocol/jinja2/utils.py
index 502a311c08..502a311c08 100644
--- a/tools/jinja2/utils.py
+++ b/tools/inspector_protocol/jinja2/utils.py
diff --git a/tools/jinja2/visitor.py b/tools/inspector_protocol/jinja2/visitor.py
index ba526dfac9..ba526dfac9 100644
--- a/tools/jinja2/visitor.py
+++ b/tools/inspector_protocol/jinja2/visitor.py
diff --git a/tools/markupsafe/AUTHORS b/tools/inspector_protocol/markupsafe/AUTHORS
index f7e2942ecc..f7e2942ecc 100644
--- a/tools/markupsafe/AUTHORS
+++ b/tools/inspector_protocol/markupsafe/AUTHORS
diff --git a/tools/markupsafe/LICENSE b/tools/inspector_protocol/markupsafe/LICENSE
index 5d2693890d..5d2693890d 100644
--- a/tools/markupsafe/LICENSE
+++ b/tools/inspector_protocol/markupsafe/LICENSE
diff --git a/tools/markupsafe/MarkupSafe-0.18.tar.gz.md5 b/tools/inspector_protocol/markupsafe/MarkupSafe-0.18.tar.gz.md5
index 1348d1eea1..1348d1eea1 100644
--- a/tools/markupsafe/MarkupSafe-0.18.tar.gz.md5
+++ b/tools/inspector_protocol/markupsafe/MarkupSafe-0.18.tar.gz.md5
diff --git a/tools/markupsafe/MarkupSafe-0.18.tar.gz.sha512 b/tools/inspector_protocol/markupsafe/MarkupSafe-0.18.tar.gz.sha512
index ab752200d5..ab752200d5 100644
--- a/tools/markupsafe/MarkupSafe-0.18.tar.gz.sha512
+++ b/tools/inspector_protocol/markupsafe/MarkupSafe-0.18.tar.gz.sha512
diff --git a/tools/markupsafe/OWNERS b/tools/inspector_protocol/markupsafe/OWNERS
index 6a57e5dc3f..6a57e5dc3f 100644
--- a/tools/markupsafe/OWNERS
+++ b/tools/inspector_protocol/markupsafe/OWNERS
diff --git a/tools/markupsafe/README.chromium b/tools/inspector_protocol/markupsafe/README.chromium
index 0fcab52fa2..0fcab52fa2 100644
--- a/tools/markupsafe/README.chromium
+++ b/tools/inspector_protocol/markupsafe/README.chromium
diff --git a/tools/markupsafe/__init__.py b/tools/inspector_protocol/markupsafe/__init__.py
index e8b4b453c1..e8b4b453c1 100644
--- a/tools/markupsafe/__init__.py
+++ b/tools/inspector_protocol/markupsafe/__init__.py
diff --git a/tools/markupsafe/_compat.py b/tools/inspector_protocol/markupsafe/_compat.py
index 29e4a3dac1..29e4a3dac1 100644
--- a/tools/markupsafe/_compat.py
+++ b/tools/inspector_protocol/markupsafe/_compat.py
diff --git a/tools/markupsafe/_constants.py b/tools/inspector_protocol/markupsafe/_constants.py
index 919bf03c50..919bf03c50 100644
--- a/tools/markupsafe/_constants.py
+++ b/tools/inspector_protocol/markupsafe/_constants.py
diff --git a/tools/markupsafe/_native.py b/tools/inspector_protocol/markupsafe/_native.py
index 5e83f10a11..5e83f10a11 100644
--- a/tools/markupsafe/_native.py
+++ b/tools/inspector_protocol/markupsafe/_native.py
diff --git a/tools/markupsafe/_speedups.c b/tools/inspector_protocol/markupsafe/_speedups.c
index 78b708e1b3..78b708e1b3 100644
--- a/tools/markupsafe/_speedups.c
+++ b/tools/inspector_protocol/markupsafe/_speedups.c
diff --git a/tools/markupsafe/get_markupsafe.sh b/tools/inspector_protocol/markupsafe/get_markupsafe.sh
index d268832df8..d268832df8 100755
--- a/tools/markupsafe/get_markupsafe.sh
+++ b/tools/inspector_protocol/markupsafe/get_markupsafe.sh
diff --git a/tools/check_macros.py b/tools/js2c_macros/check_macros.py
index 2baf0d7f41..f24d47c9ee 100644
--- a/tools/check_macros.py
+++ b/tools/js2c_macros/check_macros.py
@@ -1,3 +1,4 @@
+# flake8: noqa
 macro CHECK(x) = do { if (!(x)) (process._rawDebug("CHECK: x == true"), process.abort()) } while (0);
 macro CHECK_EQ(a, b) = CHECK((a) === (b));
 macro CHECK_GE(a, b) = CHECK((a) >= (b));
diff --git a/tools/dcheck_macros.py b/tools/js2c_macros/dcheck_macros.py
index f22c08598f..f22c08598f 100644
--- a/tools/dcheck_macros.py
+++ b/tools/js2c_macros/dcheck_macros.py
diff --git a/tools/nodcheck_macros.py b/tools/js2c_macros/nodcheck_macros.py
index 0a59001b54..0a59001b54 100644
--- a/tools/nodcheck_macros.py
+++ b/tools/js2c_macros/nodcheck_macros.py
diff --git a/src/notrace_macros.py b/tools/js2c_macros/notrace_macros.py
index 334f9c0c7f..334f9c0c7f 100644
--- a/src/notrace_macros.py
+++ b/tools/js2c_macros/notrace_macros.py