1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
|
import re
import py
import pytest
import pep8
import os
__version__ = '1.0.6'
HISTKEY = "pep8/mtimes"
def pytest_addoption(parser):
group = parser.getgroup("general")
group.addoption('--pep8', action='store_true',
help="perform some pep8 sanity checks on .py files")
parser.addini("pep8ignore", type="linelist",
help="each line specifies a glob pattern and whitespace "
"separated PEP8 errors or warnings which will be ignored, "
"example: *.py W293")
parser.addini("pep8maxlinelength",
help="max. line length (default: %d)" % pep8.MAX_LINE_LENGTH)
def pytest_sessionstart(session):
config = session.config
if config.option.pep8:
config._pep8ignore = Ignorer(config.getini("pep8ignore"))
config._pep8mtimes = config.cache.get(HISTKEY, {})
config._max_line_length = int(config.getini("pep8maxlinelength")
or pep8.MAX_LINE_LENGTH)
def pytest_collect_file(path, parent):
config = parent.config
if config.option.pep8 and path.ext == '.py':
pep8ignore = config._pep8ignore(path)
if pep8ignore is not None:
return Pep8Item(path, parent, pep8ignore, config._max_line_length)
def pytest_sessionfinish(session):
config = session.config
if hasattr(config, "_pep8mtimes"):
config.cache.set(HISTKEY, config._pep8mtimes)
class Pep8Error(Exception):
""" indicates an error during pep8 checks. """
class Pep8Item(pytest.Item, pytest.File):
def __init__(self, path, parent, pep8ignore, max_line_length):
super(Pep8Item, self).__init__(path, parent)
self.add_marker("pep8")
self.pep8ignore = pep8ignore
self.max_line_length = max_line_length
def setup(self):
pep8mtimes = self.config._pep8mtimes
self._pep8mtime = self.fspath.mtime()
old = pep8mtimes.get(str(self.fspath), (0, []))
if old == (self._pep8mtime, self.pep8ignore):
pytest.skip("file(s) previously passed PEP8 checks")
def runtest(self):
call = py.io.StdCapture.call
found_errors, out, err = call(check_file, self.fspath, self.pep8ignore,
self.max_line_length)
if found_errors:
raise Pep8Error(out, err)
# update mtime only if test passed
# otherwise failures would not be re-run next time
self.config._pep8mtimes[str(self.fspath)] = (self._pep8mtime,
self.pep8ignore)
def repr_failure(self, excinfo):
if excinfo.errisinstance(Pep8Error):
return excinfo.value.args[0]
return super(Pep8Item, self).repr_failure(excinfo)
def reportinfo(self):
if self.pep8ignore:
ignores = "(ignoring %s)" % " ".join(self.pep8ignore)
else:
ignores = ""
return (self.fspath, -1, "PEP8-check%s" % ignores)
class Ignorer:
def __init__(self, ignorelines, coderex=re.compile("[EW]\d\d\d")):
self.ignores = ignores = []
for line in ignorelines:
i = line.find("#")
if i != -1:
line = line[:i]
try:
glob, ign = line.split(None, 1)
except ValueError:
glob, ign = None, line
if glob and coderex.match(glob):
glob, ign = None, line
ign = ign.split()
if "ALL" in ign:
ign = None
if glob and "/" != os.sep and "/" in glob:
glob = glob.replace("/", os.sep)
ignores.append((glob, ign))
def __call__(self, path):
l = []
for (glob, ignlist) in self.ignores:
if not glob or path.fnmatch(glob):
if ignlist is None:
return None
l.extend(ignlist)
return l
def check_file(path, pep8ignore, max_line_length):
checker = pep8.Checker(str(path), ignore=pep8ignore, show_source=1,
max_line_length=max_line_length)
return checker.check_all()
|
