bzr branch
http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
1 |
# Copyright (C) 2006-2012, 2016 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 |
#
|
3 |
# This program is free software; you can redistribute it and/or modify
|
|
4 |
# it under the terms of the GNU General Public License as published by
|
|
5 |
# the Free Software Foundation; either version 2 of the License, or
|
|
6 |
# (at your option) any later version.
|
|
7 |
#
|
|
8 |
# This program is distributed in the hope that it will be useful,
|
|
9 |
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
10 |
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
11 |
# GNU General Public License for more details.
|
|
12 |
#
|
|
13 |
# You should have received a copy of the GNU General Public License
|
|
14 |
# along with this program; if not, write to the Free Software
|
|
4183.7.1
by Sabin Iacob
update FSF mailing address |
15 |
# 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. |
16 |
|
17 |
"""Tests for the formatting and construction of errors."""
|
|
18 |
||
5050.8.1
by Parth Malwankar
added test to ensure that BzrError subclasses dont use "message" as a name |
19 |
import inspect |
20 |
import re |
|
4634.1.2
by Martin Pool
Add test for CannotBindAddress |
21 |
import socket |
3577.1.1
by Andrew Bennetts
Cherry-pick TipChangeRejected changes from pre-branch-tip-changed-hook loom. |
22 |
import sys |
4634.1.2
by Martin Pool
Add test for CannotBindAddress |
23 |
|
6624
by Jelmer Vernooij
Merge Python3 porting work ('py3 pokes') |
24 |
from .. import ( |
6472.2.2
by Jelmer Vernooij
Use controldir rather than bzrdir in a couple more places. |
25 |
controldir, |
1948.1.6
by John Arbash Meinel
Make BzrNewError always return a str object |
26 |
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. |
27 |
osutils, |
6670.3.1
by Martin
Initial work to make errors module Python 3 compatible |
28 |
tests, |
3200.2.1
by Robert Collins
* The ``register-branch`` command will now use the public url of the branch |
29 |
urlutils, |
1948.1.6
by John Arbash Meinel
Make BzrNewError always return a str object |
30 |
)
|
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. |
31 |
|
32 |
||
6670.3.1
by Martin
Initial work to make errors module Python 3 compatible |
33 |
class TestErrors(tests.TestCase): |
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. |
34 |
|
5050.8.1
by Parth Malwankar
added test to ensure that BzrError subclasses dont use "message" as a name |
35 |
def test_no_arg_named_message(self): |
36 |
"""Ensure the __init__ and _fmt in errors do not have "message" arg. |
|
37 |
||
38 |
This test fails if __init__ or _fmt in errors has an argument
|
|
39 |
named "message" as this can cause errors in some Python versions.
|
|
40 |
Python 2.5 uses a slot for StandardError.message.
|
|
41 |
See bug #603461
|
|
42 |
"""
|
|
6798.1.1
by Jelmer Vernooij
Properly escape backslashes. |
43 |
fmt_pattern = re.compile("%\\(message\\)[sir]") |
5050.8.3
by Parth Malwankar
use __subclasses__ |
44 |
for c in errors.BzrError.__subclasses__(): |
45 |
init = getattr(c, '__init__', None) |
|
46 |
fmt = getattr(c, '_fmt', None) |
|
5050.8.1
by Parth Malwankar
added test to ensure that BzrError subclasses dont use "message" as a name |
47 |
if init: |
7479.2.1
by Jelmer Vernooij
Drop python2 support. |
48 |
args = inspect.getfullargspec(init)[0] |
5050.8.1
by Parth Malwankar
added test to ensure that BzrError subclasses dont use "message" as a name |
49 |
self.assertFalse('message' in args, |
7143.15.2
by Jelmer Vernooij
Run autopep8. |
50 |
('Argument name "message" not allowed for ' |
51 |
'"errors.%s.__init__"' % c.__name__)) |
|
5050.8.1
by Parth Malwankar
added test to ensure that BzrError subclasses dont use "message" as a name |
52 |
if fmt and fmt_pattern.search(fmt): |
53 |
self.assertFalse(True, ('"message" not allowed in ' |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
54 |
'"errors.%s._fmt"' % c.__name__)) |
5050.8.1
by Parth Malwankar
added test to ensure that BzrError subclasses dont use "message" as a name |
55 |
|
3287.20.2
by John Arbash Meinel
Raise a clear error about the offending filename when there is a filename with bad characters. |
56 |
def test_bad_filename_encoding(self): |
6670.3.1
by Martin
Initial work to make errors module Python 3 compatible |
57 |
error = errors.BadFilenameEncoding(b'bad/filen\xe5me', 'UTF-8') |
6670.3.2
by Martin
Avoid PendingDeprecationWarning from assertRegexpMatches |
58 |
self.assertContainsRe( |
6670.3.1
by Martin
Initial work to make errors module Python 3 compatible |
59 |
str(error), |
60 |
"^Filename b?'bad/filen\\\\xe5me' is not valid in your current" |
|
61 |
" filesystem encoding UTF-8$") |
|
3287.20.2
by John Arbash Meinel
Raise a clear error about the offending filename when there is a filename with bad characters. |
62 |
|
2255.7.16
by John Arbash Meinel
Make sure adding a duplicate file_id raises DuplicateFileId. |
63 |
def test_duplicate_file_id(self): |
64 |
error = errors.DuplicateFileId('a_file_id', 'foo') |
|
65 |
self.assertEqualDiff('File id {a_file_id} already exists in inventory' |
|
66 |
' as foo', str(error)) |
|
67 |
||
2432.1.19
by Robert Collins
Ensure each HelpIndex has a unique prefix. |
68 |
def test_duplicate_help_prefix(self): |
69 |
error = errors.DuplicateHelpPrefix('foo') |
|
70 |
self.assertEqualDiff('The prefix foo is in the help search path twice.', |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
71 |
str(error)) |
2432.1.19
by Robert Collins
Ensure each HelpIndex has a unique prefix. |
72 |
|
3445.1.1
by John Arbash Meinel
Start working on a new Graph api to make finding revision numbers faster. |
73 |
def test_ghost_revisions_have_no_revno(self): |
74 |
error = errors.GhostRevisionsHaveNoRevno('target', 'ghost_rev') |
|
75 |
self.assertEqualDiff("Could not determine revno for {target} because" |
|
76 |
" its ancestry shows a ghost at {ghost_rev}", |
|
77 |
str(error)) |
|
78 |
||
6672.1.2
by Jelmer Vernooij
Remove breezy.api. |
79 |
def test_incompatibleVersion(self): |
80 |
error = errors.IncompatibleVersion("module", [(4, 5, 6), (7, 8, 9)], |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
81 |
(1, 2, 3)) |
2550.2.3
by Robert Collins
Add require_api API. |
82 |
self.assertEqualDiff( |
6672.1.2
by Jelmer Vernooij
Remove breezy.api. |
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).', |
|
2550.2.3
by Robert Collins
Add require_api API. |
86 |
str(error)) |
87 |
||
3207.2.2
by John Arbash Meinel
Fix bug #187169, when an invalid delta is supplied to update_basis_by_delta |
88 |
def test_inconsistent_delta(self): |
89 |
error = errors.InconsistentDelta('path', 'file-id', 'reason for foo') |
|
90 |
self.assertEqualDiff( |
|
3221.1.8
by Martin Pool
Update error format in test_inconsistent_delta |
91 |
"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 |
92 |
"reason: reason for foo", |
93 |
str(error)) |
|
94 |
||
4505.5.1
by Robert Collins
Add more generic InconsistentDeltaDelta error class for use when the exact cause of an inconsistent delta isn't trivially accessible. |
95 |
def test_inconsistent_delta_delta(self): |
96 |
error = errors.InconsistentDeltaDelta([], 'reason') |
|
97 |
self.assertEqualDiff( |
|
98 |
"An inconsistent delta was supplied: []\nreason: reason", |
|
99 |
str(error)) |
|
100 |
||
2634.1.1
by Robert Collins
(robertc) Reinstate the accidentally backed out external_url patch. |
101 |
def test_in_process_transport(self): |
102 |
error = errors.InProcessTransport('fpp') |
|
103 |
self.assertEqualDiff( |
|
104 |
"The transport 'fpp' is only accessible within this process.", |
|
105 |
str(error)) |
|
106 |
||
3059.2.12
by Vincent Ladeuil
Spiv review feedback. |
107 |
def test_invalid_http_range(self): |
108 |
error = errors.InvalidHttpRange('path', |
|
109 |
'Content-Range: potatoes 0-00/o0oo0', |
|
110 |
'bad range') |
|
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
111 |
self.assertEqual("Invalid http range" |
112 |
" 'Content-Range: potatoes 0-00/o0oo0'"
|
|
113 |
" for path: bad range", |
|
114 |
str(error)) |
|
3059.2.12
by Vincent Ladeuil
Spiv review feedback. |
115 |
|
116 |
def test_invalid_range(self): |
|
117 |
error = errors.InvalidRange('path', 12, 'bad range') |
|
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
118 |
self.assertEqual("Invalid range access in path at 12: bad range", |
119 |
str(error)) |
|
3059.2.12
by Vincent Ladeuil
Spiv review feedback. |
120 |
|
1986.5.3
by Robert Collins
New method ``WorkingTree.flush()`` which will write the current memory |
121 |
def test_inventory_modified(self): |
122 |
error = errors.InventoryModified("a tree to be repred") |
|
123 |
self.assertEqualDiff("The current inventory for the tree 'a tree to " |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
124 |
"be repred' has been modified, so a clean inventory cannot be "
|
125 |
"read without data loss.", |
|
126 |
str(error)) |
|
1986.5.3
by Robert Collins
New method ``WorkingTree.flush()`` which will write the current memory |
127 |
|
4294.2.8
by Robert Collins
Reduce round trips pushing new branches substantially. |
128 |
def test_jail_break(self): |
129 |
error = errors.JailBreak("some url") |
|
130 |
self.assertEqualDiff("An attempt to access a url outside the server" |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
131 |
" jail was made: 'some url'.", |
132 |
str(error)) |
|
4294.2.8
by Robert Collins
Reduce round trips pushing new branches substantially. |
133 |
|
2255.2.145
by Robert Collins
Support unbreakable locks for trees. |
134 |
def test_lock_active(self): |
135 |
error = errors.LockActive("lock description") |
|
136 |
self.assertEqualDiff("The lock for 'lock description' is in use and " |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
137 |
"cannot be broken.", |
138 |
str(error)) |
|
2255.2.145
by Robert Collins
Support unbreakable locks for trees. |
139 |
|
4634.161.1
by Andrew Bennetts
Add LockCorrupt error, and use it to provide nicer handling of unparseable lock/held/info files. |
140 |
def test_lock_corrupt(self): |
141 |
error = errors.LockCorrupt("corruption info") |
|
142 |
self.assertEqualDiff("Lock is apparently held, but corrupted: " |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
143 |
"corruption info\n" |
144 |
"Use 'brz break-lock' to clear it", |
|
145 |
str(error)) |
|
4634.161.1
by Andrew Bennetts
Add LockCorrupt error, and use it to provide nicer handling of unparseable lock/held/info files. |
146 |
|
2018.2.3
by Andrew Bennetts
Starting factoring out the smart server client "medium" from the protocol. |
147 |
def test_medium_not_connected(self): |
148 |
error = errors.MediumNotConnected("a medium") |
|
149 |
self.assertEqualDiff( |
|
150 |
"The medium 'a medium' is not connected.", str(error)) |
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
151 |
|
2018.2.3
by Andrew Bennetts
Starting factoring out the smart server client "medium" from the protocol. |
152 |
def test_no_smart_medium(self): |
153 |
error = errors.NoSmartMedium("a transport") |
|
154 |
self.assertEqualDiff("The transport 'a transport' cannot tunnel the " |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
155 |
"smart protocol.", |
156 |
str(error)) |
|
2018.2.3
by Andrew Bennetts
Starting factoring out the smart server client "medium" from the protocol. |
157 |
|
1988.2.1
by Robert Collins
WorkingTree has a new api ``unversion`` which allow the unversioning of |
158 |
def test_no_such_id(self): |
159 |
error = errors.NoSuchId("atree", "anid") |
|
2745.3.2
by Daniel Watkins
Updated tests to reflect new error text. |
160 |
self.assertEqualDiff("The file id \"anid\" is not present in the tree " |
7143.15.2
by Jelmer Vernooij
Run autopep8. |
161 |
"atree.", |
162 |
str(error)) |
|
1534.5.7
by Robert Collins
Start factoring out the upgrade policy logic. |
163 |
|
1908.11.1
by Robert Collins
Add a new method ``Tree.revision_tree`` which allows access to cached |
164 |
def test_no_such_revision_in_tree(self): |
165 |
error = errors.NoSuchRevisionInTree("atree", "anid") |
|
2745.3.3
by Daniel Watkins
Changed to remove need for escaping of quotes. |
166 |
self.assertEqualDiff("The revision id {anid} is not present in the" |
167 |
" tree atree.", str(error)) |
|
1908.11.1
by Robert Collins
Add a new method ``Tree.revision_tree`` which allows access to cached |
168 |
self.assertIsInstance(error, errors.NoSuchRevision) |
169 |
||
3221.11.2
by Robert Collins
Create basic stackable branch facility. |
170 |
def test_not_stacked(self): |
171 |
error = errors.NotStacked('a branch') |
|
172 |
self.assertEqualDiff("The branch 'a branch' is not stacked.", |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
173 |
str(error)) |
3221.11.2
by Robert Collins
Create basic stackable branch facility. |
174 |
|
1986.5.3
by Robert Collins
New method ``WorkingTree.flush()`` which will write the current memory |
175 |
def test_not_write_locked(self): |
176 |
error = errors.NotWriteLocked('a thing to repr') |
|
177 |
self.assertEqualDiff("'a thing to repr' is not write locked but needs " |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
178 |
"to be.", |
179 |
str(error)) |
|
1986.5.3
by Robert Collins
New method ``WorkingTree.flush()`` which will write the current memory |
180 |
|
2872.5.1
by Martin Pool
Avoid internal error tracebacks on failure to lock on readonly transport (#129701). |
181 |
def test_lock_failed(self): |
7143.15.2
by Jelmer Vernooij
Run autopep8. |
182 |
error = errors.LockFailed( |
183 |
'http://canonical.com/', 'readonly transport') |
|
2872.5.1
by Martin Pool
Avoid internal error tracebacks on failure to lock on readonly transport (#129701). |
184 |
self.assertEqualDiff("Cannot lock http://canonical.com/: readonly transport", |
7143.15.2
by Jelmer Vernooij
Run autopep8. |
185 |
str(error)) |
2872.5.1
by Martin Pool
Avoid internal error tracebacks on failure to lock on readonly transport (#129701). |
186 |
self.assertFalse(error.internal_error) |
187 |
||
2018.2.4
by Robert Collins
separate out the client medium from the client encoding protocol for the smart server. |
188 |
def test_too_many_concurrent_requests(self): |
189 |
error = errors.TooManyConcurrentRequests("a medium") |
|
190 |
self.assertEqualDiff("The medium 'a medium' has reached its concurrent " |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
191 |
"request limit. Be sure to finish_writing and finish_reading on "
|
192 |
"the currently open request.", |
|
193 |
str(error)) |
|
2018.2.4
by Robert Collins
separate out the client medium from the client encoding protocol for the smart server. |
194 |
|
3350.3.3
by Robert Collins
Functional get_record_stream interface tests covering full interface. |
195 |
def test_unavailable_representation(self): |
7143.15.2
by Jelmer Vernooij
Run autopep8. |
196 |
error = errors.UnavailableRepresentation( |
197 |
('key',), "mpdiff", "fulltext") |
|
3350.3.3
by Robert Collins
Functional get_record_stream interface tests covering full interface. |
198 |
self.assertEqualDiff("The encoding 'mpdiff' is not available for key " |
7143.15.2
by Jelmer Vernooij
Run autopep8. |
199 |
"('key',) which is encoded as 'fulltext'.", |
200 |
str(error)) |
|
3350.3.3
by Robert Collins
Functional get_record_stream interface tests covering full interface. |
201 |
|
4462.3.2
by Robert Collins
Do not stack on the same branch/repository anymore. This was never supported and would generally result in infinite recursion. Fixes bug 376243. |
202 |
def test_unstackable_location(self): |
203 |
error = errors.UnstackableLocationError('foo', 'bar') |
|
204 |
self.assertEqualDiff("The branch 'foo' cannot be stacked on 'bar'.", |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
205 |
str(error)) |
4462.3.2
by Robert Collins
Do not stack on the same branch/repository anymore. This was never supported and would generally result in infinite recursion. Fixes bug 376243. |
206 |
|
3221.11.2
by Robert Collins
Create basic stackable branch facility. |
207 |
def test_unstackable_repository_format(self): |
208 |
format = u'foo' |
|
209 |
url = "/foo" |
|
210 |
error = errors.UnstackableRepositoryFormat(format, url) |
|
211 |
self.assertEqualDiff( |
|
212 |
"The repository '/foo'(foo) is not a stackable format. "
|
|
213 |
"You will need to upgrade the repository to permit branch stacking.", |
|
214 |
str(error)) |
|
215 |
||
1534.5.7
by Robert Collins
Start factoring out the upgrade policy logic. |
216 |
def test_up_to_date(self): |
5582.10.51
by Jelmer Vernooij
Remove use of BzrDirFormat4 in test_errors. |
217 |
error = errors.UpToDateFormat("someformat") |
218 |
self.assertEqualDiff( |
|
219 |
"The branch format someformat is already at the most "
|
|
220 |
"recent format.", str(error)) |
|
1570.1.13
by Robert Collins
Check for incorrect revision parentage in the weave during revision access. |
221 |
|
2052.6.1
by Robert Collins
``Transport.get`` has had its interface made more clear for ease of use. |
222 |
def test_read_error(self): |
223 |
# a unicode path to check that %r is being used.
|
|
224 |
path = u'a path' |
|
225 |
error = errors.ReadError(path) |
|
6670.3.2
by Martin
Avoid PendingDeprecationWarning from assertRegexpMatches |
226 |
self.assertContainsRe(str(error), "^Error reading from u?'a path'.$") |
2592.1.12
by Robert Collins
Handle basic node adds. |
227 |
|
2067.3.1
by Martin Pool
Clean up BzrNewError, other exception classes and users. |
228 |
def test_bzrerror_from_literal_string(self): |
229 |
# Some code constructs BzrError from a literal string, in which case
|
|
230 |
# no further formatting is done. (I'm not sure raising the base class
|
|
231 |
# is a great idea, but if the exception is not intended to be caught
|
|
232 |
# perhaps no more is needed.)
|
|
233 |
try: |
|
234 |
raise errors.BzrError('this is my errors; %d is not expanded') |
|
6619.3.2
by Jelmer Vernooij
Apply 2to3 except fix. |
235 |
except errors.BzrError as e: |
2067.3.1
by Martin Pool
Clean up BzrNewError, other exception classes and users. |
236 |
self.assertEqual('this is my errors; %d is not expanded', str(e)) |
237 |
||
2018.2.4
by Robert Collins
separate out the client medium from the client encoding protocol for the smart server. |
238 |
def test_reading_completed(self): |
239 |
error = errors.ReadingCompleted("a request") |
|
240 |
self.assertEqualDiff("The MediumRequest 'a request' has already had " |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
241 |
"finish_reading called upon it - the request has been completed and"
|
242 |
" no more data may be read.", |
|
243 |
str(error)) |
|
2018.2.4
by Robert Collins
separate out the client medium from the client encoding protocol for the smart server. |
244 |
|
245 |
def test_writing_completed(self): |
|
246 |
error = errors.WritingCompleted("a request") |
|
247 |
self.assertEqualDiff("The MediumRequest 'a request' has already had " |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
248 |
"finish_writing called upon it - accept bytes may not be called "
|
249 |
"anymore.", |
|
250 |
str(error)) |
|
2018.2.4
by Robert Collins
separate out the client medium from the client encoding protocol for the smart server. |
251 |
|
252 |
def test_writing_not_completed(self): |
|
253 |
error = errors.WritingNotComplete("a request") |
|
254 |
self.assertEqualDiff("The MediumRequest 'a request' has not has " |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
255 |
"finish_writing called upon it - until the write phase is complete"
|
256 |
" no data may be read.", |
|
257 |
str(error)) |
|
2018.2.4
by Robert Collins
separate out the client medium from the client encoding protocol for the smart server. |
258 |
|
2052.6.1
by Robert Collins
``Transport.get`` has had its interface made more clear for ease of use. |
259 |
def test_transport_not_possible(self): |
260 |
error = errors.TransportNotPossible('readonly', 'original error') |
|
261 |
self.assertEqualDiff('Transport operation not possible:' |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
262 |
' readonly original error', str(error)) |
2052.4.4
by John Arbash Meinel
Create a SocketConnectionError to make creating nice errors easier |
263 |
|
264 |
def assertSocketConnectionError(self, expected, *args, **kwargs): |
|
265 |
"""Check the formatting of a SocketConnectionError exception""" |
|
266 |
e = errors.SocketConnectionError(*args, **kwargs) |
|
267 |
self.assertEqual(expected, str(e)) |
|
268 |
||
269 |
def test_socket_connection_error(self): |
|
270 |
"""Test the formatting of SocketConnectionError""" |
|
271 |
||
272 |
# There should be a default msg about failing to connect
|
|
273 |
# we only require a host name.
|
|
274 |
self.assertSocketConnectionError( |
|
275 |
'Failed to connect to ahost', |
|
276 |
'ahost') |
|
277 |
||
278 |
# If port is None, we don't put :None
|
|
279 |
self.assertSocketConnectionError( |
|
280 |
'Failed to connect to ahost', |
|
281 |
'ahost', port=None) |
|
282 |
# But if port is supplied we include it
|
|
283 |
self.assertSocketConnectionError( |
|
284 |
'Failed to connect to ahost:22', |
|
285 |
'ahost', port=22) |
|
286 |
||
287 |
# We can also supply extra information about the error
|
|
288 |
# with or without a port
|
|
289 |
self.assertSocketConnectionError( |
|
290 |
'Failed to connect to ahost:22; bogus error', |
|
291 |
'ahost', port=22, orig_error='bogus error') |
|
292 |
self.assertSocketConnectionError( |
|
293 |
'Failed to connect to ahost; bogus error', |
|
294 |
'ahost', orig_error='bogus error') |
|
295 |
# An exception object can be passed rather than a string
|
|
296 |
orig_error = ValueError('bad value') |
|
297 |
self.assertSocketConnectionError( |
|
298 |
'Failed to connect to ahost; %s' % (str(orig_error),), |
|
299 |
host='ahost', orig_error=orig_error) |
|
300 |
||
301 |
# And we can supply a custom failure message
|
|
302 |
self.assertSocketConnectionError( |
|
303 |
'Unable to connect to ssh host ahost:444; my_error', |
|
304 |
host='ahost', port=444, msg='Unable to connect to ssh host', |
|
305 |
orig_error='my_error') |
|
306 |
||
3535.8.2
by James Westby
Incorporate spiv's feedback. |
307 |
def test_target_not_branch(self): |
308 |
"""Test the formatting of TargetNotBranch.""" |
|
309 |
error = errors.TargetNotBranch('foo') |
|
310 |
self.assertEqual( |
|
311 |
"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. |
312 |
"order to merge this merge directive and the target "
|
3535.8.3
by James Westby
Use location instead of branch as suggested by Robert. |
313 |
"location specified in the merge directive is not a branch: "
|
3535.8.2
by James Westby
Incorporate spiv's feedback. |
314 |
"foo.", str(error)) |
315 |
||
2018.5.163
by Andrew Bennetts
Deal with various review comments from Robert. |
316 |
def test_unexpected_smart_server_response(self): |
317 |
e = errors.UnexpectedSmartServerResponse(('not yes',)) |
|
318 |
self.assertEqual( |
|
319 |
"Could not understand response from smart server: ('not yes',)", |
|
320 |
str(e)) |
|
2052.4.4
by John Arbash Meinel
Create a SocketConnectionError to make creating nice errors easier |
321 |
|
2506.2.1
by Andrew Bennetts
Start implementing container format reading and writing. |
322 |
def test_unknown_container_format(self): |
323 |
"""Test the formatting of UnknownContainerFormatError.""" |
|
324 |
e = errors.UnknownContainerFormatError('bad format string') |
|
325 |
self.assertEqual( |
|
326 |
"Unrecognised container format: 'bad format string'", |
|
327 |
str(e)) |
|
328 |
||
329 |
def test_unexpected_end_of_container(self): |
|
330 |
"""Test the formatting of UnexpectedEndOfContainerError.""" |
|
331 |
e = errors.UnexpectedEndOfContainerError() |
|
332 |
self.assertEqual( |
|
333 |
"Unexpected end of container stream", str(e)) |
|
334 |
||
335 |
def test_unknown_record_type(self): |
|
336 |
"""Test the formatting of UnknownRecordTypeError.""" |
|
337 |
e = errors.UnknownRecordTypeError("X") |
|
338 |
self.assertEqual( |
|
339 |
"Unknown record type: 'X'", |
|
340 |
str(e)) |
|
341 |
||
2506.3.1
by Andrew Bennetts
More progress: |
342 |
def test_invalid_record(self): |
343 |
"""Test the formatting of InvalidRecordError.""" |
|
344 |
e = errors.InvalidRecordError("xxx") |
|
345 |
self.assertEqual( |
|
346 |
"Invalid record: xxx", |
|
347 |
str(e)) |
|
348 |
||
2506.2.6
by Andrew Bennetts
Add validate method to ContainerReader and BytesRecordReader. |
349 |
def test_container_has_excess_data(self): |
350 |
"""Test the formatting of ContainerHasExcessDataError.""" |
|
351 |
e = errors.ContainerHasExcessDataError("excess bytes") |
|
352 |
self.assertEqual( |
|
353 |
"Container has data after end marker: 'excess bytes'", |
|
354 |
str(e)) |
|
355 |
||
2506.6.1
by Andrew Bennetts
Return a callable instead of a str from read, and add more validation. |
356 |
def test_duplicate_record_name_error(self): |
357 |
"""Test the formatting of DuplicateRecordNameError.""" |
|
6670.3.1
by Martin
Initial work to make errors module Python 3 compatible |
358 |
e = errors.DuplicateRecordNameError(b"n\xc3\xa5me") |
2506.6.1
by Andrew Bennetts
Return a callable instead of a str from read, and add more validation. |
359 |
self.assertEqual( |
6670.3.1
by Martin
Initial work to make errors module Python 3 compatible |
360 |
u"Container has multiple records with the same name: n\xe5me", |
7479.2.1
by Jelmer Vernooij
Drop python2 support. |
361 |
str(e)) |
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
362 |
|
2854.1.1
by Martin Pool
Fix "unprintable error" message for BzrCheckError and others |
363 |
def test_check_error(self): |
364 |
e = errors.BzrCheckError('example check failure') |
|
365 |
self.assertEqual( |
|
366 |
"Internal check failed: example check failure", |
|
367 |
str(e)) |
|
368 |
self.assertTrue(e.internal_error) |
|
2506.6.1
by Andrew Bennetts
Return a callable instead of a str from read, and add more validation. |
369 |
|
2535.3.40
by Andrew Bennetts
Tidy up more XXXs. |
370 |
def test_repository_data_stream_error(self): |
371 |
"""Test the formatting of RepositoryDataStreamError.""" |
|
372 |
e = errors.RepositoryDataStreamError(u"my reason") |
|
373 |
self.assertEqual( |
|
374 |
"Corrupt or incompatible data stream: my reason", str(e)) |
|
375 |
||
2978.2.1
by Alexander Belchenko
fix formatting of ImmortalPendingDeletion error message. |
376 |
def test_immortal_pending_deletion_message(self): |
377 |
err = errors.ImmortalPendingDeletion('foo') |
|
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
378 |
self.assertEqual( |
2978.2.1
by Alexander Belchenko
fix formatting of ImmortalPendingDeletion error message. |
379 |
"Unable to delete transform temporary directory foo. "
|
380 |
"Please examine foo to see if it contains any files "
|
|
381 |
"you wish to keep, and delete it when you are done.", |
|
382 |
str(err)) |
|
383 |
||
2692.1.1
by Andrew Bennetts
Add translate_client_path method to SmartServerRequest. |
384 |
def test_invalid_url_join(self): |
385 |
"""Test the formatting of InvalidURLJoin.""" |
|
6729.6.1
by Jelmer Vernooij
Move urlutils errors. |
386 |
e = urlutils.InvalidURLJoin('Reason', 'base path', ('args',)) |
2692.1.1
by Andrew Bennetts
Add translate_client_path method to SmartServerRequest. |
387 |
self.assertEqual( |
388 |
"Invalid URL join request: Reason: 'base path' + ('args',)", |
|
389 |
str(e)) |
|
390 |
||
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. |
391 |
def test_unable_encode_path(self): |
392 |
err = errors.UnableEncodePath('foo', 'executable') |
|
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
393 |
self.assertEqual("Unable to encode executable path 'foo' in " |
394 |
"user encoding " + osutils.get_user_encoding(), |
|
395 |
str(err)) |
|
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. |
396 |
|
3246.3.4
by Daniel Watkins
Added test. |
397 |
def test_unknown_format(self): |
398 |
err = errors.UnknownFormatError('bar', kind='foo') |
|
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
399 |
self.assertEqual("Unknown foo format: 'bar'", str(err)) |
3398.1.29
by Ian Clatworthy
add UnknownRules class & test |
400 |
|
3577.1.1
by Andrew Bennetts
Cherry-pick TipChangeRejected changes from pre-branch-tip-changed-hook loom. |
401 |
def test_tip_change_rejected(self): |
402 |
err = errors.TipChangeRejected(u'Unicode message\N{INTERROBANG}') |
|
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
403 |
self.assertEqual( |
3577.1.1
by Andrew Bennetts
Cherry-pick TipChangeRejected changes from pre-branch-tip-changed-hook loom. |
404 |
u'Tip change rejected: Unicode message\N{INTERROBANG}', |
7479.2.1
by Jelmer Vernooij
Drop python2 support. |
405 |
str(err)) |
3577.1.1
by Andrew Bennetts
Cherry-pick TipChangeRejected changes from pre-branch-tip-changed-hook loom. |
406 |
|
3690.1.1
by Andrew Bennetts
Unexpected error responses from a smart server no longer cause the client to traceback. |
407 |
def test_error_from_smart_server(self): |
408 |
error_tuple = ('error', 'tuple') |
|
409 |
err = errors.ErrorFromSmartServer(error_tuple) |
|
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
410 |
self.assertEqual( |
3690.1.1
by Andrew Bennetts
Unexpected error responses from a smart server no longer cause the client to traceback. |
411 |
"Error received from smart server: ('error', 'tuple')", str(err)) |
412 |
||
413 |
def test_untranslateable_error_from_smart_server(self): |
|
414 |
error_tuple = ('error', 'tuple') |
|
415 |
orig_err = errors.ErrorFromSmartServer(error_tuple) |
|
3690.1.2
by Andrew Bennetts
Rename UntranslateableErrorFromSmartServer -> UnknownErrorFromSmartServer. |
416 |
err = errors.UnknownErrorFromSmartServer(orig_err) |
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
417 |
self.assertEqual( |
3690.1.1
by Andrew Bennetts
Unexpected error responses from a smart server no longer cause the client to traceback. |
418 |
"Server sent an unexpected error: ('error', 'tuple')", str(err)) |
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
419 |
|
3883.2.3
by Andrew Bennetts
Add test, tweak traceback formatting. |
420 |
def test_smart_message_handler_error(self): |
421 |
# Make an exc_info tuple.
|
|
422 |
try: |
|
423 |
raise Exception("example error") |
|
424 |
except Exception: |
|
5340.15.2
by John Arbash Meinel
supercede 2.4-613247-cleanup-tests |
425 |
err = errors.SmartMessageHandlerError(sys.exc_info()) |
426 |
# GZ 2010-11-08: Should not store exc_info in exception instances.
|
|
427 |
try: |
|
428 |
self.assertStartsWith( |
|
429 |
str(err), "The message handler raised an exception:\n") |
|
430 |
self.assertEndsWith(str(err), "Exception: example error\n") |
|
431 |
finally: |
|
432 |
del err |
|
3690.1.1
by Andrew Bennetts
Unexpected error responses from a smart server no longer cause the client to traceback. |
433 |
|
4002.1.7
by Andrew Bennetts
Rename UnresumableWriteGroups to UnresumableWriteGroup. |
434 |
def test_unresumable_write_group(self): |
4002.1.1
by Andrew Bennetts
Implement suspend_write_group/resume_write_group. |
435 |
repo = "dummy repo" |
436 |
wg_tokens = ['token'] |
|
437 |
reason = "a reason" |
|
4002.1.7
by Andrew Bennetts
Rename UnresumableWriteGroups to UnresumableWriteGroup. |
438 |
err = errors.UnresumableWriteGroup(repo, wg_tokens, reason) |
4002.1.1
by Andrew Bennetts
Implement suspend_write_group/resume_write_group. |
439 |
self.assertEqual( |
4002.1.7
by Andrew Bennetts
Rename UnresumableWriteGroups to UnresumableWriteGroup. |
440 |
"Repository dummy repo cannot resume write group "
|
4002.1.1
by Andrew Bennetts
Implement suspend_write_group/resume_write_group. |
441 |
"['token']: a reason", str(err)) |
442 |
||
443 |
def test_unsuspendable_write_group(self): |
|
444 |
repo = "dummy repo" |
|
445 |
err = errors.UnsuspendableWriteGroup(repo) |
|
446 |
self.assertEqual( |
|
447 |
'Repository dummy repo cannot suspend a write group.', str(err)) |
|
448 |
||
4734.4.9
by Andrew Bennetts
More tests and comments. |
449 |
def test_not_branch_no_args(self): |
450 |
err = errors.NotBranchError('path') |
|
451 |
self.assertEqual('Not a branch: "path".', str(err)) |
|
452 |
||
5050.46.1
by Andrew Bennetts
Suppress unexpected errors during NotBranchError's call to open_repository. |
453 |
def test_not_branch_bzrdir_with_recursive_not_branch_error(self): |
454 |
class FakeBzrDir(object): |
|
455 |
def open_repository(self): |
|
456 |
# str() on the NotBranchError will trigger a call to this,
|
|
457 |
# which in turn will another, identical NotBranchError.
|
|
6653.6.6
by Jelmer Vernooij
Fix remaining tests. |
458 |
raise errors.NotBranchError('path', controldir=FakeBzrDir()) |
459 |
err = errors.NotBranchError('path', controldir=FakeBzrDir()) |
|
6670.3.1
by Martin
Initial work to make errors module Python 3 compatible |
460 |
self.assertEqual('Not a branch: "path": NotBranchError.', str(err)) |
5339.1.1
by Parth Malwankar
fixes errors.InvalidPattern to work on Python2.5 |
461 |
|
5050.7.6
by Parth Malwankar
fixed test name |
462 |
def test_recursive_bind(self): |
5050.7.5
by Parth Malwankar
better error message for RecursiveBind |
463 |
error = errors.RecursiveBind('foo_bar_branch') |
464 |
msg = ('Branch "foo_bar_branch" appears to be bound to itself. ' |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
465 |
'Please use `brz unbind` to fix.') |
5050.7.5
by Parth Malwankar
better error message for RecursiveBind |
466 |
self.assertEqualDiff(msg, str(error)) |
467 |
||
5609.58.1
by Andrew Bennetts
Fix 'Unprintable exception' when displaying RetryWithNewPacks error. |
468 |
def test_retry_with_new_packs(self): |
469 |
fake_exc_info = ('{exc type}', '{exc value}', '{exc traceback}') |
|
470 |
error = errors.RetryWithNewPacks( |
|
471 |
'{context}', reload_occurred=False, exc_info=fake_exc_info) |
|
472 |
self.assertEqual( |
|
473 |
'Pack files have changed, reload and retry. context: '
|
|
474 |
'{context} {exc value}', str(error)) |
|
475 |
||
2116.3.1
by John Arbash Meinel
Cleanup error tests |
476 |
|
2067.3.1
by Martin Pool
Clean up BzrNewError, other exception classes and users. |
477 |
class PassThroughError(errors.BzrError): |
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
478 |
|
2067.3.1
by Martin Pool
Clean up BzrNewError, other exception classes and users. |
479 |
_fmt = """Pass through %(foo)s and %(bar)s""" |
2116.3.1
by John Arbash Meinel
Cleanup error tests |
480 |
|
481 |
def __init__(self, foo, bar): |
|
2067.3.1
by Martin Pool
Clean up BzrNewError, other exception classes and users. |
482 |
errors.BzrError.__init__(self, foo=foo, bar=bar) |
483 |
||
484 |
||
485 |
class ErrorWithBadFormat(errors.BzrError): |
|
486 |
||
487 |
_fmt = """One format specifier: %(thing)s""" |
|
488 |
||
489 |
||
490 |
class ErrorWithNoFormat(errors.BzrError): |
|
5131.2.6
by Martin
Fix more tests which were failing under -OO that had been missed earlier |
491 |
__doc__ = """This class has a docstring but no format string.""" |
2116.3.1
by John Arbash Meinel
Cleanup error tests |
492 |
|
493 |
||
6670.3.1
by Martin
Initial work to make errors module Python 3 compatible |
494 |
class TestErrorFormatting(tests.TestCase): |
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
495 |
|
2116.3.1
by John Arbash Meinel
Cleanup error tests |
496 |
def test_always_str(self): |
497 |
e = PassThroughError(u'\xb5', 'bar') |
|
498 |
self.assertIsInstance(e.__str__(), str) |
|
7065.3.6
by Jelmer Vernooij
Fix some more tests. |
499 |
# In Python 2 str(foo) *must* return a real byte string
|
2116.3.1
by John Arbash Meinel
Cleanup error tests |
500 |
# not a Unicode string. The following line would raise a
|
501 |
# Unicode error, because it tries to call str() on the string
|
|
502 |
# returned from e.__str__(), and it has non ascii characters
|
|
503 |
s = str(e) |
|
7479.2.1
by Jelmer Vernooij
Drop python2 support. |
504 |
self.assertEqual('Pass through \xb5 and bar', s) |
2116.3.1
by John Arbash Meinel
Cleanup error tests |
505 |
|
2067.3.1
by Martin Pool
Clean up BzrNewError, other exception classes and users. |
506 |
def test_missing_format_string(self): |
507 |
e = ErrorWithNoFormat(param='randomvalue') |
|
6318.2.1
by Martin Packman
Remove deprecated classes and practices from bzrlib.errors |
508 |
self.assertStartsWith(str(e), |
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
509 |
"Unprintable exception ErrorWithNoFormat") |
2067.3.1
by Martin Pool
Clean up BzrNewError, other exception classes and users. |
510 |
|
2116.3.1
by John Arbash Meinel
Cleanup error tests |
511 |
def test_mismatched_format_args(self): |
512 |
# Even though ErrorWithBadFormat's format string does not match the
|
|
513 |
# arguments we constructing it with, we can still stringify an instance
|
|
514 |
# of this exception. The resulting string will say its unprintable.
|
|
515 |
e = ErrorWithBadFormat(not_thing='x') |
|
516 |
self.assertStartsWith( |
|
2067.3.1
by Martin Pool
Clean up BzrNewError, other exception classes and users. |
517 |
str(e), 'Unprintable exception ErrorWithBadFormat') |
4634.1.2
by Martin Pool
Add test for CannotBindAddress |
518 |
|
519 |
def test_cannot_bind_address(self): |
|
5243.1.2
by Martin
Point launchpad links in comments at production server rather than edge |
520 |
# see <https://bugs.launchpad.net/bzr/+bug/286871>
|
4634.1.2
by Martin Pool
Add test for CannotBindAddress |
521 |
e = errors.CannotBindAddress('example.com', 22, |
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
522 |
socket.error(13, 'Permission denied')) |
523 |
self.assertContainsRe( |
|
524 |
str(e), |
|
4634.1.2
by Martin Pool
Add test for CannotBindAddress |
525 |
r'Cannot bind address "example\.com:22":.*Permission denied') |
4976.1.1
by Jelmer Vernooij
Add FileTimestampUnavailable exception. |
526 |
|
5186.2.6
by Martin Pool
Add formatting test for TransformRenameFailed |
527 |
def test_transform_rename_failed(self): |
5186.2.7
by Martin Pool
Update other cases where transform detects failure to rename |
528 |
e = errors.TransformRenameFailed(u"from", u"to", "readonly file", 2) |
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
529 |
self.assertEqual( |
5186.2.6
by Martin Pool
Add formatting test for TransformRenameFailed |
530 |
u"Failed to rename from to to: readonly file", |
531 |
str(e)) |
|
6670.3.1
by Martin
Initial work to make errors module Python 3 compatible |
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) |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
547 |
self.assertNotEqual(-1, |
548 |
str(error).find((dir.transport.clone('..').base))) |
|
6670.3.1
by Martin
Initial work to make errors module Python 3 compatible |
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." % |
|
6653.6.6
by Jelmer Vernooij
Fix remaining tests. |
556 |
repo.controldir.root_transport.base, |
6670.3.1
by Martin
Initial work to make errors module Python 3 compatible |
557 |
str(error)) |
558 |
||
559 |
def test_not_branch_bzrdir_with_repo(self): |
|
6653.6.6
by Jelmer Vernooij
Fix remaining tests. |
560 |
controldir = self.make_repository('repo').controldir |
561 |
err = errors.NotBranchError('path', controldir=controldir) |
|
6670.3.1
by Martin
Initial work to make errors module Python 3 compatible |
562 |
self.assertEqual( |
563 |
'Not a branch: "path": location is a repository.', str(err)) |
|
564 |
||
565 |
def test_not_branch_bzrdir_without_repo(self): |
|
6653.6.6
by Jelmer Vernooij
Fix remaining tests. |
566 |
controldir = self.make_controldir('bzrdir') |
567 |
err = errors.NotBranchError('path', controldir=controldir) |
|
6670.3.1
by Martin
Initial work to make errors module Python 3 compatible |
568 |
self.assertEqual('Not a branch: "path".', str(err)) |
569 |
||
570 |
def test_not_branch_laziness(self): |
|
6653.6.5
by Jelmer Vernooij
Rename make_bzrdir to make_controldir. |
571 |
real_bzrdir = self.make_controldir('path') |
7143.15.2
by Jelmer Vernooij
Run autopep8. |
572 |
|
6670.3.1
by Martin
Initial work to make errors module Python 3 compatible |
573 |
class FakeBzrDir(object): |
574 |
def __init__(self): |
|
575 |
self.calls = [] |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
576 |
|
6670.3.1
by Martin
Initial work to make errors module Python 3 compatible |
577 |
def open_repository(self): |
578 |
self.calls.append('open_repository') |
|
579 |
raise errors.NoRepositoryPresent(real_bzrdir) |
|
580 |
fake_bzrdir = FakeBzrDir() |
|
6653.6.6
by Jelmer Vernooij
Fix remaining tests. |
581 |
err = errors.NotBranchError('path', controldir=fake_bzrdir) |
6670.3.1
by Martin
Initial work to make errors module Python 3 compatible |
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) |