Drop "incremental" marker.

Previous commits removed its two only uses, replaced by an autouse fixture.
Other possible uses would be covered (with a more fine-grained approach)
by pytest-dependency.

This frees the global pytest_runtest_makereport hook for potential reuse.

Signed-off-by: Yann Dirson <[email protected]>

Author: ydirson

conftest.py

@@ -19,21 +19,6 @@
 # need to import them in the global conftest.py so that they are recognized as fixtures.
 from pkgfixtures import formatted_and_mounted_ext4_disk, sr_disk_wiped
 
-# *** Support for incremental tests in test classes ***
-# From https://stackoverflow.com/questions/12411431/how-to-skip-the-rest-of-tests-in-the-class-if-one-has-failed
-def pytest_runtest_makereport(item, call):
-    if "incremental" in item.keywords:
-        if call.excinfo is not None:
-            parent = item.parent
-            parent._previousfailed = item
-
-def pytest_runtest_setup(item):
-    previousfailed = getattr(item.parent, "_previousfailed", None)
-    if previousfailed is not None:
-        pytest.skip("previous test failed (%s)" % previousfailed.name)
-
-# *** End of: Support for incremental tests ***
-
 def pytest_addoption(parser):
     parser.addoption(
         "--hosts",

pytest.ini

@@ -3,7 +3,6 @@ addopts = -ra --maxfail=1 -s
 markers =
     # *** Markers that change test behaviour ***
     default_vm: mark a test with a default VM in case no --vm parameter was given.
-    incremental: mark a class so that any test failure skips the rest of the tests in the class.
 
     # *** Markers used to select tests at collect stage ***