test_transport_sftp backtraces if paramiko isn't present

I can confirm this in 2.0. If you change the line to this it will work:

        except errors.ParamikoNotPresent:

It's been fixed in bzr.dev:

<http://bazaar.launchpad.net/~bzr-pqm/bzr/bzr.dev/revision/4645.2.1>

Thanks for the quick response.

When I apply the patch below I get

"""
ERROR: test_transport_sftp (bzrlib.tests.test_selftest.TestSelftest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/var/tmp/paludis/build/dev-scm-bzr-2.0.1/work/bzr-2.0.1/bzrlib/tests/test_selftest.py", line 1881, in test_transport_sftp
    raise TestSkipped("Paramiko not present")
NameError: global name 'TestSkipped' is not defined
"""

Will these bugfixes be fixed wit the bzr-2.0.2 release?

Thanks again!

Ingmar Vanhassel wrote:
> Thanks for the quick response.
>
> When I apply the patch below I get
>
> """
> ERROR: test_transport_sftp (bzrlib.tests.test_selftest.TestSelftest)
> ----------------------------------------------------------------------
> Traceback (most recent call last):
> File "/var/tmp/paludis/build/dev-scm-bzr-2.0.1/work/bzr-2.0.1/bzrlib/tests/test_selftest.py", line 1881, in test_transport_sftp
> raise TestSkipped("Paramiko not present")
> NameError: global name 'TestSkipped' is not defined
> """

Argh. That was fixed in bzr.dev too. Change it to:

            raise tests.TestSkipped("Paramiko not present")

> Will these bugfixes be fixed wit the bzr-2.0.2 release?

No idea. That's not my department.

> Thanks again!
--

This looks like something that would be perfectly acceptable as a patch to the 2.0.x series. It just requires someone to actually care to submit one.