-
Notifications
You must be signed in to change notification settings - Fork 3
/
plugin.py
215 lines (163 loc) · 7.11 KB
/
plugin.py
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
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
"""
quickunit.plugin
~~~~~~~~~~~~~~~~
:copyright: 2012 DISQUS.
:license: Apache 2.0
"""
from __future__ import absolute_import
import inspect
import logging
import os
import sys
from collections import defaultdict
from nose.plugins.base import Plugin
from subprocess import Popen, PIPE, STDOUT
from quickunit.diff import DiffParser
from quickunit.utils import is_py_script
def clean_bytecode_extension(filename):
"""
Replaces Python bytecode extensions (``.pyc``) with their source extension.
"""
path, extension = os.path.splitext(filename)
if extension == '.pyc':
filename = '%s.py' % path
return filename
class FileAcceptedCache(dict):
def __init__(self, prefixes, pending_files, diff_data, root=None):
self.prefixes = prefixes
self.pending_files = pending_files
self.diff_data = diff_data
self.root = root
dict.__init__(self)
def __missing__(self, filename):
self[filename] = self.check(filename)
return self[filename]
def check(self, filename):
filename = clean_bytecode_extension(filename)
# check if this test was modified (e.g. added/changed)
if self.root and filename.startswith(self.root):
filename = filename[len(self.root):]
# # check to see if this is a modified test
# diff_data = self.diff_data[filename]
# if diff_data:
# lines, startlineno = inspect.getsourcelines(method)
# for lineno in xrange(startlineno, len(lines) + startlineno):
# if lineno not in diff_data:
# continue
# return True
# if the filename is actually the changed file
if filename in self.diff_data:
return True
filepath = os.path.join(filename.rsplit('/', 1)[0])
for prefix in self.prefixes:
if not filename.startswith(prefix):
continue
for pending in self.pending_files:
# if the filename is /foo/bar/tests.py and pending is /foo/bar, run it
# if the filename is /foo/tests.py and pending is /foo, run it
if pending.startswith(filepath):
return True
elif filepath.startswith(pending):
return True
return False
class QuickUnitPlugin(Plugin):
"""
We find the diff with the parent revision for diff-tests with::
git diff `git merge-base HEAD master`
If you run with the discover flag, it will attempt to discovery
any tests that are required to run to test the changes in your current
branch, against those of the parent commit.
"""
score = 1000
name = 'quickunit'
def _get_name_from_test(self, test):
test_method_name = test._testMethodName
# We need to determine the *actual* test path (as thats what nose gives us in wantMethod)
# for example, maybe a test was imported in foo.bar.tests, but originated as foo.bar.something.MyTest
# in this case, we'd need to identify that its *actually* foo.bar.something.MyTest to record the
# proper coverage
test_ = getattr(sys.modules[test.__module__], test.__class__.__name__)
test_name = '%s:%s.%s' % (test_.__module__, test_.__name__,
test_method_name)
return test_name
def options(self, parser, env):
Plugin.options(self, parser, env)
parser.add_option("--quickunit-prefix", dest="quickunit_prefix", action="append")
def configure(self, options, config):
Plugin.configure(self, options, config)
if not self.enabled:
return
self.verbosity = options.verbosity
if options.quickunit_prefix:
self.prefixes = options.quickunit_prefix
if len(self.prefixes) == 1:
self.prefixes = self.prefixes[0].split('\n')
else:
self.prefixes = ["tests/"]
self.parent = 'master'
self.logger = logging.getLogger(__name__)
# pending files becomes a set of directorie pieces that represent
# the file locations changed in this diff
# for example, if /foo/bar/baz.py was changed, pending files would contain
# set([('foo', 'bar', 'baz')])
self.pending_files = set()
# diff is a mapping of filename->set(linenos)
self.diff_data = defaultdict(set)
# store a list of filenames that should be accepted
self.file_cache = FileAcceptedCache(self.prefixes, self.pending_files, self.diff_data)
def begin(self):
# XXX: this is pretty hacky
if self.verbosity > 1:
self.logger.info("Parsing parent commit..")
proc = Popen(['git', 'merge-base', 'HEAD', self.parent], stdout=PIPE, stderr=STDOUT)
self.parent_revision = proc.stdout.read().strip()
if self.verbosity > 1:
self.logger.info("Parent commit identified as %s", self.parent_revision)
# pull in our diff
# git diff `git merge-base HEAD master`
if self.verbosity > 1:
self.logger.info("Parsing diff..")
proc = Popen(['git', 'diff', self.parent_revision], stdout=PIPE, stderr=STDOUT)
diff = proc.stdout.read().strip()
parser = DiffParser(diff)
files = list(parser.parse())
diff = self.diff_data
for file in files:
# we dont care about headers
if file['is_header']:
continue
# file was removed
if file['new_filename'] == '/dev/null':
continue
is_new_file = (file['old_filename'] == '/dev/null')
if is_new_file:
filename = file['new_filename']
if not filename.startswith('b/'):
continue
else:
filename = file['old_filename']
if not filename.startswith('a/'):
continue # ??
filename = filename[2:]
if self.file_cache.root is None:
self.file_cache.root = os.path.abspath(filename)[:-len(filename)]
# Ignore non python files
if not is_py_script(filename):
continue
new_filename = file['new_filename'][2:]
# file is new, only record diff state
for chunk in file['chunks']:
linenos = filter(bool, (l['new_lineno'] for l in chunk if l['line']))
diff[new_filename].update(linenos)
if is_new_file:
continue
for prefix in self.prefixes:
self.pending_files.add(os.path.join(prefix, new_filename.rsplit('.', 1)[0]))
if self.verbosity > 1:
self.logger.info("Found %d changed file(s) and %d possible test paths..", len(diff), len(self.pending_files))
def wantMethod(self, method):
# only works with unittest compatible functions currently
method = getattr(sys.modules[method.im_class.__module__], method.im_class.__name__)
# check if this test was modified (e.g. added/changed)
filename = inspect.getfile(method)
return self.file_cache[filename]