-
Notifications
You must be signed in to change notification settings - Fork 0
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
Automated testing is a PITA #77
Comments
Possibly in addition to above, maybe there should be a It would also need to be disallowed to be defined in |
- adds ability to set a newly created account as default - adds `use_default_account` setting to networks.yml - accounts with this setting will autodeploy with the default network(if autodeploy_allowed is set, anyway) - standardized `sb test` exit codes - `sb test` autocompiles now - account attribute is now not required for an instance of deployer, only for certain uses - adds "stateless" test and deploy tests without using ganache Issue #77
Alright, so, Also, The |
|
Automated testing is a PITA and shouldn't be. The only required command to test your contracts shouldn't have to be more than
sb test [network]
.Currently, the most annoying process is:
That's dumb. It should autocompile and autodeploy.
The account part is a little trickier. Using one provided by eth_tester, ganache, or whatever would deprive the user of a test account, and would probably be unexpected. And, since generally sb expects a local account to be used for deployment, would require a bunch of refactoring to do that.
Easiest solution to implement would be to allow the
sb accounts create
to set the first count as default(or an option to set the account as default).The best solution may be to add a setting to
networks.yml
that allows accounts to be auto created and funded as needed. That would remove the whole block of code thatsolidbyte-test-project
uses to autofund thedeployer_account
.If that's done, an optional value setting (in wei) should be available as well.
The text was updated successfully, but these errors were encountered: