feat(test,cli): --skip-summary for phase1 account counting with enginex

Author: fselmo

.github/configs/feature.yaml

@@ -1,27 +1,27 @@
 # Unless filling for special features, all features should fill for previous forks (starting from Frontier) too
 stable:
   evm-type: stable
-  fill-params: --no-html --until=Prague --fill-static-tests --ignore=tests/static/state_tests/stQuadraticComplexityTest
+  fill-params: --no-html --skip-summary --until=Prague --fill-static-tests --ignore=tests/static/state_tests/stQuadraticComplexityTest
 
 develop:
   evm-type: develop
-  fill-params: --no-html --until=BPO4 --fill-static-tests --ignore=tests/static/state_tests/stQuadraticComplexityTest
+  fill-params: --no-html --skip-summary --until=BPO4 --fill-static-tests --ignore=tests/static/state_tests/stQuadraticComplexityTest
 
 benchmark:
   evm-type: benchmark
-  fill-params: --no-html --fork=Prague --gas-benchmark-values 1,5,10,30,60,100,150 -m benchmark ./tests/benchmark
+  fill-params: --no-html --skip-summary --fork=Prague --gas-benchmark-values 1,5,10,30,60,100,150 -m benchmark ./tests/benchmark
 
 benchmark_develop:
   evm-type: benchmark
-  fill-params: --no-html --fork=Osaka --gas-benchmark-values 1,5,10,30,60,100,150 -m "benchmark" ./tests/benchmark
+  fill-params: --no-html --skip-summary --fork=Osaka --gas-benchmark-values 1,5,10,30,60,100,150 -m "benchmark" ./tests/benchmark
   feature_only: true
 
 benchmark_fast:
   evm-type: benchmark
-  fill-params: --no-html --fork=Prague --gas-benchmark-values 100 -m "benchmark" ./tests/benchmark
+  fill-params: --no-html --skip-summary --fork=Prague --gas-benchmark-values 100 -m "benchmark" ./tests/benchmark
   feature_only: true
 
 bal:
   evm-type: develop
-  fill-params: --no-html --fork=Amsterdam --fill-static-tests
+  fill-params: --no-html --skip-summary --fork=Amsterdam --fill-static-tests
   feature_only: true

packages/testing/src/execution_testing/cli/pytest_commands/plugins/filler/filler.py

@@ -599,6 +599,16 @@ def pytest_addoption(parser: pytest.Parser) -> None:
             "groups, phase 2 generates all supported fixture formats."
         ),
     )
+    test_group.addoption(
+        "--skip-summary",
+        action="store_true",
+        dest="skip_summary",
+        default=False,
+        help=(
+            "Skip computing expensive summary statistics "
+            "(e.g., total account counts)."
+        ),
+    )
 
     optimize_gas_group = parser.getgroup(
         "optimize gas",
@@ -694,6 +704,9 @@ def pytest_configure(config: pytest.Config) -> None:
         config
     )
 
+    # Capture start time for duration reporting
+    config.session_start_time = datetime.datetime.now()  # type: ignore[attr-defined]
+
     if is_help_or_collectonly_mode(config):
         return
 
@@ -835,33 +848,63 @@ def pytest_terminal_summary(
         return
     stats = terminalreporter.stats
     if "passed" in stats and stats["passed"]:
+        # Calculate duration
+        duration = datetime.datetime.now() - config.session_start_time  # type: ignore[attr-defined]
+        total_seconds = int(duration.total_seconds())
+        hours, remainder = divmod(total_seconds, 3600)
+        minutes, seconds = divmod(remainder, 60)
+        duration_str = f"{hours:02d}:{minutes:02d}:{seconds:02d}"
+
         # Custom message for Phase 1 (pre-allocation group generation)
         session_instance: FillingSession = config.filling_session  # type: ignore[attr-defined]
         if session_instance.phase_manager.is_pre_alloc_generation:
-            # Generate summary stats
-            pre_alloc_groups: PreAllocGroups
-            if config.pluginmanager.hasplugin("xdist"):
-                # Load pre-allocation groups from disk
-                pre_alloc_groups = PreAllocGroups.from_folder(
-                    config.fixture_output.pre_alloc_groups_folder_path,  # type: ignore[attr-defined]
-                    lazy_load=False,
+            skip_summary = config.getoption("skip_summary")
+
+            if skip_summary:
+                # Fast path: only count groups without loading file contents
+                pre_alloc_groups: PreAllocGroups
+                if config.pluginmanager.hasplugin("xdist"):
+                    pre_alloc_groups = PreAllocGroups.from_folder(
+                        config.fixture_output.pre_alloc_groups_folder_path,  # type: ignore[attr-defined]
+                        lazy_load=True,
+                    )
+                else:
+                    assert session_instance.pre_alloc_groups is not None
+                    pre_alloc_groups = session_instance.pre_alloc_groups
+                total_groups = len(pre_alloc_groups.root)
+                terminalreporter.write_sep(
+                    "=",
+                    f" Phase 1 Complete ({duration_str}): "
+                    f"Generated {total_groups} pre-alloc groups ",
+                    bold=True,
+                    green=True,
                 )
             else:
-                assert session_instance.pre_alloc_groups is not None
-                pre_alloc_groups = session_instance.pre_alloc_groups
+                # Full summary: load all groups and count accounts
+                pre_alloc_groups_full: PreAllocGroups
+                if config.pluginmanager.hasplugin("xdist"):
+                    pre_alloc_groups_full = PreAllocGroups.from_folder(
+                        config.fixture_output.pre_alloc_groups_folder_path,  # type: ignore[attr-defined]
+                        lazy_load=False,
+                    )
+                else:
+                    assert session_instance.pre_alloc_groups is not None
+                    pre_alloc_groups_full = session_instance.pre_alloc_groups
 
-            total_groups = len(pre_alloc_groups.root)
-            total_accounts = sum(
-                group.pre_account_count for group in pre_alloc_groups.values()
-            )
+                total_groups = len(pre_alloc_groups_full.root)
+                total_accounts = sum(
+                    group.pre_account_count
+                    for group in pre_alloc_groups_full.values()
+                )
 
-            terminalreporter.write_sep(
-                "=",
-                f" Phase 1 Complete: Generated {total_groups} pre-alloc "
-                f"groups ({total_accounts} total accounts) ",
-                bold=True,
-                green=True,
-            )
+                terminalreporter.write_sep(
+                    "=",
+                    f" Phase 1 Complete ({duration_str}): "
+                    f"Generated {total_groups} pre-alloc groups "
+                    f"({total_accounts} total accounts) ",
+                    bold=True,
+                    green=True,
+                )
 
         else:
             # Normal message for fixture generation