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
|
|
4183.7.1
by Sabin Iacob
update FSF mailing address |
17 |
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
|
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. |
18 |
|
19 |
"""Tests for the formatting and construction of errors."""
|
|
20 |
||
3577.1.1
by Andrew Bennetts
Cherry-pick TipChangeRejected changes from pre-branch-tip-changed-hook loom. |
21 |
import sys |
1948.1.6
by John Arbash Meinel
Make BzrNewError always return a str object |
22 |
from bzrlib import ( |
23 |
bzrdir, |
|
24 |
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. |
25 |
osutils, |
2872.5.1
by Martin Pool
Avoid internal error tracebacks on failure to lock on readonly transport (#129701). |
26 |
symbol_versioning, |
3200.2.1
by Robert Collins
* The ``register-branch`` command will now use the public url of the branch |
27 |
urlutils, |
1948.1.6
by John Arbash Meinel
Make BzrNewError always return a str object |
28 |
)
|
29 |
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. |
30 |
|
31 |
||
32 |
class TestErrors(TestCaseWithTransport): |
|
33 |
||
3287.20.2
by John Arbash Meinel
Raise a clear error about the offending filename when there is a filename with bad characters. |
34 |
def test_bad_filename_encoding(self): |
35 |
error = errors.BadFilenameEncoding('bad/filen\xe5me', 'UTF-8') |
|
36 |
self.assertEqualDiff( |
|
37 |
"Filename 'bad/filen\\xe5me' is not valid in your current" |
|
38 |
" filesystem encoding UTF-8", |
|
39 |
str(error)) |
|
40 |
||
3207.2.1
by jameinel
Add a test that _iter_changes raises a clearer error when we encounter an invalid rename. |
41 |
def test_corrupt_dirstate(self): |
42 |
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. |
43 |
self.assertEqualDiff( |
44 |
"Inconsistency in dirstate file path/to/dirstate.\n" |
|
45 |
"Error: the reason why", |
|
46 |
str(error)) |
|
3207.2.1
by jameinel
Add a test that _iter_changes raises a clearer error when we encounter an invalid rename. |
47 |
|
3640.2.5
by John Arbash Meinel
Change from using AssertionError to using DirstateCorrupt in a few places |
48 |
def test_dirstate_corrupt(self): |
49 |
error = errors.DirstateCorrupt('.bzr/checkout/dirstate', |
|
50 |
'trailing garbage: "x"') |
|
51 |
self.assertEqualDiff("The dirstate file (.bzr/checkout/dirstate)" |
|
52 |
" appears to be corrupt: trailing garbage: \"x\"", |
|
53 |
str(error)) |
|
54 |
||
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) |
55 |
def test_disabled_method(self): |
56 |
error = errors.DisabledMethod("class name") |
|
57 |
self.assertEqualDiff( |
|
58 |
"The smart server method 'class name' is disabled.", str(error)) |
|
59 |
||
2255.7.16
by John Arbash Meinel
Make sure adding a duplicate file_id raises DuplicateFileId. |
60 |
def test_duplicate_file_id(self): |
61 |
error = errors.DuplicateFileId('a_file_id', 'foo') |
|
62 |
self.assertEqualDiff('File id {a_file_id} already exists in inventory' |
|
63 |
' as foo', str(error)) |
|
64 |
||
2432.1.19
by Robert Collins
Ensure each HelpIndex has a unique prefix. |
65 |
def test_duplicate_help_prefix(self): |
66 |
error = errors.DuplicateHelpPrefix('foo') |
|
67 |
self.assertEqualDiff('The prefix foo is in the help search path twice.', |
|
68 |
str(error)) |
|
69 |
||
3445.1.1
by John Arbash Meinel
Start working on a new Graph api to make finding revision numbers faster. |
70 |
def test_ghost_revisions_have_no_revno(self): |
71 |
error = errors.GhostRevisionsHaveNoRevno('target', 'ghost_rev') |
|
72 |
self.assertEqualDiff("Could not determine revno for {target} because" |
|
73 |
" its ancestry shows a ghost at {ghost_rev}", |
|
74 |
str(error)) |
|
75 |
||
2550.2.3
by Robert Collins
Add require_api API. |
76 |
def test_incompatibleAPI(self): |
77 |
error = errors.IncompatibleAPI("module", (1, 2, 3), (4, 5, 6), (7, 8, 9)) |
|
78 |
self.assertEqualDiff( |
|
79 |
'The API for "module" is not compatible with "(1, 2, 3)". '
|
|
80 |
'It supports versions "(4, 5, 6)" to "(7, 8, 9)".', |
|
81 |
str(error)) |
|
82 |
||
3207.2.2
by John Arbash Meinel
Fix bug #187169, when an invalid delta is supplied to update_basis_by_delta |
83 |
def test_inconsistent_delta(self): |
84 |
error = errors.InconsistentDelta('path', 'file-id', 'reason for foo') |
|
85 |
self.assertEqualDiff( |
|
3221.1.8
by Martin Pool
Update error format in test_inconsistent_delta |
86 |
"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 |
87 |
"reason: reason for foo", |
88 |
str(error)) |
|
89 |
||
2634.1.1
by Robert Collins
(robertc) Reinstate the accidentally backed out external_url patch. |
90 |
def test_in_process_transport(self): |
91 |
error = errors.InProcessTransport('fpp') |
|
92 |
self.assertEqualDiff( |
|
93 |
"The transport 'fpp' is only accessible within this process.", |
|
94 |
str(error)) |
|
95 |
||
3059.2.12
by Vincent Ladeuil
Spiv review feedback. |
96 |
def test_invalid_http_range(self): |
97 |
error = errors.InvalidHttpRange('path', |
|
98 |
'Content-Range: potatoes 0-00/o0oo0', |
|
99 |
'bad range') |
|
100 |
self.assertEquals("Invalid http range" |
|
101 |
" 'Content-Range: potatoes 0-00/o0oo0'"
|
|
102 |
" for path: bad range", |
|
103 |
str(error)) |
|
104 |
||
105 |
def test_invalid_range(self): |
|
106 |
error = errors.InvalidRange('path', 12, 'bad range') |
|
107 |
self.assertEquals("Invalid range access in path at 12: bad range", |
|
108 |
str(error)) |
|
109 |
||
1986.5.3
by Robert Collins
New method ``WorkingTree.flush()`` which will write the current memory |
110 |
def test_inventory_modified(self): |
111 |
error = errors.InventoryModified("a tree to be repred") |
|
112 |
self.assertEqualDiff("The current inventory for the tree 'a tree to " |
|
113 |
"be repred' has been modified, so a clean inventory cannot be "
|
|
114 |
"read without data loss.", |
|
115 |
str(error)) |
|
116 |
||
2116.3.1
by John Arbash Meinel
Cleanup error tests |
117 |
def test_install_failed(self): |
118 |
error = errors.InstallFailed(['rev-one']) |
|
119 |
self.assertEqual("Could not install revisions:\nrev-one", str(error)) |
|
120 |
error = errors.InstallFailed(['rev-one', 'rev-two']) |
|
121 |
self.assertEqual("Could not install revisions:\nrev-one, rev-two", |
|
122 |
str(error)) |
|
123 |
error = errors.InstallFailed([None]) |
|
124 |
self.assertEqual("Could not install revisions:\nNone", str(error)) |
|
125 |
||
2255.2.145
by Robert Collins
Support unbreakable locks for trees. |
126 |
def test_lock_active(self): |
127 |
error = errors.LockActive("lock description") |
|
128 |
self.assertEqualDiff("The lock for 'lock description' is in use and " |
|
129 |
"cannot be broken.", |
|
130 |
str(error)) |
|
131 |
||
2535.3.4
by Andrew Bennetts
Simple implementation of Knit.insert_data_stream. |
132 |
def test_knit_data_stream_incompatible(self): |
133 |
error = errors.KnitDataStreamIncompatible( |
|
134 |
'stream format', 'target format') |
|
135 |
self.assertEqual('Cannot insert knit data stream of format ' |
|
136 |
'"stream format" into knit of format '
|
|
137 |
'"target format".', str(error)) |
|
138 |
||
3052.2.1
by Robert Collins
Add a new KnitDataStreamUnknown error class for showing formats we can't understand. |
139 |
def test_knit_data_stream_unknown(self): |
140 |
error = errors.KnitDataStreamUnknown( |
|
141 |
'stream format') |
|
142 |
self.assertEqual('Cannot parse knit data stream of format ' |
|
143 |
'"stream format".', str(error)) |
|
144 |
||
2171.1.1
by John Arbash Meinel
Knit index files should ignore empty indexes rather than consider them corrupt. |
145 |
def test_knit_header_error(self): |
146 |
error = errors.KnitHeaderError('line foo\n', 'path/to/file') |
|
147 |
self.assertEqual("Knit header error: 'line foo\\n' unexpected" |
|
2745.3.2
by Daniel Watkins
Updated tests to reflect new error text. |
148 |
" 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. |
149 |
|
2196.2.5
by John Arbash Meinel
Add an exception class when the knit index storage method is unknown, and properly test for it |
150 |
def test_knit_index_unknown_method(self): |
151 |
error = errors.KnitIndexUnknownMethod('http://host/foo.kndx', |
|
152 |
['bad', 'no-eol']) |
|
153 |
self.assertEqual("Knit index http://host/foo.kndx does not have a" |
|
154 |
" known method in options: ['bad', 'no-eol']", |
|
155 |
str(error)) |
|
156 |
||
2018.2.3
by Andrew Bennetts
Starting factoring out the smart server client "medium" from the protocol. |
157 |
def test_medium_not_connected(self): |
158 |
error = errors.MediumNotConnected("a medium") |
|
159 |
self.assertEqualDiff( |
|
160 |
"The medium 'a medium' is not connected.", str(error)) |
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
161 |
|
3200.2.1
by Robert Collins
* The ``register-branch`` command will now use the public url of the branch |
162 |
def test_no_public_branch(self): |
163 |
b = self.make_branch('.') |
|
164 |
error = errors.NoPublicBranch(b) |
|
165 |
url = urlutils.unescape_for_display(b.base, 'ascii') |
|
166 |
self.assertEqualDiff( |
|
167 |
'There is no public branch set for "%s".' % url, str(error)) |
|
168 |
||
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. |
169 |
def test_no_repo(self): |
170 |
dir = bzrdir.BzrDir.create(self.get_url()) |
|
171 |
error = errors.NoRepositoryPresent(dir) |
|
1740.5.6
by Martin Pool
Clean up many exception classes. |
172 |
self.assertNotEqual(-1, str(error).find((dir.transport.clone('..').base))) |
173 |
self.assertEqual(-1, str(error).find((dir.transport.base))) |
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
174 |
|
2018.2.3
by Andrew Bennetts
Starting factoring out the smart server client "medium" from the protocol. |
175 |
def test_no_smart_medium(self): |
176 |
error = errors.NoSmartMedium("a transport") |
|
177 |
self.assertEqualDiff("The transport 'a transport' cannot tunnel the " |
|
178 |
"smart protocol.", |
|
179 |
str(error)) |
|
180 |
||
2432.1.4
by Robert Collins
Add an explicit error for missing help topics. |
181 |
def test_no_help_topic(self): |
182 |
error = errors.NoHelpTopic("topic") |
|
183 |
self.assertEqualDiff("No help could be found for 'topic'. " |
|
184 |
"Please use 'bzr help topics' to obtain a list of topics.", |
|
185 |
str(error)) |
|
186 |
||
1988.2.1
by Robert Collins
WorkingTree has a new api ``unversion`` which allow the unversioning of |
187 |
def test_no_such_id(self): |
188 |
error = errors.NoSuchId("atree", "anid") |
|
2745.3.2
by Daniel Watkins
Updated tests to reflect new error text. |
189 |
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. |
190 |
"atree.", |
1988.2.1
by Robert Collins
WorkingTree has a new api ``unversion`` which allow the unversioning of |
191 |
str(error)) |
1534.5.7
by Robert Collins
Start factoring out the upgrade policy logic. |
192 |
|
1908.11.1
by Robert Collins
Add a new method ``Tree.revision_tree`` which allows access to cached |
193 |
def test_no_such_revision_in_tree(self): |
194 |
error = errors.NoSuchRevisionInTree("atree", "anid") |
|
2745.3.3
by Daniel Watkins
Changed to remove need for escaping of quotes. |
195 |
self.assertEqualDiff("The revision id {anid} is not present in the" |
196 |
" tree atree.", str(error)) |
|
1908.11.1
by Robert Collins
Add a new method ``Tree.revision_tree`` which allows access to cached |
197 |
self.assertIsInstance(error, errors.NoSuchRevision) |
198 |
||
3221.11.2
by Robert Collins
Create basic stackable branch facility. |
199 |
def test_not_stacked(self): |
200 |
error = errors.NotStacked('a branch') |
|
201 |
self.assertEqualDiff("The branch 'a branch' is not stacked.", |
|
202 |
str(error)) |
|
203 |
||
1986.5.3
by Robert Collins
New method ``WorkingTree.flush()`` which will write the current memory |
204 |
def test_not_write_locked(self): |
205 |
error = errors.NotWriteLocked('a thing to repr') |
|
206 |
self.assertEqualDiff("'a thing to repr' is not write locked but needs " |
|
207 |
"to be.", |
|
208 |
str(error)) |
|
209 |
||
2872.5.1
by Martin Pool
Avoid internal error tracebacks on failure to lock on readonly transport (#129701). |
210 |
def test_lock_failed(self): |
211 |
error = errors.LockFailed('http://canonical.com/', 'readonly transport') |
|
212 |
self.assertEqualDiff("Cannot lock http://canonical.com/: readonly transport", |
|
213 |
str(error)) |
|
214 |
self.assertFalse(error.internal_error) |
|
215 |
||
2018.2.4
by Robert Collins
separate out the client medium from the client encoding protocol for the smart server. |
216 |
def test_too_many_concurrent_requests(self): |
217 |
error = errors.TooManyConcurrentRequests("a medium") |
|
218 |
self.assertEqualDiff("The medium 'a medium' has reached its concurrent " |
|
219 |
"request limit. Be sure to finish_writing and finish_reading on "
|
|
2018.5.134
by Andrew Bennetts
Fix the TooManyConcurrentRequests error message. |
220 |
"the currently open request.", |
2018.2.4
by Robert Collins
separate out the client medium from the client encoding protocol for the smart server. |
221 |
str(error)) |
222 |
||
3350.3.3
by Robert Collins
Functional get_record_stream interface tests covering full interface. |
223 |
def test_unavailable_representation(self): |
224 |
error = errors.UnavailableRepresentation(('key',), "mpdiff", "fulltext") |
|
225 |
self.assertEqualDiff("The encoding 'mpdiff' is not available for key " |
|
226 |
"('key',) which is encoded as 'fulltext'.", |
|
227 |
str(error)) |
|
228 |
||
2245.1.3
by Robert Collins
Add install_hook to the BranchHooks class as the official means for installing a hook. |
229 |
def test_unknown_hook(self): |
230 |
error = errors.UnknownHook("branch", "foo") |
|
231 |
self.assertEqualDiff("The branch hook 'foo' is unknown in this version" |
|
232 |
" of bzrlib.", |
|
233 |
str(error)) |
|
234 |
error = errors.UnknownHook("tree", "bar") |
|
235 |
self.assertEqualDiff("The tree hook 'bar' is unknown in this version" |
|
236 |
" of bzrlib.", |
|
237 |
str(error)) |
|
238 |
||
3221.11.2
by Robert Collins
Create basic stackable branch facility. |
239 |
def test_unstackable_branch_format(self): |
240 |
format = u'foo' |
|
241 |
url = "/foo" |
|
242 |
error = errors.UnstackableBranchFormat(format, url) |
|
243 |
self.assertEqualDiff( |
|
244 |
"The branch '/foo'(foo) is not a stackable format. "
|
|
245 |
"You will need to upgrade the branch to permit branch stacking.", |
|
246 |
str(error)) |
|
247 |
||
248 |
def test_unstackable_repository_format(self): |
|
249 |
format = u'foo' |
|
250 |
url = "/foo" |
|
251 |
error = errors.UnstackableRepositoryFormat(format, url) |
|
252 |
self.assertEqualDiff( |
|
253 |
"The repository '/foo'(foo) is not a stackable format. "
|
|
254 |
"You will need to upgrade the repository to permit branch stacking.", |
|
255 |
str(error)) |
|
256 |
||
1534.5.7
by Robert Collins
Start factoring out the upgrade policy logic. |
257 |
def test_up_to_date(self): |
258 |
error = errors.UpToDateFormat(bzrdir.BzrDirFormat4()) |
|
4032.3.2
by Robert Collins
Create and use a RPC call to create branches on bzr servers rather than using VFS calls. |
259 |
self.assertEqualDiff("The branch format All-in-one " |
260 |
"format 4 is already at the most "
|
|
1534.5.9
by Robert Collins
Advise users running upgrade on a checkout to also run it on the branch. |
261 |
"recent format.", |
262 |
str(error)) |
|
1570.1.13
by Robert Collins
Check for incorrect revision parentage in the weave during revision access. |
263 |
|
264 |
def test_corrupt_repository(self): |
|
265 |
repo = self.make_repository('.') |
|
266 |
error = errors.CorruptRepository(repo) |
|
267 |
self.assertEqualDiff("An error has been detected in the repository %s.\n" |
|
268 |
"Please run bzr reconcile on this repository." % |
|
269 |
repo.bzrdir.root_transport.base, |
|
270 |
str(error)) |
|
1948.1.6
by John Arbash Meinel
Make BzrNewError always return a str object |
271 |
|
2052.6.1
by Robert Collins
``Transport.get`` has had its interface made more clear for ease of use. |
272 |
def test_read_error(self): |
273 |
# a unicode path to check that %r is being used.
|
|
274 |
path = u'a path' |
|
275 |
error = errors.ReadError(path) |
|
276 |
self.assertEqualDiff("Error reading from u'a path'.", str(error)) |
|
277 |
||
2592.1.7
by Robert Collins
A validate that goes boom. |
278 |
def test_bad_index_format_signature(self): |
279 |
error = errors.BadIndexFormatSignature("foo", "bar") |
|
280 |
self.assertEqual("foo is not an index of type bar.", |
|
281 |
str(error)) |
|
2052.6.2
by Robert Collins
Merge bzr.dev. |
282 |
|
2592.1.11
by Robert Collins
Detect truncated indices. |
283 |
def test_bad_index_data(self): |
284 |
error = errors.BadIndexData("foo") |
|
285 |
self.assertEqual("Error in data for index foo.", |
|
286 |
str(error)) |
|
287 |
||
2592.1.15
by Robert Collins
Detect duplicate key insertion. |
288 |
def test_bad_index_duplicate_key(self): |
289 |
error = errors.BadIndexDuplicateKey("foo", "bar") |
|
290 |
self.assertEqual("The key 'foo' is already in index 'bar'.", |
|
291 |
str(error)) |
|
292 |
||
2592.1.12
by Robert Collins
Handle basic node adds. |
293 |
def test_bad_index_key(self): |
294 |
error = errors.BadIndexKey("foo") |
|
295 |
self.assertEqual("The key 'foo' is not a valid key.", |
|
296 |
str(error)) |
|
297 |
||
2592.1.10
by Robert Collins
Make validate detect node reference parsing errors. |
298 |
def test_bad_index_options(self): |
299 |
error = errors.BadIndexOptions("foo") |
|
300 |
self.assertEqual("Could not parse options for index foo.", |
|
301 |
str(error)) |
|
302 |
||
2592.1.12
by Robert Collins
Handle basic node adds. |
303 |
def test_bad_index_value(self): |
304 |
error = errors.BadIndexValue("foo") |
|
305 |
self.assertEqual("The value 'foo' is not a valid value.", |
|
306 |
str(error)) |
|
307 |
||
2067.3.1
by Martin Pool
Clean up BzrNewError, other exception classes and users. |
308 |
def test_bzrnewerror_is_deprecated(self): |
309 |
class DeprecatedError(errors.BzrNewError): |
|
310 |
pass
|
|
2067.3.6
by Martin Pool
Update deprecation version |
311 |
self.callDeprecated(['BzrNewError was deprecated in bzr 0.13; ' |
2067.3.1
by Martin Pool
Clean up BzrNewError, other exception classes and users. |
312 |
'please convert DeprecatedError to use BzrError instead'], |
313 |
DeprecatedError) |
|
314 |
||
315 |
def test_bzrerror_from_literal_string(self): |
|
316 |
# Some code constructs BzrError from a literal string, in which case
|
|
317 |
# no further formatting is done. (I'm not sure raising the base class
|
|
318 |
# is a great idea, but if the exception is not intended to be caught
|
|
319 |
# perhaps no more is needed.)
|
|
320 |
try: |
|
321 |
raise errors.BzrError('this is my errors; %d is not expanded') |
|
322 |
except errors.BzrError, e: |
|
323 |
self.assertEqual('this is my errors; %d is not expanded', str(e)) |
|
324 |
||
2018.2.4
by Robert Collins
separate out the client medium from the client encoding protocol for the smart server. |
325 |
def test_reading_completed(self): |
326 |
error = errors.ReadingCompleted("a request") |
|
327 |
self.assertEqualDiff("The MediumRequest 'a request' has already had " |
|
328 |
"finish_reading called upon it - the request has been completed and"
|
|
329 |
" no more data may be read.", |
|
330 |
str(error)) |
|
331 |
||
332 |
def test_writing_completed(self): |
|
333 |
error = errors.WritingCompleted("a request") |
|
334 |
self.assertEqualDiff("The MediumRequest 'a request' has already had " |
|
335 |
"finish_writing called upon it - accept bytes may not be called "
|
|
336 |
"anymore.", |
|
337 |
str(error)) |
|
338 |
||
339 |
def test_writing_not_completed(self): |
|
340 |
error = errors.WritingNotComplete("a request") |
|
341 |
self.assertEqualDiff("The MediumRequest 'a request' has not has " |
|
342 |
"finish_writing called upon it - until the write phase is complete"
|
|
343 |
" no data may be read.", |
|
344 |
str(error)) |
|
345 |
||
2052.6.1
by Robert Collins
``Transport.get`` has had its interface made more clear for ease of use. |
346 |
def test_transport_not_possible(self): |
347 |
error = errors.TransportNotPossible('readonly', 'original error') |
|
348 |
self.assertEqualDiff('Transport operation not possible:' |
|
349 |
' readonly original error', str(error)) |
|
2052.4.4
by John Arbash Meinel
Create a SocketConnectionError to make creating nice errors easier |
350 |
|
351 |
def assertSocketConnectionError(self, expected, *args, **kwargs): |
|
352 |
"""Check the formatting of a SocketConnectionError exception"""
|
|
353 |
e = errors.SocketConnectionError(*args, **kwargs) |
|
354 |
self.assertEqual(expected, str(e)) |
|
355 |
||
356 |
def test_socket_connection_error(self): |
|
357 |
"""Test the formatting of SocketConnectionError"""
|
|
358 |
||
359 |
# There should be a default msg about failing to connect
|
|
360 |
# we only require a host name.
|
|
361 |
self.assertSocketConnectionError( |
|
362 |
'Failed to connect to ahost', |
|
363 |
'ahost') |
|
364 |
||
365 |
# If port is None, we don't put :None
|
|
366 |
self.assertSocketConnectionError( |
|
367 |
'Failed to connect to ahost', |
|
368 |
'ahost', port=None) |
|
369 |
# But if port is supplied we include it
|
|
370 |
self.assertSocketConnectionError( |
|
371 |
'Failed to connect to ahost:22', |
|
372 |
'ahost', port=22) |
|
373 |
||
374 |
# We can also supply extra information about the error
|
|
375 |
# with or without a port
|
|
376 |
self.assertSocketConnectionError( |
|
377 |
'Failed to connect to ahost:22; bogus error', |
|
378 |
'ahost', port=22, orig_error='bogus error') |
|
379 |
self.assertSocketConnectionError( |
|
380 |
'Failed to connect to ahost; bogus error', |
|
381 |
'ahost', orig_error='bogus error') |
|
382 |
# An exception object can be passed rather than a string
|
|
383 |
orig_error = ValueError('bad value') |
|
384 |
self.assertSocketConnectionError( |
|
385 |
'Failed to connect to ahost; %s' % (str(orig_error),), |
|
386 |
host='ahost', orig_error=orig_error) |
|
387 |
||
388 |
# And we can supply a custom failure message
|
|
389 |
self.assertSocketConnectionError( |
|
390 |
'Unable to connect to ssh host ahost:444; my_error', |
|
391 |
host='ahost', port=444, msg='Unable to connect to ssh host', |
|
392 |
orig_error='my_error') |
|
393 |
||
3535.8.2
by James Westby
Incorporate spiv's feedback. |
394 |
def test_target_not_branch(self): |
395 |
"""Test the formatting of TargetNotBranch."""
|
|
396 |
error = errors.TargetNotBranch('foo') |
|
397 |
self.assertEqual( |
|
398 |
"Your branch does not have all of the revisions required in "
|
|
3535.8.4
by James Westby
Replace "however" with "and" at John's request. |
399 |
"order to merge this merge directive and the target "
|
3535.8.3
by James Westby
Use location instead of branch as suggested by Robert. |
400 |
"location specified in the merge directive is not a branch: "
|
3535.8.2
by James Westby
Incorporate spiv's feedback. |
401 |
"foo.", str(error)) |
402 |
||
2376.4.26
by Jonathan Lange
Tests for MalformedBugIdentifier and new error UnknownBugTrackerAbbreviation. |
403 |
def test_malformed_bug_identifier(self): |
404 |
"""Test the formatting of MalformedBugIdentifier."""
|
|
405 |
error = errors.MalformedBugIdentifier('bogus', 'reason for bogosity') |
|
406 |
self.assertEqual( |
|
3535.10.1
by James Westby
Point to "bzr help bugs" from MalformedBugIdentifier. |
407 |
'Did not understand bug identifier bogus: reason for bogosity. '
|
3535.10.11
by James Westby
Fix test for change in error message. |
408 |
'See "bzr help bugs" for more information on this feature.', |
2376.4.26
by Jonathan Lange
Tests for MalformedBugIdentifier and new error UnknownBugTrackerAbbreviation. |
409 |
str(error)) |
410 |
||
411 |
def test_unknown_bug_tracker_abbreviation(self): |
|
412 |
"""Test the formatting of UnknownBugTrackerAbbreviation."""
|
|
2376.4.27
by Jonathan Lange
Include branch information in UnknownBugTrackerAbbreviation |
413 |
branch = self.make_branch('some_branch') |
414 |
error = errors.UnknownBugTrackerAbbreviation('xxx', branch) |
|
2376.4.26
by Jonathan Lange
Tests for MalformedBugIdentifier and new error UnknownBugTrackerAbbreviation. |
415 |
self.assertEqual( |
2376.4.27
by Jonathan Lange
Include branch information in UnknownBugTrackerAbbreviation |
416 |
"Cannot find registered bug tracker called xxx on %s" % branch, |
2376.4.26
by Jonathan Lange
Tests for MalformedBugIdentifier and new error UnknownBugTrackerAbbreviation. |
417 |
str(error)) |
2052.4.4
by John Arbash Meinel
Create a SocketConnectionError to make creating nice errors easier |
418 |
|
2018.5.163
by Andrew Bennetts
Deal with various review comments from Robert. |
419 |
def test_unexpected_smart_server_response(self): |
420 |
e = errors.UnexpectedSmartServerResponse(('not yes',)) |
|
421 |
self.assertEqual( |
|
422 |
"Could not understand response from smart server: ('not yes',)", |
|
423 |
str(e)) |
|
2052.4.4
by John Arbash Meinel
Create a SocketConnectionError to make creating nice errors easier |
424 |
|
2506.2.1
by Andrew Bennetts
Start implementing container format reading and writing. |
425 |
def test_unknown_container_format(self): |
426 |
"""Test the formatting of UnknownContainerFormatError."""
|
|
427 |
e = errors.UnknownContainerFormatError('bad format string') |
|
428 |
self.assertEqual( |
|
429 |
"Unrecognised container format: 'bad format string'", |
|
430 |
str(e)) |
|
431 |
||
432 |
def test_unexpected_end_of_container(self): |
|
433 |
"""Test the formatting of UnexpectedEndOfContainerError."""
|
|
434 |
e = errors.UnexpectedEndOfContainerError() |
|
435 |
self.assertEqual( |
|
436 |
"Unexpected end of container stream", str(e)) |
|
437 |
||
438 |
def test_unknown_record_type(self): |
|
439 |
"""Test the formatting of UnknownRecordTypeError."""
|
|
440 |
e = errors.UnknownRecordTypeError("X") |
|
441 |
self.assertEqual( |
|
442 |
"Unknown record type: 'X'", |
|
443 |
str(e)) |
|
444 |
||
2506.3.1
by Andrew Bennetts
More progress: |
445 |
def test_invalid_record(self): |
446 |
"""Test the formatting of InvalidRecordError."""
|
|
447 |
e = errors.InvalidRecordError("xxx") |
|
448 |
self.assertEqual( |
|
449 |
"Invalid record: xxx", |
|
450 |
str(e)) |
|
451 |
||
2506.2.6
by Andrew Bennetts
Add validate method to ContainerReader and BytesRecordReader. |
452 |
def test_container_has_excess_data(self): |
453 |
"""Test the formatting of ContainerHasExcessDataError."""
|
|
454 |
e = errors.ContainerHasExcessDataError("excess bytes") |
|
455 |
self.assertEqual( |
|
456 |
"Container has data after end marker: 'excess bytes'", |
|
457 |
str(e)) |
|
458 |
||
2506.6.1
by Andrew Bennetts
Return a callable instead of a str from read, and add more validation. |
459 |
def test_duplicate_record_name_error(self): |
460 |
"""Test the formatting of DuplicateRecordNameError."""
|
|
461 |
e = errors.DuplicateRecordNameError(u"n\xe5me".encode('utf-8')) |
|
462 |
self.assertEqual( |
|
2745.3.3
by Daniel Watkins
Changed to remove need for escaping of quotes. |
463 |
"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. |
464 |
str(e)) |
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
465 |
|
2854.1.1
by Martin Pool
Fix "unprintable error" message for BzrCheckError and others |
466 |
def test_check_error(self): |
467 |
# This has a member called 'message', which is problematic in
|
|
468 |
# python2.5 because that is a slot on the base Exception class
|
|
469 |
e = errors.BzrCheckError('example check failure') |
|
470 |
self.assertEqual( |
|
471 |
"Internal check failed: example check failure", |
|
472 |
str(e)) |
|
473 |
self.assertTrue(e.internal_error) |
|
2506.6.1
by Andrew Bennetts
Return a callable instead of a str from read, and add more validation. |
474 |
|
2535.3.40
by Andrew Bennetts
Tidy up more XXXs. |
475 |
def test_repository_data_stream_error(self): |
476 |
"""Test the formatting of RepositoryDataStreamError."""
|
|
477 |
e = errors.RepositoryDataStreamError(u"my reason") |
|
478 |
self.assertEqual( |
|
479 |
"Corrupt or incompatible data stream: my reason", str(e)) |
|
480 |
||
2978.2.1
by Alexander Belchenko
fix formatting of ImmortalPendingDeletion error message. |
481 |
def test_immortal_pending_deletion_message(self): |
482 |
err = errors.ImmortalPendingDeletion('foo') |
|
483 |
self.assertEquals( |
|
484 |
"Unable to delete transform temporary directory foo. "
|
|
485 |
"Please examine foo to see if it contains any files "
|
|
486 |
"you wish to keep, and delete it when you are done.", |
|
487 |
str(err)) |
|
488 |
||
3006.2.2
by Alexander Belchenko
tests added. |
489 |
def test_unable_create_symlink(self): |
490 |
err = errors.UnableCreateSymlink() |
|
491 |
self.assertEquals( |
|
492 |
"Unable to create symlink on this platform", |
|
493 |
str(err)) |
|
494 |
err = errors.UnableCreateSymlink(path=u'foo') |
|
495 |
self.assertEquals( |
|
496 |
"Unable to create symlink 'foo' on this platform", |
|
497 |
str(err)) |
|
498 |
err = errors.UnableCreateSymlink(path=u'\xb5') |
|
499 |
self.assertEquals( |
|
500 |
"Unable to create symlink u'\\xb5' on this platform", |
|
501 |
str(err)) |
|
502 |
||
2692.1.1
by Andrew Bennetts
Add translate_client_path method to SmartServerRequest. |
503 |
def test_invalid_url_join(self): |
504 |
"""Test the formatting of InvalidURLJoin."""
|
|
505 |
e = errors.InvalidURLJoin('Reason', 'base path', ('args',)) |
|
506 |
self.assertEqual( |
|
507 |
"Invalid URL join request: Reason: 'base path' + ('args',)", |
|
508 |
str(e)) |
|
509 |
||
3035.3.2
by Lukáš Lalinský
Add tests for InvalidBugTrackerURL. |
510 |
def test_incorrect_url(self): |
511 |
err = errors.InvalidBugTrackerURL('foo', 'http://bug.com/') |
|
512 |
self.assertEquals( |
|
513 |
("The URL for bug tracker \"foo\" doesn't contain {id}: " |
|
514 |
"http://bug.com/"), |
|
515 |
str(err)) |
|
516 |
||
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. |
517 |
def test_unable_encode_path(self): |
518 |
err = errors.UnableEncodePath('foo', 'executable') |
|
3234.2.8
by Alexander Belchenko
fix grammar in formatting string of UnableEncodePath error. |
519 |
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. |
520 |
"user encoding " + osutils.get_user_encoding(), |
521 |
str(err)) |
|
522 |
||
3246.3.4
by Daniel Watkins
Added test. |
523 |
def test_unknown_format(self): |
524 |
err = errors.UnknownFormatError('bar', kind='foo') |
|
525 |
self.assertEquals("Unknown foo format: 'bar'", str(err)) |
|
526 |
||
3398.1.29
by Ian Clatworthy
add UnknownRules class & test |
527 |
def test_unknown_rules(self): |
528 |
err = errors.UnknownRules(['foo', 'bar']) |
|
529 |
self.assertEquals("Unknown rules detected: foo, bar.", str(err)) |
|
530 |
||
3577.1.1
by Andrew Bennetts
Cherry-pick TipChangeRejected changes from pre-branch-tip-changed-hook loom. |
531 |
def test_hook_failed(self): |
532 |
# Create an exc_info tuple by raising and catching an exception.
|
|
533 |
try: |
|
534 |
1/0 |
|
535 |
except ZeroDivisionError: |
|
536 |
exc_info = sys.exc_info() |
|
537 |
err = errors.HookFailed('hook stage', 'hook name', exc_info) |
|
538 |
self.assertStartsWith( |
|
539 |
str(err), 'Hook \'hook name\' during hook stage failed:\n') |
|
540 |
self.assertEndsWith( |
|
541 |
str(err), 'integer division or modulo by zero') |
|
542 |
||
543 |
def test_tip_change_rejected(self): |
|
544 |
err = errors.TipChangeRejected(u'Unicode message\N{INTERROBANG}') |
|
545 |
self.assertEquals( |
|
546 |
u'Tip change rejected: Unicode message\N{INTERROBANG}', |
|
547 |
unicode(err)) |
|
548 |
self.assertEquals( |
|
549 |
'Tip change rejected: Unicode message\xe2\x80\xbd', |
|
550 |
str(err)) |
|
551 |
||
3690.1.1
by Andrew Bennetts
Unexpected error responses from a smart server no longer cause the client to traceback. |
552 |
def test_error_from_smart_server(self): |
553 |
error_tuple = ('error', 'tuple') |
|
554 |
err = errors.ErrorFromSmartServer(error_tuple) |
|
555 |
self.assertEquals( |
|
556 |
"Error received from smart server: ('error', 'tuple')", str(err)) |
|
557 |
||
558 |
def test_untranslateable_error_from_smart_server(self): |
|
559 |
error_tuple = ('error', 'tuple') |
|
560 |
orig_err = errors.ErrorFromSmartServer(error_tuple) |
|
3690.1.2
by Andrew Bennetts
Rename UntranslateableErrorFromSmartServer -> UnknownErrorFromSmartServer. |
561 |
err = errors.UnknownErrorFromSmartServer(orig_err) |
3690.1.1
by Andrew Bennetts
Unexpected error responses from a smart server no longer cause the client to traceback. |
562 |
self.assertEquals( |
563 |
"Server sent an unexpected error: ('error', 'tuple')", str(err)) |
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
564 |
|
3883.2.3
by Andrew Bennetts
Add test, tweak traceback formatting. |
565 |
def test_smart_message_handler_error(self): |
566 |
# Make an exc_info tuple.
|
|
567 |
try: |
|
568 |
raise Exception("example error") |
|
569 |
except Exception: |
|
570 |
exc_info = sys.exc_info() |
|
571 |
err = errors.SmartMessageHandlerError(exc_info) |
|
572 |
self.assertStartsWith( |
|
573 |
str(err), "The message handler raised an exception:\n") |
|
574 |
self.assertEndsWith(str(err), "Exception: example error\n") |
|
3690.1.1
by Andrew Bennetts
Unexpected error responses from a smart server no longer cause the client to traceback. |
575 |
|
3983.1.8
by Daniel Watkins
Added MustHaveWorkingTree error and accompanying test. |
576 |
def test_must_have_working_tree(self): |
577 |
err = errors.MustHaveWorkingTree('foo', 'bar') |
|
3983.1.10
by Daniel Watkins
Made exception message slightly better. |
578 |
self.assertEqual(str(err), "Branching 'bar'(foo) must create a" |
579 |
" working tree.") |
|
3983.1.8
by Daniel Watkins
Added MustHaveWorkingTree error and accompanying test. |
580 |
|
3586.1.1
by Ian Clatworthy
add view-related errors and tests |
581 |
def test_no_such_view(self): |
582 |
err = errors.NoSuchView('foo') |
|
583 |
self.assertEquals("No such view: foo.", str(err)) |
|
584 |
||
585 |
def test_views_not_supported(self): |
|
586 |
err = errors.ViewsNotSupported('atree') |
|
587 |
err_str = str(err) |
|
588 |
self.assertStartsWith(err_str, "Views are not supported by ") |
|
589 |
self.assertEndsWith(err_str, "; use 'bzr upgrade' to change your " |
|
590 |
"tree to a later format.") |
|
591 |
||
3586.1.9
by Ian Clatworthy
first cut at view command |
592 |
def test_file_outside_view(self): |
593 |
err = errors.FileOutsideView('baz', ['foo', 'bar']) |
|
594 |
self.assertEquals('Specified file "baz" is outside the current view: ' |
|
595 |
'foo, bar', str(err)) |
|
596 |
||
3990.2.2
by Daniel Watkins
Added InvalidShelfId error and accompanying test. |
597 |
def test_invalid_shelf_id(self): |
598 |
invalid_id = "foo" |
|
599 |
err = errors.InvalidShelfId(invalid_id) |
|
3999.1.1
by Ian Clatworthy
Improve shelf documentation & fix backtrace (Daniel Watkins) |
600 |
self.assertEqual('"foo" is not a valid shelf id, ' |
601 |
'try a number instead.', str(err)) |
|
3990.2.2
by Daniel Watkins
Added InvalidShelfId error and accompanying test. |
602 |
|
4002.1.7
by Andrew Bennetts
Rename UnresumableWriteGroups to UnresumableWriteGroup. |
603 |
def test_unresumable_write_group(self): |
4002.1.1
by Andrew Bennetts
Implement suspend_write_group/resume_write_group. |
604 |
repo = "dummy repo" |
605 |
wg_tokens = ['token'] |
|
606 |
reason = "a reason" |
|
4002.1.7
by Andrew Bennetts
Rename UnresumableWriteGroups to UnresumableWriteGroup. |
607 |
err = errors.UnresumableWriteGroup(repo, wg_tokens, reason) |
4002.1.1
by Andrew Bennetts
Implement suspend_write_group/resume_write_group. |
608 |
self.assertEqual( |
4002.1.7
by Andrew Bennetts
Rename UnresumableWriteGroups to UnresumableWriteGroup. |
609 |
"Repository dummy repo cannot resume write group "
|
4002.1.1
by Andrew Bennetts
Implement suspend_write_group/resume_write_group. |
610 |
"['token']: a reason", str(err)) |
611 |
||
612 |
def test_unsuspendable_write_group(self): |
|
613 |
repo = "dummy repo" |
|
614 |
err = errors.UnsuspendableWriteGroup(repo) |
|
615 |
self.assertEqual( |
|
616 |
'Repository dummy repo cannot suspend a write group.', str(err)) |
|
617 |
||
2116.3.1
by John Arbash Meinel
Cleanup error tests |
618 |
|
2067.3.1
by Martin Pool
Clean up BzrNewError, other exception classes and users. |
619 |
class PassThroughError(errors.BzrError): |
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
620 |
|
2067.3.1
by Martin Pool
Clean up BzrNewError, other exception classes and users. |
621 |
_fmt = """Pass through %(foo)s and %(bar)s""" |
2116.3.1
by John Arbash Meinel
Cleanup error tests |
622 |
|
623 |
def __init__(self, foo, bar): |
|
2067.3.1
by Martin Pool
Clean up BzrNewError, other exception classes and users. |
624 |
errors.BzrError.__init__(self, foo=foo, bar=bar) |
625 |
||
626 |
||
627 |
class ErrorWithBadFormat(errors.BzrError): |
|
628 |
||
629 |
_fmt = """One format specifier: %(thing)s""" |
|
630 |
||
631 |
||
632 |
class ErrorWithNoFormat(errors.BzrError): |
|
633 |
"""This class has a docstring but no format string."""
|
|
2116.3.1
by John Arbash Meinel
Cleanup error tests |
634 |
|
635 |
||
636 |
class TestErrorFormatting(TestCase): |
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
637 |
|
2116.3.1
by John Arbash Meinel
Cleanup error tests |
638 |
def test_always_str(self): |
639 |
e = PassThroughError(u'\xb5', 'bar') |
|
640 |
self.assertIsInstance(e.__str__(), str) |
|
641 |
# In Python str(foo) *must* return a real byte string
|
|
642 |
# not a Unicode string. The following line would raise a
|
|
643 |
# Unicode error, because it tries to call str() on the string
|
|
644 |
# returned from e.__str__(), and it has non ascii characters
|
|
645 |
s = str(e) |
|
646 |
self.assertEqual('Pass through \xc2\xb5 and bar', s) |
|
647 |
||
2067.3.1
by Martin Pool
Clean up BzrNewError, other exception classes and users. |
648 |
def test_missing_format_string(self): |
649 |
e = ErrorWithNoFormat(param='randomvalue') |
|
2067.3.3
by Martin Pool
merge bzr.dev and reconcile several changes, also some test fixes |
650 |
s = self.callDeprecated( |
651 |
['ErrorWithNoFormat uses its docstring as a format, it should use _fmt instead'], |
|
652 |
lambda x: str(x), e) |
|
653 |
## s = str(e)
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
654 |
self.assertEqual(s, |
2067.3.3
by Martin Pool
merge bzr.dev and reconcile several changes, also some test fixes |
655 |
"This class has a docstring but no format string.") |
2067.3.1
by Martin Pool
Clean up BzrNewError, other exception classes and users. |
656 |
|
2116.3.1
by John Arbash Meinel
Cleanup error tests |
657 |
def test_mismatched_format_args(self): |
658 |
# Even though ErrorWithBadFormat's format string does not match the
|
|
659 |
# arguments we constructing it with, we can still stringify an instance
|
|
660 |
# of this exception. The resulting string will say its unprintable.
|
|
661 |
e = ErrorWithBadFormat(not_thing='x') |
|
662 |
self.assertStartsWith( |
|
2067.3.1
by Martin Pool
Clean up BzrNewError, other exception classes and users. |
663 |
str(e), 'Unprintable exception ErrorWithBadFormat') |