Skip to content

Commit

Permalink
multihost: remove unused parameter from MultihostHost.get_connection()
Browse files Browse the repository at this point in the history
  • Loading branch information
pbrezina committed Oct 24, 2024
1 parent 3f770fc commit 7ddc20c
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions pytest_mh/_private/multihost.py
Original file line number Diff line number Diff line change
Expand Up @@ -529,7 +529,7 @@ def __init__(self, domain: DomainType, confdict: dict[str, Any]):

# Connection to the host
self.conn: Connection[Process[ProcessResult, ProcessInputBuffer], ProcessResult[ProcessError]] = (
self.get_connection(self.shell)
self.get_connection()
)
"""Connection to the host."""

Expand Down Expand Up @@ -606,16 +606,14 @@ def get_artifacts_list(self, host: MultihostHost, artifacts_type: MultihostArtif
"""
return self.configured_artifacts.get(artifacts_type) | self.artifacts.get(artifacts_type)

def get_connection(self, shell: Shell) -> Connection:
def get_connection(self) -> Connection:
"""
Get connection object to the host with given shell.
This creates a connection object using the information from the
multihost configuration. The caller should not make any assumptions
about the connection mechanism.
:param shell: Shell that will be used to run commands.
:type shell: Shell
:return: Generic connection to the host.
:rtype: Connection
"""
Expand Down

0 comments on commit 7ddc20c

Please sign in to comment.