/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_source.py

Prepare test framework for pyftpdlib injection.

* bzrlib/tests/ftp_server/__init__.py:
Provide a single front-end for ftp test servers.

* bzrlib/tests/test_ftp_transport.py: 
Use ftp_server import.

* bzrlib/transport/ftp/__init__.py:
(get_test_permutations): Simplified.

* bzrlib/tests/__init__.py:
(FTPServerFeature): Moved to bzrlib/tests/ftp_server/__init__.py.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/test_source.py
1
 
# Copyright (C) 2005-2010 Canonical Ltd
 
1
# Copyright (C) 2005, 2006, 2008 Canonical Ltd
 
2
#   Authors: Robert Collins <robert.collins@canonical.com>
 
3
#            and others
2
4
#
3
5
# This program is free software; you can redistribute it and/or modify
4
6
# it under the terms of the GNU General Public License as published by
12
14
#
13
15
# You should have received a copy of the GNU General Public License
14
16
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
17
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
18
 
17
19
"""These tests are tests about the source code of bzrlib itself.
18
20
 
40
42
 
41
43
# Files which are listed here will be skipped when testing for Copyright (or
42
44
# GPL) statements.
43
 
COPYRIGHT_EXCEPTIONS = ['bzrlib/lsprof.py', 'bzrlib/_bencode_py.py',
44
 
    'bzrlib/doc_generate/sphinx_conf.py']
 
45
COPYRIGHT_EXCEPTIONS = ['bzrlib/lsprof.py']
45
46
 
46
 
LICENSE_EXCEPTIONS = ['bzrlib/lsprof.py', 'bzrlib/_bencode_py.py',
47
 
    'bzrlib/doc_generate/sphinx_conf.py']
 
47
LICENSE_EXCEPTIONS = ['bzrlib/lsprof.py']
48
48
# Technically, 'bzrlib/lsprof.py' should be 'bzrlib/util/lsprof.py',
49
49
# (we do not check bzrlib/util/, since that is code bundled from elsewhere)
50
50
# but for compatibility with previous releases, we don't want to move it.
51
 
#
52
 
# sphinx_conf is semi-autogenerated.
53
51
 
54
52
 
55
53
class TestSourceHelper(TestCase):
111
109
                              % source_dir)
112
110
        return source_dir
113
111
 
114
 
    def get_source_files(self, extensions=None):
 
112
    def get_source_files(self):
115
113
        """Yield all source files for bzr and bzrlib
116
114
 
117
115
        :param our_files_only: If true, exclude files from included libraries
118
116
            or plugins.
119
117
        """
120
118
        bzrlib_dir = self.get_bzrlib_dir()
121
 
        if extensions is None:
122
 
            extensions = ('.py',)
123
119
 
124
120
        # This is the front-end 'bzr' script
125
121
        bzr_path = self.get_bzr_path()
130
126
                if d.endswith('.tmp'):
131
127
                    dirs.remove(d)
132
128
            for f in files:
133
 
                for extension in extensions:
134
 
                    if f.endswith(extension):
135
 
                        break
136
 
                else:
137
 
                    # Did not match the accepted extensions
 
129
                if not f.endswith('.py'):
138
130
                    continue
139
131
                yield osutils.pathjoin(root, f)
140
132
 
141
 
    def get_source_file_contents(self, extensions=None):
142
 
        for fname in self.get_source_files(extensions=extensions):
 
133
    def get_source_file_contents(self):
 
134
        for fname in self.get_source_files():
143
135
            f = open(fname, 'rb')
144
136
            try:
145
137
                text = f.read()
183
175
                          % filename)
184
176
 
185
177
    def test_copyright(self):
186
 
        """Test that all .py and .pyx files have a valid copyright statement"""
 
178
        """Test that all .py files have a valid copyright statement"""
 
179
        # These are files which contain a different copyright statement
 
180
        # and that is okay.
187
181
        incorrect = []
188
182
 
189
183
        copyright_re = re.compile('#\\s*copyright.*(?=\n)', re.I)
193
187
            r'.*Canonical Ltd' # And containing 'Canonical Ltd'
194
188
            )
195
189
 
196
 
        for fname, text in self.get_source_file_contents(
197
 
                extensions=('.py', '.pyx')):
 
190
        for fname, text in self.get_source_file_contents():
198
191
            if self.is_copyright_exception(fname):
199
192
                continue
200
193
            match = copyright_canonical_re.search(text)
229
222
            self.fail('\n'.join(help_text))
230
223
 
231
224
    def test_gpl(self):
232
 
        """Test that all .py and .pyx files have a GPL disclaimer."""
 
225
        """Test that all .py files have a GPL disclaimer"""
233
226
        incorrect = []
234
227
 
235
228
        gpl_txt = """
245
238
#
246
239
# You should have received a copy of the GNU General Public License
247
240
# along with this program; if not, write to the Free Software
248
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
241
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
249
242
"""
250
243
        gpl_re = re.compile(re.escape(gpl_txt), re.MULTILINE)
251
244
 
252
 
        for fname, text in self.get_source_file_contents(
253
 
                extensions=('.py', '.pyx')):
 
245
        for fname, text in self.get_source_file_contents():
254
246
            if self.is_license_exception(fname):
255
247
                continue
256
248
            if not gpl_re.search(text):
285
277
    def test_coding_style(self):
286
278
        """Check if bazaar code conforms to some coding style conventions.
287
279
 
288
 
        Currently we assert that the following is not present:
 
280
        Currently we check for:
289
281
         * any tab characters
 
282
         * trailing white space
290
283
         * non-unix newlines
291
284
         * no newline at end of files
292
 
 
293
 
        Print how many files have
294
 
         * trailing white space
295
285
         * lines longer than 79 chars
 
286
           (only print how many files and lines are in violation)
296
287
        """
297
288
        tabs = {}
298
289
        trailing_ws = {}
299
290
        illegal_newlines = {}
300
291
        long_lines = {}
301
292
        no_newline_at_eof = []
302
 
        for fname, text in self.get_source_file_contents(
303
 
                extensions=('.py', '.pyx')):
 
293
        for fname, text in self.get_source_file_contents():
304
294
            if not self.is_our_code(fname):
305
295
                continue
306
296
            lines = text.splitlines(True)
323
313
                'Tab characters were found in the following source files.'
324
314
                '\nThey should either be replaced by "\\t" or by spaces:'))
325
315
        if trailing_ws:
326
 
            print ("There are %i lines with trailing white space in %i files."
327
 
                % (sum([len(lines) for f, lines in trailing_ws.items()]),
328
 
                    len(trailing_ws)))
 
316
            problems.append(self._format_message(trailing_ws,
 
317
                'Trailing white space was found in the following source files:'
 
318
                ))
329
319
        if illegal_newlines:
330
320
            problems.append(self._format_message(illegal_newlines,
331
321
                'Non-unix newlines were found in the following source files:'))
359
349
                    return True
360
350
            return False
361
351
        badfiles = []
362
 
        assert_re = re.compile(r'\bassert\b')
363
352
        for fname, text in self.get_source_file_contents():
364
353
            if not self.is_our_code(fname):
365
354
                continue
366
 
            if not assert_re.search(text):
367
 
                continue
368
 
            ast = parser.ast2tuple(parser.suite(text))
 
355
            ast = parser.ast2tuple(parser.suite(''.join(text)))
369
356
            if search(ast):
370
357
                badfiles.append(fname)
371
358
        if badfiles:
372
359
            self.fail(
373
360
                "these files contain an assert statement and should not:\n%s"
374
361
                % '\n'.join(badfiles))
375
 
 
376
 
    def test_extension_exceptions(self):
377
 
        """Extension functions should propagate exceptions.
378
 
 
379
 
        Either they should return an object, have an 'except' clause, or have a
380
 
        "# cannot_raise" to indicate that we've audited them and defined them as not
381
 
        raising exceptions.
382
 
        """
383
 
        both_exc_and_no_exc = []
384
 
        missing_except = []
385
 
        class_re = re.compile(r'^(cdef\s+)?(public\s+)?(api\s+)?class (\w+).*:',
386
 
                              re.MULTILINE)
387
 
        except_re = re.compile(r'cdef\s+' # start with cdef
388
 
                               r'([\w *]*?)\s*' # this is the return signature
389
 
                               r'(\w+)\s*\(' # the function name
390
 
                               r'[^)]*\)\s*' # parameters
391
 
                               r'(.*)\s*:' # the except clause
392
 
                               r'\s*(#\s*cannot[- _]raise)?' # cannot raise comment
393
 
                              )
394
 
        for fname, text in self.get_source_file_contents(
395
 
                extensions=('.pyx',)):
396
 
            known_classes = set([m[-1] for m in class_re.findall(text)])
397
 
            cdefs = except_re.findall(text)
398
 
            for sig, func, exc_clause, no_exc_comment in cdefs:
399
 
                if sig.startswith('api '):
400
 
                    sig = sig[4:]
401
 
                if not sig or sig in known_classes:
402
 
                    sig = 'object'
403
 
                if 'nogil' in exc_clause:
404
 
                    exc_clause = exc_clause.replace('nogil', '').strip()
405
 
                if exc_clause and no_exc_comment:
406
 
                    both_exc_and_no_exc.append((fname, func))
407
 
                if sig != 'object' and not (exc_clause or no_exc_comment):
408
 
                    missing_except.append((fname, func))
409
 
        error_msg = []
410
 
        if both_exc_and_no_exc:
411
 
            error_msg.append('The following functions had "cannot raise" comments'
412
 
                             ' but did have an except clause set:')
413
 
            for fname, func in both_exc_and_no_exc:
414
 
                error_msg.append('%s:%s' % (fname, func))
415
 
            error_msg.extend(('', ''))
416
 
        if missing_except:
417
 
            error_msg.append('The following functions have fixed return types,'
418
 
                             ' but no except clause.')
419
 
            error_msg.append('Either add an except or append "# cannot_raise".')
420
 
            for fname, func in missing_except:
421
 
                error_msg.append('%s:%s' % (fname, func))
422
 
            error_msg.extend(('', ''))
423
 
        if error_msg:
424
 
            self.fail('\n'.join(error_msg))