~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/transport/remote.py

  • Committer: Martin Pool
  • Date: 2005-05-10 06:07:16 UTC
  • Revision ID: mbp@sourcefrog.net-20050510060716-0f939ce3ddea5d15
- New command update-stat-cache for testing
- work-cache always stored with unix newlines and in ascii

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2006 Canonical Ltd
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
15
 
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
 
 
17
 
"""RemoteTransport client for the smart-server.
18
 
 
19
 
This module shouldn't be accessed directly.  The classes defined here should be
20
 
imported from bzrlib.smart.
21
 
"""
22
 
 
23
 
__all__ = ['RemoteTransport', 'RemoteTCPTransport', 'RemoteSSHTransport']
24
 
 
25
 
from cStringIO import StringIO
26
 
 
27
 
from bzrlib import (
28
 
    config,
29
 
    debug,
30
 
    errors,
31
 
    remote,
32
 
    trace,
33
 
    transport,
34
 
    urlutils,
35
 
    )
36
 
from bzrlib.smart import client, medium
37
 
from bzrlib.symbol_versioning import (deprecated_method, one_four)
38
 
 
39
 
 
40
 
class _SmartStat(object):
41
 
 
42
 
    def __init__(self, size, mode):
43
 
        self.st_size = size
44
 
        self.st_mode = mode
45
 
 
46
 
 
47
 
class RemoteTransport(transport.ConnectedTransport):
48
 
    """Connection to a smart server.
49
 
 
50
 
    The connection holds references to the medium that can be used to send
51
 
    requests to the server.
52
 
 
53
 
    The connection has a notion of the current directory to which it's
54
 
    connected; this is incorporated in filenames passed to the server.
55
 
    
56
 
    This supports some higher-level RPC operations and can also be treated 
57
 
    like a Transport to do file-like operations.
58
 
 
59
 
    The connection can be made over a tcp socket, an ssh pipe or a series of
60
 
    http requests.  There are concrete subclasses for each type:
61
 
    RemoteTCPTransport, etc.
62
 
    """
63
 
 
64
 
    # When making a readv request, cap it at requesting 5MB of data
65
 
    _max_readv_bytes = 5*1024*1024
66
 
 
67
 
    # IMPORTANT FOR IMPLEMENTORS: RemoteTransport MUST NOT be given encoding
68
 
    # responsibilities: Put those on SmartClient or similar. This is vital for
69
 
    # the ability to support multiple versions of the smart protocol over time:
70
 
    # RemoteTransport is an adapter from the Transport object model to the 
71
 
    # SmartClient model, not an encoder.
72
 
 
73
 
    # FIXME: the medium parameter should be private, only the tests requires
74
 
    # it. It may be even clearer to define a TestRemoteTransport that handles
75
 
    # the specific cases of providing a _client and/or a _medium, and leave
76
 
    # RemoteTransport as an abstract class.
77
 
    def __init__(self, url, _from_transport=None, medium=None, _client=None):
78
 
        """Constructor.
79
 
 
80
 
        :param _from_transport: Another RemoteTransport instance that this
81
 
            one is being cloned from.  Attributes such as the medium will
82
 
            be reused.
83
 
 
84
 
        :param medium: The medium to use for this RemoteTransport.  If None,
85
 
            the medium from the _from_transport is shared.  If both this
86
 
            and _from_transport are None, a new medium will be built.
87
 
            _from_transport and medium cannot both be specified.
88
 
 
89
 
        :param _client: Override the _SmartClient used by this transport.  This
90
 
            should only be used for testing purposes; normally this is
91
 
            determined from the medium.
92
 
        """
93
 
        super(RemoteTransport, self).__init__(url,
94
 
                                              _from_transport=_from_transport)
95
 
 
96
 
        # The medium is the connection, except when we need to share it with
97
 
        # other objects (RemoteBzrDir, RemoteRepository etc). In these cases
98
 
        # what we want to share is really the shared connection.
99
 
 
100
 
        if _from_transport is None:
101
 
            # If no _from_transport is specified, we need to intialize the
102
 
            # shared medium.
103
 
            credentials = None
104
 
            if medium is None:
105
 
                medium, credentials = self._build_medium()
106
 
                if 'hpss' in debug.debug_flags:
107
 
                    trace.mutter('hpss: Built a new medium: %s',
108
 
                                 medium.__class__.__name__)
109
 
            self._shared_connection = transport._SharedConnection(medium,
110
 
                                                                  credentials,
111
 
                                                                  self.base)
112
 
        elif medium is None:
113
 
            # No medium was specified, so share the medium from the
114
 
            # _from_transport.
115
 
            medium = self._shared_connection.connection
116
 
        else:
117
 
            raise AssertionError(
118
 
                "Both _from_transport (%r) and medium (%r) passed to "
119
 
                "RemoteTransport.__init__, but these parameters are mutally "
120
 
                "exclusive." % (_from_transport, medium))
121
 
 
122
 
        if _client is None:
123
 
            self._client = client._SmartClient(medium)
124
 
        else:
125
 
            self._client = _client
126
 
 
127
 
    def _build_medium(self):
128
 
        """Create the medium if _from_transport does not provide one.
129
 
 
130
 
        The medium is analogous to the connection for ConnectedTransport: it
131
 
        allows connection sharing.
132
 
        """
133
 
        # No credentials
134
 
        return None, None
135
 
 
136
 
    def is_readonly(self):
137
 
        """Smart server transport can do read/write file operations."""
138
 
        try:
139
 
            resp = self._call2('Transport.is_readonly')
140
 
        except errors.UnknownSmartMethod:
141
 
            # XXX: nasty hack: servers before 0.16 don't have a
142
 
            # 'Transport.is_readonly' verb, so we do what clients before 0.16
143
 
            # did: assume False.
144
 
            return False
145
 
        if resp == ('yes', ):
146
 
            return True
147
 
        elif resp == ('no', ):
148
 
            return False
149
 
        else:
150
 
            raise errors.UnexpectedSmartServerResponse(resp)
151
 
 
152
 
    def get_smart_client(self):
153
 
        return self._get_connection()
154
 
 
155
 
    def get_smart_medium(self):
156
 
        return self._get_connection()
157
 
 
158
 
    @deprecated_method(one_four)
159
 
    def get_shared_medium(self):
160
 
        return self._get_shared_connection()
161
 
 
162
 
    def _remote_path(self, relpath):
163
 
        """Returns the Unicode version of the absolute path for relpath."""
164
 
        return self._combine_paths(self._path, relpath)
165
 
 
166
 
    def _call(self, method, *args):
167
 
        resp = self._call2(method, *args)
168
 
        self._ensure_ok(resp)
169
 
 
170
 
    def _call2(self, method, *args):
171
 
        """Call a method on the remote server."""
172
 
        try:
173
 
            return self._client.call(method, *args)
174
 
        except errors.ErrorFromSmartServer, err:
175
 
            # The first argument, if present, is always a path.
176
 
            if args:
177
 
                context = {'relpath': args[0]}
178
 
            else:
179
 
                context = {}
180
 
            self._translate_error(err, **context)
181
 
 
182
 
    def _call_with_body_bytes(self, method, args, body):
183
 
        """Call a method on the remote server with body bytes."""
184
 
        try:
185
 
            return self._client.call_with_body_bytes(method, args, body)
186
 
        except errors.ErrorFromSmartServer, err:
187
 
            # The first argument, if present, is always a path.
188
 
            if args:
189
 
                context = {'relpath': args[0]}
190
 
            else:
191
 
                context = {}
192
 
            self._translate_error(err, **context)
193
 
 
194
 
    def has(self, relpath):
195
 
        """Indicate whether a remote file of the given name exists or not.
196
 
 
197
 
        :see: Transport.has()
198
 
        """
199
 
        resp = self._call2('has', self._remote_path(relpath))
200
 
        if resp == ('yes', ):
201
 
            return True
202
 
        elif resp == ('no', ):
203
 
            return False
204
 
        else:
205
 
            raise errors.UnexpectedSmartServerResponse(resp)
206
 
 
207
 
    def get(self, relpath):
208
 
        """Return file-like object reading the contents of a remote file.
209
 
        
210
 
        :see: Transport.get_bytes()/get_file()
211
 
        """
212
 
        return StringIO(self.get_bytes(relpath))
213
 
 
214
 
    def get_bytes(self, relpath):
215
 
        remote = self._remote_path(relpath)
216
 
        try:
217
 
            resp, response_handler = self._client.call_expecting_body('get', remote)
218
 
        except errors.ErrorFromSmartServer, err:
219
 
            self._translate_error(err, relpath)
220
 
        if resp != ('ok', ):
221
 
            response_handler.cancel_read_body()
222
 
            raise errors.UnexpectedSmartServerResponse(resp)
223
 
        return response_handler.read_body_bytes()
224
 
 
225
 
    def _serialise_optional_mode(self, mode):
226
 
        if mode is None:
227
 
            return ''
228
 
        else:
229
 
            return '%d' % mode
230
 
 
231
 
    def mkdir(self, relpath, mode=None):
232
 
        resp = self._call2('mkdir', self._remote_path(relpath),
233
 
            self._serialise_optional_mode(mode))
234
 
 
235
 
    def open_write_stream(self, relpath, mode=None):
236
 
        """See Transport.open_write_stream."""
237
 
        self.put_bytes(relpath, "", mode)
238
 
        result = transport.AppendBasedFileStream(self, relpath)
239
 
        transport._file_streams[self.abspath(relpath)] = result
240
 
        return result
241
 
 
242
 
    def put_bytes(self, relpath, upload_contents, mode=None):
243
 
        # FIXME: upload_file is probably not safe for non-ascii characters -
244
 
        # should probably just pass all parameters as length-delimited
245
 
        # strings?
246
 
        if type(upload_contents) is unicode:
247
 
            # Although not strictly correct, we raise UnicodeEncodeError to be
248
 
            # compatible with other transports.
249
 
            raise UnicodeEncodeError(
250
 
                'undefined', upload_contents, 0, 1,
251
 
                'put_bytes must be given bytes, not unicode.')
252
 
        resp = self._call_with_body_bytes('put',
253
 
            (self._remote_path(relpath), self._serialise_optional_mode(mode)),
254
 
            upload_contents)
255
 
        self._ensure_ok(resp)
256
 
        return len(upload_contents)
257
 
 
258
 
    def put_bytes_non_atomic(self, relpath, bytes, mode=None,
259
 
                             create_parent_dir=False,
260
 
                             dir_mode=None):
261
 
        """See Transport.put_bytes_non_atomic."""
262
 
        # FIXME: no encoding in the transport!
263
 
        create_parent_str = 'F'
264
 
        if create_parent_dir:
265
 
            create_parent_str = 'T'
266
 
 
267
 
        resp = self._call_with_body_bytes(
268
 
            'put_non_atomic',
269
 
            (self._remote_path(relpath), self._serialise_optional_mode(mode),
270
 
             create_parent_str, self._serialise_optional_mode(dir_mode)),
271
 
            bytes)
272
 
        self._ensure_ok(resp)
273
 
 
274
 
    def put_file(self, relpath, upload_file, mode=None):
275
 
        # its not ideal to seek back, but currently put_non_atomic_file depends
276
 
        # on transports not reading before failing - which is a faulty
277
 
        # assumption I think - RBC 20060915
278
 
        pos = upload_file.tell()
279
 
        try:
280
 
            return self.put_bytes(relpath, upload_file.read(), mode)
281
 
        except:
282
 
            upload_file.seek(pos)
283
 
            raise
284
 
 
285
 
    def put_file_non_atomic(self, relpath, f, mode=None,
286
 
                            create_parent_dir=False,
287
 
                            dir_mode=None):
288
 
        return self.put_bytes_non_atomic(relpath, f.read(), mode=mode,
289
 
                                         create_parent_dir=create_parent_dir,
290
 
                                         dir_mode=dir_mode)
291
 
 
292
 
    def append_file(self, relpath, from_file, mode=None):
293
 
        return self.append_bytes(relpath, from_file.read(), mode)
294
 
        
295
 
    def append_bytes(self, relpath, bytes, mode=None):
296
 
        resp = self._call_with_body_bytes(
297
 
            'append',
298
 
            (self._remote_path(relpath), self._serialise_optional_mode(mode)),
299
 
            bytes)
300
 
        if resp[0] == 'appended':
301
 
            return int(resp[1])
302
 
        raise errors.UnexpectedSmartServerResponse(resp)
303
 
 
304
 
    def delete(self, relpath):
305
 
        resp = self._call2('delete', self._remote_path(relpath))
306
 
        self._ensure_ok(resp)
307
 
 
308
 
    def external_url(self):
309
 
        """See bzrlib.transport.Transport.external_url."""
310
 
        # the external path for RemoteTransports is the base
311
 
        return self.base
312
 
 
313
 
    def recommended_page_size(self):
314
 
        """Return the recommended page size for this transport."""
315
 
        return 64 * 1024
316
 
        
317
 
    def _readv(self, relpath, offsets):
318
 
        if not offsets:
319
 
            return
320
 
 
321
 
        offsets = list(offsets)
322
 
 
323
 
        sorted_offsets = sorted(offsets)
324
 
        coalesced = list(self._coalesce_offsets(sorted_offsets,
325
 
                               limit=self._max_readv_combine,
326
 
                               fudge_factor=self._bytes_to_read_before_seek))
327
 
 
328
 
        # now that we've coallesced things, avoid making enormous requests
329
 
        requests = []
330
 
        cur_request = []
331
 
        cur_len = 0
332
 
        for c in coalesced:
333
 
            if c.length + cur_len > self._max_readv_bytes:
334
 
                requests.append(cur_request)
335
 
                cur_request = [c]
336
 
                cur_len = c.length
337
 
                continue
338
 
            cur_request.append(c)
339
 
            cur_len += c.length
340
 
        if cur_request:
341
 
            requests.append(cur_request)
342
 
        if 'hpss' in debug.debug_flags:
343
 
            trace.mutter('%s.readv %s offsets => %s coalesced'
344
 
                         ' => %s requests (%s)',
345
 
                         self.__class__.__name__, len(offsets), len(coalesced),
346
 
                         len(requests), sum(map(len, requests)))
347
 
        # Cache the results, but only until they have been fulfilled
348
 
        data_map = {}
349
 
        # turn the list of offsets into a single stack to iterate
350
 
        offset_stack = iter(offsets)
351
 
        # using a list so it can be modified when passing down and coming back
352
 
        next_offset = [offset_stack.next()]
353
 
        for cur_request in requests:
354
 
            try:
355
 
                result = self._client.call_with_body_readv_array(
356
 
                    ('readv', self._remote_path(relpath),),
357
 
                    [(c.start, c.length) for c in cur_request])
358
 
                resp, response_handler = result
359
 
            except errors.ErrorFromSmartServer, err:
360
 
                self._translate_error(err, relpath)
361
 
 
362
 
            if resp[0] != 'readv':
363
 
                # This should raise an exception
364
 
                response_handler.cancel_read_body()
365
 
                raise errors.UnexpectedSmartServerResponse(resp)
366
 
 
367
 
            for res in self._handle_response(offset_stack, cur_request,
368
 
                                             response_handler,
369
 
                                             data_map,
370
 
                                             next_offset):
371
 
                yield res
372
 
 
373
 
    def _handle_response(self, offset_stack, coalesced, response_handler,
374
 
                         data_map, next_offset):
375
 
        cur_offset_and_size = next_offset[0]
376
 
        # FIXME: this should know how many bytes are needed, for clarity.
377
 
        data = response_handler.read_body_bytes()
378
 
        data_offset = 0
379
 
        for c_offset in coalesced:
380
 
            if len(data) < c_offset.length:
381
 
                raise errors.ShortReadvError(relpath, c_offset.start,
382
 
                            c_offset.length, actual=len(data))
383
 
            for suboffset, subsize in c_offset.ranges:
384
 
                key = (c_offset.start+suboffset, subsize)
385
 
                this_data = data[data_offset+suboffset:
386
 
                                 data_offset+suboffset+subsize]
387
 
                # Special case when the data is in-order, rather than packing
388
 
                # into a map and then back out again. Benchmarking shows that
389
 
                # this has 100% hit rate, but leave in the data_map work just
390
 
                # in case.
391
 
                # TODO: Could we get away with using buffer() to avoid the
392
 
                #       memory copy?  Callers would need to realize they may
393
 
                #       not have a real string.
394
 
                if key == cur_offset_and_size:
395
 
                    yield cur_offset_and_size[0], this_data
396
 
                    cur_offset_and_size = next_offset[0] = offset_stack.next()
397
 
                else:
398
 
                    data_map[key] = this_data
399
 
            data_offset += c_offset.length
400
 
 
401
 
            # Now that we've read some data, see if we can yield anything back
402
 
            while cur_offset_and_size in data_map:
403
 
                this_data = data_map.pop(cur_offset_and_size)
404
 
                yield cur_offset_and_size[0], this_data
405
 
                cur_offset_and_size = next_offset[0] = offset_stack.next()
406
 
 
407
 
    def rename(self, rel_from, rel_to):
408
 
        self._call('rename',
409
 
                   self._remote_path(rel_from),
410
 
                   self._remote_path(rel_to))
411
 
 
412
 
    def move(self, rel_from, rel_to):
413
 
        self._call('move',
414
 
                   self._remote_path(rel_from),
415
 
                   self._remote_path(rel_to))
416
 
 
417
 
    def rmdir(self, relpath):
418
 
        resp = self._call('rmdir', self._remote_path(relpath))
419
 
 
420
 
    def _ensure_ok(self, resp):
421
 
        if resp[0] != 'ok':
422
 
            raise errors.UnexpectedSmartServerResponse(resp)
423
 
        
424
 
    def _translate_error(self, err, relpath=None):
425
 
        remote._translate_error(err, path=relpath)
426
 
 
427
 
    def disconnect(self):
428
 
        self.get_smart_medium().disconnect()
429
 
 
430
 
    def stat(self, relpath):
431
 
        resp = self._call2('stat', self._remote_path(relpath))
432
 
        if resp[0] == 'stat':
433
 
            return _SmartStat(int(resp[1]), int(resp[2], 8))
434
 
        raise errors.UnexpectedSmartServerResponse(resp)
435
 
 
436
 
    ## def lock_read(self, relpath):
437
 
    ##     """Lock the given file for shared (read) access.
438
 
    ##     :return: A lock object, which should be passed to Transport.unlock()
439
 
    ##     """
440
 
    ##     # The old RemoteBranch ignore lock for reading, so we will
441
 
    ##     # continue that tradition and return a bogus lock object.
442
 
    ##     class BogusLock(object):
443
 
    ##         def __init__(self, path):
444
 
    ##             self.path = path
445
 
    ##         def unlock(self):
446
 
    ##             pass
447
 
    ##     return BogusLock(relpath)
448
 
 
449
 
    def listable(self):
450
 
        return True
451
 
 
452
 
    def list_dir(self, relpath):
453
 
        resp = self._call2('list_dir', self._remote_path(relpath))
454
 
        if resp[0] == 'names':
455
 
            return [name.encode('ascii') for name in resp[1:]]
456
 
        raise errors.UnexpectedSmartServerResponse(resp)
457
 
 
458
 
    def iter_files_recursive(self):
459
 
        resp = self._call2('iter_files_recursive', self._remote_path(''))
460
 
        if resp[0] == 'names':
461
 
            return resp[1:]
462
 
        raise errors.UnexpectedSmartServerResponse(resp)
463
 
 
464
 
 
465
 
class RemoteTCPTransport(RemoteTransport):
466
 
    """Connection to smart server over plain tcp.
467
 
    
468
 
    This is essentially just a factory to get 'RemoteTransport(url,
469
 
        SmartTCPClientMedium).
470
 
    """
471
 
 
472
 
    def _build_medium(self):
473
 
        client_medium = medium.SmartTCPClientMedium(
474
 
            self._host, self._port, self.base)
475
 
        return client_medium, None
476
 
 
477
 
 
478
 
class RemoteTCPTransportV2Only(RemoteTransport):
479
 
    """Connection to smart server over plain tcp with the client hard-coded to
480
 
    assume protocol v2 and remote server version <= 1.6.
481
 
 
482
 
    This should only be used for testing.
483
 
    """
484
 
 
485
 
    def _build_medium(self):
486
 
        client_medium = medium.SmartTCPClientMedium(
487
 
            self._host, self._port, self.base)
488
 
        client_medium._protocol_version = 2
489
 
        client_medium._remember_remote_is_before((1, 6))
490
 
        return client_medium, None
491
 
 
492
 
 
493
 
class RemoteSSHTransport(RemoteTransport):
494
 
    """Connection to smart server over SSH.
495
 
 
496
 
    This is essentially just a factory to get 'RemoteTransport(url,
497
 
        SmartSSHClientMedium).
498
 
    """
499
 
 
500
 
    def _build_medium(self):
501
 
        location_config = config.LocationConfig(self.base)
502
 
        bzr_remote_path = location_config.get_bzr_remote_path()
503
 
        user = self._user
504
 
        if user is None:
505
 
            auth = config.AuthenticationConfig()
506
 
            user = auth.get_user('ssh', self._host, self._port)
507
 
        client_medium = medium.SmartSSHClientMedium(self._host, self._port,
508
 
            user, self._password, self.base,
509
 
            bzr_remote_path=bzr_remote_path)
510
 
        return client_medium, (user, self._password)
511
 
 
512
 
 
513
 
class RemoteHTTPTransport(RemoteTransport):
514
 
    """Just a way to connect between a bzr+http:// url and http://.
515
 
    
516
 
    This connection operates slightly differently than the RemoteSSHTransport.
517
 
    It uses a plain http:// transport underneath, which defines what remote
518
 
    .bzr/smart URL we are connected to. From there, all paths that are sent are
519
 
    sent as relative paths, this way, the remote side can properly
520
 
    de-reference them, since it is likely doing rewrite rules to translate an
521
 
    HTTP path into a local path.
522
 
    """
523
 
 
524
 
    def __init__(self, base, _from_transport=None, http_transport=None):
525
 
        if http_transport is None:
526
 
            # FIXME: the password may be lost here because it appears in the
527
 
            # url only for an intial construction (when the url came from the
528
 
            # command-line).
529
 
            http_url = base[len('bzr+'):]
530
 
            self._http_transport = transport.get_transport(http_url)
531
 
        else:
532
 
            self._http_transport = http_transport
533
 
        super(RemoteHTTPTransport, self).__init__(
534
 
            base, _from_transport=_from_transport)
535
 
 
536
 
    def _build_medium(self):
537
 
        # We let http_transport take care of the credentials
538
 
        return self._http_transport.get_smart_medium(), None
539
 
 
540
 
    def _remote_path(self, relpath):
541
 
        """After connecting, HTTP Transport only deals in relative URLs."""
542
 
        # Adjust the relpath based on which URL this smart transport is
543
 
        # connected to.
544
 
        http_base = urlutils.normalize_url(self.get_smart_medium().base)
545
 
        url = urlutils.join(self.base[len('bzr+'):], relpath)
546
 
        url = urlutils.normalize_url(url)
547
 
        return urlutils.relative_url(http_base, url)
548
 
 
549
 
    def clone(self, relative_url):
550
 
        """Make a new RemoteHTTPTransport related to me.
551
 
 
552
 
        This is re-implemented rather than using the default
553
 
        RemoteTransport.clone() because we must be careful about the underlying
554
 
        http transport.
555
 
 
556
 
        Also, the cloned smart transport will POST to the same .bzr/smart
557
 
        location as this transport (although obviously the relative paths in the
558
 
        smart requests may be different).  This is so that the server doesn't
559
 
        have to handle .bzr/smart requests at arbitrary places inside .bzr
560
 
        directories, just at the initial URL the user uses.
561
 
        """
562
 
        if relative_url:
563
 
            abs_url = self.abspath(relative_url)
564
 
        else:
565
 
            abs_url = self.base
566
 
        return RemoteHTTPTransport(abs_url,
567
 
                                   _from_transport=self,
568
 
                                   http_transport=self._http_transport)
569
 
 
570
 
 
571
 
def get_test_permutations():
572
 
    """Return (transport, server) permutations for testing."""
573
 
    ### We may need a little more test framework support to construct an
574
 
    ### appropriate RemoteTransport in the future.
575
 
    from bzrlib.smart import server
576
 
    return [(RemoteTCPTransport, server.SmartTCPServer_for_testing)]