Skip to content
PortalNetwork

Update HistoryNetwork types to match specs #2546

Sign in to view logs

Update HistoryNetwork types to match specs

Update HistoryNetwork types to match specs #2546

Triggered via pull request April 28, 2025 17:06
@acolytec3acolytec3
synchronize #775
align-history-types
Status Failure
Total duration 1m 22s
Artifacts –

portalnetwork-build.yml

on: pull_request
Matrix: test-integration-portalnetwork
Matrix: test-unit-portalnetwork
Fit to window
Zoom out
Zoom in

Annotations

9 errors
test-unit-portalnetwork (22)
Process completed with exit code 1.
test/networks/history/historyNetwork.spec.ts > Header Tests > should validate and store a post merge header proof: packages/portalnetwork/src/networks/history/history.ts#L234
Error: Unable to validate proof for post-merge header ❯ HistoryNetwork.validateHeader src/networks/history/history.ts:234:15 ❯ HistoryNetwork.store src/networks/history/history.ts:549:22 ❯ test/networks/history/historyNetwork.spec.ts:159:19
test/networks/history/headerProof.spec.ts > it should verify a post-Capella header proof > should verify a post-capella header proof: packages/portalnetwork/test/networks/history/headerProof.spec.ts#L235
Error: Incorrect vector length 12 expected 13 ❯ assertValidArrayLength ../../node_modules/@chainsafe/ssz/src/type/arrayBasic.ts:244:13 ❯ Object.value_fromJsonArray ../../node_modules/@chainsafe/ssz/src/type/arrayBasic.ts:154:3 ❯ VectorCompositeType.fromJson ../../node_modules/@chainsafe/ssz/src/type/array.ts:120:12 ❯ ContainerType.fromJson ../../node_modules/@chainsafe/ssz/src/type/container.ts:349:36 ❯ test/networks/history/headerProof.spec.ts:235:55
test/networks/history/headerProof.spec.ts > it should verify a post-Capella header proof > should instantiate a proof from json: packages/portalnetwork/test/networks/history/headerProof.spec.ts#L225
Error: Incorrect vector length 12 expected 13 ❯ assertValidArrayLength ../../node_modules/@chainsafe/ssz/src/type/arrayBasic.ts:244:13 ❯ Object.value_fromJsonArray ../../node_modules/@chainsafe/ssz/src/type/arrayBasic.ts:154:3 ❯ VectorCompositeType.fromJson ../../node_modules/@chainsafe/ssz/src/type/array.ts:120:12 ❯ ContainerType.fromJson ../../node_modules/@chainsafe/ssz/src/type/container.ts:349:36 ❯ test/networks/history/headerProof.spec.ts:225:55
test/networks/history/headerProof.spec.ts > Bellatrix - Capella header proof tests > should verify a fluffy proof: packages/portalnetwork/test/networks/history/headerProof.spec.ts#L205
Error: Incorrect vector length 11 expected 14 ❯ assertValidArrayLength ../../node_modules/@chainsafe/ssz/src/type/arrayBasic.ts:244:13 ❯ Object.value_fromJsonArray ../../node_modules/@chainsafe/ssz/src/type/arrayBasic.ts:154:3 ❯ VectorCompositeType.fromJson ../../node_modules/@chainsafe/ssz/src/type/array.ts:120:12 ❯ ContainerType.fromJson ../../node_modules/@chainsafe/ssz/src/type/container.ts:349:36 ❯ test/networks/history/headerProof.spec.ts:205:51
test/networks/history/headerProof.spec.ts > Bellatrix - Capella header proof tests > should deserialize proof: packages/portalnetwork/test/networks/history/headerProof.spec.ts#L165
TypeError: Cannot read properties of undefined (reading 'slice') ❯ createNodeFromSingleProof ../../node_modules/@lodestar/types/node_modules/@chainsafe/persistent-merkle-tree/src/proof/single.ts:25:23 ❯ createNodeFromProof ../../node_modules/@lodestar/types/node_modules/@chainsafe/persistent-merkle-tree/src/proof/index.ts:139:14 ❯ Function.createFromProof ../../node_modules/@lodestar/types/node_modules/@chainsafe/persistent-merkle-tree/src/tree.ts:74:21 ❯ ContainerType.createFromProof ../../node_modules/@lodestar/types/node_modules/@chainsafe/ssz/src/type/composite.ts:273:36 ❯ test/networks/history/headerProof.spec.ts:165:59
Unhandled error: packages/portalnetwork/src/networks/history/history.ts#L234
Error: Unable to validate proof for post-merge header ❯ HistoryNetwork.validateHeader src/networks/history/history.ts:234:15 ❯ test/networks/history/historyNetwork.spec.ts:152:27 ❯ ../../node_modules/@vitest/runner/dist/index.js:103:11 ❯ ../../node_modules/@vitest/runner/dist/index.js:596:26 ❯ ../../node_modules/@vitest/runner/dist/index.js:879:20 ❯ runWithTimeout ../../node_modules/@vitest/runner/dist/index.js:852:10 ❯ runTest ../../node_modules/@vitest/runner/dist/index.js:1360:12 ❯ runSuite ../../node_modules/@vitest/runner/dist/index.js:1507:8 ❯ runSuite ../../node_modules/@vitest/runner/dist/index.js:1507:8 This error originated in "test/networks/history/historyNetwork.spec.ts" test file. It doesn't mean the error was thrown inside the file itself, but while it was running. The latest test that might've caused the error is "should verify a pre-merge header proof". It might mean one of the following: - The error was thrown, while Vitest was running this test. - If the error occurred after the test had been completed, this was the last documented test before it was thrown.
test-integration-portalnetwork (22)
Process completed with exit code 1.
test/integration/postCapellaHeaderProof.spec.ts > Block Bridge Data Test > should store and retrieve block header data: packages/portalnetwork/src/networks/history/history.ts#L244
Error: invalid proof for block 0x1e98ea9bdf6e44eaed730041682e7db748812d5baef84a38435c8ad5f6c5d1e2 ❯ HistoryNetwork.validateHeader src/networks/history/history.ts:244:15 ❯ HistoryNetwork.store src/networks/history/history.ts:549:22 ❯ test/integration/postCapellaHeaderProof.spec.ts:111:20