Rewrite test_extract to *_fail variant that tests an expected failure…

… in rpm2cpio

test/test_pkg.py

@@ -1,5 +1,8 @@
 import pytest
 import rpm
+import shutil
+import subprocess
+import unittest.mock as mock
 from rpmlint.pkg import parse_deps, rangeCompare
 
 from Testing import get_tested_package
@@ -24,6 +27,16 @@ def test_range_compare():
         assert not rangeCompare(req, prov)
 
 
+def mock_which_rpm2archive(cmd, *args, **kwargs):
+    # pretend that there's no rpm2archive because we're testing a rpm2cpio failure
+    if cmd == "rpm2archive":
+        return None
+    return shutil.which(cmd, *args, **kwargs)
+
+
+@mock.patch('shutil.which', new=mock_which_rpm2archive)
 @pytest.mark.parametrize('package', ['binary/python311-pytest-xprocess'])
-def test_extract(package, tmp_path):
-    get_tested_package(package, tmp_path)
+def test_extract_fail(package, tmp_path):
+    # the package cannot be extracted using rpm2cpio because it contains a directory without 'x' permission
+    with pytest.raises(subprocess.CalledProcessError) as exc:
+        get_tested_package(package, tmp_path)