~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_errors.py

  • Committer: John Arbash Meinel
  • Date: 2007-03-15 22:35:35 UTC
  • mto: This revision was merged to the branch mainline in revision 2363.
  • Revision ID: john@arbash-meinel.com-20070315223535-d3d4964oe1hc8zhg
Add an overzealous test, for Unicode support of _iter_changes.
For both knowns and unknowns.
And include a basic, if suboptimal, fix.
I would rather defer the decoding until we've determined that we are going to return the tuple.
There is still something broken with added files, but I'll get to that.

Show diffs side-by-side

added added

removed removed

Lines of Context:
 
1
# Copyright (C) 2006 Canonical Ltd
 
2
#   Authors: Robert Collins <robert.collins@canonical.com>
 
3
#
 
4
# This program is free software; you can redistribute it and/or modify
 
5
# it under the terms of the GNU General Public License as published by
 
6
# the Free Software Foundation; either version 2 of the License, or
 
7
# (at your option) any later version.
 
8
#
 
9
# This program is distributed in the hope that it will be useful,
 
10
# but WITHOUT ANY WARRANTY; without even the implied warranty of
 
11
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 
12
# GNU General Public License for more details.
 
13
#
 
14
# You should have received a copy of the GNU General Public License
 
15
# along with this program; if not, write to the Free Software
 
16
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
17
 
 
18
"""Tests for the formatting and construction of errors."""
 
19
 
 
20
from bzrlib import (
 
21
    bzrdir,
 
22
    errors,
 
23
    )
 
24
from bzrlib.tests import TestCase, TestCaseWithTransport
 
25
 
 
26
 
 
27
# TODO: Make sure builtin exception class formats are consistent - e.g. should
 
28
# or shouldn't end with a full stop, etc.
 
29
 
 
30
 
 
31
class TestErrors(TestCaseWithTransport):
 
32
 
 
33
    def test_duplicate_file_id(self):
 
34
        error = errors.DuplicateFileId('a_file_id', 'foo')
 
35
        self.assertEqualDiff('File id {a_file_id} already exists in inventory'
 
36
                             ' as foo', str(error))
 
37
 
 
38
    def test_inventory_modified(self):
 
39
        error = errors.InventoryModified("a tree to be repred")
 
40
        self.assertEqualDiff("The current inventory for the tree 'a tree to "
 
41
            "be repred' has been modified, so a clean inventory cannot be "
 
42
            "read without data loss.",
 
43
            str(error))
 
44
 
 
45
    def test_install_failed(self):
 
46
        error = errors.InstallFailed(['rev-one'])
 
47
        self.assertEqual("Could not install revisions:\nrev-one", str(error))
 
48
        error = errors.InstallFailed(['rev-one', 'rev-two'])
 
49
        self.assertEqual("Could not install revisions:\nrev-one, rev-two",
 
50
                         str(error))
 
51
        error = errors.InstallFailed([None])
 
52
        self.assertEqual("Could not install revisions:\nNone", str(error))
 
53
 
 
54
    def test_lock_active(self):
 
55
        error = errors.LockActive("lock description")
 
56
        self.assertEqualDiff("The lock for 'lock description' is in use and "
 
57
            "cannot be broken.",
 
58
            str(error))
 
59
 
 
60
    def test_knit_header_error(self):
 
61
        error = errors.KnitHeaderError('line foo\n', 'path/to/file')
 
62
        self.assertEqual("Knit header error: 'line foo\\n' unexpected"
 
63
                         " for file path/to/file", str(error))
 
64
 
 
65
    def test_knit_index_unknown_method(self):
 
66
        error = errors.KnitIndexUnknownMethod('http://host/foo.kndx',
 
67
                                              ['bad', 'no-eol'])
 
68
        self.assertEqual("Knit index http://host/foo.kndx does not have a"
 
69
                         " known method in options: ['bad', 'no-eol']",
 
70
                         str(error))
 
71
 
 
72
    def test_medium_not_connected(self):
 
73
        error = errors.MediumNotConnected("a medium")
 
74
        self.assertEqualDiff(
 
75
            "The medium 'a medium' is not connected.", str(error))
 
76
        
 
77
    def test_no_repo(self):
 
78
        dir = bzrdir.BzrDir.create(self.get_url())
 
79
        error = errors.NoRepositoryPresent(dir)
 
80
        self.assertNotEqual(-1, str(error).find((dir.transport.clone('..').base)))
 
81
        self.assertEqual(-1, str(error).find((dir.transport.base)))
 
82
        
 
83
    def test_no_smart_medium(self):
 
84
        error = errors.NoSmartMedium("a transport")
 
85
        self.assertEqualDiff("The transport 'a transport' cannot tunnel the "
 
86
            "smart protocol.",
 
87
            str(error))
 
88
 
 
89
    def test_no_such_id(self):
 
90
        error = errors.NoSuchId("atree", "anid")
 
91
        self.assertEqualDiff("The file id anid is not present in the tree "
 
92
            "atree.",
 
93
            str(error))
 
94
 
 
95
    def test_no_such_revision_in_tree(self):
 
96
        error = errors.NoSuchRevisionInTree("atree", "anid")
 
97
        self.assertEqualDiff("The revision id anid is not present in the tree "
 
98
            "atree.",
 
99
            str(error))
 
100
        self.assertIsInstance(error, errors.NoSuchRevision)
 
101
 
 
102
    def test_not_write_locked(self):
 
103
        error = errors.NotWriteLocked('a thing to repr')
 
104
        self.assertEqualDiff("'a thing to repr' is not write locked but needs "
 
105
            "to be.",
 
106
            str(error))
 
107
 
 
108
    def test_read_only_lock_error(self):
 
109
        error = errors.ReadOnlyLockError('filename', 'error message')
 
110
        self.assertEqualDiff("Cannot acquire write lock on filename."
 
111
                             " error message", str(error))
 
112
 
 
113
    def test_too_many_concurrent_requests(self):
 
114
        error = errors.TooManyConcurrentRequests("a medium")
 
115
        self.assertEqualDiff("The medium 'a medium' has reached its concurrent "
 
116
            "request limit. Be sure to finish_writing and finish_reading on "
 
117
            "the current request that is open.",
 
118
            str(error))
 
119
 
 
120
    def test_unknown_hook(self):
 
121
        error = errors.UnknownHook("branch", "foo")
 
122
        self.assertEqualDiff("The branch hook 'foo' is unknown in this version"
 
123
            " of bzrlib.",
 
124
            str(error))
 
125
        error = errors.UnknownHook("tree", "bar")
 
126
        self.assertEqualDiff("The tree hook 'bar' is unknown in this version"
 
127
            " of bzrlib.",
 
128
            str(error))
 
129
 
 
130
    def test_up_to_date(self):
 
131
        error = errors.UpToDateFormat(bzrdir.BzrDirFormat4())
 
132
        self.assertEqualDiff("The branch format Bazaar-NG branch, "
 
133
                             "format 0.0.4 is already at the most "
 
134
                             "recent format.",
 
135
                             str(error))
 
136
 
 
137
    def test_corrupt_repository(self):
 
138
        repo = self.make_repository('.')
 
139
        error = errors.CorruptRepository(repo)
 
140
        self.assertEqualDiff("An error has been detected in the repository %s.\n"
 
141
                             "Please run bzr reconcile on this repository." %
 
142
                             repo.bzrdir.root_transport.base,
 
143
                             str(error))
 
144
 
 
145
    def test_bzrnewerror_is_deprecated(self):
 
146
        class DeprecatedError(errors.BzrNewError):
 
147
            pass
 
148
        self.callDeprecated(['BzrNewError was deprecated in bzr 0.13; '
 
149
             'please convert DeprecatedError to use BzrError instead'],
 
150
            DeprecatedError)
 
151
 
 
152
    def test_bzrerror_from_literal_string(self):
 
153
        # Some code constructs BzrError from a literal string, in which case
 
154
        # no further formatting is done.  (I'm not sure raising the base class
 
155
        # is a great idea, but if the exception is not intended to be caught
 
156
        # perhaps no more is needed.)
 
157
        try:
 
158
            raise errors.BzrError('this is my errors; %d is not expanded')
 
159
        except errors.BzrError, e:
 
160
            self.assertEqual('this is my errors; %d is not expanded', str(e))
 
161
 
 
162
    def test_reading_completed(self):
 
163
        error = errors.ReadingCompleted("a request")
 
164
        self.assertEqualDiff("The MediumRequest 'a request' has already had "
 
165
            "finish_reading called upon it - the request has been completed and"
 
166
            " no more data may be read.",
 
167
            str(error))
 
168
 
 
169
    def test_writing_completed(self):
 
170
        error = errors.WritingCompleted("a request")
 
171
        self.assertEqualDiff("The MediumRequest 'a request' has already had "
 
172
            "finish_writing called upon it - accept bytes may not be called "
 
173
            "anymore.",
 
174
            str(error))
 
175
 
 
176
    def test_writing_not_completed(self):
 
177
        error = errors.WritingNotComplete("a request")
 
178
        self.assertEqualDiff("The MediumRequest 'a request' has not has "
 
179
            "finish_writing called upon it - until the write phase is complete"
 
180
            " no data may be read.",
 
181
            str(error))
 
182
 
 
183
    def test_transport_not_possible(self):
 
184
        e = errors.TransportNotPossible('readonly', 'original error')
 
185
        self.assertEqual('Transport operation not possible:'
 
186
                         ' readonly original error', str(e))
 
187
 
 
188
    def assertSocketConnectionError(self, expected, *args, **kwargs):
 
189
        """Check the formatting of a SocketConnectionError exception"""
 
190
        e = errors.SocketConnectionError(*args, **kwargs)
 
191
        self.assertEqual(expected, str(e))
 
192
 
 
193
    def test_socket_connection_error(self):
 
194
        """Test the formatting of SocketConnectionError"""
 
195
 
 
196
        # There should be a default msg about failing to connect
 
197
        # we only require a host name.
 
198
        self.assertSocketConnectionError(
 
199
            'Failed to connect to ahost',
 
200
            'ahost')
 
201
 
 
202
        # If port is None, we don't put :None
 
203
        self.assertSocketConnectionError(
 
204
            'Failed to connect to ahost',
 
205
            'ahost', port=None)
 
206
        # But if port is supplied we include it
 
207
        self.assertSocketConnectionError(
 
208
            'Failed to connect to ahost:22',
 
209
            'ahost', port=22)
 
210
 
 
211
        # We can also supply extra information about the error
 
212
        # with or without a port
 
213
        self.assertSocketConnectionError(
 
214
            'Failed to connect to ahost:22; bogus error',
 
215
            'ahost', port=22, orig_error='bogus error')
 
216
        self.assertSocketConnectionError(
 
217
            'Failed to connect to ahost; bogus error',
 
218
            'ahost', orig_error='bogus error')
 
219
        # An exception object can be passed rather than a string
 
220
        orig_error = ValueError('bad value')
 
221
        self.assertSocketConnectionError(
 
222
            'Failed to connect to ahost; %s' % (str(orig_error),),
 
223
            host='ahost', orig_error=orig_error)
 
224
 
 
225
        # And we can supply a custom failure message
 
226
        self.assertSocketConnectionError(
 
227
            'Unable to connect to ssh host ahost:444; my_error',
 
228
            host='ahost', port=444, msg='Unable to connect to ssh host',
 
229
            orig_error='my_error')
 
230
 
 
231
 
 
232
 
 
233
class PassThroughError(errors.BzrError):
 
234
    
 
235
    _fmt = """Pass through %(foo)s and %(bar)s"""
 
236
 
 
237
    def __init__(self, foo, bar):
 
238
        errors.BzrError.__init__(self, foo=foo, bar=bar)
 
239
 
 
240
 
 
241
class ErrorWithBadFormat(errors.BzrError):
 
242
 
 
243
    _fmt = """One format specifier: %(thing)s"""
 
244
 
 
245
 
 
246
class ErrorWithNoFormat(errors.BzrError):
 
247
    """This class has a docstring but no format string."""
 
248
 
 
249
 
 
250
class TestErrorFormatting(TestCase):
 
251
    
 
252
    def test_always_str(self):
 
253
        e = PassThroughError(u'\xb5', 'bar')
 
254
        self.assertIsInstance(e.__str__(), str)
 
255
        # In Python str(foo) *must* return a real byte string
 
256
        # not a Unicode string. The following line would raise a
 
257
        # Unicode error, because it tries to call str() on the string
 
258
        # returned from e.__str__(), and it has non ascii characters
 
259
        s = str(e)
 
260
        self.assertEqual('Pass through \xc2\xb5 and bar', s)
 
261
 
 
262
    def test_missing_format_string(self):
 
263
        e = ErrorWithNoFormat(param='randomvalue')
 
264
        s = self.callDeprecated(
 
265
                ['ErrorWithNoFormat uses its docstring as a format, it should use _fmt instead'],
 
266
                lambda x: str(x), e)
 
267
        ## s = str(e)
 
268
        self.assertEqual(s, 
 
269
                "This class has a docstring but no format string.")
 
270
 
 
271
    def test_mismatched_format_args(self):
 
272
        # Even though ErrorWithBadFormat's format string does not match the
 
273
        # arguments we constructing it with, we can still stringify an instance
 
274
        # of this exception. The resulting string will say its unprintable.
 
275
        e = ErrorWithBadFormat(not_thing='x')
 
276
        self.assertStartsWith(
 
277
            str(e), 'Unprintable exception ErrorWithBadFormat')
 
278