--runinstalled is not necessary anymore with bootstrap convention, we are always installed

Author: benoit74

.github/workflows/Tests.yaml

@@ -35,7 +35,7 @@ jobs:
           pip install -e .[test,scripts]
 
       - name: Run the tests
-        run: inv coverage --args "--runslow --runinstalled -vvv"
+        run: inv coverage --args "--runslow -vvv"
 
       - name: Upload coverage report to codecov
         if: matrix.python == '3.12'

tests/conftest.py

@@ -10,28 +10,16 @@ def pytest_addoption(parser):
     parser.addoption(
         "--runslow", action="store_true", default=False, help="run slow tests"
     )
-    parser.addoption(
-        "--runinstalled",
-        action="store_true",
-        default=False,
-        help="run tests checking for installed features",
-    )
 
 
 def pytest_configure(config):
     config.addinivalue_line("markers", "slow: mark test as slow to run")
-    config.addinivalue_line(
-        "markers", "installed: mark test as testing installed features"
-    )
 
 
 def pytest_collection_modifyitems(config, items):
     skip_slow = pytest.mark.skip(reason="need --runslow option to run")
-    skip_installed = pytest.mark.skip(reason="need --runinstalled option to run")
 
     for item in items:
-        if "installed" in item.keywords and not config.getoption("--runinstalled"):
-            item.add_marker(skip_installed)
         if "slow" in item.keywords and not config.getoption("--runslow"):
             item.add_marker(skip_slow)
 

tests/ogvjs/test_ogvjs.py

@@ -44,7 +44,6 @@ def prepare_ogvjs_folder(tmp_path, videojs_url, ogvjs_url, videojs_ogvjs_url):
     tmp_path.joinpath(member).rename(tmp_path.joinpath("videojs-ogvjs.js"))
 
 
-@pytest.mark.installed
 def test_installed_script():
     kwargs = {"universal_newlines": True, "stdout": subprocess.PIPE}
     script = subprocess.run(["/usr/bin/env", "fix_ogvjs_dist"], **kwargs, check=False)