Skip to content

Commit

Permalink
update paramiko_ssh unit tests load the connection with plugin loader (
Browse files Browse the repository at this point in the history
…ansible#80157)

* Load the paramiko_ssh connection plugin properly using plugin loader so it can use config manager when _connect() is called
  • Loading branch information
s-hertel authored Mar 7, 2023
1 parent a1bff41 commit 9a03c8f
Showing 1 changed file with 8 additions and 6 deletions.
14 changes: 8 additions & 6 deletions test/units/plugins/connection/test_paramiko.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,8 @@
from io import StringIO
import pytest

from ansible.plugins.connection import paramiko_ssh
from ansible.plugins.connection import paramiko_ssh as paramiko_ssh_module
from ansible.plugins.loader import connection_loader
from ansible.playbook.play_context import PlayContext


Expand All @@ -44,13 +45,14 @@ def in_stream():

def test_paramiko_connection_module(play_context, in_stream):
assert isinstance(
paramiko_ssh.Connection(play_context, in_stream),
paramiko_ssh.Connection)
connection_loader.get('paramiko_ssh', play_context, in_stream),
paramiko_ssh_module.Connection)


def test_paramiko_connect(play_context, in_stream, mocker):
mocker.patch.object(paramiko_ssh.Connection, '_connect_uncached')
connection = paramiko_ssh.Connection(play_context, in_stream)._connect()
paramiko_ssh = connection_loader.get('paramiko_ssh', play_context, in_stream)
mocker.patch.object(paramiko_ssh, '_connect_uncached')
connection = paramiko_ssh._connect()

assert isinstance(connection, paramiko_ssh.Connection)
assert isinstance(connection, paramiko_ssh_module.Connection)
assert connection._connected is True

0 comments on commit 9a03c8f

Please sign in to comment.