Skip to content

test: cover targeted auth for milestone and escrow entrypoints - #755

Merged
chonilius merged 1 commit into
MergeFi:mainfrom
Mikey-222:test/targeted-auth-release-issue-contribute-fund-create-milestone
Oct 20, 2026
Merged

chonilius merged 1 commit into
MergeFi:mainfrom
Mikey-222:test/targeted-auth-release-issue-contribute-fund-create-milestone

Conversation

@Mikey-222

Copy link
Copy Markdown
Contributor

Add positive and negative mock_auths coverage for milestones create_milestone and release_issue, and escrow fund and contribute. The negative tests assert the host Auth/InvalidAction error and that no state changed, so removing or misdirecting a require_auth call now fails the suite.

Summary

Related issues

Changes

Test plan

  • Existing tests pass (make test)
  • New tests added for changed behaviour
  • Manually verified against a local node / testnet where applicable

Checklist

  • Branch is up to date with main
  • No debug / dead code left in
  • cargo fmt and cargo clippy are clean

Add positive and negative mock_auths coverage for milestones create_milestone and release_issue, and escrow fund and contribute. The negative tests assert the host Auth/InvalidAction error and that no state changed, so removing or misdirecting a require_auth call now fails the suite.
@Mikey-222
Mikey-222 requested a review from chonilius as a code owner October 20, 2026 12:45
@drips-wave

drips-wave Bot commented Oct 20, 2026

Copy link
Copy Markdown

@Mikey-222 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@vercel

vercel Bot commented Oct 20, 2026

Copy link
Copy Markdown

@Mikey-222 is attempting to deploy a commit to the chonilius' projects Team on Vercel.

A member of the Team first needs to authorize it.

@chonilius
chonilius merged commit 686a2be into MergeFi:main Oct 20, 2026
0 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

2 participants