test: improve ssz tests consistency (#6776)

* test: improve ssz tests consistency * chore: address comments
ChainSafe · May 16, 2024 · 68ded3d · 68ded3d
1 parent 44d9599
commit 68ded3d
Showing 1 changed file with 18 additions and 15 deletions.
diff --git a/packages/beacon-node/test/spec/presets/ssz_static.test.ts b/packages/beacon-node/test/spec/presets/ssz_static.test.ts
@@ -1,6 +1,6 @@
 import fs from "node:fs";
 import path from "node:path";
-import {it, vi} from "vitest";
+import {expect, it, vi} from "vitest";
 import {Type} from "@chainsafe/ssz";
 import {ssz} from "@lodestar/types";
 import {ACTIVE_PRESET, ForkName, ForkLightClient, ForkExecution} from "@lodestar/params";
@@ -56,23 +56,26 @@ const sszStatic =
       (ssz.bellatrix as Types)[typeName] ||
       (ssz.altair as Types)[typeName] ||
       (ssz.phase0 as Types)[typeName];
-    if (!sszType) {
-      throw Error(`No type for ${typeName}`);
-    }
 
-    const sszTypeNoUint = replaceUintTypeWithUintBigintType(sszType);
+    if (!sszType) {
+      expect.fail(
+        `Missing SSZ type definition for ${typeName}; this will prevent associated ssz_static tests to be executed`
+      );
+    } else {
+      const sszTypeNoUint = replaceUintTypeWithUintBigintType(sszType);
 
-    for (const testCase of fs.readdirSync(testSuiteDirpath)) {
-      // Do not manually skip tests here, do it in packages/beacon-node/test/spec/presets/index.test.ts
-      it(testCase, function () {
-        // Mainnet must deal with big full states and hash each one multiple times
-        if (ACTIVE_PRESET === "mainnet") {
-          vi.setConfig({testTimeout: 30 * 1000});
-        }
+      for (const testCase of fs.readdirSync(testSuiteDirpath)) {
+        // Do not manually skip tests here, do it in packages/beacon-node/test/spec/presets/index.test.ts
+        it(testCase, function () {
+          // Mainnet must deal with big full states and hash each one multiple times
+          if (ACTIVE_PRESET === "mainnet") {
+            vi.setConfig({testTimeout: 30 * 1000});
+          }
 
-        const testData = parseSszStaticTestcase(path.join(testSuiteDirpath, testCase));
-        runValidSszTest(sszTypeNoUint, testData);
-      });
+          const testData = parseSszStaticTestcase(path.join(testSuiteDirpath, testCase));
+          runValidSszTest(sszTypeNoUint, testData);
+        });
+      }
     }
   };