Merge pull request #2657 from IntersectMBO/long_on_byron_cluster

feat(tests): add conditional markers for Byron tests

cardano_node_tests/tests/common.py

@@ -9,6 +9,7 @@
 from cardano_node_tests.tests import issues
 from cardano_node_tests.utils import cluster_nodes
 from cardano_node_tests.utils import clusterlib_utils
+from cardano_node_tests.utils import configuration
 from cardano_node_tests.utils import pytest_utils
 from cardano_node_tests.utils.versions import VERSIONS
 
@@ -18,6 +19,11 @@
 MAX_INT64 = (2**63) - 1
 MAX_UINT64 = (2**64) - 1
 
+ORDER5_BYRON = (
+    pytest.mark.order(5) if "_fast" not in configuration.SCRIPTS_DIRNAME else pytest.mark.noop
+)
+LONG_BYRON = pytest.mark.long if "_fast" not in configuration.SCRIPTS_DIRNAME else pytest.mark.noop
+
 
 _BLD_SKIP_REASON = ""
 if VERSIONS.transaction_era != VERSIONS.cluster_era:

cardano_node_tests/tests/test_configuration.py

@@ -129,8 +129,8 @@ def check_epoch_length(cluster_obj: clusterlib.ClusterLib) -> None:
 # It takes long time to setup the cluster instance (when starting from Byron).
 # We mark the tests as "long" and set the highest priority, so the setup is done at the
 # beginning of the testrun, instead of needing to respin a cluster that is already running.
-@pytest.mark.order(5)
-@pytest.mark.long
+@common.ORDER5_BYRON
+@common.LONG_BYRON
 class TestBasic:
     """Basic tests for node configuration."""
 

cardano_node_tests/tests/test_kes.py

@@ -157,8 +157,8 @@ class TestKES:
     # It takes long time to setup the cluster instance (when starting from Byron).
     # We mark the tests as "long" and set the highest priority, so the setup is done at the
     # beginning of the testrun, instead of needing to respin a cluster that is already running.
-    @pytest.mark.order(5)
-    @pytest.mark.long
+    @common.ORDER5_BYRON
+    @common.LONG_BYRON
     def test_expired_kes(
         self,
         cluster_kes: clusterlib.ClusterLib,

cardano_node_tests/tests/test_pools.py

@@ -1776,8 +1776,8 @@ def test_pool_registration_deregistration(
 # It takes long time to setup the cluster instance (when starting from Byron).
 # We mark the tests as "long" and set the highest priority, so the setup is done at the
 # beginning of the testrun, instead of needing to respin a cluster that is already running.
-@pytest.mark.order(5)
-@pytest.mark.long
+@common.ORDER5_BYRON
+@common.LONG_BYRON
 class TestPoolCost:
     """Tests for stake pool cost."""
 

pyproject.toml

@@ -136,4 +136,5 @@ markers = [
     "upgrade: test(s) for upgrade testing",
     "plutus: test(s) for plutus",
     "disabled: temporarily disabled test(s)",
+    "noop: placeholder marker",
 ]