Skip to content

Commit

Permalink
Rename SaltMaster.get_salt_cli to SaltMaster.salt_cli
Browse files Browse the repository at this point in the history
Forgotten in saltstack#50
  • Loading branch information
s0undt3ch committed Jun 17, 2021
1 parent 7e510bf commit 393dac9
Show file tree
Hide file tree
Showing 8 changed files with 10 additions and 9 deletions.
1 change: 1 addition & 0 deletions changelog/70.breaking.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Rename the ``SaltMaster.get_salt_cli`` to ``SaltMaster.salt_cli``, forgotten on `#50 <https://github.com/saltstack/pytest-salt-factories/issues/50>`_
2 changes: 1 addition & 1 deletion examples/echo-extension/tests/integration/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ def salt_run_cli(master):

@pytest.fixture
def salt_cli(master):
return master.get_salt_cli()
return master.salt_cli()


@pytest.fixture
Expand Down
4 changes: 2 additions & 2 deletions src/saltfactories/daemons/master.py
Original file line number Diff line number Diff line change
Expand Up @@ -360,7 +360,7 @@ def salt_cloud_cli(
**factory_class_kwargs
)

def get_salt_cli(self, factory_class=cli.salt.Salt, **factory_class_kwargs):
def salt_cli(self, factory_class=cli.salt.Salt, **factory_class_kwargs):
"""
Return a `salt` CLI process for this master instance
"""
Expand Down Expand Up @@ -508,7 +508,7 @@ def salt_ssh_cli(
**factory_class_kwargs
)

def get_salt_client(
def salt_client(
self,
functions_known_to_return_none=None,
factory_class=client.LocalClient,
Expand Down
2 changes: 1 addition & 1 deletion tests/functional/factories/cli/test_salt.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@


def test_version_info(salt_master, salt_version):
cli = salt_master.get_salt_cli()
cli = salt_master.salt_cli()
ret = cli.run("--version")
assert ret.exitcode == 0, ret
assert ret.stdout.strip() == "{} {}".format(pathlib.Path(cli.script_name).name, salt_version)
2 changes: 1 addition & 1 deletion tests/integration/factories/cli/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -38,4 +38,4 @@ def salt_minion(salt_factories, minion_id, salt_master):

@pytest.fixture(scope="package")
def salt_cli(salt_master):
return salt_master.get_salt_cli()
return salt_master.salt_cli()
2 changes: 1 addition & 1 deletion tests/integration/factories/daemons/minion/test_minion.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ def minion(master):

@pytest.fixture
def salt_cli(master):
return master.get_salt_cli()
return master.salt_cli()


@pytest.fixture
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ def proxy_minion(master):

@pytest.fixture
def salt_cli(master):
return master.get_salt_cli()
return master.salt_cli()


@pytest.fixture
Expand Down
4 changes: 2 additions & 2 deletions tests/integration/factories/daemons/syndic/test_syndic.py
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ def master_of_masters_salt_cli(master_of_masters, minion_1):
"""
assert master_of_masters.is_running()
assert minion_1.is_running()
return master_of_masters.get_salt_cli()
return master_of_masters.salt_cli()


@pytest.fixture(scope="module")
Expand All @@ -93,7 +93,7 @@ def syndic_master_salt_cli(syndic_master, syndic_minion, minion_2):
assert syndic_master.is_running()
assert syndic_minion.is_running()
assert minion_2.is_running()
return syndic_master.get_salt_cli()
return syndic_master.salt_cli()


@pytest.fixture(scope="module")
Expand Down

0 comments on commit 393dac9

Please sign in to comment.