Skip to content

Conversation

@YoshihitoAso
Copy link
Member

@YoshihitoAso YoshihitoAso commented Jan 22, 2026

📌 Description

This pull request introduces automated testing infrastructure.
And also, refines terminology and logging for clarity in EIP-7702 example scripts.

✅ Related Issues

None

🔄 Changes

Testing Infrastructure

  • Added a pytest test suite with fixtures for contract deployments, accounts, and web3 setup in tests/conftest.py.
  • Introduced a GitHub Actions workflow (.github/workflows/pytest.yml) to automatically run tests on pull requests, including environment setup and log artifact upload on failure.
  • Updated pyproject.toml to include pytest as a development dependency.
  • Added a make test command to the Makefile for running the test suite.

📌 Checklist

  • I have added tests where necessary.
  • I have updated the documentation where necessary.

@YoshihitoAso YoshihitoAso marked this pull request as ready for review January 22, 2026 12:33
@YoshihitoAso YoshihitoAso requested a review from Copilot January 22, 2026 12:33
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a pytest-based testing setup for the EIP-7702 example flow (including positive and negative-path tests), and introduces CI automation to run the suite on pull requests.

Changes:

  • Added pytest tests + fixtures for connecting to the local Hardhat chain and interacting with deployed contracts.
  • Added a GitHub Actions workflow to run make test on PRs.
  • Added pytest to Python dev dependencies and exposed make test.

Reviewed changes

Copilot reviewed 9 out of 11 changed files in this pull request and generated no comments.

Show a summary per file
File Description
uv.lock Adds pytest (and transitive deps) to the resolved dev environment.
pyproject.toml Adds pytest to the dev dependency group.
tests/conftest.py Adds session fixtures for RPC/web3 setup and contract ABI/address wiring.
tests/test_eip7702_tx.py Adds initial EIP-7702 “delegation then calldata execution” test coverage.
.github/workflows/pytest.yml Adds CI job to install deps, start Hardhat, and run pytest.
Makefile Adds make test to run pytest (after deploy).
package.json Changes Hardhat + plugin versions (major downgrade) and updates ethers version.
package-lock.json Locks the new Node dependency graph after the Hardhat/tooling change.
example/eip7702_one_tx.py Logging/terminology refinement around delegation and eth_getCode output.
example/eip7702_separate_tx.py Logging/terminology refinement around delegation and eth_getCode output.
example/RESULTS.md Updates example output to match revised logging text.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@YoshihitoAso YoshihitoAso merged commit eb69fd8 into main Jan 22, 2026
9 checks passed
@YoshihitoAso YoshihitoAso deleted the add-unit-test branch January 22, 2026 12:39
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.

1 participant