~bzr-pqm/bzr/bzr.dev

3221.1.3 by Martin Pool
Review cleanups for CorruptDirstate: use the path everywhere rather than the object, and use more standard phrasing.
1
# Copyright (C) 2006, 2007, 2008 Canonical Ltd
1534.4.50 by Robert Collins
Got the bzrdir api straightened out, plenty of refactoring to use it pending, but the api is up and running.
2
#   Authors: Robert Collins <robert.collins@canonical.com>
2018.18.22 by Martin Pool
merge bzr.dev
3
#            and others
1534.4.50 by Robert Collins
Got the bzrdir api straightened out, plenty of refactoring to use it pending, but the api is up and running.
4
#
5
# This program is free software; you can redistribute it and/or modify
6
# it under the terms of the GNU General Public License as published by
7
# the Free Software Foundation; either version 2 of the License, or
8
# (at your option) any later version.
9
#
10
# This program is distributed in the hope that it will be useful,
11
# but WITHOUT ANY WARRANTY; without even the implied warranty of
12
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
13
# GNU General Public License for more details.
14
#
15
# You should have received a copy of the GNU General Public License
16
# along with this program; if not, write to the Free Software
17
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
18
19
"""Tests for the formatting and construction of errors."""
20
1948.1.6 by John Arbash Meinel
Make BzrNewError always return a str object
21
from bzrlib import (
22
    bzrdir,
23
    errors,
3234.2.6 by Alexander Belchenko
because every mail client has different rules to compose command line we should encode arguments to 8 bit string only when needed.
24
    osutils,
2872.5.1 by Martin Pool
Avoid internal error tracebacks on failure to lock on readonly transport (#129701).
25
    symbol_versioning,
3200.2.1 by Robert Collins
* The ``register-branch`` command will now use the public url of the branch
26
    urlutils,
1948.1.6 by John Arbash Meinel
Make BzrNewError always return a str object
27
    )
28
from bzrlib.tests import TestCase, TestCaseWithTransport
1534.4.50 by Robert Collins
Got the bzrdir api straightened out, plenty of refactoring to use it pending, but the api is up and running.
29
30
31
class TestErrors(TestCaseWithTransport):
32
3287.20.2 by John Arbash Meinel
Raise a clear error about the offending filename when there is a filename with bad characters.
33
    def test_bad_filename_encoding(self):
34
        error = errors.BadFilenameEncoding('bad/filen\xe5me', 'UTF-8')
35
        self.assertEqualDiff(
36
            "Filename 'bad/filen\\xe5me' is not valid in your current"
37
            " filesystem encoding UTF-8",
38
            str(error))
39
3207.2.1 by jameinel
Add a test that _iter_changes raises a clearer error when we encounter an invalid rename.
40
    def test_corrupt_dirstate(self):
41
        error = errors.CorruptDirstate('path/to/dirstate', 'the reason why')
3221.1.3 by Martin Pool
Review cleanups for CorruptDirstate: use the path everywhere rather than the object, and use more standard phrasing.
42
        self.assertEqualDiff(
43
            "Inconsistency in dirstate file path/to/dirstate.\n"
44
            "Error: the reason why",
45
            str(error))
3207.2.1 by jameinel
Add a test that _iter_changes raises a clearer error when we encounter an invalid rename.
46
2018.5.24 by Andrew Bennetts
Setting NO_SMART_VFS in environment will disable VFS methods in the smart server. (Robert Collins, John Arbash Meinel, Andrew Bennetts)
47
    def test_disabled_method(self):
48
        error = errors.DisabledMethod("class name")
49
        self.assertEqualDiff(
50
            "The smart server method 'class name' is disabled.", str(error))
51
2255.7.16 by John Arbash Meinel
Make sure adding a duplicate file_id raises DuplicateFileId.
52
    def test_duplicate_file_id(self):
53
        error = errors.DuplicateFileId('a_file_id', 'foo')
54
        self.assertEqualDiff('File id {a_file_id} already exists in inventory'
55
                             ' as foo', str(error))
56
2432.1.19 by Robert Collins
Ensure each HelpIndex has a unique prefix.
57
    def test_duplicate_help_prefix(self):
58
        error = errors.DuplicateHelpPrefix('foo')
59
        self.assertEqualDiff('The prefix foo is in the help search path twice.',
60
            str(error))
61
3445.1.1 by John Arbash Meinel
Start working on a new Graph api to make finding revision numbers faster.
62
    def test_ghost_revisions_have_no_revno(self):
63
        error = errors.GhostRevisionsHaveNoRevno('target', 'ghost_rev')
64
        self.assertEqualDiff("Could not determine revno for {target} because"
65
                             " its ancestry shows a ghost at {ghost_rev}",
66
                             str(error))
67
2550.2.3 by Robert Collins
Add require_api API.
68
    def test_incompatibleAPI(self):
69
        error = errors.IncompatibleAPI("module", (1, 2, 3), (4, 5, 6), (7, 8, 9))
70
        self.assertEqualDiff(
71
            'The API for "module" is not compatible with "(1, 2, 3)". '
72
            'It supports versions "(4, 5, 6)" to "(7, 8, 9)".',
73
            str(error))
74
3207.2.2 by John Arbash Meinel
Fix bug #187169, when an invalid delta is supplied to update_basis_by_delta
75
    def test_inconsistent_delta(self):
76
        error = errors.InconsistentDelta('path', 'file-id', 'reason for foo')
77
        self.assertEqualDiff(
3221.1.8 by Martin Pool
Update error format in test_inconsistent_delta
78
            "An inconsistent delta was supplied involving 'path', 'file-id'\n"
3207.2.2 by John Arbash Meinel
Fix bug #187169, when an invalid delta is supplied to update_basis_by_delta
79
            "reason: reason for foo",
80
            str(error))
81
2634.1.1 by Robert Collins
(robertc) Reinstate the accidentally backed out external_url patch.
82
    def test_in_process_transport(self):
83
        error = errors.InProcessTransport('fpp')
84
        self.assertEqualDiff(
85
            "The transport 'fpp' is only accessible within this process.",
86
            str(error))
87
3059.2.12 by Vincent Ladeuil
Spiv review feedback.
88
    def test_invalid_http_range(self):
89
        error = errors.InvalidHttpRange('path',
90
                                        'Content-Range: potatoes 0-00/o0oo0',
91
                                        'bad range')
92
        self.assertEquals("Invalid http range"
93
                          " 'Content-Range: potatoes 0-00/o0oo0'"
94
                          " for path: bad range",
95
                          str(error))
96
97
    def test_invalid_range(self):
98
        error = errors.InvalidRange('path', 12, 'bad range')
99
        self.assertEquals("Invalid range access in path at 12: bad range",
100
                          str(error))
101
1986.5.3 by Robert Collins
New method ``WorkingTree.flush()`` which will write the current memory
102
    def test_inventory_modified(self):
103
        error = errors.InventoryModified("a tree to be repred")
104
        self.assertEqualDiff("The current inventory for the tree 'a tree to "
105
            "be repred' has been modified, so a clean inventory cannot be "
106
            "read without data loss.",
107
            str(error))
108
2116.3.1 by John Arbash Meinel
Cleanup error tests
109
    def test_install_failed(self):
110
        error = errors.InstallFailed(['rev-one'])
111
        self.assertEqual("Could not install revisions:\nrev-one", str(error))
112
        error = errors.InstallFailed(['rev-one', 'rev-two'])
113
        self.assertEqual("Could not install revisions:\nrev-one, rev-two",
114
                         str(error))
115
        error = errors.InstallFailed([None])
116
        self.assertEqual("Could not install revisions:\nNone", str(error))
117
2255.2.145 by Robert Collins
Support unbreakable locks for trees.
118
    def test_lock_active(self):
119
        error = errors.LockActive("lock description")
120
        self.assertEqualDiff("The lock for 'lock description' is in use and "
121
            "cannot be broken.",
122
            str(error))
123
2535.3.4 by Andrew Bennetts
Simple implementation of Knit.insert_data_stream.
124
    def test_knit_data_stream_incompatible(self):
125
        error = errors.KnitDataStreamIncompatible(
126
            'stream format', 'target format')
127
        self.assertEqual('Cannot insert knit data stream of format '
128
                         '"stream format" into knit of format '
129
                         '"target format".', str(error))
130
3052.2.1 by Robert Collins
Add a new KnitDataStreamUnknown error class for showing formats we can't understand.
131
    def test_knit_data_stream_unknown(self):
132
        error = errors.KnitDataStreamUnknown(
133
            'stream format')
134
        self.assertEqual('Cannot parse knit data stream of format '
135
                         '"stream format".', str(error))
136
2171.1.1 by John Arbash Meinel
Knit index files should ignore empty indexes rather than consider them corrupt.
137
    def test_knit_header_error(self):
138
        error = errors.KnitHeaderError('line foo\n', 'path/to/file')
139
        self.assertEqual("Knit header error: 'line foo\\n' unexpected"
2745.3.2 by Daniel Watkins
Updated tests to reflect new error text.
140
                         " for file \"path/to/file\".", str(error))
2171.1.1 by John Arbash Meinel
Knit index files should ignore empty indexes rather than consider them corrupt.
141
2196.2.5 by John Arbash Meinel
Add an exception class when the knit index storage method is unknown, and properly test for it
142
    def test_knit_index_unknown_method(self):
143
        error = errors.KnitIndexUnknownMethod('http://host/foo.kndx',
144
                                              ['bad', 'no-eol'])
145
        self.assertEqual("Knit index http://host/foo.kndx does not have a"
146
                         " known method in options: ['bad', 'no-eol']",
147
                         str(error))
148
2018.2.3 by Andrew Bennetts
Starting factoring out the smart server client "medium" from the protocol.
149
    def test_medium_not_connected(self):
150
        error = errors.MediumNotConnected("a medium")
151
        self.assertEqualDiff(
152
            "The medium 'a medium' is not connected.", str(error))
3200.2.1 by Robert Collins
* The ``register-branch`` command will now use the public url of the branch
153
 
154
    def test_no_public_branch(self):
155
        b = self.make_branch('.')
156
        error = errors.NoPublicBranch(b)
157
        url = urlutils.unescape_for_display(b.base, 'ascii')
158
        self.assertEqualDiff(
159
            'There is no public branch set for "%s".' % url, str(error))
160
1534.4.50 by Robert Collins
Got the bzrdir api straightened out, plenty of refactoring to use it pending, but the api is up and running.
161
    def test_no_repo(self):
162
        dir = bzrdir.BzrDir.create(self.get_url())
163
        error = errors.NoRepositoryPresent(dir)
1740.5.6 by Martin Pool
Clean up many exception classes.
164
        self.assertNotEqual(-1, str(error).find((dir.transport.clone('..').base)))
165
        self.assertEqual(-1, str(error).find((dir.transport.base)))
1988.2.1 by Robert Collins
WorkingTree has a new api ``unversion`` which allow the unversioning of
166
        
2018.2.3 by Andrew Bennetts
Starting factoring out the smart server client "medium" from the protocol.
167
    def test_no_smart_medium(self):
168
        error = errors.NoSmartMedium("a transport")
169
        self.assertEqualDiff("The transport 'a transport' cannot tunnel the "
170
            "smart protocol.",
171
            str(error))
172
2432.1.4 by Robert Collins
Add an explicit error for missing help topics.
173
    def test_no_help_topic(self):
174
        error = errors.NoHelpTopic("topic")
175
        self.assertEqualDiff("No help could be found for 'topic'. "
176
            "Please use 'bzr help topics' to obtain a list of topics.",
177
            str(error))
178
1988.2.1 by Robert Collins
WorkingTree has a new api ``unversion`` which allow the unversioning of
179
    def test_no_such_id(self):
180
        error = errors.NoSuchId("atree", "anid")
2745.3.2 by Daniel Watkins
Updated tests to reflect new error text.
181
        self.assertEqualDiff("The file id \"anid\" is not present in the tree "
2745.3.3 by Daniel Watkins
Changed to remove need for escaping of quotes.
182
            "atree.",
1988.2.1 by Robert Collins
WorkingTree has a new api ``unversion`` which allow the unversioning of
183
            str(error))
1534.5.7 by Robert Collins
Start factoring out the upgrade policy logic.
184
1908.11.1 by Robert Collins
Add a new method ``Tree.revision_tree`` which allows access to cached
185
    def test_no_such_revision_in_tree(self):
186
        error = errors.NoSuchRevisionInTree("atree", "anid")
2745.3.3 by Daniel Watkins
Changed to remove need for escaping of quotes.
187
        self.assertEqualDiff("The revision id {anid} is not present in the"
188
                             " tree atree.", str(error))
1908.11.1 by Robert Collins
Add a new method ``Tree.revision_tree`` which allows access to cached
189
        self.assertIsInstance(error, errors.NoSuchRevision)
190
3221.11.2 by Robert Collins
Create basic stackable branch facility.
191
    def test_not_stacked(self):
192
        error = errors.NotStacked('a branch')
193
        self.assertEqualDiff("The branch 'a branch' is not stacked.",
194
            str(error))
195
1986.5.3 by Robert Collins
New method ``WorkingTree.flush()`` which will write the current memory
196
    def test_not_write_locked(self):
197
        error = errors.NotWriteLocked('a thing to repr')
198
        self.assertEqualDiff("'a thing to repr' is not write locked but needs "
199
            "to be.",
200
            str(error))
201
2872.5.1 by Martin Pool
Avoid internal error tracebacks on failure to lock on readonly transport (#129701).
202
    def test_lock_failed(self):
203
        error = errors.LockFailed('http://canonical.com/', 'readonly transport')
204
        self.assertEqualDiff("Cannot lock http://canonical.com/: readonly transport",
205
            str(error))
206
        self.assertFalse(error.internal_error)
207
2018.2.4 by Robert Collins
separate out the client medium from the client encoding protocol for the smart server.
208
    def test_too_many_concurrent_requests(self):
209
        error = errors.TooManyConcurrentRequests("a medium")
210
        self.assertEqualDiff("The medium 'a medium' has reached its concurrent "
211
            "request limit. Be sure to finish_writing and finish_reading on "
2018.5.134 by Andrew Bennetts
Fix the TooManyConcurrentRequests error message.
212
            "the currently open request.",
2018.2.4 by Robert Collins
separate out the client medium from the client encoding protocol for the smart server.
213
            str(error))
214
3350.3.3 by Robert Collins
Functional get_record_stream interface tests covering full interface.
215
    def test_unavailable_representation(self):
216
        error = errors.UnavailableRepresentation(('key',), "mpdiff", "fulltext")
217
        self.assertEqualDiff("The encoding 'mpdiff' is not available for key "
218
            "('key',) which is encoded as 'fulltext'.",
219
            str(error))
220
2245.1.3 by Robert Collins
Add install_hook to the BranchHooks class as the official means for installing a hook.
221
    def test_unknown_hook(self):
222
        error = errors.UnknownHook("branch", "foo")
223
        self.assertEqualDiff("The branch hook 'foo' is unknown in this version"
224
            " of bzrlib.",
225
            str(error))
226
        error = errors.UnknownHook("tree", "bar")
227
        self.assertEqualDiff("The tree hook 'bar' is unknown in this version"
228
            " of bzrlib.",
229
            str(error))
230
3221.11.2 by Robert Collins
Create basic stackable branch facility.
231
    def test_unstackable_branch_format(self):
232
        format = u'foo'
233
        url = "/foo"
234
        error = errors.UnstackableBranchFormat(format, url)
235
        self.assertEqualDiff(
236
            "The branch '/foo'(foo) is not a stackable format. "
237
            "You will need to upgrade the branch to permit branch stacking.",
238
            str(error))
239
240
    def test_unstackable_repository_format(self):
241
        format = u'foo'
242
        url = "/foo"
243
        error = errors.UnstackableRepositoryFormat(format, url)
244
        self.assertEqualDiff(
245
            "The repository '/foo'(foo) is not a stackable format. "
246
            "You will need to upgrade the repository to permit branch stacking.",
247
            str(error))
248
1534.5.7 by Robert Collins
Start factoring out the upgrade policy logic.
249
    def test_up_to_date(self):
250
        error = errors.UpToDateFormat(bzrdir.BzrDirFormat4())
1534.5.9 by Robert Collins
Advise users running upgrade on a checkout to also run it on the branch.
251
        self.assertEqualDiff("The branch format Bazaar-NG branch, "
252
                             "format 0.0.4 is already at the most "
253
                             "recent format.",
254
                             str(error))
1570.1.13 by Robert Collins
Check for incorrect revision parentage in the weave during revision access.
255
256
    def test_corrupt_repository(self):
257
        repo = self.make_repository('.')
258
        error = errors.CorruptRepository(repo)
259
        self.assertEqualDiff("An error has been detected in the repository %s.\n"
260
                             "Please run bzr reconcile on this repository." %
261
                             repo.bzrdir.root_transport.base,
262
                             str(error))
1948.1.6 by John Arbash Meinel
Make BzrNewError always return a str object
263
2052.6.1 by Robert Collins
``Transport.get`` has had its interface made more clear for ease of use.
264
    def test_read_error(self):
265
        # a unicode path to check that %r is being used.
266
        path = u'a path'
267
        error = errors.ReadError(path)
268
        self.assertEqualDiff("Error reading from u'a path'.", str(error))
269
2592.1.7 by Robert Collins
A validate that goes boom.
270
    def test_bad_index_format_signature(self):
271
        error = errors.BadIndexFormatSignature("foo", "bar")
272
        self.assertEqual("foo is not an index of type bar.",
273
            str(error))
2052.6.2 by Robert Collins
Merge bzr.dev.
274
2592.1.11 by Robert Collins
Detect truncated indices.
275
    def test_bad_index_data(self):
276
        error = errors.BadIndexData("foo")
277
        self.assertEqual("Error in data for index foo.",
278
            str(error))
279
2592.1.15 by Robert Collins
Detect duplicate key insertion.
280
    def test_bad_index_duplicate_key(self):
281
        error = errors.BadIndexDuplicateKey("foo", "bar")
282
        self.assertEqual("The key 'foo' is already in index 'bar'.",
283
            str(error))
284
2592.1.12 by Robert Collins
Handle basic node adds.
285
    def test_bad_index_key(self):
286
        error = errors.BadIndexKey("foo")
287
        self.assertEqual("The key 'foo' is not a valid key.",
288
            str(error))
289
2592.1.10 by Robert Collins
Make validate detect node reference parsing errors.
290
    def test_bad_index_options(self):
291
        error = errors.BadIndexOptions("foo")
292
        self.assertEqual("Could not parse options for index foo.",
293
            str(error))
294
2592.1.12 by Robert Collins
Handle basic node adds.
295
    def test_bad_index_value(self):
296
        error = errors.BadIndexValue("foo")
297
        self.assertEqual("The value 'foo' is not a valid value.",
298
            str(error))
299
2067.3.1 by Martin Pool
Clean up BzrNewError, other exception classes and users.
300
    def test_bzrnewerror_is_deprecated(self):
301
        class DeprecatedError(errors.BzrNewError):
302
            pass
2067.3.6 by Martin Pool
Update deprecation version
303
        self.callDeprecated(['BzrNewError was deprecated in bzr 0.13; '
2067.3.1 by Martin Pool
Clean up BzrNewError, other exception classes and users.
304
             'please convert DeprecatedError to use BzrError instead'],
305
            DeprecatedError)
306
307
    def test_bzrerror_from_literal_string(self):
308
        # Some code constructs BzrError from a literal string, in which case
309
        # no further formatting is done.  (I'm not sure raising the base class
310
        # is a great idea, but if the exception is not intended to be caught
311
        # perhaps no more is needed.)
312
        try:
313
            raise errors.BzrError('this is my errors; %d is not expanded')
314
        except errors.BzrError, e:
315
            self.assertEqual('this is my errors; %d is not expanded', str(e))
316
2018.2.4 by Robert Collins
separate out the client medium from the client encoding protocol for the smart server.
317
    def test_reading_completed(self):
318
        error = errors.ReadingCompleted("a request")
319
        self.assertEqualDiff("The MediumRequest 'a request' has already had "
320
            "finish_reading called upon it - the request has been completed and"
321
            " no more data may be read.",
322
            str(error))
323
324
    def test_writing_completed(self):
325
        error = errors.WritingCompleted("a request")
326
        self.assertEqualDiff("The MediumRequest 'a request' has already had "
327
            "finish_writing called upon it - accept bytes may not be called "
328
            "anymore.",
329
            str(error))
330
331
    def test_writing_not_completed(self):
332
        error = errors.WritingNotComplete("a request")
333
        self.assertEqualDiff("The MediumRequest 'a request' has not has "
334
            "finish_writing called upon it - until the write phase is complete"
335
            " no data may be read.",
336
            str(error))
337
2052.6.1 by Robert Collins
``Transport.get`` has had its interface made more clear for ease of use.
338
    def test_transport_not_possible(self):
339
        error = errors.TransportNotPossible('readonly', 'original error')
340
        self.assertEqualDiff('Transport operation not possible:'
341
                         ' readonly original error', str(error))
2052.4.4 by John Arbash Meinel
Create a SocketConnectionError to make creating nice errors easier
342
343
    def assertSocketConnectionError(self, expected, *args, **kwargs):
344
        """Check the formatting of a SocketConnectionError exception"""
345
        e = errors.SocketConnectionError(*args, **kwargs)
346
        self.assertEqual(expected, str(e))
347
348
    def test_socket_connection_error(self):
349
        """Test the formatting of SocketConnectionError"""
350
351
        # There should be a default msg about failing to connect
352
        # we only require a host name.
353
        self.assertSocketConnectionError(
354
            'Failed to connect to ahost',
355
            'ahost')
356
357
        # If port is None, we don't put :None
358
        self.assertSocketConnectionError(
359
            'Failed to connect to ahost',
360
            'ahost', port=None)
361
        # But if port is supplied we include it
362
        self.assertSocketConnectionError(
363
            'Failed to connect to ahost:22',
364
            'ahost', port=22)
365
366
        # We can also supply extra information about the error
367
        # with or without a port
368
        self.assertSocketConnectionError(
369
            'Failed to connect to ahost:22; bogus error',
370
            'ahost', port=22, orig_error='bogus error')
371
        self.assertSocketConnectionError(
372
            'Failed to connect to ahost; bogus error',
373
            'ahost', orig_error='bogus error')
374
        # An exception object can be passed rather than a string
375
        orig_error = ValueError('bad value')
376
        self.assertSocketConnectionError(
377
            'Failed to connect to ahost; %s' % (str(orig_error),),
378
            host='ahost', orig_error=orig_error)
379
380
        # And we can supply a custom failure message
381
        self.assertSocketConnectionError(
382
            'Unable to connect to ssh host ahost:444; my_error',
383
            host='ahost', port=444, msg='Unable to connect to ssh host',
384
            orig_error='my_error')
385
2376.4.26 by Jonathan Lange
Tests for MalformedBugIdentifier and new error UnknownBugTrackerAbbreviation.
386
    def test_malformed_bug_identifier(self):
387
        """Test the formatting of MalformedBugIdentifier."""
388
        error = errors.MalformedBugIdentifier('bogus', 'reason for bogosity')
389
        self.assertEqual(
390
            "Did not understand bug identifier bogus: reason for bogosity",
391
            str(error))
392
393
    def test_unknown_bug_tracker_abbreviation(self):
394
        """Test the formatting of UnknownBugTrackerAbbreviation."""
2376.4.27 by Jonathan Lange
Include branch information in UnknownBugTrackerAbbreviation
395
        branch = self.make_branch('some_branch')
396
        error = errors.UnknownBugTrackerAbbreviation('xxx', branch)
2376.4.26 by Jonathan Lange
Tests for MalformedBugIdentifier and new error UnknownBugTrackerAbbreviation.
397
        self.assertEqual(
2376.4.27 by Jonathan Lange
Include branch information in UnknownBugTrackerAbbreviation
398
            "Cannot find registered bug tracker called xxx on %s" % branch,
2376.4.26 by Jonathan Lange
Tests for MalformedBugIdentifier and new error UnknownBugTrackerAbbreviation.
399
            str(error))
2052.4.4 by John Arbash Meinel
Create a SocketConnectionError to make creating nice errors easier
400
2018.5.163 by Andrew Bennetts
Deal with various review comments from Robert.
401
    def test_unexpected_smart_server_response(self):
402
        e = errors.UnexpectedSmartServerResponse(('not yes',))
403
        self.assertEqual(
404
            "Could not understand response from smart server: ('not yes',)",
405
            str(e))
2052.4.4 by John Arbash Meinel
Create a SocketConnectionError to make creating nice errors easier
406
2506.2.1 by Andrew Bennetts
Start implementing container format reading and writing.
407
    def test_unknown_container_format(self):
408
        """Test the formatting of UnknownContainerFormatError."""
409
        e = errors.UnknownContainerFormatError('bad format string')
410
        self.assertEqual(
411
            "Unrecognised container format: 'bad format string'",
412
            str(e))
413
414
    def test_unexpected_end_of_container(self):
415
        """Test the formatting of UnexpectedEndOfContainerError."""
416
        e = errors.UnexpectedEndOfContainerError()
417
        self.assertEqual(
418
            "Unexpected end of container stream", str(e))
419
420
    def test_unknown_record_type(self):
421
        """Test the formatting of UnknownRecordTypeError."""
422
        e = errors.UnknownRecordTypeError("X")
423
        self.assertEqual(
424
            "Unknown record type: 'X'",
425
            str(e))
426
2506.3.1 by Andrew Bennetts
More progress:
427
    def test_invalid_record(self):
428
        """Test the formatting of InvalidRecordError."""
429
        e = errors.InvalidRecordError("xxx")
430
        self.assertEqual(
431
            "Invalid record: xxx",
432
            str(e))
433
2506.2.6 by Andrew Bennetts
Add validate method to ContainerReader and BytesRecordReader.
434
    def test_container_has_excess_data(self):
435
        """Test the formatting of ContainerHasExcessDataError."""
436
        e = errors.ContainerHasExcessDataError("excess bytes")
437
        self.assertEqual(
438
            "Container has data after end marker: 'excess bytes'",
439
            str(e))
440
2506.6.1 by Andrew Bennetts
Return a callable instead of a str from read, and add more validation.
441
    def test_duplicate_record_name_error(self):
442
        """Test the formatting of DuplicateRecordNameError."""
443
        e = errors.DuplicateRecordNameError(u"n\xe5me".encode('utf-8'))
444
        self.assertEqual(
2745.3.3 by Daniel Watkins
Changed to remove need for escaping of quotes.
445
            "Container has multiple records with the same name: n\xc3\xa5me",
2506.6.1 by Andrew Bennetts
Return a callable instead of a str from read, and add more validation.
446
            str(e))
2854.1.1 by Martin Pool
Fix "unprintable error" message for BzrCheckError and others
447
        
448
    def test_check_error(self):
449
        # This has a member called 'message', which is problematic in
450
        # python2.5 because that is a slot on the base Exception class
451
        e = errors.BzrCheckError('example check failure')
452
        self.assertEqual(
453
            "Internal check failed: example check failure",
454
            str(e))
455
        self.assertTrue(e.internal_error)
2506.6.1 by Andrew Bennetts
Return a callable instead of a str from read, and add more validation.
456
2535.3.40 by Andrew Bennetts
Tidy up more XXXs.
457
    def test_repository_data_stream_error(self):
458
        """Test the formatting of RepositoryDataStreamError."""
459
        e = errors.RepositoryDataStreamError(u"my reason")
460
        self.assertEqual(
461
            "Corrupt or incompatible data stream: my reason", str(e))
462
2978.2.1 by Alexander Belchenko
fix formatting of ImmortalPendingDeletion error message.
463
    def test_immortal_pending_deletion_message(self):
464
        err = errors.ImmortalPendingDeletion('foo')
465
        self.assertEquals(
466
            "Unable to delete transform temporary directory foo.  "
467
            "Please examine foo to see if it contains any files "
468
            "you wish to keep, and delete it when you are done.",
469
            str(err))
470
3006.2.2 by Alexander Belchenko
tests added.
471
    def test_unable_create_symlink(self):
472
        err = errors.UnableCreateSymlink()
473
        self.assertEquals(
474
            "Unable to create symlink on this platform",
475
            str(err))
476
        err = errors.UnableCreateSymlink(path=u'foo')
477
        self.assertEquals(
478
            "Unable to create symlink 'foo' on this platform",
479
            str(err))
480
        err = errors.UnableCreateSymlink(path=u'\xb5')
481
        self.assertEquals(
482
            "Unable to create symlink u'\\xb5' on this platform",
483
            str(err))
484
2692.1.1 by Andrew Bennetts
Add translate_client_path method to SmartServerRequest.
485
    def test_invalid_url_join(self):
486
        """Test the formatting of InvalidURLJoin."""
487
        e = errors.InvalidURLJoin('Reason', 'base path', ('args',))
488
        self.assertEqual(
489
            "Invalid URL join request: Reason: 'base path' + ('args',)",
490
            str(e))
491
3035.3.2 by Lukáš Lalinský
Add tests for InvalidBugTrackerURL.
492
    def test_incorrect_url(self):
493
        err = errors.InvalidBugTrackerURL('foo', 'http://bug.com/')
494
        self.assertEquals(
495
            ("The URL for bug tracker \"foo\" doesn't contain {id}: "
496
             "http://bug.com/"),
497
            str(err))
498
3234.2.6 by Alexander Belchenko
because every mail client has different rules to compose command line we should encode arguments to 8 bit string only when needed.
499
    def test_unable_encode_path(self):
500
        err = errors.UnableEncodePath('foo', 'executable')
3234.2.8 by Alexander Belchenko
fix grammar in formatting string of UnableEncodePath error.
501
        self.assertEquals("Unable to encode executable path 'foo' in "
3234.2.6 by Alexander Belchenko
because every mail client has different rules to compose command line we should encode arguments to 8 bit string only when needed.
502
            "user encoding " + osutils.get_user_encoding(),
503
            str(err))
504
3567.4.4 by John Arbash Meinel
Add the ability to 'unversion' files, and handle unknown actions.
505
    def test_unknown_build_action(self):
506
        err = errors.UnknownBuildAction('myaction')
507
        self.assertEqual("Unknown build action: myaction", str(err))
508
3246.3.4 by Daniel Watkins
Added test.
509
    def test_unknown_format(self):
510
        err = errors.UnknownFormatError('bar', kind='foo')
511
        self.assertEquals("Unknown foo format: 'bar'", str(err))
512
3398.1.29 by Ian Clatworthy
add UnknownRules class & test
513
    def test_unknown_rules(self):
514
        err = errors.UnknownRules(['foo', 'bar'])
515
        self.assertEquals("Unknown rules detected: foo, bar.", str(err))
516
2116.3.1 by John Arbash Meinel
Cleanup error tests
517
2067.3.1 by Martin Pool
Clean up BzrNewError, other exception classes and users.
518
class PassThroughError(errors.BzrError):
519
    
520
    _fmt = """Pass through %(foo)s and %(bar)s"""
2116.3.1 by John Arbash Meinel
Cleanup error tests
521
522
    def __init__(self, foo, bar):
2067.3.1 by Martin Pool
Clean up BzrNewError, other exception classes and users.
523
        errors.BzrError.__init__(self, foo=foo, bar=bar)
524
525
526
class ErrorWithBadFormat(errors.BzrError):
527
528
    _fmt = """One format specifier: %(thing)s"""
529
530
531
class ErrorWithNoFormat(errors.BzrError):
532
    """This class has a docstring but no format string."""
2116.3.1 by John Arbash Meinel
Cleanup error tests
533
534
535
class TestErrorFormatting(TestCase):
536
    
537
    def test_always_str(self):
538
        e = PassThroughError(u'\xb5', 'bar')
539
        self.assertIsInstance(e.__str__(), str)
540
        # In Python str(foo) *must* return a real byte string
541
        # not a Unicode string. The following line would raise a
542
        # Unicode error, because it tries to call str() on the string
543
        # returned from e.__str__(), and it has non ascii characters
544
        s = str(e)
545
        self.assertEqual('Pass through \xc2\xb5 and bar', s)
546
2067.3.1 by Martin Pool
Clean up BzrNewError, other exception classes and users.
547
    def test_missing_format_string(self):
548
        e = ErrorWithNoFormat(param='randomvalue')
2067.3.3 by Martin Pool
merge bzr.dev and reconcile several changes, also some test fixes
549
        s = self.callDeprecated(
550
                ['ErrorWithNoFormat uses its docstring as a format, it should use _fmt instead'],
551
                lambda x: str(x), e)
552
        ## s = str(e)
553
        self.assertEqual(s, 
554
                "This class has a docstring but no format string.")
2067.3.1 by Martin Pool
Clean up BzrNewError, other exception classes and users.
555
2116.3.1 by John Arbash Meinel
Cleanup error tests
556
    def test_mismatched_format_args(self):
557
        # Even though ErrorWithBadFormat's format string does not match the
558
        # arguments we constructing it with, we can still stringify an instance
559
        # of this exception. The resulting string will say its unprintable.
560
        e = ErrorWithBadFormat(not_thing='x')
561
        self.assertStartsWith(
2067.3.1 by Martin Pool
Clean up BzrNewError, other exception classes and users.
562
            str(e), 'Unprintable exception ErrorWithBadFormat')