Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

scrypto-test test_bindings location #1492

Open
gunther9 opened this issue Sep 10, 2023 · 1 comment
Open

scrypto-test test_bindings location #1492

gunther9 opened this issue Sep 10, 2023 · 1 comment

Comments

@gunther9
Copy link

Currently, if we have a file pool.rs (imported as a module in lib.rs) that contains a blueprint module pool, and inside it the Pool blueprint, the way to get the test bindings would be:

use pool::test_bindings

Wouldn't it be better that test_bindings was placed at the blueprint module level, instead of the file module level? So it would be:

use pool::pool::test_bindings

This would enable the user to create more than one blueprint in the same file, if for any reason it is wanted.

@0xOmarA
Copy link
Member

0xOmarA commented Sep 11, 2023

Hi @gunther9, your recommendation makes sense to me especially when thinking about having multiple blueprints in a single file, this is something that we support that all other parts of the stack so supporting/enabling it with scrypto-test makes sense. There is also another alternative where the test_bindings module could be prefixed with the blueprint name. So, for Radiswap the test bindingas module could be radiswap_test_bindings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants