mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-10-31 06:05:44 +00:00
39b88340ad
MozReview-Commit-ID: 521o0fV72SQ --HG-- rename : testing/puppeteer/.flake8 => testing/marionette/puppeteer/.flake8 rename : testing/puppeteer/firefox/MANIFEST.in => testing/marionette/puppeteer/firefox/MANIFEST.in rename : testing/puppeteer/firefox/docs/Makefile => testing/marionette/puppeteer/firefox/docs/Makefile rename : testing/puppeteer/firefox/docs/api/appinfo.rst => testing/marionette/puppeteer/firefox/docs/api/appinfo.rst rename : testing/puppeteer/firefox/docs/api/keys.rst => testing/marionette/puppeteer/firefox/docs/api/keys.rst rename : testing/puppeteer/firefox/docs/api/l10n.rst => testing/marionette/puppeteer/firefox/docs/api/l10n.rst rename : testing/puppeteer/firefox/docs/api/places.rst => testing/marionette/puppeteer/firefox/docs/api/places.rst rename : testing/puppeteer/firefox/docs/api/prefs.rst => testing/marionette/puppeteer/firefox/docs/api/prefs.rst rename : testing/puppeteer/firefox/docs/api/security.rst => testing/marionette/puppeteer/firefox/docs/api/security.rst rename : testing/puppeteer/firefox/docs/api/software_update.rst => testing/marionette/puppeteer/firefox/docs/api/software_update.rst rename : testing/puppeteer/firefox/docs/api/utils.rst => testing/marionette/puppeteer/firefox/docs/api/utils.rst rename : testing/puppeteer/firefox/docs/conf.py => testing/marionette/puppeteer/firefox/docs/conf.py rename : testing/puppeteer/firefox/docs/index.rst => testing/marionette/puppeteer/firefox/docs/index.rst rename : testing/puppeteer/firefox/docs/make.bat => testing/marionette/puppeteer/firefox/docs/make.bat rename : testing/puppeteer/firefox/docs/ui/about_window/window.rst => testing/marionette/puppeteer/firefox/docs/ui/about_window/window.rst rename : testing/puppeteer/firefox/docs/ui/browser/notifications.rst => testing/marionette/puppeteer/firefox/docs/ui/browser/notifications.rst rename : testing/puppeteer/firefox/docs/ui/browser/tabbar.rst => testing/marionette/puppeteer/firefox/docs/ui/browser/tabbar.rst rename : testing/puppeteer/firefox/docs/ui/browser/toolbars.rst => testing/marionette/puppeteer/firefox/docs/ui/browser/toolbars.rst rename : testing/puppeteer/firefox/docs/ui/browser/window.rst => testing/marionette/puppeteer/firefox/docs/ui/browser/window.rst rename : testing/puppeteer/firefox/docs/ui/deck.rst => testing/marionette/puppeteer/firefox/docs/ui/deck.rst rename : testing/puppeteer/firefox/docs/ui/menu.rst => testing/marionette/puppeteer/firefox/docs/ui/menu.rst rename : testing/puppeteer/firefox/docs/ui/pageinfo/window.rst => testing/marionette/puppeteer/firefox/docs/ui/pageinfo/window.rst rename : testing/puppeteer/firefox/docs/ui/update_wizard/dialog.rst => testing/marionette/puppeteer/firefox/docs/ui/update_wizard/dialog.rst rename : testing/puppeteer/firefox/docs/ui/windows.rst => testing/marionette/puppeteer/firefox/docs/ui/windows.rst rename : testing/puppeteer/firefox/firefox_puppeteer/__init__.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/__init__.py rename : testing/puppeteer/firefox/firefox_puppeteer/api/__init__.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/api/__init__.py rename : testing/puppeteer/firefox/firefox_puppeteer/api/appinfo.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/api/appinfo.py rename : testing/puppeteer/firefox/firefox_puppeteer/api/keys.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/api/keys.py rename : testing/puppeteer/firefox/firefox_puppeteer/api/l10n.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/api/l10n.py rename : testing/puppeteer/firefox/firefox_puppeteer/api/places.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/api/places.py rename : testing/puppeteer/firefox/firefox_puppeteer/api/prefs.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/api/prefs.py rename : testing/puppeteer/firefox/firefox_puppeteer/api/security.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/api/security.py rename : testing/puppeteer/firefox/firefox_puppeteer/api/software_update.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/api/software_update.py rename : testing/puppeteer/firefox/firefox_puppeteer/api/utils.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/api/utils.py rename : testing/puppeteer/firefox/firefox_puppeteer/base.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/base.py rename : testing/puppeteer/firefox/firefox_puppeteer/decorators.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/decorators.py rename : testing/puppeteer/firefox/firefox_puppeteer/errors.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/errors.py rename : testing/puppeteer/firefox/firefox_puppeteer/mixins.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/mixins.py rename : testing/puppeteer/firefox/firefox_puppeteer/puppeteer.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/puppeteer.py rename : testing/puppeteer/firefox/firefox_puppeteer/ui/__init__.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/ui/__init__.py rename : testing/puppeteer/firefox/firefox_puppeteer/ui/about_window/__init__.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/ui/about_window/__init__.py rename : testing/puppeteer/firefox/firefox_puppeteer/ui/about_window/deck.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/ui/about_window/deck.py rename : testing/puppeteer/firefox/firefox_puppeteer/ui/about_window/window.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/ui/about_window/window.py rename : testing/puppeteer/firefox/firefox_puppeteer/ui/base.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/ui/base.py rename : testing/puppeteer/firefox/firefox_puppeteer/ui/browser/__init__.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/ui/browser/__init__.py rename : testing/puppeteer/firefox/firefox_puppeteer/ui/browser/notifications.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/ui/browser/notifications.py rename : testing/puppeteer/firefox/firefox_puppeteer/ui/browser/tabbar.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/ui/browser/tabbar.py rename : testing/puppeteer/firefox/firefox_puppeteer/ui/browser/toolbars.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/ui/browser/toolbars.py rename : testing/puppeteer/firefox/firefox_puppeteer/ui/browser/window.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/ui/browser/window.py rename : testing/puppeteer/firefox/firefox_puppeteer/ui/deck.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/ui/deck.py rename : testing/puppeteer/firefox/firefox_puppeteer/ui/menu.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/ui/menu.py rename : testing/puppeteer/firefox/firefox_puppeteer/ui/pageinfo/__init__.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/ui/pageinfo/__init__.py rename : testing/puppeteer/firefox/firefox_puppeteer/ui/pageinfo/deck.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/ui/pageinfo/deck.py rename : testing/puppeteer/firefox/firefox_puppeteer/ui/pageinfo/window.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/ui/pageinfo/window.py rename : testing/puppeteer/firefox/firefox_puppeteer/ui/update_wizard/__init__.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/ui/update_wizard/__init__.py rename : testing/puppeteer/firefox/firefox_puppeteer/ui/update_wizard/dialog.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/ui/update_wizard/dialog.py rename : testing/puppeteer/firefox/firefox_puppeteer/ui/update_wizard/wizard.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/ui/update_wizard/wizard.py rename : testing/puppeteer/firefox/firefox_puppeteer/ui/windows.py => testing/marionette/puppeteer/firefox/firefox_puppeteer/ui/windows.py rename : testing/puppeteer/firefox/requirements-docs.txt => testing/marionette/puppeteer/firefox/requirements-docs.txt rename : testing/puppeteer/firefox/requirements.txt => testing/marionette/puppeteer/firefox/requirements.txt rename : testing/puppeteer/firefox/setup.py => testing/marionette/puppeteer/firefox/setup.py extra : rebase_source : 2c34436c2669cab33d7d3b3f2b23c46c7d3a27d7
196 lines
5.3 KiB
Python
196 lines
5.3 KiB
Python
# -*- Mode: python; indent-tabs-mode: nil; tab-width: 40 -*-
|
|
# vim: set filetype=python:
|
|
# This Source Code Form is subject to the terms of the Mozilla Public
|
|
# License, v. 2.0. If a copy of the MPL was not distributed with this
|
|
# file, You can obtain one at http://mozilla.org/MPL/2.0/.
|
|
|
|
import json
|
|
import os
|
|
import signal
|
|
import subprocess
|
|
|
|
import which
|
|
from mozprocess import ProcessHandler
|
|
|
|
from mozlint import result
|
|
|
|
|
|
here = os.path.abspath(os.path.dirname(__file__))
|
|
FLAKE8_REQUIREMENTS_PATH = os.path.join(here, 'flake8', 'flake8_requirements.txt')
|
|
|
|
FLAKE8_NOT_FOUND = """
|
|
Could not find flake8! Install flake8 and try again.
|
|
|
|
$ pip install -U --require-hashes -r {}
|
|
""".strip().format(FLAKE8_REQUIREMENTS_PATH)
|
|
|
|
|
|
FLAKE8_INSTALL_ERROR = """
|
|
Unable to install correct version of flake8
|
|
Try to install it manually with:
|
|
$ pip install -U --require-hashes -r {}
|
|
""".strip().format(FLAKE8_REQUIREMENTS_PATH)
|
|
|
|
LINE_OFFSETS = {
|
|
# continuation line under-indented for hanging indent
|
|
'E121': (-1, 2),
|
|
# continuation line missing indentation or outdented
|
|
'E122': (-1, 2),
|
|
# continuation line over-indented for hanging indent
|
|
'E126': (-1, 2),
|
|
# continuation line over-indented for visual indent
|
|
'E127': (-1, 2),
|
|
# continuation line under-indented for visual indent
|
|
'E128': (-1, 2),
|
|
# continuation line unaligned for hanging indend
|
|
'E131': (-1, 2),
|
|
# expected 1 blank line, found 0
|
|
'E301': (-1, 2),
|
|
# expected 2 blank lines, found 1
|
|
'E302': (-2, 3),
|
|
}
|
|
"""Maps a flake8 error to a lineoffset tuple.
|
|
|
|
The offset is of the form (lineno_offset, num_lines) and is passed
|
|
to the lineoffset property of `ResultContainer`.
|
|
"""
|
|
|
|
EXTENSIONS = ['.py', '.lint']
|
|
results = []
|
|
|
|
|
|
def process_line(line):
|
|
# Escape slashes otherwise JSON conversion will not work
|
|
line = line.replace('\\', '\\\\')
|
|
try:
|
|
res = json.loads(line)
|
|
except ValueError:
|
|
print('Non JSON output from linter, will not be processed: {}'.format(line))
|
|
return
|
|
|
|
if 'code' in res:
|
|
if res['code'].startswith('W'):
|
|
res['level'] = 'warning'
|
|
|
|
if res['code'] in LINE_OFFSETS:
|
|
res['lineoffset'] = LINE_OFFSETS[res['code']]
|
|
|
|
results.append(result.from_linter(LINTER, **res))
|
|
|
|
|
|
def run_process(cmdargs):
|
|
# flake8 seems to handle SIGINT poorly. Handle it here instead
|
|
# so we can kill the process without a cryptic traceback.
|
|
orig = signal.signal(signal.SIGINT, signal.SIG_IGN)
|
|
proc = ProcessHandler(cmdargs, env=os.environ,
|
|
processOutputLine=process_line)
|
|
proc.run()
|
|
signal.signal(signal.SIGINT, orig)
|
|
|
|
try:
|
|
proc.wait()
|
|
except KeyboardInterrupt:
|
|
proc.kill()
|
|
|
|
|
|
def get_flake8_binary():
|
|
"""
|
|
Returns the path of the first flake8 binary available
|
|
if not found returns None
|
|
"""
|
|
binary = os.environ.get('FLAKE8')
|
|
if binary:
|
|
return binary
|
|
|
|
try:
|
|
return which.which('flake8')
|
|
except which.WhichError:
|
|
return None
|
|
|
|
|
|
def _run_pip(*args):
|
|
"""
|
|
Helper function that runs pip with subprocess
|
|
"""
|
|
try:
|
|
subprocess.check_output(['pip'] + list(args),
|
|
stderr=subprocess.STDOUT)
|
|
return True
|
|
except subprocess.CalledProcessError as e:
|
|
print(e.output)
|
|
return False
|
|
|
|
|
|
def reinstall_flake8():
|
|
"""
|
|
Try to install flake8 at the target version, returns True on success
|
|
otherwise prints the otuput of the pip command and returns False
|
|
"""
|
|
if _run_pip('install', '-U',
|
|
'--require-hashes', '-r',
|
|
FLAKE8_REQUIREMENTS_PATH):
|
|
return True
|
|
|
|
return False
|
|
|
|
|
|
def lint(files, **lintargs):
|
|
|
|
if not reinstall_flake8():
|
|
print(FLAKE8_INSTALL_ERROR)
|
|
return 1
|
|
|
|
binary = get_flake8_binary()
|
|
|
|
cmdargs = [
|
|
binary,
|
|
'--format', '{"path":"%(path)s","lineno":%(row)s,'
|
|
'"column":%(col)s,"rule":"%(code)s","message":"%(text)s"}',
|
|
]
|
|
|
|
# Run any paths with a .flake8 file in the directory separately so
|
|
# it gets picked up. This means only .flake8 files that live in
|
|
# directories that are explicitly included will be considered.
|
|
# See bug 1277851
|
|
no_config = []
|
|
for f in files:
|
|
if not os.path.isfile(os.path.join(f, '.flake8')):
|
|
no_config.append(f)
|
|
continue
|
|
run_process(cmdargs+[f])
|
|
|
|
# XXX For some reason passing in --exclude results in flake8 not using
|
|
# the local .flake8 file. So for now only pass in --exclude if there
|
|
# is no local config.
|
|
exclude = lintargs.get('exclude')
|
|
if exclude:
|
|
cmdargs += ['--exclude', ','.join(lintargs['exclude'])]
|
|
|
|
if no_config:
|
|
run_process(cmdargs+no_config)
|
|
|
|
return results
|
|
|
|
|
|
LINTER = {
|
|
'name': "flake8",
|
|
'description': "Python linter",
|
|
'include': [
|
|
'python/mozlint',
|
|
'taskcluster',
|
|
'testing/firefox-ui',
|
|
'testing/marionette/client',
|
|
'testing/marionette/harness',
|
|
'testing/marionette/puppeteer',
|
|
'testing/mozbase',
|
|
'testing/mochitest',
|
|
'testing/talos/',
|
|
'tools/lint',
|
|
],
|
|
'exclude': ["testing/mozbase/mozdevice/mozdevice/Zeroconf.py",
|
|
'testing/mochitest/pywebsocket'],
|
|
'extensions': EXTENSIONS,
|
|
'type': 'external',
|
|
'payload': lint,
|
|
}
|