Tests hanging forever about pytest-sftpserver

[ulope]

If this is indeed a paramiko issue I assume this issue can be closed?

Note to self: Maybe adding a hint in the readme wouldn't hurt though...

[mbyio]

It looks like you're using the paramiko client? If so, then this is actually a paramiko issue. It opens Python threads in the background to do the actual communication with the server. So you need to actually close the client or it might get stuck.

[alfredopalhares]

I am having ths problem, if its a paramiko issue, is there any links ?
What is the status of this.

[christopherdoyle]

Also encountered this problem, I think sure this is the relevant paramiko issue: paramiko/paramiko#520 (the issue is more widespread than the title). As above, explicitly closing the transport or client fixes it, e.g. from the orignal post adding trans.close() to the end of the test. I think the issue can be closed? I can't see any way of dealing with this in the fixture without monkey patching paramiko.

[matteosantama]

Similar problem here. Has anyone figured out how to gracefully kill the sftpserver at the end of a test?

[alfredopalhares]

Nope, still nothing on my end. Shame, because this library is not that useful withthis bug.

[fiendish]

Yeah, this happened to me too.

As above, explicitly closing the transport or client fixes it, e.g. from the orignal post adding trans.close() to the end of the test.

Not for me. :\

[fiendish]

@ulope running the server as a thread is probably a mistake. I was able to work around this issue by spawning the server in its own process to eliminate lock contention:

import time
from multiprocessing import Process, Queue
from pytest_sftpserver.sftp.server import SFTPServer


def run_server(q, contents):
    sftpserver = SFTPServer(content_object=contents)
    sftpserver.start()
    q.put((sftpserver.host, sftpserver.port))


def test_some_stuff():
    contents = {}  # my contents structure here

    queue = Queue()
    p = Process(target=run_server, args=(queue, contents), daemon=True)
    p.start()

    for _ in range(10):
        address = queue.get()
        if address:
            break
        time.sleep(0.5)

    assert address

    # <do my testing here>

[johannes-cogitaris]

To anyone coming here: This might be an alternative: https://github.com/oz123/pytest-localftpserver

edit: ftp only :-(

[tmieulet]

see : #30 (comment)
It worked for me:

@pytest.fixture
def sftp_fixture(sftpserver):
    # https://github.com/ulope/pytest-sftpserver/issues/30
    # Tests hanging forever
    sftpserver.daemon_threads = True
    sftpserver.block_on_close = False
    yield sftpserver