Skip to content

Pectra Hiero tests#71

Merged
0x-ds merged 126 commits intomainfrom
denys/run-pectra-tests-with-hh
Mar 23, 2026
Merged

Pectra Hiero tests#71
0x-ds merged 126 commits intomainfrom
denys/run-pectra-tests-with-hh

Conversation

@0x-ds
Copy link
Copy Markdown
Contributor

@0x-ds 0x-ds commented Feb 20, 2026 •
edited
Loading

Description:

This branch expands HIP-1340/EIP-7702 functional coverage, adds Hiero-specific delegation tests, and improves CI runtime by sharding solo-network test execution. It also refactors shared test utilities and updates contract references/helpers to align with current Hedera behavior.

Key Changes:
Added comprehensive HIP-1340 test coverage:

  • New hiero.test.js suite for Hiero-specific delegation scenarios.
  • Added new HIP-1340 test utilities:
    • test/hip-1340/utils/hts.js
    • test/hip-1340/utils/setup.js
    • Updated test/hip-1340/utils/web3.js.
    • Added evm-functional-testing/contracts/hip-1340/HasFacadeSelectors.sol to mimic behavior of HAS facade
      CI optimization in .github/workflows/evm-functional-testing.yaml:
  • Sharded evm-functional-regression into parallel solo-network buckets (HIP-1215, HIP-632, HTS transfer events, HIP-1340 baseline/hiero/eip7702).
  • Added matrix-driven test selection and more reliable cleanup (if: always() for cluster teardown).

Related issue(s):

Fixes #

Checklist

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

@acuarica
Run pectra tests with HH config instead of separate Geth
c80cc0d 
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
@acuarica
Run pectra test in CI
4c8f708 
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
@acuarica
Include Relay build step
067845b 
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
@acuarica
Remove .only to allow tests to run in CI
a5fbc45 
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
@acuarica
Increase account funding amount to match test's
0cc6a6f 
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
@acuarica
Run only delegation setup tests
5511aa9 
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
@acuarica
Enable more test cases
10e92da 
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
@acuarica
Move env defs to a closer scope in workflow
0f293ce 
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
@acuarica
Fix workflow after refactor
c092aca 
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
@acuarica
Enable delegationAddress query using custom SDK
f9fb648 
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
@acuarica
Add delegated execution test
d1d425e 
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
@acuarica
Improve log message
3b0a912 
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
@acuarica
Add receipt logging
0695c25 
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
@acuarica
Add more cases for delegated execution
4596769 
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
@acuarica
Remove .only
57792a0 
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
@acuarica
Reenable more tests
5b5eda5 
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
@acuarica
Add more cases
2f4a900 
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
@acuarica
Add more cases
b4de1bc 
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
@acuarica
Replace try/catch with catch
4f6f286 
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
@0x-ds 0x-ds force-pushed the denys/run-pectra-tests-with-hh branch 2 times, most recently from 7d43d86 to 13c68e4 Compare February 20, 2026 00:56
0x-ds added 9 commits February 19, 2026 17:09
@0x-ds
working version of 'should return delegation designation to 0x167 whe…
63af181 
…n an HTS token is created' test

Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
working version of should transfer HTS tokens when two EOAs delegate …
17e4f59 
…to the same Smart Wallet and send self-sponsored transactions

Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
remove duped code
cebe8bf 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
refactor: move literals to constants
9c23899 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
refactor: move Nonce class out of the test
81fd5c9 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
refactor: extract helper functions
7543866 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
refactor: remove loop (that ends up in more code)
e036165 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
final clean up
65ed5cc 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
remove redundant logs
1098d26 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
0x-ds added 6 commits March 11, 2026 15:31
@0x-ds
remove hardcoded nonce
1b1df9a 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
update solo version
7be1b2a 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
fix seed fudning for hardhat based network
3f20d36 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
address comments - part 2
035629b 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
remove some redunant test cases
51d681a 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
manual merge
d073465 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
gkozyryatskyy
gkozyryatskyy previously approved these changes Mar 12, 2026
View reviewed changes
Comment thread .github/workflows/evm-functional-testing.yaml
Comment thread evm-functional-testing/test/hip-1340/utils/hts.js
0x-ds added 12 commits March 12, 2026 09:09
@0x-ds
post manual merge changes
a76282e 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
fix actions warning
d364af4 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
add image pull retries
556d17e 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
add image workaround
88918a8 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
pull latest image instead
ab88dce 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
use small runner for geth
4527c07 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
donwgrade geth image to 1.17.0
67ba88d 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
another workaround
a41795f 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
Merge branch 'run-pectra-tests-with-hh' of github.com:hashgraph/heder…
079e5ff 
…a-evm-testing into denys/run-pectra-tests-with-hh
@0x-ds
finish after merge refactoring
78e6286 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
return back the standard job, but make it umbrella for others
31d1961 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
@0x-ds
Merge branch 'run-pectra-tests-with-hh' into denys/run-pectra-tests-w…
b6d10c6 
…ith-hh
Base automatically changed from run-pectra-tests-with-hh to main March 12, 2026 22:10
@0x-ds 0x-ds dismissed gkozyryatskyy’s stale review March 12, 2026 22:10

The base branch was changed.

@0x-ds
manual merge
25f43c6 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
lukasz-hashgraph
lukasz-hashgraph previously approved these changes Mar 19, 2026
View reviewed changes
Copy link
Copy Markdown

@lukasz-hashgraph lukasz-hashgraph left a comment

Choose a reason for hiding this comment

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

Just a minor comments. Looks good, thanks!

Comment thread evm-functional-testing/test/hip-1340/utils/web3.js Outdated
Comment thread evm-functional-testing/test/hip-1340/utils/web3.js
0x-ds added 2 commits March 19, 2026 09:22
@0x-ds
Merge branch 'main' of github.com:hashgraph/hedera-evm-testing into d…
6bbd7a6 
…enys/run-pectra-tests-with-hh
@0x-ds
address review comments
8d6cf8d 
Signed-off-by: Denys Sinyakov <denys.sinyakov@swirldslabs.com>
gkozyryatskyy
gkozyryatskyy approved these changes Mar 19, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@gkozyryatskyy gkozyryatskyy left a comment

Choose a reason for hiding this comment

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

Reapprove. Thank you @0x-ds !

lukasz-hashgraph
lukasz-hashgraph approved these changes Mar 23, 2026
View reviewed changes
Copy link
Copy Markdown

@lukasz-hashgraph lukasz-hashgraph left a comment

Choose a reason for hiding this comment

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

Thanks Denys, LGTM!

@0x-ds 0x-ds merged commit c307f7b into main Mar 23, 2026
29 of 31 checks passed
@0x-ds 0x-ds deleted the denys/run-pectra-tests-with-hh branch March 23, 2026 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@stoyanov-st stoyanov-st stoyanov-st left review comments

@gkozyryatskyy gkozyryatskyy gkozyryatskyy approved these changes

@lukasz-hashgraph lukasz-hashgraph lukasz-hashgraph approved these changes

@rbarker-dev rbarker-dev Awaiting requested review from rbarker-dev rbarker-dev is a code owner automatically assigned from hashgraph/platform-ci

+1 more reviewer

@acuarica acuarica acuarica left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@0x-ds @acuarica @gkozyryatskyy @stoyanov-st @lukasz-hashgraph