/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 breezy/tests/test_bugtracker.py

  • Committer: Jelmer Vernooij
  • Date: 2020-04-05 19:11:34 UTC
  • mto: (7490.7.16 work)
  • mto: This revision was merged to the branch mainline in revision 7501.
  • Revision ID: jelmer@jelmer.uk-20200405191134-0aebh8ikiwygxma5
Populate the .gitignore file.

Show diffs side-by-side

added added

removed removed

Lines of Context:
breezy/tests/test_bugtracker.py
1
 
# Copyright (C) 2007 Canonical Ltd
 
1
# Copyright (C) 2007-2010 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
15
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
16
 
17
17
 
18
 
from bzrlib import bugtracker, errors, urlutils
19
 
from bzrlib.tests import TestCase, TestCaseWithMemoryTransport
 
18
from .. import bugtracker, urlutils
 
19
from . import TestCase, TestCaseWithMemoryTransport
 
20
 
 
21
 
 
22
class ErrorsTest(TestCaseWithMemoryTransport):
 
23
 
 
24
    def test_unknown_bug_tracker_abbreviation(self):
 
25
        """Test the formatting of UnknownBugTrackerAbbreviation."""
 
26
        branch = self.make_branch('some_branch')
 
27
        error = bugtracker.UnknownBugTrackerAbbreviation('xxx', branch)
 
28
        self.assertEqual(
 
29
            "Cannot find registered bug tracker called xxx on %s" % branch,
 
30
            str(error))
 
31
 
 
32
    def test_malformed_bug_identifier(self):
 
33
        """Test the formatting of MalformedBugIdentifier."""
 
34
        error = bugtracker.MalformedBugIdentifier(
 
35
            'bogus', 'reason for bogosity')
 
36
        self.assertEqual(
 
37
            'Did not understand bug identifier bogus: reason for bogosity. '
 
38
            'See "brz help bugs" for more information on this feature.',
 
39
            str(error))
 
40
 
 
41
    def test_incorrect_url(self):
 
42
        err = bugtracker.InvalidBugTrackerURL('foo', 'http://bug.example.com/')
 
43
        self.assertEqual(
 
44
            ("The URL for bug tracker \"foo\" doesn't contain {id}: "
 
45
             "http://bug.example.com/"),
 
46
            str(err))
20
47
 
21
48
 
22
49
class TestGetBugURL(TestCaseWithMemoryTransport):
34
61
 
35
62
        def get_bug_url(self, bug_id):
36
63
            self.log.append(('get_bug_url', bug_id))
37
 
            return "http://bugs.com/%s" % bug_id
 
64
            return "http://bugs.example.com/%s" % bug_id
38
65
 
39
66
    def setUp(self):
40
 
        TestCaseWithMemoryTransport.setUp(self)
 
67
        super(TestGetBugURL, self).setUp()
41
68
        self.tracker_type = TestGetBugURL.TransientTracker
42
69
        self.tracker_type.log = []
43
70
        bugtracker.tracker_registry.register('transient', self.tracker_type)
45
72
 
46
73
    def test_get_bug_url_for_transient_tracker(self):
47
74
        branch = self.make_branch('some_branch')
48
 
        self.assertEqual('http://bugs.com/1234',
 
75
        self.assertEqual('http://bugs.example.com/1234',
49
76
                         bugtracker.get_bug_url('transient', branch, '1234'))
50
77
        self.assertEqual(
51
78
            [('get', 'transient', branch), ('get_bug_url', '1234')],
54
81
    def test_unrecognized_abbreviation_raises_error(self):
55
82
        """If the abbreviation is unrecognized, then raise an error."""
56
83
        branch = self.make_branch('some_branch')
57
 
        self.assertRaises(errors.UnknownBugTrackerAbbreviation,
 
84
        self.assertRaises(bugtracker.UnknownBugTrackerAbbreviation,
58
85
                          bugtracker.get_bug_url, 'xxx', branch, '1234')
59
86
        self.assertEqual([('get', 'xxx', branch)], self.tracker_type.log)
60
87
 
92
119
        """
93
120
        branch = self.make_branch('some_branch')
94
121
        config = branch.get_config()
95
 
        config.set_user_option('trac_foo_url', 'http://bugs.com/trac')
 
122
        config.set_user_option('trac_foo_url', 'http://bugs.example.com/trac')
96
123
        tracker = bugtracker.tracker_registry.get_tracker('foo', branch)
97
 
        self.assertEqual('http://bugs.com/trac/ticket/1234',
 
124
        self.assertEqual('http://bugs.example.com/trac/ticket/1234',
98
125
                         tracker.get_bug_url('1234'))
99
126
 
100
127
    def test_bugzilla_registered(self):
104
131
        """
105
132
        branch = self.make_branch('some_branch')
106
133
        config = branch.get_config()
107
 
        config.set_user_option('bugzilla_foo_url', 'http://bugs.com')
 
134
        config.set_user_option('bugzilla_foo_url', 'http://bugs.example.com')
108
135
        tracker = bugtracker.tracker_registry.get_tracker('foo', branch)
109
 
        self.assertEqual('http://bugs.com/show_bug.cgi?id=1234',
 
136
        self.assertEqual('http://bugs.example.com/show_bug.cgi?id=1234',
110
137
                         tracker.get_bug_url('1234'))
111
138
 
 
139
    def test_github(self):
 
140
        branch = self.make_branch('some_branch')
 
141
        tracker = bugtracker.tracker_registry.get_tracker('github', branch)
 
142
        self.assertEqual('https://github.com/breezy-team/breezy/issues/1234',
 
143
                         tracker.get_bug_url('breezy-team/breezy/1234'))
 
144
 
112
145
    def test_generic_registered(self):
113
146
        branch = self.make_branch('some_branch')
114
147
        config = branch.get_config()
115
 
        config.set_user_option('bugtracker_foo_url', 'http://bugs.com/{id}/view.html')
 
148
        config.set_user_option('bugtracker_foo_url',
 
149
                               'http://bugs.example.com/{id}/view.html')
116
150
        tracker = bugtracker.tracker_registry.get_tracker('foo', branch)
117
 
        self.assertEqual('http://bugs.com/1234/view.html',
 
151
        self.assertEqual('http://bugs.example.com/1234/view.html',
118
152
                         tracker.get_bug_url('1234'))
119
153
 
 
154
    def test_generic_registered_non_integer(self):
 
155
        branch = self.make_branch('some_branch')
 
156
        config = branch.get_config()
 
157
        config.set_user_option('bugtracker_foo_url',
 
158
                               'http://bugs.example.com/{id}/view.html')
 
159
        tracker = bugtracker.tracker_registry.get_tracker('foo', branch)
 
160
        self.assertEqual('http://bugs.example.com/ABC-1234/view.html',
 
161
                         tracker.get_bug_url('ABC-1234'))
 
162
 
120
163
    def test_generic_incorrect_url(self):
121
164
        branch = self.make_branch('some_branch')
122
165
        config = branch.get_config()
123
 
        config.set_user_option('bugtracker_foo_url', 'http://bugs.com/view.html')
 
166
        config.set_user_option('bugtracker_foo_url',
 
167
                               'http://bugs.example.com/view.html')
124
168
        tracker = bugtracker.tracker_registry.get_tracker('foo', branch)
125
 
        self.assertRaises(errors.InvalidBugTrackerURL, tracker.get_bug_url, '1234')
 
169
        self.assertRaises(bugtracker.InvalidBugTrackerURL, tracker.get_bug_url,
 
170
                          '1234')
126
171
 
127
172
 
128
173
class TestUniqueIntegerBugTracker(TestCaseWithMemoryTransport):
130
175
    def test_appends_id_to_base_url(self):
131
176
        """The URL of a bug is the base URL joined to the identifier."""
132
177
        tracker = bugtracker.UniqueIntegerBugTracker('xxx',
133
 
                'http://bugs.com/foo')
134
 
        self.assertEqual('http://bugs.com/foo1234', tracker.get_bug_url('1234'))
 
178
                                                     'http://bugs.example.com/foo')
 
179
        self.assertEqual('http://bugs.example.com/foo1234',
 
180
                         tracker.get_bug_url('1234'))
135
181
 
136
182
    def test_returns_tracker_if_abbreviation_matches(self):
137
183
        """The get() method should return an instance of the tracker if the
138
184
        given abbreviation matches the tracker's abbreviated name.
139
185
        """
140
186
        tracker = bugtracker.UniqueIntegerBugTracker('xxx',
141
 
                'http://bugs.com/')
 
187
                                                     'http://bugs.example.com/')
142
188
        branch = self.make_branch('some_branch')
143
189
        self.assertIs(tracker, tracker.get('xxx', branch))
144
190
 
147
193
        doesn't match the tracker's abbreviation.
148
194
        """
149
195
        tracker = bugtracker.UniqueIntegerBugTracker('xxx',
150
 
                'http://bugs.com/')
 
196
                                                     'http://bugs.example.com/')
151
197
        branch = self.make_branch('some_branch')
152
198
        self.assertIs(None, tracker.get('yyy', branch))
153
199
 
156
202
        information.
157
203
        """
158
204
        tracker = bugtracker.UniqueIntegerBugTracker('xxx',
159
 
                'http://bugs.com/')
 
205
                                                     'http://bugs.example.com/')
160
206
        self.assertIs(tracker, tracker.get('xxx', None))
161
207
        self.assertIs(None, tracker.get('yyy', None))
162
208
 
163
209
    def test_check_bug_id_only_accepts_integers(self):
164
210
        """A UniqueIntegerBugTracker accepts integers as bug IDs."""
165
211
        tracker = bugtracker.UniqueIntegerBugTracker('xxx',
166
 
                'http://bugs.com/')
 
212
                                                     'http://bugs.example.com/')
167
213
        tracker.check_bug_id('1234')
168
214
 
169
215
    def test_check_bug_id_doesnt_accept_non_integers(self):
170
216
        """A UniqueIntegerBugTracker rejects non-integers as bug IDs."""
171
217
        tracker = bugtracker.UniqueIntegerBugTracker('xxx',
172
 
                'http://bugs.com/')
173
 
        self.assertRaises(
174
 
            errors.MalformedBugIdentifier, tracker.check_bug_id, 'red')
175
 
 
176
 
 
177
 
class TestURLParametrizedIntegerBugTracker(TestCaseWithMemoryTransport):
178
 
    """Tests for TracTracker."""
 
218
                                                     'http://bugs.example.com/')
 
219
        self.assertRaises(
 
220
            bugtracker.MalformedBugIdentifier, tracker.check_bug_id, 'red')
 
221
 
 
222
 
 
223
class TestProjectIntegerBugTracker(TestCaseWithMemoryTransport):
 
224
 
 
225
    def test_appends_id_to_base_url(self):
 
226
        """The URL of a bug is the base URL joined to the identifier."""
 
227
        tracker = bugtracker.ProjectIntegerBugTracker('xxx',
 
228
                                                      'http://bugs.example.com/{project}/{id}')
 
229
        self.assertEqual('http://bugs.example.com/foo/1234',
 
230
                         tracker.get_bug_url('foo/1234'))
 
231
 
 
232
    def test_returns_tracker_if_abbreviation_matches(self):
 
233
        """The get() method should return an instance of the tracker if the
 
234
        given abbreviation matches the tracker's abbreviated name.
 
235
        """
 
236
        tracker = bugtracker.ProjectIntegerBugTracker('xxx',
 
237
                                                      'http://bugs.example.com/{project}/{id}')
 
238
        branch = self.make_branch('some_branch')
 
239
        self.assertIs(tracker, tracker.get('xxx', branch))
 
240
 
 
241
    def test_returns_none_if_abbreviation_doesnt_match(self):
 
242
        """The get() method should return None if the given abbreviated name
 
243
        doesn't match the tracker's abbreviation.
 
244
        """
 
245
        tracker = bugtracker.ProjectIntegerBugTracker('xxx',
 
246
                                                      'http://bugs.example.com/{project}/{id}')
 
247
        branch = self.make_branch('some_branch')
 
248
        self.assertIs(None, tracker.get('yyy', branch))
 
249
 
 
250
    def test_doesnt_consult_branch(self):
 
251
        """Shouldn't consult the branch for tracker information.
 
252
        """
 
253
        tracker = bugtracker.ProjectIntegerBugTracker('xxx',
 
254
                                                      'http://bugs.example.com/{project}/{id}')
 
255
        self.assertIs(tracker, tracker.get('xxx', None))
 
256
        self.assertIs(None, tracker.get('yyy', None))
 
257
 
 
258
    def test_check_bug_id_only_accepts_project_integers(self):
 
259
        """Accepts integers as bug IDs."""
 
260
        tracker = bugtracker.ProjectIntegerBugTracker('xxx',
 
261
                                                      'http://bugs.example.com/{project}/{id}')
 
262
        tracker.check_bug_id('project/1234')
 
263
 
 
264
    def test_check_bug_id_doesnt_accept_non_project_integers(self):
 
265
        """Rejects non-integers as bug IDs."""
 
266
        tracker = bugtracker.ProjectIntegerBugTracker('xxx',
 
267
                                                      'http://bugs.example.com/{project}/{id}')
 
268
        self.assertRaises(
 
269
            bugtracker.MalformedBugIdentifier, tracker.check_bug_id, 'red')
 
270
        self.assertRaises(
 
271
            bugtracker.MalformedBugIdentifier, tracker.check_bug_id, '1234')
 
272
 
 
273
 
 
274
class TestURLParametrizedBugTracker(TestCaseWithMemoryTransport):
 
275
    """Tests for URLParametrizedBugTracker."""
179
276
 
180
277
    def setUp(self):
181
 
        TestCaseWithMemoryTransport.setUp(self)
 
278
        super(TestURLParametrizedBugTracker, self).setUp()
182
279
        self.url = 'http://twistedmatrix.com/trac'
183
 
        self.tracker = bugtracker.URLParametrizedIntegerBugTracker('some',
184
 
                                                                   'ticket/')
 
280
        self.tracker = bugtracker.URLParametrizedBugTracker('some', 'ticket/')
185
281
 
186
282
    def test_get_with_unsupported_tag(self):
187
283
        """If asked for an unrecognized or unconfigured tag, return None."""
203
299
            urlutils.join(self.url, 'ticket/') + '1234',
204
300
            tracker.get_bug_url('1234'))
205
301
 
 
302
    def test_get_bug_url_for_integer_id(self):
 
303
        self.tracker.check_bug_id('1234')
 
304
 
 
305
    def test_get_bug_url_for_non_integer_id(self):
 
306
        self.tracker.check_bug_id('ABC-1234')
 
307
 
 
308
 
 
309
class TestURLParametrizedIntegerBugTracker(TestCaseWithMemoryTransport):
 
310
    """Tests for URLParametrizedIntegerBugTracker."""
 
311
 
 
312
    def setUp(self):
 
313
        super(TestURLParametrizedIntegerBugTracker, self).setUp()
 
314
        self.url = 'http://twistedmatrix.com/trac'
 
315
        self.tracker = bugtracker.URLParametrizedIntegerBugTracker('some',
 
316
                                                                   'ticket/')
 
317
 
206
318
    def test_get_bug_url_for_bad_bug(self):
207
319
        """When given a bug identifier that is invalid for Trac, get_bug_url
208
320
        should raise an error.
209
321
        """
210
322
        self.assertRaises(
211
 
            errors.MalformedBugIdentifier, self.tracker.get_bug_url, 'bad')
 
323
            bugtracker.MalformedBugIdentifier, self.tracker.get_bug_url, 'bad')
212
324
 
213
325
 
214
326
class TestPropertyEncoding(TestCase):
217
329
    def test_encoding_one(self):
218
330
        self.assertEqual(
219
331
            'http://example.com/bugs/1 fixed',
220
 
            bugtracker.encode_fixes_bug_urls(['http://example.com/bugs/1']))
 
332
            bugtracker.encode_fixes_bug_urls(
 
333
                [('http://example.com/bugs/1', 'fixed')]))
221
334
 
222
335
    def test_encoding_zero(self):
223
336
        self.assertEqual('', bugtracker.encode_fixes_bug_urls([]))
225
338
    def test_encoding_two(self):
226
339
        self.assertEqual(
227
340
            'http://example.com/bugs/1 fixed\n'
228
 
            'http://example.com/bugs/2 fixed',
 
341
            'http://example.com/bugs/2 related',
229
342
            bugtracker.encode_fixes_bug_urls(
230
 
                ['http://example.com/bugs/1', 'http://example.com/bugs/2']))
 
343
                [('http://example.com/bugs/1', 'fixed'),
 
344
                 ('http://example.com/bugs/2', 'related')]))
 
345
 
 
346
    def test_encoding_with_space(self):
 
347
        self.assertRaises(
 
348
            bugtracker.InvalidBugUrl,
 
349
            bugtracker.encode_fixes_bug_urls,
 
350
            [('http://example.com/bugs/ 1', 'fixed')])
 
351
 
 
352
 
 
353
class TestPropertyDecoding(TestCase):
 
354
    """Tests for parsing bug revision properties."""
 
355
 
 
356
    def test_decoding_one(self):
 
357
        self.assertEqual(
 
358
            [('http://example.com/bugs/1', 'fixed')],
 
359
            list(bugtracker.decode_bug_urls(
 
360
                'http://example.com/bugs/1 fixed')))
 
361
 
 
362
    def test_decoding_zero(self):
 
363
        self.assertEqual([], list(bugtracker.decode_bug_urls('')))
 
364
 
 
365
    def test_decoding_two(self):
 
366
        self.assertEqual(
 
367
            [('http://example.com/bugs/1', 'fixed'),
 
368
             ('http://example.com/bugs/2', 'related')],
 
369
            list(bugtracker.decode_bug_urls(
 
370
                'http://example.com/bugs/1 fixed\n'
 
371
                'http://example.com/bugs/2 related')))
 
372
 
 
373
    def test_decoding_invalid(self):
 
374
        self.assertRaises(
 
375
            bugtracker.InvalidLineInBugsProperty,
 
376
            list,
 
377
            bugtracker.decode_bug_urls('http://example.com/bugs/ 1 fixed\n'))