Skip to content

test(evm): add storage createGroup regression tests on native-action base#347

Merged
phenix3443 merged 1 commit into
precompile-integration-v2from
precompile/storage-regression-on-332
Jul 14, 2026
Merged

test(evm): add storage createGroup regression tests on native-action base#347
phenix3443 merged 1 commit into
precompile-integration-v2from
precompile/storage-regression-on-332

Conversation

@phenix3443

Copy link
Copy Markdown
Contributor

What

Second test slice on top of #332 (base precompile-integration-v2). Adds x/evm/precompiles/storage/tx_evm_apply_test.go — createGroup characterization:

Uses createGroup (funded-creator only) rather than the heavy createBucket fixture; registers the control-hub account (0x…dead) for the internal group-NFT mint.

Test-only; go test ./x/evm/precompiles/storage -count=1 → ok; golangci-lint → 0 issues.

…base

Add storage characterization tests on top of #332, using createGroup as the
representative transactional write:
- TestCreateGroup_EVMDispatchSuccess: real static-precompile dispatch, group
  owned by the caller
- TestCreateGroup_RejectsContractForwarding: pins the EOA-only guard #332
  deliberately keeps
- TestCreateGroup_FailureDoesNotMutateState: duplicate createGroup reverts
  cleanly (native revert), existing group untouched

No production changes.
@phenix3443 phenix3443 merged commit fc4c029 into precompile-integration-v2 Jul 14, 2026
3 of 10 checks passed
@phenix3443 phenix3443 deleted the precompile/storage-regression-on-332 branch July 14, 2026 06:51
@github-actions github-actions Bot added the tests label Jul 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant