Skip to content

chore: Test ci third#25668

Closed
Grigorov-Georgi wants to merge 23 commits into
hiero-ledger:mainfrom
Grigorov-Georgi:test-ci-third
Closed

chore: Test ci third#25668
Grigorov-Georgi wants to merge 23 commits into
hiero-ledger:mainfrom
Grigorov-Georgi:test-ci-third

Conversation

@Grigorov-Georgi
Copy link
Copy Markdown
Contributor

Description:

Related issue(s):

Fixes #

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

petreze and others added 23 commits May 27, 2026 22:40
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
BLOCKS-mode embedded node can't sustain 10 concurrent contract
deployments after heavy preceding test classes (PrecompileSCTest).
Receipt polling times out with UNKNOWN on FileCreate/CryptoCreate.

Signed-off-by: aderevets <artem.derevets@limechain.tech>
In BLOCKS mode the legacy BlockInfo singleton is not maintained, so
ContractCallLocal queries read block number 0 from BlockRecordInfoImpl,
making blockhash(block.number - 1) underflow and revert. Mirror the
handle path (ParentTxnFactory) by deriving query block info from
BlockStreamInfo state. Read from the immutable snapshot, not the live
BlockStreamManager, whose fields are mutated off the query thread.

Signed-off-by: aderevets <artem.derevets@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: aderevets <artem.derevets@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Petar Tonev <petar.tonev@limechain.tech>
Signed-off-by: Georgi Grigorov <georgi.grigorov@limechain.tech>
@Grigorov-Georgi Grigorov-Georgi self-assigned this Jun 1, 2026
@trunk-io
Copy link
Copy Markdown

trunk-io Bot commented Jun 1, 2026

Merging to main in this repository is managed by Trunk.

  • To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

@lfdt-bot
Copy link
Copy Markdown

lfdt-bot commented Jun 1, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 1, 2026

Codecov Report

❌ Patch coverage is 43.47826% with 26 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...era/node/app/records/impl/BlockStreamInfoImpl.java 0.00% 14 Missing ⚠️
...edera/node/app/blocks/impl/BlockStreamBuilder.java 0.00% 5 Missing ⚠️
...era/node/app/workflows/query/QueryContextImpl.java 0.00% 4 Missing ⚠️
...hedera/node/app/workflows/handle/HandleOutput.java 33.33% 1 Missing and 1 partial ⚠️
.../node/app/state/recordcache/BlockRecordSource.java 92.30% 0 Missing and 1 partial ⚠️

Impacted file tree graph

@@             Coverage Diff              @@
##               main   #25668      +/-   ##
============================================
- Coverage     75.11%   74.84%   -0.27%     
- Complexity    11548    11552       +4     
============================================
  Files          2585     2586       +1     
  Lines        101419   101455      +36     
  Branches      11241    11244       +3     
============================================
- Hits          76179    75937     -242     
- Misses        21335    21616     +281     
+ Partials       3905     3902       -3     
Files with missing lines Coverage Δ Complexity Δ
...app/workflows/standalone/TransactionExecutors.java 85.82% <100.00%> (+0.34%) 0.00 <0.00> (ø)
...com/hedera/node/config/data/BlockStreamConfig.java 100.00% <ø> (ø) 0.00 <0.00> (ø)
.../node/app/state/recordcache/BlockRecordSource.java 98.11% <92.30%> (-1.89%) 0.00 <0.00> (ø)
...hedera/node/app/workflows/handle/HandleOutput.java 20.45% <33.33%> (+0.94%) 0.00 <0.00> (ø)
...era/node/app/workflows/query/QueryContextImpl.java 70.96% <0.00%> (-7.61%) 0.00 <0.00> (ø)
...edera/node/app/blocks/impl/BlockStreamBuilder.java 80.07% <0.00%> (+2.95%) 0.00 <0.00> (ø)
...era/node/app/records/impl/BlockStreamInfoImpl.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)

... and 26 files with indirect coverage changes

Impacted file tree graph

🚀 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.

@testlens-app
Copy link
Copy Markdown

testlens-app Bot commented Jun 1, 2026

✅ All tests passed ✅

🏷️ Commit: c96b813
▶️ Tests: 7068 executed
⚪️ Checks: 32/32 completed


Learn more about TestLens at testlens.app.

@Grigorov-Georgi Grigorov-Georgi deleted the test-ci-third branch June 1, 2026 15:15
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.

4 participants