← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/inter.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/inter.py
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
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
17
 
18
18
"""Inter-object utility class."""
22
22
    """This class represents operations taking place between two objects.
23
23
 
24
24
    Its instances have methods like join or copy_content or fetch, and contain
25
 
    references to the source and target objects these operations can be
 
25
    references to the source and target objects these operations can be 
26
26
    carried out between.
27
27
 
28
28
    Often we will provide convenience methods on the objects which carry out
29
29
    operations with another of similar type - they will always forward to
30
 
    a subclass of InterObject - i.e.
 
30
    a subclass of InterObject - i.e. 
31
31
    InterVersionedFile.get(other).method_name(parameters).
32
32
 
33
 
    If the source and target objects implement the locking protocol -
 
33
    If the source and target objects implement the locking protocol - 
34
34
    lock_read, lock_write, unlock, then the InterObject's lock_read,
35
35
    lock_write and unlock methods may be used (optionally in conjunction with
36
36
    the needs_read_lock and needs_write_lock decorators.)
37
 
 
38
 
    When looking for an inter, the most recently registered types are tested
39
 
    first.  So typically the most generic and slowest InterObjects should be
40
 
    registered first.
41
37
    """
42
38
 
43
39
    # _optimisers = list()
45
41
 
46
42
    def __init__(self, source, target):
47
43
        """Construct a default InterObject instance. Please use 'get'.
48
 
 
49
 
        Only subclasses of InterObject should call
 
44
        
 
45
        Only subclasses of InterObject should call 
50
46
        InterObject.__init__ - clients should call InterFOO.get where FOO
51
47
        is the base type of the objects they are interacting between. I.e.
52
48
        InterVersionedFile or InterRepository.