/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_bad_files.py

  • Committer: v.ladeuil+lp at free
  • Date: 2006-12-01 15:06:29 UTC
  • mto: (2172.3.1 bzr.73948)
  • mto: This revision was merged to the branch mainline in revision 2181.
  • Revision ID: v.ladeuil+lp@free.fr-20061201150629-zjd2an87u0r7nhhw
The tests that would have help avoid bug #73948 and all that mess :)

* bzrlib/transport/http/response.py:
(handle_response): Translate a 416 http error code into a bzr
exception.

* bzrlib/transport/http/_urllib2_wrappers.py:
(HTTPDefaultErrorHandler.http_error_default): Translate a 416 http
error code into a bzr exception.

* bzrlib/transport/http/_pycurl.py:
(PyCurlTransport._curl_perform): It could happen that pycrul
itself detect a short read.

* bzrlib/transport/http/__init__.py:
(HttpTransportBase._retry_get): New method, factorizing the retry
logic.
(HttpTransportBase.readv): We can have exception during the
initial GET worth degrading the range requirements (i.e. retrying
the GET request with either single or not ranges).

* bzrlib/tests/test_transport_implementations.py:
(TransportTests.test_readv_short_read): InvalidRange can also be
raised.

* bzrlib/tests/test_http.py:
(TestRangeRequestServer.test_readv_invalid_ranges): Was named
test_readv_short_read, the new name make the intent
clearer. Depending of the code path used (urllib or pycurl), both
exceptions can be raised.

* bzrlib/tests/HttpServer.py:
(TestingHTTPRequestHandler.do_GET): If invalid ranges are
specified, returns a 416 instead of the whole file (both are valid
according to the RFC).

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/test_bad_files.py
1
 
# Copyright (C) 2005, 2007 Canonical Ltd
 
1
# Copyright (C) 2005 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
 
 
17
 
 
18
 
"""Tests being able to ignore bad filetypes."""
19
 
 
20
 
from cStringIO import StringIO
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
16
 
 
17
 
 
18
"""Tests being able to ignore mad filetypes.
 
19
"""
 
20
 
 
21
from bzrlib.tests import TestCaseWithTransport
 
22
from bzrlib.errors import BadFileKindError
21
23
import os
22
24
 
23
 
from bzrlib import (
24
 
    add,
25
 
    errors,
26
 
    )
27
 
from bzrlib.status import show_tree_status
28
 
from bzrlib.tests import OsFifoFeature, TestCaseWithTransport
29
 
 
30
 
 
31
25
def verify_status(tester, tree, value):
32
 
    """Verify the output of show_tree_status"""
 
26
    from bzrlib.status import show_tree_status
 
27
    from cStringIO import StringIO
 
28
 
33
29
    tof = StringIO()
34
30
    show_tree_status(tree, to_file=tof)
35
31
    tof.seek(0)
36
 
    tester.assertEqual(value, tof.readlines())
 
32
    tester.assertEquals(tof.readlines(), value)
37
33
 
38
34
 
39
35
class TestBadFiles(TestCaseWithTransport):
40
 
 
 
36
    
41
37
    def test_bad_files(self):
42
38
        """Test that bzr will ignore files it doesn't like"""
43
 
        self.requireFeature(OsFifoFeature)
 
39
        from bzrlib.add import smart_add_tree
 
40
        from bzrlib.branch import Branch
44
41
 
45
42
        wt = self.make_branch_and_tree('.')
46
43
        b = wt.branch
47
44
 
48
45
        files = ['one', 'two', 'three']
49
 
        file_ids = ['one-id', 'two-id', 'three-id']
50
46
        self.build_tree(files)
51
 
        wt.add(files, file_ids)
 
47
        wt.add(files)
52
48
        wt.commit("Commit one", rev_id="a@u-0-0")
 
49
        self.build_tree(['four'])
 
50
        wt.add(['four'])
 
51
        wt.commit("Commit two", rev_id="a@u-0-1")
 
52
        self.build_tree(['five'])
 
53
        wt.add(['five'])
 
54
        wt.commit("Commit three", rev_id="a@u-0-2")
53
55
 
54
56
        # We should now have a few files, lets try to
55
57
        # put some bogus stuff in the tree
56
58
 
57
 
        # status with nothing changed
 
59
        # We can only continue if we have mkfifo
 
60
        if getattr(os, 'mkfifo', None) is None:
 
61
            # TODO: Ultimately this should be TestSkipped
 
62
            # or PlatformDeficiency
 
63
            return
 
64
 
 
65
        # status with nothing
58
66
        verify_status(self, wt, [])
59
67
 
60
68
        os.mkfifo('a-fifo')
65
73
                           '  a-fifo\n',
66
74
                           '  six\n'
67
75
                           ])
68
 
 
69
 
        # We should raise an error if we are adding a bogus file
70
 
        self.assertRaises(errors.BadFileKindError, wt.smart_add, ['a-fifo'])
71
 
 
72
 
        # And the list of files shouldn't have been modified
73
 
        verify_status(self, wt,
74
 
                          ['unknown:\n',
75
 
                           '  a-fifo\n',
76
 
                           '  six\n'
77
 
                           ])
78
 
 
 
76
        
79
77
        # Make sure smart_add can handle having a bogus
80
78
        # file in the way
81
 
        wt.smart_add([])
 
79
        smart_add_tree(wt, '.')
82
80
        verify_status(self, wt,
83
81
                          ['added:\n',
84
82
                           '  six\n',
91
89
                          ['unknown:\n',
92
90
                           '  a-fifo\n',
93
91
                           ])
 
92
 
 
93
        # We should raise an error if we are adding a bogus file
 
94
        # Is there a way to test the actual error that should be raised?
 
95
        self.run_bzr('add', 'a-fifo', retcode=3)
 
96