-
Notifications
You must be signed in to change notification settings - Fork 164
Conversation
|
||
|
||
pytestmark = [pytest.mark.usefixtures("start_evmserverd_after_module")] | ||
|
||
|
||
@pytest.mark.tier(1) | ||
def test_evmserverd_stop(ssh_client): | ||
def test_evmserverd_stop(appliance): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why replace ssh_client there?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because it should be gotten from appliance
not from some mad global ssh_client
@@ -30,26 +30,28 @@ def test_evmserverd_stop(ssh_client): | |||
* For 5.5+: Really call ``service evmserverd status`` and check that the mentions of | |||
stopping the service are present. | |||
""" | |||
server_names = {server["Server Name"] for server in ssh_client.status["servers"]} | |||
assert ssh_client.run_command("service evmserverd stop").rc == 0 | |||
server_names = {server["Server Name"] for server in appliance.ssh_client.status["servers"]} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is raising a KeyError upstream for 'Server Name' but that's outside your intended scope here.
@@ -30,9 +27,8 @@ def test_login(method): | |||
|
|||
@pytest.mark.tier(2) | |||
@pytest.mark.sauce | |||
def test_bad_password(request): | |||
def test_bad_password(request, appliance): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
failing in 5.6 and 5.7 PRT but outside the scope of this change.
Adding appliance fixture