Properly skip on Suse.

tests/integration/states/test_pkg.py

@@ -9,12 +9,6 @@
 import os
 import time
 
-# linux_distribution deprecated in py3.7
-try:
-    from platform import linux_distribution
-except ImportError:
-    from distro import linux_distribution
-
 # Import Salt Testing libs
 from tests.support.case import ModuleCase
 from tests.support.mixins import SaltReturnAssertsMixin
@@ -31,6 +25,7 @@
 import salt.utils.platform
 
 # Import 3rd-party libs
+import pytest
 from salt.ext import six
 from salt.ext.six.moves import range  # pylint: disable=import-error,redefined-builtin
 
@@ -70,10 +65,6 @@
 
 _WILDCARDS_SUPPORTED = ('Arch', 'Debian', 'RedHat')
 
-ON_SUSE = False
-if 'SuSE' in linux_distribution(full_distribution_name=False):
-    ON_SUSE = True
-
 
 def pkgmgr_avail(run_function, grains):
     '''
@@ -188,7 +179,7 @@ def setUp(self):
             self.run_function('pkg.refresh_db')
             self.ctx['refresh'] = True
 
-    @skipIf(ON_SUSE, 'WAR ROOM SKIP FRIDAY')
+    @pytest.mark.skipif('grains["os_family"] in ("Suse",)', reason='WAR ROOM SKIP FRIDAY')
     @requires_system_grains
     def test_pkg_001_installed(self, grains):
         '''
@@ -290,7 +281,7 @@ def test_pkg_003_installed_multipkg(self, grains):
             ret = self.run_state('pkg.removed', name=None, pkgs=pkg_targets)
             self.assertSaltTrueReturn(ret)
 
-    @skipIf(ON_SUSE, 'WAR ROOM SKIP FRIDAY')
+    @pytest.mark.skipif('grains["os_family"] in ("Suse",)', reason='WAR ROOM SKIP FRIDAY')
     @requires_system_grains
     def test_pkg_004_installed_multipkg_with_version(self, grains):
         '''

tests/unit/modules/test_boto_lambda.py

@@ -8,12 +8,6 @@
 import os
 from tempfile import NamedTemporaryFile
 
-# linux_distribution deprecated in py3.7
-try:
-    from platform import linux_distribution
-except ImportError:
-    from distro import linux_distribution
-
 # Import Salt Testing libs
 from tests.support.mixins import LoaderModuleMockMixin
 from tests.support.unit import skipIf, TestCase
@@ -44,10 +38,6 @@
 except ImportError:
     HAS_BOTO = False
 
-ON_SUSE = False
-if 'SuSE' in linux_distribution(full_distribution_name=False):
-    ON_SUSE = True
-
 # pylint: enable=import-error,no-name-in-module
 
 # the boto_lambda module relies on the connect_to_region() method

tests/unit/modules/test_cmdmod.py

@@ -9,11 +9,8 @@
 import sys
 import tempfile
 
-# linux_distribution deprecated in py3.7
-try:
-    from platform import linux_distribution
-except ImportError:
-    from distro import linux_distribution
+# Import 3rd-party libs
+import pytest
 
 # Import Salt Libs
 import salt.utils.files
@@ -75,11 +72,6 @@ def stderr(self):
         return self._stderr
 
 
-ON_SUSE = False
-if 'SuSE' in linux_distribution(full_distribution_name=False):
-    ON_SUSE = True
-
-
 @skipIf(NO_MOCK, NO_MOCK_REASON)
 class CMDMODTestCase(TestCase, LoaderModuleMockMixin):
     '''
@@ -356,8 +348,7 @@ def test_run_cwd_doesnt_exist_issue_7154(self):
             raise RuntimeError
 
     @skipIf(True, 'WAR ROOM SKIP FRIDAY')
-    @skipIf(salt.utils.platform.is_windows(), 'Do not run on Windows')
-    @skipIf(salt.utils.platform.is_darwin(), 'Do not run on MacOS')
+    @pytest.mark.skipif('grains["os_family"] in ("MacOS", "Windows")', reason='Do not run on Windows or MacOS')
     def test_run_cwd_in_combination_with_runas(self):
         '''
         cmd.run executes command in the cwd directory

tests/unit/transport/test_zeromq.py

@@ -12,13 +12,8 @@
 import ctypes
 from concurrent.futures.thread import ThreadPoolExecutor
 
-# linux_distribution deprecated in py3.7
-try:
-    from platform import linux_distribution
-except ImportError:
-    from distro import linux_distribution
-
 # Import 3rd-party libs
+import pytest
 import zmq.eventloop.ioloop
 # support pyzmq 13.0.x, TODO: remove once we force people to 14.0.x
 if not hasattr(zmq.eventloop.ioloop, 'ZMQIOLoop'):
@@ -46,10 +41,6 @@
 from tests.support.mock import MagicMock, patch
 from tests.unit.transport.mixins import PubChannelMixin, ReqChannelMixin
 
-ON_SUSE = False
-if 'SuSE' in linux_distribution(full_distribution_name=False):
-    ON_SUSE = True
-
 
 class BaseZMQReqCase(TestCase, AdaptedConfigurationTestCaseMixin):
     '''
@@ -158,7 +149,8 @@ def test_master_uri_override(self):
 
 
 @flaky
-@skipIf(ON_SUSE, 'Skipping until https://github.com/saltstack/salt/issues/32902 gets fixed')
+@pytest.mark.skipif('grains["os_family"] in ("Suse",)',
+                    reason='Skipping until https://github.com/saltstack/salt/issues/32902 gets fixed')
 class AESReqTestCases(BaseZMQReqCase, ReqChannelMixin):
     def setUp(self):
         self.channel = salt.transport.client.ReqChannel.factory(self.minion_config)