Use skipif instead of skip_if as it seems not behaving equally

openSUSE · Feb 15, 2024 · a591123 · a591123
1 parent 425fdd3
commit a591123
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/tests/pytests/functional/modules/test_system.py b/tests/pytests/functional/modules/test_system.py
@@ -14,7 +14,7 @@
 pytestmark = [
     pytest.mark.skip_unless_on_linux,
     pytest.mark.slow_test,
-    pytest.mark.skip_if(INSIDE_CONTAINER, reason="No systemd in container."),
+    pytest.mark.skipif(INSIDE_CONTAINER, reason="No systemd in container."),
 ]
 
 log = logging.getLogger(__name__)

diff --git a/tests/pytests/integration/ssh/test_master.py b/tests/pytests/integration/ssh/test_master.py
@@ -15,7 +15,7 @@
 INSIDE_CONTAINER = os.getenv("HOSTNAME", "") == "salt-test-container"
 
 
-@pytest.mark.skip_if(INSIDE_CONTAINER, reason="No systemd in container.")
+@pytest.mark.skipif(INSIDE_CONTAINER, reason="No systemd in container.")
 @pytest.mark.skip_if_not_root
 def test_service(salt_ssh_cli, grains):
     service = "cron"

diff --git a/tests/pytests/scenarios/setup/test_install.py b/tests/pytests/scenarios/setup/test_install.py
@@ -24,7 +24,7 @@
     pytest.mark.windows_whitelisted,
     pytest.mark.skip_initial_onedir_failure,
     pytest.mark.skip_if_binaries_missing(*KNOWN_BINARY_NAMES, check_all=False),
-    pytest.mark.skip_if(INSIDE_CONTAINER, reason="No gcc and python3-devel in container."),
+    pytest.mark.skipif(INSIDE_CONTAINER, reason="No gcc and python3-devel in container."),
 ]