Add manifest.yaml to each test vector to make metadata self-contained and test cases easier to share and verify

Author: Sandijigs

tests/core/pyspec/eth2spec/gen_helpers/gen_base/dumper.py

@@ -70,6 +70,18 @@ def dump_ssz(self, test_case: TestCase, name: str, data: bytes) -> None:
         with path.open("wb") as f:
             f.write(compress(data))
 
+    def dump_manifest(self, test_case: TestCase) -> None:
+        """Write manifest.yml file containing test case metadata."""
+        manifest_data = {
+            'config_name': test_case.preset_name,
+            'fork_name': test_case.fork_name,
+            'runner_name': test_case.runner_name,
+            'handler_name': test_case.handler_name,
+            'suite_name': test_case.suite_name,
+            'case_name': test_case.case_name,
+        }
+        self._dump_yaml(test_case, "manifest", manifest_data, self.default_yaml)
+
     def _dump_yaml(self, test_case: TestCase, name: str, data: any, yaml_encoder: YAML) -> None:
         """Helper to write YAML files for test case."""
         path = test_case.dir / f"{name}.yaml"

tests/core/pyspec/eth2spec/gen_helpers/gen_base/gen_runner.py

@@ -109,6 +109,9 @@ def execute_test(test_case: TestCase, dumper: Dumper):
     if meta:
         dumper.dump_meta(test_case, meta)
 
+    # Always write manifest.yml for every test case
+    dumper.dump_manifest(test_case)
+
 
 def run_generator(input_test_cases: Iterable[TestCase], args=None):
     start_time = time.time()

tests/formats/README.md

@@ -22,6 +22,7 @@ testing.
     - [Common output formats](#common-output-formats)
     - [Special output parts](#special-output-parts)
       - [`meta.yaml`](#metayaml)
+      - [`manifest.yaml`](#manifestyaml)
       - [`config.yaml`](#configyaml)
 - [Config sourcing](#config-sourcing)
 - [Note for implementers](#note-for-implementers)
@@ -213,6 +214,28 @@ bls_setting: int     -- optional, can have 3 different values:
                             2: known as "BLS ignored"  - if the test validity is strictly dependent on BLS being OFF
 ```
 
+##### `manifest.yaml`
+
+Included in every test case. Contains metadata that identifies the test vector:
+
+```yaml
+config_name: minimal      # Configuration preset (mainnet, minimal, general)
+fork_name: phase0         # Fork/phase name
+runner_name: bls          # Test runner category
+handler_name: eth_aggregate_pubkeys  # Specific handler
+suite_name: bls           # Test suite name
+case_name: eth_aggregate_pubkeys_valid_0  # Individual test case name
+```
+
+This metadata duplicates what is already encoded in the directory path:
+
+```
+tests/<config_name>/<fork_name>/<runner_name>/<handler_name>/<suite_name>/<case_name>/
+```
+
+Having it in a file means test vectors can be moved or distributed without
+losing context.
+
 ##### `config.yaml`
 
 The runtime-configurables may be different for specific tests. When present,
@@ -252,7 +275,7 @@ The basic pattern for test-suite loading and running is:
    `operations > deposits`). Again, repeat for each if running all.
 4. Select a test suite. Or repeat for each.
 5. Select a test case. Or repeat for each.
-6. Load the parts of the case. And `meta.yaml` if present.
+6. Load the parts of the case. `manifest.yaml` and `meta.yaml` if present.
 7. Run the test, as defined by the test format.
 
 Step 1 may be a step with compile time selection of a configuration, if desired