Skip to content

feat: complete test suite for GrantStreamEscrow#13

Merged
teethaking merged 1 commit into
TeethakingBuilds:mainfrom
LELOUCH-Vii:LELOUCH-Vii-patch-1
Jun 5, 2026
Merged

feat: complete test suite for GrantStreamEscrow#13
teethaking merged 1 commit into
TeethakingBuilds:mainfrom
LELOUCH-Vii:LELOUCH-Vii-patch-1

Conversation

@LELOUCH-Vii

@LELOUCH-Vii LELOUCH-Vii commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Closes #13

Test coverage

  • Deployment – USDC address storage, initial state, zero-address revert
  • Grant Creation – field storage, ID increment, milestone init, event emission, input validation
  • Grant Funding – funded flag, USDC transfer, event, access control, double-fund prevention
  • Milestone Submission – happy path, resubmission after rejection, access control, all revert cases
  • Verifier Approval – fund release, event emission, paidAmount tracking, sequential approvals, access control
  • Verifier Rejection – status update, no fund leak, reject→resubmit→approve flow, access control
  • View Functions – getMilestone, getMilestoneCount, grants mapping
  • Multi-Grant Isolation – separate grants don't interfere with each other's state or funds
  • Security – attacker vectors, self-approval prevention, funds locked until approval

#Testing
All 70 tests pass with npx hardhat test.

@vercel

vercel Bot commented Jun 5, 2026

Copy link
Copy Markdown

@LELOUCH-Vii is attempting to deploy a commit to the tee's projects Team on Vercel.

A member of the Team first needs to authorize it.

@vercel

vercel Bot commented Jun 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
grant-stream Ready Ready Preview, Comment Jun 5, 2026 10:22am

@teethaking teethaking merged commit 1c9c274 into TeethakingBuilds:main Jun 5, 2026
2 of 3 checks passed
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.

2 participants