Skip to content

Commit

Permalink
Fix broken skip logic
Browse files Browse the repository at this point in the history
These tests are supposed to be skipped if we CANNOT be root, not if we
*can*.
  • Loading branch information
lunkwill42 committed Dec 8, 2023
1 parent 3440e00 commit f7b13c0
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions tests/integration/pping_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ def timeout_command_line_program_exists():
reason="{} is not available".format(TIMEOUT_COMMAND_LINE),
)
@pytest.mark.skipif(
can_be_root(), reason="pping can only be tested with root privileges"
not can_be_root(), reason="pping can only be tested with root privileges"
)
def test_pping_localhost_should_work(localhost, pping_test_config):
output = get_pping_output()
Expand All @@ -50,7 +50,7 @@ def test_pping_localhost_should_work(localhost, pping_test_config):
reason="{} is not available".format(TIMEOUT_COMMAND_LINE),
)
@pytest.mark.skipif(
can_be_root(), reason="pping can only be tested with root privileges"
not can_be_root(), reason="pping can only be tested with root privileges"
)
def test_pping_nonavailable_host_should_fail(
host_expected_to_be_down, pping_test_config
Expand All @@ -68,7 +68,7 @@ def test_pping_nonavailable_host_should_fail(
reason="{} is not available".format(TIMEOUT_COMMAND_LINE),
)
@pytest.mark.skipif(
can_be_root(), reason="pping can only be tested with root privileges"
not can_be_root(), reason="pping can only be tested with root privileges"
)
def test_pping_should_post_event_when_host_is_unreachable(
host_expected_to_be_down, pping_test_config
Expand Down

0 comments on commit f7b13c0

Please sign in to comment.