Small improvements to the formulas tests

tests/pytests/functional/formulas/conftest.py

@@ -1,16 +1,28 @@
 import shutil
+import zipfile
 
 import pytest
 
-from tests.support.pytest.formulas import SaltStackFormula
+
+@pytest.fixture(scope="module")
+def formula():
+    pytest.fail("The module scoped 'formula' fixture should have been overridden.")
 
 
 @pytest.fixture(scope="module")
-def saltstack_formula(tmp_path_factory, base_env_state_tree_root_dir):
-    zipfiles_dir = tmp_path_factory.mktemp("fomulas-zips")
+def modules(loaders, formula, tmp_path_factory, base_env_state_tree_root_dir):
+    url = f"https://github.com/saltstack-formulas/{formula.name}/archive/refs/tags/v{formula.tag}.zip"
+    zipfiles_dir = tmp_path_factory.mktemp(f"unzipped-{formula.name}-{formula.tag}")
     try:
-        yield SaltStackFormula.with_default_paths(
-            zipfiles_dir, base_env_state_tree_root_dir
+        target_path = base_env_state_tree_root_dir / f"{formula.name}-{formula.tag}"
+        zipfile_path = pytest.helpers.download_file(
+            url, zipfiles_dir / url.split("/")[-1]
         )
+        with zipfile.ZipFile(zipfile_path) as zip_obj:
+            zip_obj.extractall(zipfiles_dir)
+        shutil.move(zipfiles_dir / f"{formula.name}-{formula.tag}", target_path)
+
+        loaders.opts["file_roots"]["base"].append(str(target_path))
+        yield loaders.modules
     finally:
         shutil.rmtree(zipfiles_dir, ignore_errors=True)

tests/pytests/functional/formulas/test_nginx.py

@@ -2,32 +2,25 @@
 Tests using nginx formula
 """
 
+import types
+
 import pytest
 
 pytestmark = [
-    pytest.mark.timeout_unless_on_windows(120),
+    pytest.mark.skip_on_windows,
+    pytest.mark.destructive_test,
+    pytest.mark.timeout_unless_on_windows(240),
 ]
 
 
 @pytest.fixture(scope="module")
-def _formula(saltstack_formula):
-    with saltstack_formula(name="nginx-formula", tag="2.8.1") as formula:
-        yield formula
-
-
-@pytest.fixture(scope="module")
-def modules(loaders, _formula):
-    loaders.opts["file_roots"]["base"].append(
-        str(_formula.state_tree_path / f"{_formula.name}-{_formula.tag}")
-    )
-    return loaders.modules
+def formula():
+    return types.SimpleNamespace(name="nginx-formula", tag="2.8.1")
 
 
-@pytest.mark.skip_on_windows
-@pytest.mark.destructive_test
 def test_formula(modules):
     ret = modules.state.sls("nginx")
     assert not ret.errors
-    assert not ret.failed
+    assert ret.failed is False
     for staterun in ret:
         assert staterun.result is True

tests/pytests/functional/formulas/test_sudoers.py

@@ -2,25 +2,21 @@
 Tests using sudoers formula
 """
 
-import pytest
+import types
 
+import pytest
 
-@pytest.fixture(scope="module")
-def _formula(saltstack_formula):
-    with saltstack_formula(name="sudoers-formula", tag="0.25.0") as formula:
-        yield formula
+pytestmark = [
+    pytest.mark.skip_on_windows,
+    pytest.mark.destructive_test,
+]
 
 
 @pytest.fixture(scope="module")
-def modules(loaders, _formula):
-    loaders.opts["file_roots"]["base"].append(
-        str(_formula.state_tree_path / f"{_formula.name}-{_formula.tag}")
-    )
-    return loaders.modules
+def formula():
+    return types.SimpleNamespace(name="sudoers-formula", tag="0.25.0")
 
 
-@pytest.mark.skip_on_windows
-@pytest.mark.destructive_test
 def test_sudoers_formula(modules):
     ret = modules.state.sls("sudoers")
     assert not ret.errors

tests/pytests/functional/formulas/test_users.py

@@ -2,38 +2,32 @@
 Tests using users formula
 """
 
-import pytest
+import types
 
+import pytest
 
-@pytest.fixture(scope="module")
-def _formula(saltstack_formula):
-    with saltstack_formula(name="users-formula", tag="0.48.8") as formula:
-        yield formula
+pytestmark = [
+    pytest.mark.skip_on_windows,
+    pytest.mark.destructive_test,
+]
 
 
 @pytest.fixture(scope="module")
-def modules(loaders, _formula):
-    loaders.opts["file_roots"]["base"].append(
-        str(_formula.state_tree_path / f"{_formula.name}-{_formula.tag}")
-    )
-    return loaders.modules
+def formula():
+    return types.SimpleNamespace(name="users-formula", tag="0.48.8")
 
 
-@pytest.mark.skip_on_windows
-@pytest.mark.destructive_test
-def test_users_formula(modules):
-    # sudo
+def test_users_sudo_formula(modules):
     ret = modules.state.sls("users.sudo")
     assert not ret.errors
-    assert not ret.failed
+    assert ret.failed is False
     for staterun in ret:
         assert staterun.result is True
 
-    # bashrc
+
+def test_users_bashrc_formula(modules):
     ret = modules.state.sls("users.bashrc")
-    for staterun in ret:
-        assert not staterun.result.failed
     assert not ret.errors
-    assert not ret.failed
+    assert ret.failed is False
     for staterun in ret:
         assert staterun.result is True

tests/pytests/functional/formulas/test_vim.py

@@ -2,29 +2,23 @@
 Tests using vim formula
 """
 
-import pytest
+import types
 
+import pytest
 
-@pytest.fixture(scope="module")
-def _formula(saltstack_formula):
-    with saltstack_formula(name="vim-formula", tag="0.15.5") as formula:
-        yield formula
+pytestmark = [
+    pytest.mark.skip_on_windows,
+    pytest.mark.destructive_test,
+]
 
 
 @pytest.fixture(scope="module")
-def modules(loaders, _formula):
-    loaders.opts["file_roots"]["base"].append(
-        str(_formula.state_tree_path / f"{_formula.name}-{_formula.tag}")
-    )
-    return loaders.modules
+def formula(grains):
+    if grains["oscodename"] == "Photon":
+        pytest.skip(reason="vim package not available for this distribution")
+    return types.SimpleNamespace(name="vim-formula", tag="0.15.5")
 
 
-@pytest.mark.skip_on_windows
-@pytest.mark.destructive_test
-@pytest.mark.skipif(
-    'grains["oscodename"] == "Photon"',
-    reason="vim package not available for this distrubition",
-)
 def test_vim_formula(modules):
     ret = modules.state.sls("vim")
     assert not ret.errors