Skip to content

Asset macro rework - #445

Open
willemneal wants to merge 1 commit into
mainfrom
fix/parse-asset-args-v2
Open

Asset macro rework#445
willemneal wants to merge 1 commit into
mainfrom
fix/parse-asset-args-v2

Conversation

@willemneal

Copy link
Copy Markdown
Contributor

Summary

  • Reworks the asset macro to properly parse arguments using syn::Parse
  • Introduces separate test and production modules for the asset macro
    • Test mod: deploys a new SAC and uses its instance to get clients — useful for mocking actual SACs (e.g. XLM) in tests
    • Production mod: deploys an actual SAC contract and gets the contract ID from the serialized asset code
  • Adds test contract and macro-level tests

Based on #315

Test plan

  • Verify import_asset!("xlm") works in test and production contexts
  • Run cargo test for macro-level tests
  • Verify SAC deployment in test environment

🤖 Generated with Claude Code

…t/production modes

Introduces separate test and production modules for the asset macro. The test mod
deploys a new SAC and uses its instance for clients, while the production mod deploys
an actual SAC contract and gets the contract ID from the serialized asset code.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

1 participant