hiero-ledger / hiero-ledger/hiero-consensus-node

Refactor VirtualMapStateTest test setup methods

Open Beginner friendly
#21,750 2 comments 0 reactions 0 assignees View on GitHub
Platform Tech Debt Reduced Test Development
Dominant language
Java
Stars
407
Forks
226
Avg merge
3d 4h
Merged PRs (30d)
210

Description

**Description:**
Multiple test methods in `VirtualMapStateTest` contain the comment `// FUTURE WORK: refactor after MerkleStateRootTest will be removed`. It was removed in https://github.com/hiero-ledger/hiero-consensus-node/pull/20699 so now it is possible to do.

Contributor guide

Open the contributing guide

Research direction

Start by locating the VirtualMapStateTest methods containing the `FUTURE WORK` comment and review the MerkleStateRootTest removal in PR #20699. Refactor those setup methods using the now-removed dependency, then run VirtualMapStateTest and confirm its tests still pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
testing-qa
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
68/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.