18
18
"""Tests for the formatting and construction of errors."""
20
import bzrlib.bzrdir as bzrdir
21
import bzrlib.errors as errors
22
from bzrlib.tests import TestCaseWithTransport
24
from bzrlib.tests import TestCase, TestCaseWithTransport
27
# TODO: Make sure builtin exception class formats are consistent - e.g. should
28
# or shouldn't end with a full stop, etc.
25
31
class TestErrors(TestCaseWithTransport):
33
def test_inventory_modified(self):
34
error = errors.InventoryModified("a tree to be repred")
35
self.assertEqualDiff("The current inventory for the tree 'a tree to "
36
"be repred' has been modified, so a clean inventory cannot be "
37
"read without data loss.",
40
def test_install_failed(self):
41
error = errors.InstallFailed(['rev-one'])
42
self.assertEqual("Could not install revisions:\nrev-one", str(error))
43
error = errors.InstallFailed(['rev-one', 'rev-two'])
44
self.assertEqual("Could not install revisions:\nrev-one, rev-two",
46
error = errors.InstallFailed([None])
47
self.assertEqual("Could not install revisions:\nNone", str(error))
49
def test_knit_header_error(self):
50
error = errors.KnitHeaderError('line foo\n', 'path/to/file')
51
self.assertEqual("Knit header error: 'line foo\\n' unexpected"
52
" for file path/to/file", str(error))
54
def test_knit_index_unknown_method(self):
55
error = errors.KnitIndexUnknownMethod('http://host/foo.kndx',
57
self.assertEqual("Knit index http://host/foo.kndx does not have a"
58
" known method in options: ['bad', 'no-eol']",
61
def test_medium_not_connected(self):
62
error = errors.MediumNotConnected("a medium")
64
"The medium 'a medium' is not connected.", str(error))
27
66
def test_no_repo(self):
28
67
dir = bzrdir.BzrDir.create(self.get_url())
29
68
error = errors.NoRepositoryPresent(dir)
30
self.assertNotEqual(-1, str(error).find(repr(dir.transport.clone('..').base)))
31
self.assertEqual(-1, str(error).find(repr(dir.transport.base)))
69
self.assertNotEqual(-1, str(error).find((dir.transport.clone('..').base)))
70
self.assertEqual(-1, str(error).find((dir.transport.base)))
72
def test_no_smart_medium(self):
73
error = errors.NoSmartMedium("a transport")
74
self.assertEqualDiff("The transport 'a transport' cannot tunnel the "
78
def test_no_such_id(self):
79
error = errors.NoSuchId("atree", "anid")
80
self.assertEqualDiff("The file id anid is not present in the tree "
84
def test_no_such_revision_in_tree(self):
85
error = errors.NoSuchRevisionInTree("atree", "anid")
86
self.assertEqualDiff("The revision id anid is not present in the tree "
89
self.assertIsInstance(error, errors.NoSuchRevision)
91
def test_not_write_locked(self):
92
error = errors.NotWriteLocked('a thing to repr')
93
self.assertEqualDiff("'a thing to repr' is not write locked but needs "
97
def test_too_many_concurrent_requests(self):
98
error = errors.TooManyConcurrentRequests("a medium")
99
self.assertEqualDiff("The medium 'a medium' has reached its concurrent "
100
"request limit. Be sure to finish_writing and finish_reading on "
101
"the current request that is open.",
33
104
def test_up_to_date(self):
34
105
error = errors.UpToDateFormat(bzrdir.BzrDirFormat4())
44
115
"Please run bzr reconcile on this repository." %
45
116
repo.bzrdir.root_transport.base,
119
def test_bzrnewerror_is_deprecated(self):
120
class DeprecatedError(errors.BzrNewError):
122
self.callDeprecated(['BzrNewError was deprecated in bzr 0.13; '
123
'please convert DeprecatedError to use BzrError instead'],
126
def test_bzrerror_from_literal_string(self):
127
# Some code constructs BzrError from a literal string, in which case
128
# no further formatting is done. (I'm not sure raising the base class
129
# is a great idea, but if the exception is not intended to be caught
130
# perhaps no more is needed.)
132
raise errors.BzrError('this is my errors; %d is not expanded')
133
except errors.BzrError, e:
134
self.assertEqual('this is my errors; %d is not expanded', str(e))
136
def test_reading_completed(self):
137
error = errors.ReadingCompleted("a request")
138
self.assertEqualDiff("The MediumRequest 'a request' has already had "
139
"finish_reading called upon it - the request has been completed and"
140
" no more data may be read.",
143
def test_writing_completed(self):
144
error = errors.WritingCompleted("a request")
145
self.assertEqualDiff("The MediumRequest 'a request' has already had "
146
"finish_writing called upon it - accept bytes may not be called "
150
def test_writing_not_completed(self):
151
error = errors.WritingNotComplete("a request")
152
self.assertEqualDiff("The MediumRequest 'a request' has not has "
153
"finish_writing called upon it - until the write phase is complete"
154
" no data may be read.",
157
def test_transport_not_possible(self):
158
e = errors.TransportNotPossible('readonly', 'original error')
159
self.assertEqual('Transport operation not possible:'
160
' readonly original error', str(e))
162
def assertSocketConnectionError(self, expected, *args, **kwargs):
163
"""Check the formatting of a SocketConnectionError exception"""
164
e = errors.SocketConnectionError(*args, **kwargs)
165
self.assertEqual(expected, str(e))
167
def test_socket_connection_error(self):
168
"""Test the formatting of SocketConnectionError"""
170
# There should be a default msg about failing to connect
171
# we only require a host name.
172
self.assertSocketConnectionError(
173
'Failed to connect to ahost',
176
# If port is None, we don't put :None
177
self.assertSocketConnectionError(
178
'Failed to connect to ahost',
180
# But if port is supplied we include it
181
self.assertSocketConnectionError(
182
'Failed to connect to ahost:22',
185
# We can also supply extra information about the error
186
# with or without a port
187
self.assertSocketConnectionError(
188
'Failed to connect to ahost:22; bogus error',
189
'ahost', port=22, orig_error='bogus error')
190
self.assertSocketConnectionError(
191
'Failed to connect to ahost; bogus error',
192
'ahost', orig_error='bogus error')
193
# An exception object can be passed rather than a string
194
orig_error = ValueError('bad value')
195
self.assertSocketConnectionError(
196
'Failed to connect to ahost; %s' % (str(orig_error),),
197
host='ahost', orig_error=orig_error)
199
# And we can supply a custom failure message
200
self.assertSocketConnectionError(
201
'Unable to connect to ssh host ahost:444; my_error',
202
host='ahost', port=444, msg='Unable to connect to ssh host',
203
orig_error='my_error')
207
class PassThroughError(errors.BzrError):
209
_fmt = """Pass through %(foo)s and %(bar)s"""
211
def __init__(self, foo, bar):
212
errors.BzrError.__init__(self, foo=foo, bar=bar)
215
class ErrorWithBadFormat(errors.BzrError):
217
_fmt = """One format specifier: %(thing)s"""
220
class ErrorWithNoFormat(errors.BzrError):
221
"""This class has a docstring but no format string."""
224
class TestErrorFormatting(TestCase):
226
def test_always_str(self):
227
e = PassThroughError(u'\xb5', 'bar')
228
self.assertIsInstance(e.__str__(), str)
229
# In Python str(foo) *must* return a real byte string
230
# not a Unicode string. The following line would raise a
231
# Unicode error, because it tries to call str() on the string
232
# returned from e.__str__(), and it has non ascii characters
234
self.assertEqual('Pass through \xc2\xb5 and bar', s)
236
def test_missing_format_string(self):
237
e = ErrorWithNoFormat(param='randomvalue')
238
s = self.callDeprecated(
239
['ErrorWithNoFormat uses its docstring as a format, it should use _fmt instead'],
243
"This class has a docstring but no format string.")
245
def test_mismatched_format_args(self):
246
# Even though ErrorWithBadFormat's format string does not match the
247
# arguments we constructing it with, we can still stringify an instance
248
# of this exception. The resulting string will say its unprintable.
249
e = ErrorWithBadFormat(not_thing='x')
250
self.assertStartsWith(
251
str(e), 'Unprintable exception ErrorWithBadFormat')
added
removed
bzrlib/tests/test_errors.py
