Skip to content

Add upgrade migration state compatibility tests#674

Open
dekaptain2 wants to merge 1 commit into
Pulsefy:mainfrom
dekaptain2:feature/573-upgrade-migration-test-harness
Open

Add upgrade migration state compatibility tests#674
dekaptain2 wants to merge 1 commit into
Pulsefy:mainfrom
dekaptain2:feature/573-upgrade-migration-test-harness

Conversation

@dekaptain2

Copy link
Copy Markdown

PR: Upgrade/Migration Test Harness

Summary

This PR adds regression coverage for Aid Escrow contract upgrade and migration behavior.

What changed

  • Added migration progression tests covering a real version path from v1 to v2 to v3.
  • Added state compatibility assertions to ensure migrations preserve core contract state:
    • contract version
    • admin identity
    • config values
    • pause state
  • Added invalid-transition coverage so repeated or downgraded versions fail with a clear InvalidState error.

Why

These tests make upgrade and migration rules explicit and help prevent regressions where state compatibility breaks across contract versions.

Testing

  • Added/updated Rust contract tests in the Aid Escrow versioning suite.
  • Verification was attempted locally, but the current Windows Rust toolchain environment blocked cargo execution with a local toolchain error.

Notes

  • Branch: feature/573-upgrade-migration-test-harness
  • Commit: c3ca6c5

Closes #573

@vercel

vercel Bot commented Jun 28, 2026

Copy link
Copy Markdown

@AbilityTechy is attempting to deploy a commit to the Cedarich's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Jun 28, 2026

Copy link
Copy Markdown

@dekaptain2 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

@Cedarich

Copy link
Copy Markdown
Contributor

@dekaptain2 fix workflow

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.

Upgrade/Migration Test Harness

3 participants