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
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
|
# Copyright (C) 2007 Canonical Ltd
#
# This program is free software; you can redistribute it and/or modify
# it under the terms of the GNU General Public License as published by
# the Free Software Foundation; either version 2 of the License, or
# (at your option) any later version.
#
# This program is distributed in the hope that it will be useful,
# but WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU General Public License for more details.
#
# You should have received a copy of the GNU General Public License
# along with this program; if not, write to the Free Software
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
from bzrlib import bugtracker, errors, urlutils
from bzrlib.tests import TestCase, TestCaseWithMemoryTransport
class TestGetBugURL(TestCaseWithMemoryTransport):
"""Tests for bugtracker.get_bug_url"""
class TransientTracker(object):
"""An transient tracker used for testing."""
@classmethod
def get(klass, abbreviation, branch):
klass.log.append(('get', abbreviation, branch))
if abbreviation != 'transient':
return None
return klass()
def get_bug_url(self, bug_id):
self.log.append(('get_bug_url', bug_id))
return "http://bugs.com/%s" % bug_id
def setUp(self):
TestCaseWithMemoryTransport.setUp(self)
self.tracker_type = TestGetBugURL.TransientTracker
self.tracker_type.log = []
bugtracker.tracker_registry.register('transient', self.tracker_type)
self.addCleanup(bugtracker.tracker_registry.remove, 'transient')
def test_get_bug_url_for_transient_tracker(self):
branch = self.make_branch('some_branch')
self.assertEqual('http://bugs.com/1234',
bugtracker.get_bug_url('transient', branch, '1234'))
self.assertEqual(
[('get', 'transient', branch), ('get_bug_url', '1234')],
self.tracker_type.log)
def test_unrecognized_abbreviation_raises_error(self):
"""If the abbreviation is unrecognized, then raise an error."""
branch = self.make_branch('some_branch')
self.assertRaises(errors.UnknownBugTrackerAbbreviation,
bugtracker.get_bug_url, 'xxx', branch, '1234')
self.assertEqual([('get', 'xxx', branch)], self.tracker_type.log)
class TestBuiltinTrackers(TestCaseWithMemoryTransport):
"""Test that the builtin trackers are registered and return sane URLs."""
def test_launchpad_registered(self):
"""The Launchpad bug tracker should be registered by default and
generate Launchpad bug page URLs.
"""
branch = self.make_branch('some_branch')
tracker = bugtracker.tracker_registry.get_tracker('lp', branch)
self.assertEqual('https://launchpad.net/bugs/1234',
tracker.get_bug_url('1234'))
def test_debian_registered(self):
"""The Debian bug tracker should be registered by default and generate
bugs.debian.org bug page URLs.
"""
branch = self.make_branch('some_branch')
tracker = bugtracker.tracker_registry.get_tracker('deb', branch)
self.assertEqual('http://bugs.debian.org/1234',
tracker.get_bug_url('1234'))
def test_gnome_registered(self):
branch = self.make_branch('some_branch')
tracker = bugtracker.tracker_registry.get_tracker('gnome', branch)
self.assertEqual('http://bugzilla.gnome.org/show_bug.cgi?id=1234',
tracker.get_bug_url('1234'))
def test_trac_registered(self):
"""The Trac bug tracker should be registered by default and generate
Trac bug page URLs when the appropriate configuration is present.
"""
branch = self.make_branch('some_branch')
config = branch.get_config()
config.set_user_option('trac_foo_url', 'http://bugs.com/trac')
tracker = bugtracker.tracker_registry.get_tracker('foo', branch)
self.assertEqual('http://bugs.com/trac/ticket/1234',
tracker.get_bug_url('1234'))
def test_bugzilla_registered(self):
"""The Bugzilla bug tracker should be registered by default and
generate Bugzilla bug page URLs when the appropriate configuration is
present.
"""
branch = self.make_branch('some_branch')
config = branch.get_config()
config.set_user_option('bugzilla_foo_url', 'http://bugs.com')
tracker = bugtracker.tracker_registry.get_tracker('foo', branch)
self.assertEqual('http://bugs.com/show_bug.cgi?id=1234',
tracker.get_bug_url('1234'))
def test_generic_registered(self):
branch = self.make_branch('some_branch')
config = branch.get_config()
config.set_user_option('bugtracker_foo_url', 'http://bugs.com/{id}/view.html')
tracker = bugtracker.tracker_registry.get_tracker('foo', branch)
self.assertEqual('http://bugs.com/1234/view.html',
tracker.get_bug_url('1234'))
def test_generic_registered_non_integer(self):
branch = self.make_branch('some_branch')
config = branch.get_config()
config.set_user_option('bugtracker_foo_url', 'http://bugs.com/{id}/view.html')
tracker = bugtracker.tracker_registry.get_tracker('foo', branch)
self.assertEqual('http://bugs.com/ABC-1234/view.html',
tracker.get_bug_url('ABC-1234'))
def test_generic_incorrect_url(self):
branch = self.make_branch('some_branch')
config = branch.get_config()
config.set_user_option('bugtracker_foo_url', 'http://bugs.com/view.html')
tracker = bugtracker.tracker_registry.get_tracker('foo', branch)
self.assertRaises(errors.InvalidBugTrackerURL, tracker.get_bug_url, '1234')
class TestUniqueIntegerBugTracker(TestCaseWithMemoryTransport):
def test_appends_id_to_base_url(self):
"""The URL of a bug is the base URL joined to the identifier."""
tracker = bugtracker.UniqueIntegerBugTracker('xxx',
'http://bugs.com/foo')
self.assertEqual('http://bugs.com/foo1234', tracker.get_bug_url('1234'))
def test_returns_tracker_if_abbreviation_matches(self):
"""The get() method should return an instance of the tracker if the
given abbreviation matches the tracker's abbreviated name.
"""
tracker = bugtracker.UniqueIntegerBugTracker('xxx',
'http://bugs.com/')
branch = self.make_branch('some_branch')
self.assertIs(tracker, tracker.get('xxx', branch))
def test_returns_none_if_abbreviation_doesnt_match(self):
"""The get() method should return None if the given abbreviated name
doesn't match the tracker's abbreviation.
"""
tracker = bugtracker.UniqueIntegerBugTracker('xxx',
'http://bugs.com/')
branch = self.make_branch('some_branch')
self.assertIs(None, tracker.get('yyy', branch))
def test_doesnt_consult_branch(self):
"""A UniqueIntegerBugTracker shouldn't consult the branch for tracker
information.
"""
tracker = bugtracker.UniqueIntegerBugTracker('xxx',
'http://bugs.com/')
self.assertIs(tracker, tracker.get('xxx', None))
self.assertIs(None, tracker.get('yyy', None))
def test_check_bug_id_only_accepts_integers(self):
"""A UniqueIntegerBugTracker accepts integers as bug IDs."""
tracker = bugtracker.UniqueIntegerBugTracker('xxx',
'http://bugs.com/')
tracker.check_bug_id('1234')
def test_check_bug_id_doesnt_accept_non_integers(self):
"""A UniqueIntegerBugTracker rejects non-integers as bug IDs."""
tracker = bugtracker.UniqueIntegerBugTracker('xxx',
'http://bugs.com/')
self.assertRaises(
errors.MalformedBugIdentifier, tracker.check_bug_id, 'red')
class TestURLParametrizedBugTracker(TestCaseWithMemoryTransport):
"""Tests for URLParametrizedBugTracker."""
def setUp(self):
TestCaseWithMemoryTransport.setUp(self)
self.url = 'http://twistedmatrix.com/trac'
self.tracker = bugtracker.URLParametrizedBugTracker('some', 'ticket/')
def test_get_with_unsupported_tag(self):
"""If asked for an unrecognized or unconfigured tag, return None."""
branch = self.make_branch('some_branch')
self.assertEqual(None, self.tracker.get('lp', branch))
self.assertEqual(None, self.tracker.get('twisted', branch))
def test_get_with_supported_tag(self):
"""If asked for a valid tag, return a tracker instance that can map bug
IDs to <base_url>/<bug_area> + <bug_id>."""
bugtracker.tracker_registry.register('some', self.tracker)
self.addCleanup(bugtracker.tracker_registry.remove, 'some')
branch = self.make_branch('some_branch')
config = branch.get_config()
config.set_user_option('some_twisted_url', self.url)
tracker = self.tracker.get('twisted', branch)
self.assertEqual(
urlutils.join(self.url, 'ticket/') + '1234',
tracker.get_bug_url('1234'))
def test_get_bug_url_for_integer_id(self):
self.tracker.check_bug_id('1234')
def test_get_bug_url_for_non_integer_id(self):
self.tracker.check_bug_id('ABC-1234')
class TestURLParametrizedIntegerBugTracker(TestCaseWithMemoryTransport):
"""Tests for URLParametrizedIntegerBugTracker."""
def setUp(self):
TestCaseWithMemoryTransport.setUp(self)
self.url = 'http://twistedmatrix.com/trac'
self.tracker = bugtracker.URLParametrizedIntegerBugTracker('some',
'ticket/')
def test_get_bug_url_for_bad_bug(self):
"""When given a bug identifier that is invalid for Trac, get_bug_url
should raise an error.
"""
self.assertRaises(
errors.MalformedBugIdentifier, self.tracker.get_bug_url, 'bad')
class TestPropertyEncoding(TestCase):
"""Tests for how the bug URLs are encoded as revision properties."""
def test_encoding_one(self):
self.assertEqual(
'http://example.com/bugs/1 fixed',
bugtracker.encode_fixes_bug_urls(['http://example.com/bugs/1']))
def test_encoding_zero(self):
self.assertEqual('', bugtracker.encode_fixes_bug_urls([]))
def test_encoding_two(self):
self.assertEqual(
'http://example.com/bugs/1 fixed\n'
'http://example.com/bugs/2 fixed',
bugtracker.encode_fixes_bug_urls(
['http://example.com/bugs/1', 'http://example.com/bugs/2']))
|