Skip to content

feat: fix debug_getRawBlock acceptance test#4981

Open
natanasow wants to merge 2 commits intomainfrom
4890-fix-getrawblock-acceptancetest
Open

feat: fix debug_getRawBlock acceptance test#4981
natanasow wants to merge 2 commits intomainfrom
4890-fix-getrawblock-acceptancetest

Conversation

@natanasow
Copy link
Member

Description

Fix the assertion on failing acceptance test on debug_getRawBlock.

Related issue(s)

Fixes #4890

Testing Guide

Changes from original design (optional)

N/A

Additional work needed (optional)

N/A

Checklist

  • I've assigned an assignee to this PR and related issue(s) (if applicable)
  • I've assigned a label to this PR and related issue(s) (if applicable)
  • I've assigned a milestone to this PR and related issue(s) (if applicable)
  • I've updated documentation (code comments, README, etc. if applicable)
  • I've done sufficient testing (unit, integration, etc.)

Signed-off-by: nikolay <n.atanasow94@gmail.com>
@natanasow natanasow added this to the 0.75.0 milestone Feb 24, 2026
@natanasow natanasow self-assigned this Feb 24, 2026
@natanasow natanasow requested review from a team as code owners February 24, 2026 14:58
@natanasow natanasow added the enhancement New feature or request label Feb 24, 2026
Signed-off-by: nikolay <n.atanasow94@gmail.com>
@github-actions
Copy link

github-actions bot commented Feb 24, 2026

Test Results

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit 85d2fd9. ± Comparison against base commit eb0e545.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@jasuwienas jasuwienas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it should fix debug accceptance tests, thank you!

@natanasow natanasow requested a review from jasuwienas February 24, 2026 15:15
@codecov
Copy link

codecov bot commented Feb 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@            Coverage Diff             @@
##             main    #4981      +/-   ##
==========================================
+ Coverage   95.97%   96.02%   +0.05%     
==========================================
  Files         143      143              
  Lines       24063    24064       +1     
  Branches     1910     1912       +2     
==========================================
+ Hits        23094    23108      +14     
+ Misses        943      932      -11     
+ Partials       26       24       -2     
Flag Coverage Δ
config-service 98.63% <ø> (+<0.01%) ⬆️
relay 92.42% <ø> (+0.04%) ⬆️
server 88.39% <ø> (ø)
ws-server 97.90% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Contributor

@BartoszSolkaBD BartoszSolkaBD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey, just one question

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add debug_getRawBlock method

3 participants