Skip to content

fix(prover): update rlp_blocks.bin path in shnarf_calculator tests#3129

Merged
gusiri merged 1 commit into
mainfrom
prover/fix-blockrlpbin-path
May 18, 2026
Merged

fix(prover): update rlp_blocks.bin path in shnarf_calculator tests#3129
gusiri merged 1 commit into
mainfrom
prover/fix-blockrlpbin-path

Conversation

@gusiri
Copy link
Copy Markdown
Contributor

@gusiri gusiri commented May 18, 2026

The test data file moved from jvm-libs/blob-compressor/ to coordinator/core/src/test/resources/.

This PR implements issue(s) #

Checklist

  • I wrote new tests for my new core changes.
  • I have successfully ran tests, style checker and build against my new changes locally.
  • If this change is deployed to any environment (including Devnet), E2E test coverage exists or is included in this
    PR.
  • I have informed the team of any breaking changes if there are any.

Note

Low Risk
Low risk: only updates a test fixture file path to match a repository move; no production logic changes.

Overview
Updates shnarf_calculator_test.go to load rlp_blocks.bin from its new location under coordinator/core/src/test/resources/... instead of the old jvm-libs/blob-compressor/... path, keeping the shnarf calculator tests working after the fixture move.

Reviewed by Cursor Bugbot for commit 9e3a42f. Bugbot is set up for automated code reviews on this repo. Configure here.

The test data file moved from jvm-libs/blob-compressor/ to
coordinator/core/src/test/resources/.
@github-actions
Copy link
Copy Markdown
Contributor

tx-exclusion-api Changelog Preview (informational)

[Unreleased] diff (commits touching tx-exclusion-api/** since latest releases/tx-exclusion-api/v* tag)

[unreleased]

⚙️ Miscellaneous Tasks

Generated by git-cliff-action using cliff.toml. This comment is informational and does not gate the PR.

@github-actions
Copy link
Copy Markdown
Contributor

postman Changelog Preview (informational)

[Unreleased] diff (commits touching postman/** since latest releases/postman/v* tag)

[unreleased]

Generated by git-cliff-action using cliff.toml. This comment is informational and does not gate the PR.

@github-actions
Copy link
Copy Markdown
Contributor

prover Changelog Preview (informational)

[Unreleased] diff (commits touching prover/** since latest releases/prover/v* tag)

[unreleased]

🐛 Bug Fixes

  • (prover) Update rlp_blocks.bin path in shnarf_calculator tests

Generated by git-cliff-action using cliff.toml. This comment is informational and does not gate the PR.

@github-actions
Copy link
Copy Markdown
Contributor

coordinator Changelog Preview (informational)

[Unreleased] diff (commits touching coordinator/** since latest releases/coordinator/v* tag)

[unreleased]

Generated by git-cliff-action using cliff.toml. This comment is informational and does not gate the PR.

@github-actions
Copy link
Copy Markdown
Contributor

linea-besu Changelog Preview (informational)

[Unreleased] diff (commits touching linea-besu/** since latest releases/linea-besu/v* tag)

[unreleased]

Generated by git-cliff-action using cliff.toml. This comment is informational and does not gate the PR.

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.23%. Comparing base (f1a0b28) to head (9e3a42f).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #3129   +/-   ##
=========================================
  Coverage     77.23%   77.23%           
  Complexity     7005     7005           
=========================================
  Files          1118     1118           
  Lines         44394    44394           
  Branches       5343     5343           
=========================================
  Hits          34286    34286           
  Misses         8749     8749           
  Partials       1359     1359           
Flag Coverage Δ *Carryforward flag
hardhat 96.17% <ø> (ø)
kotlin 55.84% <ø> (ø) Carriedforward from f1a0b28
lido-governance-monitor 97.61% <ø> (ø) Carriedforward from f1a0b28
linea-native-libs 90.69% <ø> (ø) Carriedforward from f1a0b28
linea-shared-utils 96.18% <ø> (ø) Carriedforward from f1a0b28
native-yield-automation-service 97.68% <ø> (ø) Carriedforward from f1a0b28
postman 99.92% <ø> (ø) Carriedforward from f1a0b28
sdk-core 98.09% <ø> (ø) Carriedforward from f1a0b28
sdk-ethers 89.83% <ø> (ø) Carriedforward from f1a0b28
sdk-viem 99.45% <ø> (ø) Carriedforward from f1a0b28
tracer 88.55% <ø> (ø) Carriedforward from f1a0b28

*This pull request uses carry forward flags. Click here to find out more.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants