← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_transport.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2010-09-01 08:02:42 UTC
  • mfrom: (5390.3.3 faster-revert-593560)
  • Revision ID: pqm@pqm.ubuntu.com-20100901080242-esg62ody4frwmy66
(spiv) Avoid repeatedly calling self.target.all_file_ids() in
 InterTree.iter_changes. (Andrew Bennetts)

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/test_transport.py
900
900
 
901
901
        bzr+ssh:// should cause bzr to run a remote bzr smart server over SSH.
902
902
        """
 
903
        raise tests.TestSkipped('this test was recently broken,'
 
904
                                ' see bug #626876')
903
905
        # This test actually causes a bzr instance to be invoked, which is very
904
906
        # expensive: it should be the only such test in the test suite.
905
907
        # A reasonable evolution for this would be to simply check inside
955
957
        ssh_server = stub_sftp.SFTPFullAbsoluteServer(StubSSHServer)
956
958
        # We *don't* want to override the default SSH vendor: the detected one
957
959
        # is the one to use.
 
960
 
 
961
        # FIXME: I don't understand the above comment, SFTPFullAbsoluteServer
 
962
        # inherits from SFTPServer which forces the SSH vendor to
 
963
        # ssh.ParamikoVendor(). So it's forced, not detected. --vila 20100623
958
964
        self.start_server(ssh_server)
959
 
        port = ssh_server._listener.port
 
965
        port = ssh_server.port
960
966
 
961
967
        if sys.platform == 'win32':
962
968
            bzr_remote_path = sys.executable + ' ' + self.get_bzr_path()