/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 brzlib/tests/test_errors.py

  • Committer: Jelmer Vernooij
  • Date: 2017-05-21 12:41:27 UTC
  • mto: This revision was merged to the branch mainline in revision 6623.
  • Revision ID: jelmer@jelmer.uk-20170521124127-iv8etg0vwymyai6y
s/bzr/brz/ in apport config.

Show diffs side-by-side

added added

removed removed

Lines of Context:
21
21
import socket
22
22
import sys
23
23
 
24
 
from .. import (
 
24
from brzlib import (
25
25
    controldir,
26
26
    errors,
27
27
    osutils,
28
 
    tests,
29
28
    urlutils,
30
29
    )
31
 
 
32
 
 
33
 
class TestErrors(tests.TestCase):
 
30
from brzlib.tests import (
 
31
    TestCase,
 
32
    TestCaseWithTransport,
 
33
    TestSkipped,
 
34
    )
 
35
 
 
36
 
 
37
class TestErrors(TestCaseWithTransport):
34
38
 
35
39
    def test_no_arg_named_message(self):
36
40
        """Ensure the __init__ and _fmt in errors do not have "message" arg.
40
44
        Python 2.5 uses a slot for StandardError.message.
41
45
        See bug #603461
42
46
        """
43
 
        fmt_pattern = re.compile("%\\(message\\)[sir]")
 
47
        fmt_pattern = re.compile("%\(message\)[sir]")
 
48
        subclasses_present = getattr(errors.BzrError, '__subclasses__', None)
 
49
        if not subclasses_present:
 
50
            raise TestSkipped('__subclasses__ attribute required for classes. '
 
51
                'Requires Python 2.5 or later.')
44
52
        for c in errors.BzrError.__subclasses__():
45
53
            init = getattr(c, '__init__', None)
46
54
            fmt = getattr(c, '_fmt', None)
47
55
            if init:
48
 
                args = inspect.getfullargspec(init)[0]
 
56
                args = inspect.getargspec(init)[0]
49
57
                self.assertFalse('message' in args,
50
 
                                 ('Argument name "message" not allowed for '
51
 
                                  '"errors.%s.__init__"' % c.__name__))
 
58
                    ('Argument name "message" not allowed for '
 
59
                    '"errors.%s.__init__"' % c.__name__))
52
60
            if fmt and fmt_pattern.search(fmt):
53
61
                self.assertFalse(True, ('"message" not allowed in '
54
 
                                        '"errors.%s._fmt"' % c.__name__))
 
62
                    '"errors.%s._fmt"' % c.__name__))
55
63
 
56
64
    def test_bad_filename_encoding(self):
57
 
        error = errors.BadFilenameEncoding(b'bad/filen\xe5me', 'UTF-8')
58
 
        self.assertContainsRe(
59
 
            str(error),
60
 
            "^Filename b?'bad/filen\\\\xe5me' is not valid in your current"
61
 
            " filesystem encoding UTF-8$")
 
65
        error = errors.BadFilenameEncoding('bad/filen\xe5me', 'UTF-8')
 
66
        self.assertEqualDiff(
 
67
            "Filename 'bad/filen\\xe5me' is not valid in your current"
 
68
            " filesystem encoding UTF-8",
 
69
            str(error))
 
70
 
 
71
    def test_corrupt_dirstate(self):
 
72
        error = errors.CorruptDirstate('path/to/dirstate', 'the reason why')
 
73
        self.assertEqualDiff(
 
74
            "Inconsistency in dirstate file path/to/dirstate.\n"
 
75
            "Error: the reason why",
 
76
            str(error))
 
77
 
 
78
    def test_dirstate_corrupt(self):
 
79
        error = errors.DirstateCorrupt('.bzr/checkout/dirstate',
 
80
                                       'trailing garbage: "x"')
 
81
        self.assertEqualDiff("The dirstate file (.bzr/checkout/dirstate)"
 
82
            " appears to be corrupt: trailing garbage: \"x\"",
 
83
            str(error))
 
84
 
 
85
    def test_disabled_method(self):
 
86
        error = errors.DisabledMethod("class name")
 
87
        self.assertEqualDiff(
 
88
            "The smart server method 'class name' is disabled.", str(error))
62
89
 
63
90
    def test_duplicate_file_id(self):
64
91
        error = errors.DuplicateFileId('a_file_id', 'foo')
68
95
    def test_duplicate_help_prefix(self):
69
96
        error = errors.DuplicateHelpPrefix('foo')
70
97
        self.assertEqualDiff('The prefix foo is in the help search path twice.',
71
 
                             str(error))
 
98
            str(error))
72
99
 
73
100
    def test_ghost_revisions_have_no_revno(self):
74
101
        error = errors.GhostRevisionsHaveNoRevno('target', 'ghost_rev')
76
103
                             " its ancestry shows a ghost at {ghost_rev}",
77
104
                             str(error))
78
105
 
79
 
    def test_incompatibleVersion(self):
80
 
        error = errors.IncompatibleVersion("module", [(4, 5, 6), (7, 8, 9)],
81
 
                                           (1, 2, 3))
 
106
    def test_incompatibleAPI(self):
 
107
        error = errors.IncompatibleAPI("module", (1, 2, 3), (4, 5, 6), (7, 8, 9))
82
108
        self.assertEqualDiff(
83
 
            'API module is not compatible; one of versions '
84
 
            '[(4, 5, 6), (7, 8, 9)] is required, but current version is '
85
 
            '(1, 2, 3).',
 
109
            'The API for "module" is not compatible with "(1, 2, 3)". '
 
110
            'It supports versions "(4, 5, 6)" to "(7, 8, 9)".',
86
111
            str(error))
87
112
 
88
113
    def test_inconsistent_delta(self):
121
146
    def test_inventory_modified(self):
122
147
        error = errors.InventoryModified("a tree to be repred")
123
148
        self.assertEqualDiff("The current inventory for the tree 'a tree to "
124
 
                             "be repred' has been modified, so a clean inventory cannot be "
125
 
                             "read without data loss.",
126
 
                             str(error))
 
149
            "be repred' has been modified, so a clean inventory cannot be "
 
150
            "read without data loss.",
 
151
            str(error))
127
152
 
128
153
    def test_jail_break(self):
129
154
        error = errors.JailBreak("some url")
130
155
        self.assertEqualDiff("An attempt to access a url outside the server"
131
 
                             " jail was made: 'some url'.",
132
 
                             str(error))
 
156
            " jail was made: 'some url'.",
 
157
            str(error))
133
158
 
134
159
    def test_lock_active(self):
135
160
        error = errors.LockActive("lock description")
136
161
        self.assertEqualDiff("The lock for 'lock description' is in use and "
137
 
                             "cannot be broken.",
138
 
                             str(error))
 
162
            "cannot be broken.",
 
163
            str(error))
139
164
 
140
165
    def test_lock_corrupt(self):
141
166
        error = errors.LockCorrupt("corruption info")
142
167
        self.assertEqualDiff("Lock is apparently held, but corrupted: "
143
 
                             "corruption info\n"
144
 
                             "Use 'brz break-lock' to clear it",
145
 
                             str(error))
 
168
            "corruption info\n"
 
169
            "Use 'bzr break-lock' to clear it",
 
170
            str(error))
 
171
 
 
172
    def test_knit_data_stream_incompatible(self):
 
173
        error = errors.KnitDataStreamIncompatible(
 
174
            'stream format', 'target format')
 
175
        self.assertEqual('Cannot insert knit data stream of format '
 
176
                         '"stream format" into knit of format '
 
177
                         '"target format".', str(error))
 
178
 
 
179
    def test_knit_data_stream_unknown(self):
 
180
        error = errors.KnitDataStreamUnknown(
 
181
            'stream format')
 
182
        self.assertEqual('Cannot parse knit data stream of format '
 
183
                         '"stream format".', str(error))
 
184
 
 
185
    def test_knit_header_error(self):
 
186
        error = errors.KnitHeaderError('line foo\n', 'path/to/file')
 
187
        self.assertEqual("Knit header error: 'line foo\\n' unexpected"
 
188
                         " for file \"path/to/file\".", str(error))
 
189
 
 
190
    def test_knit_index_unknown_method(self):
 
191
        error = errors.KnitIndexUnknownMethod('http://host/foo.kndx',
 
192
                                              ['bad', 'no-eol'])
 
193
        self.assertEqual("Knit index http://host/foo.kndx does not have a"
 
194
                         " known method in options: ['bad', 'no-eol']",
 
195
                         str(error))
146
196
 
147
197
    def test_medium_not_connected(self):
148
198
        error = errors.MediumNotConnected("a medium")
149
199
        self.assertEqualDiff(
150
200
            "The medium 'a medium' is not connected.", str(error))
151
201
 
 
202
    def test_no_public_branch(self):
 
203
        b = self.make_branch('.')
 
204
        error = errors.NoPublicBranch(b)
 
205
        url = urlutils.unescape_for_display(b.base, 'ascii')
 
206
        self.assertEqualDiff(
 
207
            'There is no public branch set for "%s".' % url, str(error))
 
208
 
 
209
    def test_no_repo(self):
 
210
        dir = controldir.ControlDir.create(self.get_url())
 
211
        error = errors.NoRepositoryPresent(dir)
 
212
        self.assertNotEqual(-1, str(error).find((dir.transport.clone('..').base)))
 
213
        self.assertEqual(-1, str(error).find((dir.transport.base)))
 
214
 
152
215
    def test_no_smart_medium(self):
153
216
        error = errors.NoSmartMedium("a transport")
154
217
        self.assertEqualDiff("The transport 'a transport' cannot tunnel the "
155
 
                             "smart protocol.",
156
 
                             str(error))
 
218
            "smart protocol.",
 
219
            str(error))
 
220
 
 
221
    def test_no_help_topic(self):
 
222
        error = errors.NoHelpTopic("topic")
 
223
        self.assertEqualDiff("No help could be found for 'topic'. "
 
224
            "Please use 'bzr help topics' to obtain a list of topics.",
 
225
            str(error))
157
226
 
158
227
    def test_no_such_id(self):
159
228
        error = errors.NoSuchId("atree", "anid")
160
229
        self.assertEqualDiff("The file id \"anid\" is not present in the tree "
161
 
                             "atree.",
162
 
                             str(error))
 
230
            "atree.",
 
231
            str(error))
163
232
 
164
233
    def test_no_such_revision_in_tree(self):
165
234
        error = errors.NoSuchRevisionInTree("atree", "anid")
170
239
    def test_not_stacked(self):
171
240
        error = errors.NotStacked('a branch')
172
241
        self.assertEqualDiff("The branch 'a branch' is not stacked.",
173
 
                             str(error))
 
242
            str(error))
174
243
 
175
244
    def test_not_write_locked(self):
176
245
        error = errors.NotWriteLocked('a thing to repr')
177
246
        self.assertEqualDiff("'a thing to repr' is not write locked but needs "
178
 
                             "to be.",
179
 
                             str(error))
 
247
            "to be.",
 
248
            str(error))
180
249
 
181
250
    def test_lock_failed(self):
182
 
        error = errors.LockFailed(
183
 
            'http://canonical.com/', 'readonly transport')
 
251
        error = errors.LockFailed('http://canonical.com/', 'readonly transport')
184
252
        self.assertEqualDiff("Cannot lock http://canonical.com/: readonly transport",
185
 
                             str(error))
 
253
            str(error))
186
254
        self.assertFalse(error.internal_error)
187
255
 
188
256
    def test_too_many_concurrent_requests(self):
189
257
        error = errors.TooManyConcurrentRequests("a medium")
190
258
        self.assertEqualDiff("The medium 'a medium' has reached its concurrent "
191
 
                             "request limit. Be sure to finish_writing and finish_reading on "
192
 
                             "the currently open request.",
193
 
                             str(error))
 
259
            "request limit. Be sure to finish_writing and finish_reading on "
 
260
            "the currently open request.",
 
261
            str(error))
194
262
 
195
263
    def test_unavailable_representation(self):
196
 
        error = errors.UnavailableRepresentation(
197
 
            ('key',), "mpdiff", "fulltext")
 
264
        error = errors.UnavailableRepresentation(('key',), "mpdiff", "fulltext")
198
265
        self.assertEqualDiff("The encoding 'mpdiff' is not available for key "
199
 
                             "('key',) which is encoded as 'fulltext'.",
200
 
                             str(error))
 
266
            "('key',) which is encoded as 'fulltext'.",
 
267
            str(error))
 
268
 
 
269
    def test_unknown_hook(self):
 
270
        error = errors.UnknownHook("branch", "foo")
 
271
        self.assertEqualDiff("The branch hook 'foo' is unknown in this version"
 
272
            " of brzlib.",
 
273
            str(error))
 
274
        error = errors.UnknownHook("tree", "bar")
 
275
        self.assertEqualDiff("The tree hook 'bar' is unknown in this version"
 
276
            " of brzlib.",
 
277
            str(error))
 
278
 
 
279
    def test_unstackable_branch_format(self):
 
280
        format = u'foo'
 
281
        url = "/foo"
 
282
        error = errors.UnstackableBranchFormat(format, url)
 
283
        self.assertEqualDiff(
 
284
            "The branch '/foo'(foo) is not a stackable format. "
 
285
            "You will need to upgrade the branch to permit branch stacking.",
 
286
            str(error))
201
287
 
202
288
    def test_unstackable_location(self):
203
289
        error = errors.UnstackableLocationError('foo', 'bar')
204
290
        self.assertEqualDiff("The branch 'foo' cannot be stacked on 'bar'.",
205
 
                             str(error))
 
291
            str(error))
206
292
 
207
293
    def test_unstackable_repository_format(self):
208
294
        format = u'foo'
219
305
            "The branch format someformat is already at the most "
220
306
            "recent format.", str(error))
221
307
 
 
308
    def test_corrupt_repository(self):
 
309
        repo = self.make_repository('.')
 
310
        error = errors.CorruptRepository(repo)
 
311
        self.assertEqualDiff("An error has been detected in the repository %s.\n"
 
312
                             "Please run bzr reconcile on this repository." %
 
313
                             repo.bzrdir.root_transport.base,
 
314
                             str(error))
 
315
 
222
316
    def test_read_error(self):
223
317
        # a unicode path to check that %r is being used.
224
318
        path = u'a path'
225
319
        error = errors.ReadError(path)
226
 
        self.assertContainsRe(str(error), "^Error reading from u?'a path'.$")
 
320
        self.assertEqualDiff("Error reading from u'a path'.", str(error))
 
321
 
 
322
    def test_bad_index_format_signature(self):
 
323
        error = errors.BadIndexFormatSignature("foo", "bar")
 
324
        self.assertEqual("foo is not an index of type bar.",
 
325
            str(error))
 
326
 
 
327
    def test_bad_index_data(self):
 
328
        error = errors.BadIndexData("foo")
 
329
        self.assertEqual("Error in data for index foo.",
 
330
            str(error))
 
331
 
 
332
    def test_bad_index_duplicate_key(self):
 
333
        error = errors.BadIndexDuplicateKey("foo", "bar")
 
334
        self.assertEqual("The key 'foo' is already in index 'bar'.",
 
335
            str(error))
 
336
 
 
337
    def test_bad_index_key(self):
 
338
        error = errors.BadIndexKey("foo")
 
339
        self.assertEqual("The key 'foo' is not a valid key.",
 
340
            str(error))
 
341
 
 
342
    def test_bad_index_options(self):
 
343
        error = errors.BadIndexOptions("foo")
 
344
        self.assertEqual("Could not parse options for index foo.",
 
345
            str(error))
 
346
 
 
347
    def test_bad_index_value(self):
 
348
        error = errors.BadIndexValue("foo")
 
349
        self.assertEqual("The value 'foo' is not a valid value.",
 
350
            str(error))
227
351
 
228
352
    def test_bzrerror_from_literal_string(self):
229
353
        # Some code constructs BzrError from a literal string, in which case
232
356
        # perhaps no more is needed.)
233
357
        try:
234
358
            raise errors.BzrError('this is my errors; %d is not expanded')
235
 
        except errors.BzrError as e:
 
359
        except errors.BzrError, e:
236
360
            self.assertEqual('this is my errors; %d is not expanded', str(e))
237
361
 
238
362
    def test_reading_completed(self):
239
363
        error = errors.ReadingCompleted("a request")
240
364
        self.assertEqualDiff("The MediumRequest 'a request' has already had "
241
 
                             "finish_reading called upon it - the request has been completed and"
242
 
                             " no more data may be read.",
243
 
                             str(error))
 
365
            "finish_reading called upon it - the request has been completed and"
 
366
            " no more data may be read.",
 
367
            str(error))
244
368
 
245
369
    def test_writing_completed(self):
246
370
        error = errors.WritingCompleted("a request")
247
371
        self.assertEqualDiff("The MediumRequest 'a request' has already had "
248
 
                             "finish_writing called upon it - accept bytes may not be called "
249
 
                             "anymore.",
250
 
                             str(error))
 
372
            "finish_writing called upon it - accept bytes may not be called "
 
373
            "anymore.",
 
374
            str(error))
251
375
 
252
376
    def test_writing_not_completed(self):
253
377
        error = errors.WritingNotComplete("a request")
254
378
        self.assertEqualDiff("The MediumRequest 'a request' has not has "
255
 
                             "finish_writing called upon it - until the write phase is complete"
256
 
                             " no data may be read.",
257
 
                             str(error))
 
379
            "finish_writing called upon it - until the write phase is complete"
 
380
            " no data may be read.",
 
381
            str(error))
258
382
 
259
383
    def test_transport_not_possible(self):
260
384
        error = errors.TransportNotPossible('readonly', 'original error')
261
385
        self.assertEqualDiff('Transport operation not possible:'
262
 
                             ' readonly original error', str(error))
 
386
                         ' readonly original error', str(error))
263
387
 
264
388
    def assertSocketConnectionError(self, expected, *args, **kwargs):
265
389
        """Check the formatting of a SocketConnectionError exception"""
313
437
            "location specified in the merge directive is not a branch: "
314
438
            "foo.", str(error))
315
439
 
 
440
    def test_malformed_bug_identifier(self):
 
441
        """Test the formatting of MalformedBugIdentifier."""
 
442
        error = errors.MalformedBugIdentifier('bogus', 'reason for bogosity')
 
443
        self.assertEqual(
 
444
            'Did not understand bug identifier bogus: reason for bogosity. '
 
445
            'See "bzr help bugs" for more information on this feature.',
 
446
            str(error))
 
447
 
 
448
    def test_unknown_bug_tracker_abbreviation(self):
 
449
        """Test the formatting of UnknownBugTrackerAbbreviation."""
 
450
        branch = self.make_branch('some_branch')
 
451
        error = errors.UnknownBugTrackerAbbreviation('xxx', branch)
 
452
        self.assertEqual(
 
453
            "Cannot find registered bug tracker called xxx on %s" % branch,
 
454
            str(error))
 
455
 
316
456
    def test_unexpected_smart_server_response(self):
317
457
        e = errors.UnexpectedSmartServerResponse(('not yes',))
318
458
        self.assertEqual(
355
495
 
356
496
    def test_duplicate_record_name_error(self):
357
497
        """Test the formatting of DuplicateRecordNameError."""
358
 
        e = errors.DuplicateRecordNameError(b"n\xc3\xa5me")
 
498
        e = errors.DuplicateRecordNameError(u"n\xe5me".encode('utf-8'))
359
499
        self.assertEqual(
360
 
            u"Container has multiple records with the same name: n\xe5me",
 
500
            "Container has multiple records with the same name: n\xc3\xa5me",
361
501
            str(e))
362
502
 
363
503
    def test_check_error(self):
 
504
        # This has a member called 'message', which is problematic in
 
505
        # python2.5 because that is a slot on the base Exception class
364
506
        e = errors.BzrCheckError('example check failure')
365
507
        self.assertEqual(
366
508
            "Internal check failed: example check failure",
381
523
            "you wish to keep, and delete it when you are done.",
382
524
            str(err))
383
525
 
 
526
    def test_unable_create_symlink(self):
 
527
        err = errors.UnableCreateSymlink()
 
528
        self.assertEqual(
 
529
            "Unable to create symlink on this platform",
 
530
            str(err))
 
531
        err = errors.UnableCreateSymlink(path=u'foo')
 
532
        self.assertEqual(
 
533
            "Unable to create symlink 'foo' on this platform",
 
534
            str(err))
 
535
        err = errors.UnableCreateSymlink(path=u'\xb5')
 
536
        self.assertEqual(
 
537
            "Unable to create symlink u'\\xb5' on this platform",
 
538
            str(err))
 
539
 
384
540
    def test_invalid_url_join(self):
385
541
        """Test the formatting of InvalidURLJoin."""
386
 
        e = urlutils.InvalidURLJoin('Reason', 'base path', ('args',))
 
542
        e = errors.InvalidURLJoin('Reason', 'base path', ('args',))
387
543
        self.assertEqual(
388
544
            "Invalid URL join request: Reason: 'base path' + ('args',)",
389
545
            str(e))
390
546
 
 
547
    def test_incorrect_url(self):
 
548
        err = errors.InvalidBugTrackerURL('foo', 'http://bug.com/')
 
549
        self.assertEqual(
 
550
            ("The URL for bug tracker \"foo\" doesn't contain {id}: "
 
551
             "http://bug.com/"),
 
552
            str(err))
 
553
 
391
554
    def test_unable_encode_path(self):
392
555
        err = errors.UnableEncodePath('foo', 'executable')
393
556
        self.assertEqual("Unable to encode executable path 'foo' in "
398
561
        err = errors.UnknownFormatError('bar', kind='foo')
399
562
        self.assertEqual("Unknown foo format: 'bar'", str(err))
400
563
 
 
564
    def test_unknown_rules(self):
 
565
        err = errors.UnknownRules(['foo', 'bar'])
 
566
        self.assertEqual("Unknown rules detected: foo, bar.", str(err))
 
567
 
401
568
    def test_tip_change_rejected(self):
402
569
        err = errors.TipChangeRejected(u'Unicode message\N{INTERROBANG}')
403
570
        self.assertEqual(
404
571
            u'Tip change rejected: Unicode message\N{INTERROBANG}',
 
572
            unicode(err))
 
573
        self.assertEqual(
 
574
            'Tip change rejected: Unicode message\xe2\x80\xbd',
405
575
            str(err))
406
576
 
407
577
    def test_error_from_smart_server(self):
431
601
        finally:
432
602
            del err
433
603
 
 
604
    def test_must_have_working_tree(self):
 
605
        err = errors.MustHaveWorkingTree('foo', 'bar')
 
606
        self.assertEqual(str(err), "Branching 'bar'(foo) must create a"
 
607
                                   " working tree.")
 
608
 
 
609
    def test_no_such_view(self):
 
610
        err = errors.NoSuchView('foo')
 
611
        self.assertEqual("No such view: foo.", str(err))
 
612
 
 
613
    def test_views_not_supported(self):
 
614
        err = errors.ViewsNotSupported('atree')
 
615
        err_str = str(err)
 
616
        self.assertStartsWith(err_str, "Views are not supported by ")
 
617
        self.assertEndsWith(err_str, "; use 'bzr upgrade' to change your "
 
618
            "tree to a later format.")
 
619
 
 
620
    def test_file_outside_view(self):
 
621
        err = errors.FileOutsideView('baz', ['foo', 'bar'])
 
622
        self.assertEqual('Specified file "baz" is outside the current view: '
 
623
                         'foo, bar', str(err))
 
624
 
 
625
    def test_invalid_shelf_id(self):
 
626
        invalid_id = "foo"
 
627
        err = errors.InvalidShelfId(invalid_id)
 
628
        self.assertEqual('"foo" is not a valid shelf id, '
 
629
                         'try a number instead.', str(err))
 
630
 
434
631
    def test_unresumable_write_group(self):
435
632
        repo = "dummy repo"
436
633
        wg_tokens = ['token']
450
647
        err = errors.NotBranchError('path')
451
648
        self.assertEqual('Not a branch: "path".', str(err))
452
649
 
 
650
    def test_not_branch_bzrdir_with_repo(self):
 
651
        bzrdir = self.make_repository('repo').bzrdir
 
652
        err = errors.NotBranchError('path', bzrdir=bzrdir)
 
653
        self.assertEqual(
 
654
            'Not a branch: "path": location is a repository.', str(err))
 
655
 
 
656
    def test_not_branch_bzrdir_without_repo(self):
 
657
        bzrdir = self.make_bzrdir('bzrdir')
 
658
        err = errors.NotBranchError('path', bzrdir=bzrdir)
 
659
        self.assertEqual('Not a branch: "path".', str(err))
 
660
 
453
661
    def test_not_branch_bzrdir_with_recursive_not_branch_error(self):
454
662
        class FakeBzrDir(object):
455
663
            def open_repository(self):
456
664
                # str() on the NotBranchError will trigger a call to this,
457
665
                # which in turn will another, identical NotBranchError.
458
 
                raise errors.NotBranchError('path', controldir=FakeBzrDir())
459
 
        err = errors.NotBranchError('path', controldir=FakeBzrDir())
460
 
        self.assertEqual('Not a branch: "path": NotBranchError.', str(err))
 
666
                raise errors.NotBranchError('path', bzrdir=FakeBzrDir())
 
667
        err = errors.NotBranchError('path', bzrdir=FakeBzrDir())
 
668
        self.assertEqual('Not a branch: "path".', str(err))
 
669
 
 
670
    def test_not_branch_laziness(self):
 
671
        real_bzrdir = self.make_bzrdir('path')
 
672
        class FakeBzrDir(object):
 
673
            def __init__(self):
 
674
                self.calls = []
 
675
            def open_repository(self):
 
676
                self.calls.append('open_repository')
 
677
                raise errors.NoRepositoryPresent(real_bzrdir)
 
678
        fake_bzrdir = FakeBzrDir()
 
679
        err = errors.NotBranchError('path', bzrdir=fake_bzrdir)
 
680
        self.assertEqual([], fake_bzrdir.calls)
 
681
        str(err)
 
682
        self.assertEqual(['open_repository'], fake_bzrdir.calls)
 
683
        # Stringifying twice doesn't try to open a repository twice.
 
684
        str(err)
 
685
        self.assertEqual(['open_repository'], fake_bzrdir.calls)
 
686
 
 
687
    def test_invalid_pattern(self):
 
688
        error = errors.InvalidPattern('Bad pattern msg.')
 
689
        self.assertEqualDiff("Invalid pattern(s) found. Bad pattern msg.",
 
690
            str(error))
461
691
 
462
692
    def test_recursive_bind(self):
463
693
        error = errors.RecursiveBind('foo_bar_branch')
464
694
        msg = ('Branch "foo_bar_branch" appears to be bound to itself. '
465
 
               'Please use `brz unbind` to fix.')
 
695
            'Please use `bzr unbind` to fix.')
466
696
        self.assertEqualDiff(msg, str(error))
467
697
 
468
698
    def test_retry_with_new_packs(self):
491
721
    __doc__ = """This class has a docstring but no format string."""
492
722
 
493
723
 
494
 
class TestErrorFormatting(tests.TestCase):
 
724
class TestErrorFormatting(TestCase):
495
725
 
496
726
    def test_always_str(self):
497
727
        e = PassThroughError(u'\xb5', 'bar')
498
728
        self.assertIsInstance(e.__str__(), str)
499
 
        # In Python 2 str(foo) *must* return a real byte string
 
729
        # In Python str(foo) *must* return a real byte string
500
730
        # not a Unicode string. The following line would raise a
501
731
        # Unicode error, because it tries to call str() on the string
502
732
        # returned from e.__str__(), and it has non ascii characters
503
733
        s = str(e)
504
 
        self.assertEqual('Pass through \xb5 and bar', s)
 
734
        self.assertEqual('Pass through \xc2\xb5 and bar', s)
505
735
 
506
736
    def test_missing_format_string(self):
507
737
        e = ErrorWithNoFormat(param='randomvalue')
524
754
            str(e),
525
755
            r'Cannot bind address "example\.com:22":.*Permission denied')
526
756
 
 
757
    def test_file_timestamp_unavailable(self):
 
758
        e = errors.FileTimestampUnavailable("/path/foo")
 
759
        self.assertEqual("The filestamp for /path/foo is not available.",
 
760
                         str(e))
 
761
 
527
762
    def test_transform_rename_failed(self):
528
763
        e = errors.TransformRenameFailed(u"from", u"to", "readonly file", 2)
529
764
        self.assertEqual(
530
765
            u"Failed to rename from to to: readonly file",
531
766
            str(e))
532
 
 
533
 
 
534
 
class TestErrorsUsingTransport(tests.TestCaseWithMemoryTransport):
535
 
    """Tests for errors that need to use a branch or repo."""
536
 
 
537
 
    def test_no_public_branch(self):
538
 
        b = self.make_branch('.')
539
 
        error = errors.NoPublicBranch(b)
540
 
        url = urlutils.unescape_for_display(b.base, 'ascii')
541
 
        self.assertEqualDiff(
542
 
            'There is no public branch set for "%s".' % url, str(error))
543
 
 
544
 
    def test_no_repo(self):
545
 
        dir = controldir.ControlDir.create(self.get_url())
546
 
        error = errors.NoRepositoryPresent(dir)
547
 
        self.assertNotEqual(-1,
548
 
                            str(error).find((dir.transport.clone('..').base)))
549
 
        self.assertEqual(-1, str(error).find((dir.transport.base)))
550
 
 
551
 
    def test_corrupt_repository(self):
552
 
        repo = self.make_repository('.')
553
 
        error = errors.CorruptRepository(repo)
554
 
        self.assertEqualDiff("An error has been detected in the repository %s.\n"
555
 
                             "Please run brz reconcile on this repository." %
556
 
                             repo.controldir.root_transport.base,
557
 
                             str(error))
558
 
 
559
 
    def test_not_branch_bzrdir_with_repo(self):
560
 
        controldir = self.make_repository('repo').controldir
561
 
        err = errors.NotBranchError('path', controldir=controldir)
562
 
        self.assertEqual(
563
 
            'Not a branch: "path": location is a repository.', str(err))
564
 
 
565
 
    def test_not_branch_bzrdir_without_repo(self):
566
 
        controldir = self.make_controldir('bzrdir')
567
 
        err = errors.NotBranchError('path', controldir=controldir)
568
 
        self.assertEqual('Not a branch: "path".', str(err))
569
 
 
570
 
    def test_not_branch_laziness(self):
571
 
        real_bzrdir = self.make_controldir('path')
572
 
 
573
 
        class FakeBzrDir(object):
574
 
            def __init__(self):
575
 
                self.calls = []
576
 
 
577
 
            def open_repository(self):
578
 
                self.calls.append('open_repository')
579
 
                raise errors.NoRepositoryPresent(real_bzrdir)
580
 
        fake_bzrdir = FakeBzrDir()
581
 
        err = errors.NotBranchError('path', controldir=fake_bzrdir)
582
 
        self.assertEqual([], fake_bzrdir.calls)
583
 
        str(err)
584
 
        self.assertEqual(['open_repository'], fake_bzrdir.calls)
585
 
        # Stringifying twice doesn't try to open a repository twice.
586
 
        str(err)
587
 
        self.assertEqual(['open_repository'], fake_bzrdir.calls)