Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: account signature tests #24

Merged
merged 15 commits into from
Mar 14, 2024
Merged

feat: account signature tests #24

merged 15 commits into from
Mar 14, 2024

Conversation

VGabriel45
Copy link
Collaborator

@VGabriel45 VGabriel45 commented Mar 14, 2024

Added tests for 6492 and 712 signatures for deployed and undeployed smart accounts.


PR-Codex overview

This PR updates test file paths to ep6 directory and fixes import paths in test files related to paymaster, bundler, ecdsaOwnershipModule, account, and account_signature.

Detailed summary

  • Updated test file paths to ep6 directory
  • Fixed import paths in test files for paymaster, bundler, ecdsaOwnershipModule, account, and account_signature

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@VGabriel45 VGabriel45 requested a review from joepegler March 14, 2024 09:06
Copy link

Coverage Report

Status Category Percentage Covered / Total
🟢 Lines 93.14% (🎯 80%) 1305 / 1401
🟢 Statements 93.14% (🎯 80%) 1305 / 1401
🟢 Functions 81.39% (🎯 70%) 35 / 43
🟢 Branches 69.01% (🎯 60%) 49 / 71
File CoverageNo changed files found.
Generated in workflow #103

Copy link

size-limit report 📦

Path Size
core (esm) 8.26 KB (0%)
core (cjs) 19.97 KB (0%)
smartAccount (tree-shaking) 3.19 KB (0%)
bundler (tree-shaking) 110 B (0%)
paymaster (tree-shaking) 92 B (0%)

@joepegler joepegler merged commit d26a37a into main Mar 14, 2024
5 checks passed
@joepegler joepegler deleted the feat/account_signature_tests branch March 14, 2024 09:50
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.

3 participants