Skip to content

Conversation

@Shubbu03
Copy link
Contributor

@Shubbu03 Shubbu03 commented Dec 4, 2025

Add integration tests for Token‑2022 support and document current metadata limitations.

  • Token‑2022 lifecycle: create mint, initialize, create ATAs, mint, transfer, burn, and close accounts (test_token2022_full_lifecycle).
  • Error handling: transferring/burning more than balance and closing accounts with non‑zero balance (test_token2022_error_cases).
  • Delegation: approve delegate, delegated transfer, revoke, and ensure post‑revoke transfers fail (test_token2022_delegate_operations).
  • Freeze/thaw: freeze accounts, verify transfers fail while frozen, thaw, and confirm transfers succeed afterward (test_token2022_freeze_thaw).

completes requested tests by @lgalabru in #431

@Shubbu03
Copy link
Contributor Author

Shubbu03 commented Dec 4, 2025

also i tried the InvalidRealloc issue around Token‑2022 metadata init. I added an integration test that tries to replicate the spl-token CLI flow, but in our Surfpool/LiteSVM environment the metadata instruction currently fails earlier with “incorrect program id for instruction" rather than InvalidRealloc so currently stuck there

@MicaiahReid MicaiahReid changed the title feat(tests): add comprehensive tests for Token-2022 tests: add comprehensive tests for Token-2022 Dec 4, 2025
Copy link
Member

@MicaiahReid MicaiahReid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice, thanks @Shubbu03!

@lgalabru
Copy link
Member

lgalabru commented Dec 5, 2025

Thanks @Shubbu03! Could you rebase your branch with main? That flaky test should be fixed now.

@Shubbu03
Copy link
Contributor Author

Shubbu03 commented Dec 5, 2025

@lgalabru ser can I do this after the 10th? currently not at my desk

…cases, delegate operations, and freeze/thaw functionality
@MicaiahReid MicaiahReid force-pushed the feat(tkn22)/tkn22-test-coverage branch from b4f40ba to 7fac0c6 Compare December 5, 2025 18:45
@MicaiahReid
Copy link
Member

All good, I rebased for you @Shubbu03

@lgalabru lgalabru merged commit a1de755 into txtx:main Dec 6, 2025
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.

3 participants