/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

  • Committer: John Arbash Meinel
  • Date: 2006-10-16 04:38:20 UTC
  • mto: This revision was merged to the branch mainline in revision 2080.
  • Revision ID: john@arbash-meinel.com-20061016043820-ff374b87ac84e2df
Add an entry about copyright to HACKING

Show diffs side-by-side

added added

removed removed

Lines of Context:
 
1
# Copyright (C) 2005 Canonical Ltd
 
2
#   Authors: Robert Collins <robert.collins@canonical.com>
 
3
#
 
4
# This program is free software; you can redistribute it and/or modify
 
5
# it under the terms of the GNU General Public License as published by
 
6
# the Free Software Foundation; either version 2 of the License, or
 
7
# (at your option) any later version.
 
8
#
 
9
# This program is distributed in the hope that it will be useful,
 
10
# but WITHOUT ANY WARRANTY; without even the implied warranty of
 
11
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 
12
# GNU General Public License for more details.
 
13
#
 
14
# You should have received a copy of the GNU General Public License
 
15
# along with this program; if not, write to the Free Software
 
16
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
17
 
 
18
"""These tests are tests about the source code of bzrlib itself.
 
19
 
 
20
They are useful for testing code quality, checking coverage metric etc.
 
21
"""
 
22
 
 
23
# import system imports here
 
24
import os
 
25
import re
 
26
import sys
 
27
 
 
28
#import bzrlib specific imports here
 
29
from bzrlib import (
 
30
    osutils,
 
31
    )
 
32
import bzrlib.branch
 
33
from bzrlib.tests import TestCase, TestSkipped
 
34
 
 
35
 
 
36
# Files which are listed here will be skipped when testing for Copyright (or
 
37
# GPL) statements.
 
38
COPYRIGHT_EXCEPTIONS = ['bzrlib/lsprof.py']
 
39
 
 
40
LICENSE_EXCEPTIONS = ['bzrlib/lsprof.py']
 
41
# Technically, 'bzrlib/lsprof.py' should be 'bzrlib/util/lsprof.py',
 
42
# (we do not check bzrlib/util/, since that is code bundled from elsewhere)
 
43
# but for compatibility with previous releases, we don't want to move it.
 
44
 
 
45
 
 
46
class TestSourceHelper(TestCase):
 
47
 
 
48
    def source_file_name(self, package):
 
49
        """Return the path of the .py file for package."""
 
50
        path = package.__file__
 
51
        if path[-1] in 'co':
 
52
            return path[:-1]
 
53
        else:
 
54
            return path
 
55
 
 
56
 
 
57
class TestApiUsage(TestSourceHelper):
 
58
 
 
59
    def find_occurences(self, rule, filename):
 
60
        """Find the number of occurences of rule in a file."""
 
61
        occurences = 0
 
62
        source = file(filename, 'r')
 
63
        for line in source:
 
64
            if line.find(rule) > -1:
 
65
                occurences += 1
 
66
        return occurences
 
67
 
 
68
    def test_branch_working_tree(self):
 
69
        """Test that the number of uses of working_tree in branch is stable."""
 
70
        occurences = self.find_occurences('self.working_tree()',
 
71
                                          self.source_file_name(bzrlib.branch))
 
72
        # do not even think of increasing this number. If you think you need to
 
73
        # increase it, then you almost certainly are doing something wrong as
 
74
        # the relationship from working_tree to branch is one way.
 
75
        # Note that this is an exact equality so that when the number drops, 
 
76
        #it is not given a buffer but rather has this test updated immediately.
 
77
        self.assertEqual(0, occurences)
 
78
 
 
79
    def test_branch_WorkingTree(self):
 
80
        """Test that the number of uses of working_tree in branch is stable."""
 
81
        occurences = self.find_occurences('WorkingTree',
 
82
                                          self.source_file_name(bzrlib.branch))
 
83
        # do not even think of increasing this number. If you think you need to
 
84
        # increase it, then you almost certainly are doing something wrong as
 
85
        # the relationship from working_tree to branch is one way.
 
86
        # This number should be 4 (import NoWorkingTree and WorkingTree, 
 
87
        # raise NoWorkingTree from working_tree(), and construct a working tree
 
88
        # there) but a merge that regressed this was done before this test was
 
89
        # written. Note that this is an exact equality so that when the number
 
90
        # drops, it is not given a buffer but rather this test updated
 
91
        # immediately.
 
92
        self.assertEqual(2, occurences)
 
93
 
 
94
 
 
95
class TestSource(TestSourceHelper):
 
96
 
 
97
    def get_bzrlib_dir(self):
 
98
        """Get the path to the root of bzrlib"""
 
99
        source = self.source_file_name(bzrlib)
 
100
        source_dir = os.path.dirname(source)
 
101
 
 
102
        # Avoid the case when bzrlib is packaged in a zip file
 
103
        if not os.path.isdir(source_dir):
 
104
            raise TestSkipped('Cannot find bzrlib source directory. Expected %s'
 
105
                              % source_dir)
 
106
        return source_dir
 
107
 
 
108
    def get_source_files(self):
 
109
        """yield all source files for bzr and bzrlib"""
 
110
        bzrlib_dir = self.get_bzrlib_dir()
 
111
 
 
112
        # This is the front-end 'bzr' script
 
113
        bzr_path = self.get_bzr_path()
 
114
        yield bzr_path
 
115
 
 
116
        for root, dirs, files in os.walk(bzrlib_dir):
 
117
            for f in files:
 
118
                if not f.endswith('.py'):
 
119
                    continue
 
120
                yield osutils.pathjoin(root, f)
 
121
 
 
122
    def get_source_file_contents(self):
 
123
        for fname in self.get_source_files():
 
124
            f = open(fname, 'rb')
 
125
            try:
 
126
                text = f.read()
 
127
            finally:
 
128
                f.close()
 
129
            yield fname, text
 
130
 
 
131
    def is_copyright_exception(self, fname):
 
132
        """Certain files are allowed to be different"""
 
133
        if '/util/' in fname:
 
134
            # We don't require external utilities to be (C) Canonical Ltd
 
135
            return True
 
136
 
 
137
        for exc in COPYRIGHT_EXCEPTIONS:
 
138
            if fname.endswith(exc):
 
139
                return True
 
140
 
 
141
        return False
 
142
 
 
143
    def is_license_exception(self, fname):
 
144
        """Certain files are allowed to be different"""
 
145
        if '/util/' in fname:
 
146
            # We don't require external utilities to be (C) Canonical Ltd
 
147
            return True
 
148
 
 
149
        for exc in LICENSE_EXCEPTIONS:
 
150
            if fname.endswith(exc):
 
151
                return True
 
152
 
 
153
        return False
 
154
 
 
155
    def test_copyright(self):
 
156
        """Test that all .py files have a valid copyright statement"""
 
157
        # These are files which contain a different copyright statement
 
158
        # and that is okay.
 
159
        incorrect = []
 
160
 
 
161
        copyright_re = re.compile('#\\s*copyright.*(?=\n)', re.I)
 
162
        copyright_canonical_re = re.compile(
 
163
            r'# Copyright \(C\) ' # Opening "# Copyright (C)"
 
164
            r'(\d+)(, \d+)*' # Followed by a series of dates
 
165
            r'.*Canonical Ltd' # And containing 'Canonical Ltd'
 
166
            )
 
167
 
 
168
        for fname, text in self.get_source_file_contents():
 
169
            if self.is_copyright_exception(fname):
 
170
                continue
 
171
            match = copyright_canonical_re.search(text)
 
172
            if not match:
 
173
                match = copyright_re.search(text)
 
174
                if match:
 
175
                    incorrect.append((fname, 'found: %s' % (match.group(),)))
 
176
                else:
 
177
                    incorrect.append((fname, 'no copyright line found\n'))
 
178
            else:
 
179
                if 'by Canonical' in match.group():
 
180
                    incorrect.append((fname,
 
181
                        'should not have: "by Canonical": %s'
 
182
                        % (match.group(),)))
 
183
 
 
184
        if incorrect:
 
185
            help_text = ["Some files have missing or incorrect copyright"
 
186
                         " statements.",
 
187
                         "",
 
188
                         "Please either add them to the list of"
 
189
                         " COPYRIGHT_EXCEPTIONS in"
 
190
                         " bzrlib/tests/test_source.py",
 
191
                         # this is broken to prevent a false match
 
192
                         "or add '# Copyright (C)"
 
193
                         " 2006 Canonical Ltd' to these files:",
 
194
                         "",
 
195
                        ]
 
196
            for fname, comment in incorrect:
 
197
                help_text.append(fname)
 
198
                help_text.append((' '*4) + comment)
 
199
 
 
200
            self.fail('\n'.join(help_text))
 
201
 
 
202
    def test_gpl(self):
 
203
        """Test that all .py files have a GPL disclaimer"""
 
204
        incorrect = []
 
205
 
 
206
        gpl_txt = """
 
207
# This program is free software; you can redistribute it and/or modify
 
208
# it under the terms of the GNU General Public License as published by
 
209
# the Free Software Foundation; either version 2 of the License, or
 
210
# (at your option) any later version.
 
211
#
 
212
# This program is distributed in the hope that it will be useful,
 
213
# but WITHOUT ANY WARRANTY; without even the implied warranty of
 
214
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 
215
# GNU General Public License for more details.
 
216
#
 
217
# You should have received a copy of the GNU General Public License
 
218
# along with this program; if not, write to the Free Software
 
219
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
220
"""
 
221
        gpl_re = re.compile(re.escape(gpl_txt), re.MULTILINE)
 
222
 
 
223
        for fname, text in self.get_source_file_contents():
 
224
            if self.is_license_exception(fname):
 
225
                continue
 
226
            if not gpl_re.search(text):
 
227
                incorrect.append(fname)
 
228
 
 
229
        if incorrect:
 
230
            help_text = ['Some files have missing or incomplete GPL statement',
 
231
                         "",
 
232
                         "Please either add them to the list of"
 
233
                         " LICENSE_EXCEPTIONS in"
 
234
                         " bzrlib/tests/test_source.py",
 
235
                         "Or add the following text to the beginning:",
 
236
                         gpl_txt
 
237
                        ]
 
238
            for fname in incorrect:
 
239
                help_text.append((' '*4) + fname)
 
240
 
 
241
            self.fail('\n'.join(help_text))