Skip to content

Comments

feat(dio): constructTest#97

Merged
wjmelements merged 9 commits intomasterfrom
constructTest
Feb 23, 2026
Merged

feat(dio): constructTest#97
wjmelements merged 9 commits intomasterfrom
constructTest

Conversation

@wjmelements
Copy link
Owner

Closes #80

Changes

  • Add constructTest entry field to dio JSON format for testing constructor execution (gas, deployed code, logs, status)
  • Extract shared helpers printEntryHeader, reportResult, jsonScanTestEntry to eliminate duplication
  • Document in README and docs/manual.md

wjmelements and others added 2 commits February 23, 2026 01:55
Adds a "constructTest" entry-level field that tests constructor
execution — gasUsed, deployed code (output), logs, and status —
with full support for the -u -w auto-update workflow.

Also extracts shared helpers printEntryHeader, reportResult, and
jsonScanTestEntry to eliminate duplication between tests and
constructTest parsing/reporting.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@wjmelements wjmelements added the enhancement New feature or request label Feb 23, 2026
@wjmelements
Copy link
Owner Author

I get a segfault when -uw that doesn't happen with -w so I think there is a bug in the gasUsed updating logic.

@wjmelements wjmelements changed the title Dio: constructTest feat(dio): constructTest Feb 23, 2026
@wjmelements wjmelements merged commit 96e52fa into master Feb 23, 2026
2 checks passed
wjmelements added a commit to wjmelements/erc8167 that referenced this pull request Feb 23, 2026
wjmelements added a commit to wjmelements/erc8167 that referenced this pull request Feb 23, 2026
* initial gasUsed

* add failing test for missing log

* add missing log3 and update gasUsed

* save 325 gas with delegatecall

* update evm for constructTest support (wjmelements/evm#97)

* testConstructorEvents

* dio test for configure and implementation, add dio testing to `make test`
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.

Dio: initcode as test case

1 participant