Add rpm_vercmp python library for version comparison

changelog/60814.added

@@ -0,0 +1 @@
+Add the python rpm-vercmp library in the rpm_lowpkg.py module.

requirements/static/ci/linux.in

@@ -4,6 +4,7 @@ pygit2<1.1.0; python_version <= '3.8'
 pygit2>=1.4.0; python_version > '3.8'
 pymysql==0.9.3; python_version <= '3.5'
 pymysql>=1.0.2; python_version > '3.5'
+rpm-vercmp
 ansible
 twilio
 python-telegram-bot==12.8; python_version <= '3.5'

requirements/static/ci/py3.10/linux.txt

@@ -818,6 +818,8 @@ responses==0.10.6
     # via moto
 rfc3987==1.3.8
     # via -r requirements/static/ci/common.in
+rpm-vercmp==0.1.2
+    # via -r requirements/static/ci/linux.in
 rsa==4.7.2
     # via
     #   google-auth

requirements/static/ci/py3.5/linux.txt

@@ -801,6 +801,8 @@ responses==0.10.6
     # via moto
 rfc3987==1.3.8
     # via -r requirements/static/ci/common.in
+rpm-vercmp==0.1.2
+    # via -r requirements/static/ci/linux.in
 rsa==4.7.2
     # via
     #   google-auth

requirements/static/ci/py3.6/linux.txt

@@ -804,6 +804,8 @@ responses==0.10.6
     # via moto
 rfc3987==1.3.8
     # via -r requirements/static/ci/common.in
+rpm-vercmp==0.1.2
+    # via -r requirements/static/ci/linux.in
 rsa==4.7.2
     # via
     #   google-auth

requirements/static/ci/py3.7/linux.txt

@@ -825,6 +825,8 @@ responses==0.10.6
     # via moto
 rfc3987==1.3.8
     # via -r requirements/static/ci/common.in
+rpm-vercmp==0.1.2
+    # via -r requirements/static/ci/linux.in
 rsa==4.7.2
     # via
     #   google-auth

requirements/static/ci/py3.8/linux.txt

@@ -818,6 +818,8 @@ responses==0.10.6
     # via moto
 rfc3987==1.3.8
     # via -r requirements/static/ci/common.in
+rpm-vercmp==0.1.2
+    # via -r requirements/static/ci/linux.in
 rsa==4.7.2
     # via
     #   google-auth

requirements/static/ci/py3.9/linux.txt

@@ -820,6 +820,8 @@ responses==0.10.6
     # via moto
 rfc3987==1.3.8
     # via -r requirements/static/ci/common.in
+rpm-vercmp==0.1.2
+    # via -r requirements/static/ci/linux.in
 rsa==4.7.2
     # via
     #   google-auth

salt/modules/rpm_lowpkg.py

@@ -29,6 +29,13 @@ try:
 except ImportError:
     HAS_RPMUTILS = False
 
+try:
+    import rpm_vercmp
+
+    HAS_PY_RPM = True
+except ImportError:
+    HAS_PY_RPM = False
+
 
 log = logging.getLogger(__name__)
 
@@ -720,6 +727,8 @@ def version_cmp(ver1, ver2, ignore_epoch=False):
                     "labelCompare function. Not using rpm.labelCompare for "
                     "version comparison."
                 )
+        elif HAS_PY_RPM:
+            cmp_func = rpm_vercmp.vercmp
         else:
             log.warning(
                 "Please install a package that provides rpm.labelCompare for "
@@ -790,7 +799,17 @@ def version_cmp(ver1, ver2, ignore_epoch=False):
             if not ver1_r or not ver2_r:
                 ver1_r = ver2_r = ""
 
-            cmp_result = cmp_func((ver1_e, ver1_v, ver1_r), (ver2_e, ver2_v, ver2_r))
+            if HAS_PY_RPM:
+                # handle epoch version comparison first
+                # rpm_vercmp.vercmp does not handle epoch version comparison
+                ret = salt.utils.versions.version_cmp(ver1_e, ver2_e)
+                if ret in (1, -1):
+                    return ret
+                cmp_result = cmp_func(ver1, ver2)
+            else:
+                cmp_result = cmp_func(
+                    (ver1_e, ver1_v, ver1_r), (ver2_e, ver2_v, ver2_r)
+                )
             if cmp_result not in (-1, 0, 1):
                 raise CommandExecutionError(
                     "Comparison result '{}' is invalid".format(cmp_result)

tests/pytests/unit/modules/test_rpm_lowpkg.py

@@ -0,0 +1,337 @@
+"""
+    :codeauthor: Jayesh Kariya <jayeshk@saltstack.com>
+"""
+
+
+import pytest
+import salt.modules.cmdmod
+import salt.modules.rpm_lowpkg as rpm
+import salt.utils.path
+from tests.support.mock import MagicMock, patch
+
+# pylint: disable=unused-import
+try:
+    import rpm as rpm_lib
+
+    HAS_RPM = True
+except ImportError:
+    HAS_RPM = False
+
+try:
+    import rpm_vercmp
+
+    HAS_PY_RPM = True
+except ImportError:
+    HAS_PY_RPM = False
+# pylint: enable=unused-import
+
+
+def _called_with_root(mock):
+    cmd = " ".join(mock.call_args[0][0])
+    return cmd.startswith("rpm --root /")
+
+
+@pytest.fixture
+def configure_loader_modules():
+    return {rpm: {"rpm": MagicMock(return_value=MagicMock)}}
+
+
+# 'list_pkgs' function tests: 2
+
+
+def test_list_pkgs():
+    """
+    Test if it list the packages currently installed in a dict
+    """
+    mock = MagicMock(return_value="")
+    with patch.dict(rpm.__salt__, {"cmd.run": mock}):
+        assert rpm.list_pkgs() == {}
+        assert not _called_with_root(mock)
+
+
+def test_list_pkgs_root():
+    """
+    Test if it list the packages currently installed in a dict,
+    called with root parameter
+    """
+    mock = MagicMock(return_value="")
+    with patch.dict(rpm.__salt__, {"cmd.run": mock}):
+        rpm.list_pkgs(root="/")
+        assert _called_with_root(mock)
+
+
+# 'verify' function tests: 2
+
+
+def test_verify():
+    """
+    Test if it runs an rpm -Va on a system, and returns the
+    results in a dict
+    """
+    mock = MagicMock(
+        return_value={"stdout": "", "stderr": "", "retcode": 0, "pid": 12345}
+    )
+    with patch.dict(rpm.__salt__, {"cmd.run_all": mock}):
+        assert rpm.verify("httpd") == {}
+        assert not _called_with_root(mock)
+
+
+def test_verify_root():
+    """
+    Test if it runs an rpm -Va on a system, and returns the
+    results in a dict, called with root parameter
+    """
+    mock = MagicMock(
+        return_value={"stdout": "", "stderr": "", "retcode": 0, "pid": 12345}
+    )
+    with patch.dict(rpm.__salt__, {"cmd.run_all": mock}):
+        rpm.verify("httpd", root="/")
+        assert _called_with_root(mock)
+
+
+# 'file_list' function tests: 2
+
+
+def test_file_list():
+    """
+    Test if it list the files that belong to a package.
+    """
+    mock = MagicMock(return_value="")
+    with patch.dict(rpm.__salt__, {"cmd.run": mock}):
+        assert rpm.file_list("httpd") == {"errors": [], "files": []}
+        assert not _called_with_root(mock)
+
+
+def test_file_list_root():
+    """
+    Test if it list the files that belong to a package, using the
+    root parameter.
+    """
+
+    mock = MagicMock(return_value="")
+    with patch.dict(rpm.__salt__, {"cmd.run": mock}):
+        rpm.file_list("httpd", root="/")
+        assert _called_with_root(mock)
+
+
+# 'file_dict' function tests: 2
+
+
+def test_file_dict():
+    """
+    Test if it list the files that belong to a package
+    """
+    mock = MagicMock(return_value="")
+    with patch.dict(rpm.__salt__, {"cmd.run": mock}):
+        assert rpm.file_dict("httpd") == {"errors": [], "packages": {}}
+        assert not _called_with_root(mock)
+
+
+def test_file_dict_root():
+    """
+    Test if it list the files that belong to a package
+    """
+    mock = MagicMock(return_value="")
+    with patch.dict(rpm.__salt__, {"cmd.run": mock}):
+        rpm.file_dict("httpd", root="/")
+        assert _called_with_root(mock)
+
+
+# 'owner' function tests: 1
+
+
+def test_owner():
+    """
+    Test if it return the name of the package that owns the file.
+    """
+    assert rpm.owner() == ""
+
+    ret = "file /usr/bin/salt-jenkins-build is not owned by any package"
+    mock = MagicMock(return_value=ret)
+    with patch.dict(rpm.__salt__, {"cmd.run_stdout": mock}):
+        assert rpm.owner("/usr/bin/salt-jenkins-build") == ""
+        assert not _called_with_root(mock)
+
+    ret = {
+        "/usr/bin/vim": "vim-enhanced-7.4.160-1.e17.x86_64",
+        "/usr/bin/python": "python-2.7.5-16.e17.x86_64",
+    }
+    mock = MagicMock(
+        side_effect=[
+            "python-2.7.5-16.e17.x86_64",
+            "vim-enhanced-7.4.160-1.e17.x86_64",
+        ]
+    )
+    with patch.dict(rpm.__salt__, {"cmd.run_stdout": mock}):
+        assert rpm.owner("/usr/bin/python", "/usr/bin/vim") == ret
+        assert not _called_with_root(mock)
+
+
+def test_owner_root():
+    """
+    Test if it return the name of the package that owns the file,
+    using the parameter root.
+    """
+    assert rpm.owner() == ""
+
+    ret = "file /usr/bin/salt-jenkins-build is not owned by any package"
+    mock = MagicMock(return_value=ret)
+    with patch.dict(rpm.__salt__, {"cmd.run_stdout": mock}):
+        rpm.owner("/usr/bin/salt-jenkins-build", root="/")
+        assert _called_with_root(mock)
+
+
+# 'checksum' function tests: 2
+
+
+def test_checksum():
+    """
+    Test if checksum validate as expected
+    """
+    ret = {
+        "file1.rpm": True,
+        "file2.rpm": False,
+        "file3.rpm": False,
+    }
+
+    mock = MagicMock(side_effect=[True, 0, True, 1, False, 0])
+    with patch.dict(rpm.__salt__, {"file.file_exists": mock, "cmd.retcode": mock}):
+        assert rpm.checksum("file1.rpm", "file2.rpm", "file3.rpm") == ret
+        assert not _called_with_root(mock)
+
+
+def test_checksum_root():
+    """
+    Test if checksum validate as expected, using the parameter
+    root
+    """
+    mock = MagicMock(side_effect=[True, 0])
+    with patch.dict(rpm.__salt__, {"file.file_exists": mock, "cmd.retcode": mock}):
+        rpm.checksum("file1.rpm", root="/")
+        assert _called_with_root(mock)
+
+
+@pytest.mark.parametrize("rpm_lib", ["HAS_RPM", "HAS_PY_RPM", "rpmdev-vercmp"])
+def test_version_cmp_rpm_all_libraries(rpm_lib):
+    """
+    Test package version when each library is installed
+    """
+    rpmdev = salt.utils.path.which("rpmdev-vercmp")
+    patch_cmd = patch.dict(rpm.__salt__, {"cmd.run_all": salt.modules.cmdmod.run_all})
+    if rpm_lib == "rpmdev-vercmp":
+        if rpmdev:
+            patch_rpm = patch("salt.modules.rpm_lowpkg.HAS_RPM", False)
+            patch_py_rpm = patch("salt.modules.rpm_lowpkg.HAS_PY_RPM", False)
+        else:
+            pytest.skip("The rpmdev-vercmp binary is not installed")
+    elif rpm_lib == "HAS_RPM":
+        if HAS_RPM:
+            patch_rpm = patch("salt.modules.rpm_lowpkg.HAS_RPM", True)
+            patch_py_rpm = patch("salt.modules.rpm_lowpkg.HAS_PY_RPM", False)
+        else:
+            pytest.skip("The RPM lib is not installed, skipping")
+    elif rpm_lib == "HAS_PY_RPM":
+        if HAS_PY_RPM:
+            patch_rpm = patch("salt.modules.rpm_lowpkg.HAS_RPM", False)
+            patch_py_rpm = patch("salt.modules.rpm_lowpkg.HAS_PY_RPM", True)
+        else:
+            pytest.skip("The Python RPM lib is not installed, skipping")
+
+    with patch_rpm, patch_py_rpm, patch_cmd:
+        assert -1 == rpm.version_cmp("1", "2")
+        assert -1 == rpm.version_cmp("2.9.1-6.el7_2.3", "2.9.1-6.el7.4")
+        assert 1 == rpm.version_cmp("3.2", "3.0")
+        assert 0 == rpm.version_cmp("3.0", "3.0")
+        assert 1 == rpm.version_cmp("1:2.9.1-6.el7_2.3", "2.9.1-6.el7.4")
+        assert -1 == rpm.version_cmp("1:2.9.1-6.el7_2.3", "1:2.9.1-6.el7.4")
+        assert 1 == rpm.version_cmp("2:2.9.1-6.el7_2.3", "1:2.9.1-6.el7.4")
+        assert 0 == rpm.version_cmp("3:2.9.1-6.el7.4", "3:2.9.1-6.el7.4")
+        assert -1 == rpm.version_cmp("3:2.9.1-6.el7.4", "3:2.9.1-7.el7.4")
+        assert 1 == rpm.version_cmp("3:2.9.1-8.el7.4", "3:2.9.1-7.el7.4")
+
+
+@patch("salt.modules.rpm_lowpkg.HAS_RPM", True)
+@patch("salt.modules.rpm_lowpkg.rpm.labelCompare", return_value=-1)
+@patch("salt.modules.rpm_lowpkg.log")
+def test_version_cmp_rpm(mock_log, mock_labelCompare):
+    """
+    Test package version if RPM-Python is installed
+
+    :return:
+    """
+    assert -1 == rpm.version_cmp("1", "2")
+    assert not mock_log.warning.called
+    assert mock_labelCompare.called
+
+
+@patch("salt.modules.rpm_lowpkg.HAS_RPM", False)
+@patch("salt.modules.rpm_lowpkg.HAS_RPMUTILS", True)
+@patch("salt.modules.rpm_lowpkg.HAS_PY_RPM", False)
+@patch("salt.modules.rpm_lowpkg.rpmUtils", create=True)
+@patch("salt.modules.rpm_lowpkg.log")
+def test_version_cmp_rpmutils(mock_log, mock_rpmUtils):
+    """
+    Test package version if rpmUtils.miscutils called
+
+    :return:
+    """
+    mock_rpmUtils.miscutils = MagicMock()
+    mock_rpmUtils.miscutils.compareEVR = MagicMock(return_value=-1)
+    assert -1 == rpm.version_cmp("1", "2")
+    assert mock_log.warning.called
+    assert mock_rpmUtils.miscutils.compareEVR.called
+    assert (
+        mock_log.warning.mock_calls[0][1][0]
+        == "Please install a package that provides rpm.labelCompare for more accurate version comparisons."
+    )
+
+
+@patch("salt.modules.rpm_lowpkg.HAS_RPM", False)
+@patch("salt.modules.rpm_lowpkg.HAS_RPMUTILS", False)
+@patch("salt.modules.rpm_lowpkg.HAS_PY_RPM", False)
+@patch("salt.utils.path.which", return_value=True)
+@patch("salt.modules.rpm_lowpkg.log")
+def test_version_cmp_rpmdev_vercmp(mock_log, mock_which):
+    """
+    Test package version if rpmdev-vercmp is installed
+
+    :return:
+    """
+    mock__salt__ = MagicMock(return_value={"retcode": 12})
+    with patch.dict(rpm.__salt__, {"cmd.run_all": mock__salt__}):
+        assert -1 == rpm.version_cmp("1", "2")
+        assert mock__salt__.called
+        assert mock_log.warning.called
+        assert (
+            mock_log.warning.mock_calls[0][1][0]
+            == "Please install a package that provides rpm.labelCompare for more accurate version comparisons."
+        )
+        assert (
+            mock_log.warning.mock_calls[1][1][0]
+            == "Installing the rpmdevtools package may surface dev tools in production."
+        )
+
+
+@patch("salt.modules.rpm_lowpkg.HAS_RPM", False)
+@patch("salt.modules.rpm_lowpkg.HAS_RPMUTILS", False)
+@patch("salt.modules.rpm_lowpkg.HAS_PY_RPM", False)
+@patch("salt.utils.versions.version_cmp", return_value=-1)
+@patch("salt.utils.path.which", return_value=False)
+@patch("salt.modules.rpm_lowpkg.log")
+def test_version_cmp_python(mock_log, mock_which, mock_version_cmp):
+    """
+    Test package version if falling back to python
+
+    :return:
+    """
+    assert -1 == rpm.version_cmp("1", "2")
+    assert mock_version_cmp.called
+    assert mock_log.warning.called
+    assert (
+        mock_log.warning.mock_calls[0][1][0]
+        == "Please install a package that provides rpm.labelCompare for more accurate version comparisons."
+    )
+    assert (
+        mock_log.warning.mock_calls[1][1][0]
+        == "Falling back on salt.utils.versions.version_cmp() for version comparisons"
+    )

tests/unit/modules/test_rpm_lowpkg.py

@@ -1,266 +0,0 @@
-"""
-    :codeauthor: Jayesh Kariya <jayeshk@saltstack.com>
-"""
-
-
-import salt.modules.rpm_lowpkg as rpm
-from tests.support.mixins import LoaderModuleMockMixin
-from tests.support.mock import MagicMock, patch
-from tests.support.unit import TestCase
-
-
-def _called_with_root(mock):
-    cmd = " ".join(mock.call_args[0][0])
-    return cmd.startswith("rpm --root /")
-
-
-class RpmTestCase(TestCase, LoaderModuleMockMixin):
-    """
-    Test cases for salt.modules.rpm
-    """
-
-    def setup_loader_modules(self):
-        return {rpm: {"rpm": MagicMock(return_value=MagicMock)}}
-
-    # 'list_pkgs' function tests: 2
-
-    def test_list_pkgs(self):
-        """
-        Test if it list the packages currently installed in a dict
-        """
-        mock = MagicMock(return_value="")
-        with patch.dict(rpm.__salt__, {"cmd.run": mock}):
-            self.assertDictEqual(rpm.list_pkgs(), {})
-            self.assertFalse(_called_with_root(mock))
-
-    def test_list_pkgs_root(self):
-        """
-        Test if it list the packages currently installed in a dict,
-        called with root parameter
-        """
-        mock = MagicMock(return_value="")
-        with patch.dict(rpm.__salt__, {"cmd.run": mock}):
-            rpm.list_pkgs(root="/")
-            self.assertTrue(_called_with_root(mock))
-
-    # 'verify' function tests: 2
-
-    def test_verify(self):
-        """
-        Test if it runs an rpm -Va on a system, and returns the
-        results in a dict
-        """
-        mock = MagicMock(
-            return_value={"stdout": "", "stderr": "", "retcode": 0, "pid": 12345}
-        )
-        with patch.dict(rpm.__salt__, {"cmd.run_all": mock}):
-            self.assertDictEqual(rpm.verify("httpd"), {})
-            self.assertFalse(_called_with_root(mock))
-
-    def test_verify_root(self):
-        """
-        Test if it runs an rpm -Va on a system, and returns the
-        results in a dict, called with root parameter
-        """
-        mock = MagicMock(
-            return_value={"stdout": "", "stderr": "", "retcode": 0, "pid": 12345}
-        )
-        with patch.dict(rpm.__salt__, {"cmd.run_all": mock}):
-            rpm.verify("httpd", root="/")
-            self.assertTrue(_called_with_root(mock))
-
-    # 'file_list' function tests: 2
-
-    def test_file_list(self):
-        """
-        Test if it list the files that belong to a package.
-        """
-        mock = MagicMock(return_value="")
-        with patch.dict(rpm.__salt__, {"cmd.run": mock}):
-            self.assertDictEqual(rpm.file_list("httpd"), {"errors": [], "files": []})
-            self.assertFalse(_called_with_root(mock))
-
-    def test_file_list_root(self):
-        """
-        Test if it list the files that belong to a package, using the
-        root parameter.
-        """
-
-        mock = MagicMock(return_value="")
-        with patch.dict(rpm.__salt__, {"cmd.run": mock}):
-            rpm.file_list("httpd", root="/")
-            self.assertTrue(_called_with_root(mock))
-
-    # 'file_dict' function tests: 2
-
-    def test_file_dict(self):
-        """
-        Test if it list the files that belong to a package
-        """
-        mock = MagicMock(return_value="")
-        with patch.dict(rpm.__salt__, {"cmd.run": mock}):
-            self.assertDictEqual(rpm.file_dict("httpd"), {"errors": [], "packages": {}})
-            self.assertFalse(_called_with_root(mock))
-
-    def test_file_dict_root(self):
-        """
-        Test if it list the files that belong to a package
-        """
-        mock = MagicMock(return_value="")
-        with patch.dict(rpm.__salt__, {"cmd.run": mock}):
-            rpm.file_dict("httpd", root="/")
-            self.assertTrue(_called_with_root(mock))
-
-    # 'owner' function tests: 1
-
-    def test_owner(self):
-        """
-        Test if it return the name of the package that owns the file.
-        """
-        self.assertEqual(rpm.owner(), "")
-
-        ret = "file /usr/bin/salt-jenkins-build is not owned by any package"
-        mock = MagicMock(return_value=ret)
-        with patch.dict(rpm.__salt__, {"cmd.run_stdout": mock}):
-            self.assertEqual(rpm.owner("/usr/bin/salt-jenkins-build"), "")
-            self.assertFalse(_called_with_root(mock))
-
-        ret = {
-            "/usr/bin/vim": "vim-enhanced-7.4.160-1.e17.x86_64",
-            "/usr/bin/python": "python-2.7.5-16.e17.x86_64",
-        }
-        mock = MagicMock(
-            side_effect=[
-                "python-2.7.5-16.e17.x86_64",
-                "vim-enhanced-7.4.160-1.e17.x86_64",
-            ]
-        )
-        with patch.dict(rpm.__salt__, {"cmd.run_stdout": mock}):
-            self.assertDictEqual(rpm.owner("/usr/bin/python", "/usr/bin/vim"), ret)
-            self.assertFalse(_called_with_root(mock))
-
-    def test_owner_root(self):
-        """
-        Test if it return the name of the package that owns the file,
-        using the parameter root.
-        """
-        self.assertEqual(rpm.owner(), "")
-
-        ret = "file /usr/bin/salt-jenkins-build is not owned by any package"
-        mock = MagicMock(return_value=ret)
-        with patch.dict(rpm.__salt__, {"cmd.run_stdout": mock}):
-            rpm.owner("/usr/bin/salt-jenkins-build", root="/")
-            self.assertTrue(_called_with_root(mock))
-
-    # 'checksum' function tests: 2
-
-    def test_checksum(self):
-        """
-        Test if checksum validate as expected
-        """
-        ret = {
-            "file1.rpm": True,
-            "file2.rpm": False,
-            "file3.rpm": False,
-        }
-
-        mock = MagicMock(side_effect=[True, 0, True, 1, False, 0])
-        with patch.dict(rpm.__salt__, {"file.file_exists": mock, "cmd.retcode": mock}):
-            self.assertDictEqual(
-                rpm.checksum("file1.rpm", "file2.rpm", "file3.rpm"), ret
-            )
-            self.assertFalse(_called_with_root(mock))
-
-    def test_checksum_root(self):
-        """
-        Test if checksum validate as expected, using the parameter
-        root
-        """
-        mock = MagicMock(side_effect=[True, 0])
-        with patch.dict(rpm.__salt__, {"file.file_exists": mock, "cmd.retcode": mock}):
-            rpm.checksum("file1.rpm", root="/")
-            self.assertTrue(_called_with_root(mock))
-
-    @patch("salt.modules.rpm_lowpkg.HAS_RPM", True)
-    @patch("salt.modules.rpm_lowpkg.rpm.labelCompare", return_value=-1)
-    @patch("salt.modules.rpm_lowpkg.log")
-    def test_version_cmp_rpm(self, mock_log, mock_labelCompare):
-        """
-        Test package version if RPM-Python is installed
-
-        :return:
-        """
-        self.assertEqual(-1, rpm.version_cmp("1", "2"))
-        self.assertEqual(mock_log.warning.called, False)
-        self.assertEqual(mock_labelCompare.called, True)
-
-    @patch("salt.modules.rpm_lowpkg.HAS_RPM", False)
-    @patch("salt.modules.rpm_lowpkg.HAS_RPMUTILS", True)
-    @patch("salt.modules.rpm_lowpkg.rpmUtils", create=True)
-    @patch("salt.modules.rpm_lowpkg.log")
-    def test_version_cmp_rpmutils(self, mock_log, mock_rpmUtils):
-        """
-        Test package version if rpmUtils.miscutils called
-
-        :return:
-        """
-        mock_rpmUtils.miscutils = MagicMock()
-        mock_rpmUtils.miscutils.compareEVR = MagicMock(return_value=-1)
-        self.assertEqual(-1, rpm.version_cmp("1", "2"))
-        self.assertEqual(mock_log.warning.called, True)
-        self.assertEqual(mock_rpmUtils.miscutils.compareEVR.called, True)
-        self.assertEqual(
-            mock_log.warning.mock_calls[0][1][0],
-            "Please install a package that provides rpm.labelCompare for more accurate"
-            " version comparisons.",
-        )
-
-    @patch("salt.modules.rpm_lowpkg.HAS_RPM", False)
-    @patch("salt.modules.rpm_lowpkg.HAS_RPMUTILS", False)
-    @patch("salt.utils.path.which", return_value=True)
-    @patch("salt.modules.rpm_lowpkg.log")
-    def test_version_cmp_rpmdev_vercmp(self, mock_log, mock_which):
-        """
-        Test package version if rpmdev-vercmp is installed
-
-        :return:
-        """
-        mock__salt__ = MagicMock(return_value={"retcode": 12})
-        with patch.dict(rpm.__salt__, {"cmd.run_all": mock__salt__}):
-            self.assertEqual(-1, rpm.version_cmp("1", "2"))
-            self.assertEqual(mock__salt__.called, True)
-            self.assertEqual(mock_log.warning.called, True)
-            self.assertEqual(
-                mock_log.warning.mock_calls[0][1][0],
-                "Please install a package that provides rpm.labelCompare for more"
-                " accurate version comparisons.",
-            )
-            self.assertEqual(
-                mock_log.warning.mock_calls[1][1][0],
-                "Installing the rpmdevtools package may surface dev tools in"
-                " production.",
-            )
-
-    @patch("salt.modules.rpm_lowpkg.HAS_RPM", False)
-    @patch("salt.modules.rpm_lowpkg.HAS_RPMUTILS", False)
-    @patch("salt.utils.versions.version_cmp", return_value=-1)
-    @patch("salt.utils.path.which", return_value=False)
-    @patch("salt.modules.rpm_lowpkg.log")
-    def test_version_cmp_python(self, mock_log, mock_which, mock_version_cmp):
-        """
-        Test package version if falling back to python
-
-        :return:
-        """
-        self.assertEqual(-1, rpm.version_cmp("1", "2"))
-        self.assertEqual(mock_version_cmp.called, True)
-        self.assertEqual(mock_log.warning.called, True)
-        self.assertEqual(
-            mock_log.warning.mock_calls[0][1][0],
-            "Please install a package that provides rpm.labelCompare for more accurate"
-            " version comparisons.",
-        )
-        self.assertEqual(
-            mock_log.warning.mock_calls[1][1][0],
-            "Falling back on salt.utils.versions.version_cmp() for version comparisons",
-        )