Skip to content

test(admin): execution succeeds after quorum (#668) - #780

Open
funds0033-cmyk wants to merge 1 commit into
BCPathway:mainfrom
funds0033-cmyk:test/668-execution-succeeds-after-quorum
Open

test(admin): execution succeeds after quorum (#668)#780
funds0033-cmyk wants to merge 1 commit into
BCPathway:mainfrom
funds0033-cmyk:test/668-execution-succeeds-after-quorum

Conversation

@funds0033-cmyk

Copy link
Copy Markdown
Contributor

Add execute_upgrade for multi-sig gated WASM upgrades and unit tests covering the happy path after quorum plus expected error states.

Closes #668

What does this PR do?

Type of change

  • Bug fix
  • Feature
  • Refactor
  • Docs
  • CI
  • Chore

Checklist

  • I ran pre-commit run --all-files locally and it passed
  • All CI jobs pass on this branch
  • I added or updated tests for new behaviour
  • I updated relevant docs / comments
  • No secrets or credentials are included
  • No breaking changes to public APIs (or I've documented them)

Breaking changes?

Related issues

Closes #668

Add execute_upgrade for multi-sig gated WASM upgrades and unit tests
covering the happy path after quorum plus expected error states.

Closes BCPathway#668

Co-authored-by: Cursor <cursoragent@cursor.com>
@drips-wave

drips-wave Bot commented Aug 25, 2026

Copy link
Copy Markdown

@funds0033-cmyk 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

@p3ris0n

p3ris0n commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

@funds0033-cmyk Conflicts

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.

[Testing] Test: Execution succeeds after quorum

2 participants