Skip to content

test: verify expired allowance rejection - #361

Merged
DeFiVC merged 2 commits into
ChainLearnOfficial:mainfrom
graceful-tech12:fix/183-expired-allowance-regression
Aug 29, 2026
Merged

test: verify expired allowance rejection#361
DeFiVC merged 2 commits into
ChainLearnOfficial:mainfrom
graceful-tech12:fix/183-expired-allowance-regression

Conversation

@graceful-tech12

Copy link
Copy Markdown
Contributor

Summary

  • verify expired allowances are reported as zero before transfer rejection
  • preserve the expected insufficient allowance panic assertion

Closes #183

Verification

  • git diff --check passed
  • cargo test --workspace could not run: Cargo is not installed in the environment
  • cargo fmt --all -- --check could not run: Cargo is not installed in the environment

🤖 Generated with Codebuff
Co-Authored-By: Codebuff <noreply@codebuff.com>
@drips-wave

drips-wave Bot commented Aug 29, 2026

Copy link
Copy Markdown

@graceful-tech12 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

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.

Fix wrong expected panic string in token_tests.rs test_transfer_from_expired_allowance

2 participants