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

tester: use eosio/authority rather than tester equivalent #664

Closed
James-Mart opened this issue Feb 1, 2022 · 2 comments · Fixed by #665
Closed

tester: use eosio/authority rather than tester equivalent #664

James-Mart opened this issue Feb 1, 2022 · 2 comments · Fixed by #665

Comments

@James-Mart
Copy link
Member

struct tester_permission_level_weight, struct tester_wait_weight, and struct tester_authority are in tester.cpp, and therefore not visible to tester users.

These structs should be externally accessible, to make it easier for users to call actions in test-cases to update account permissions, etc.

@James-Mart
Copy link
Member Author

What was the reason for not simply putting them in the eosio namespace in the header?

@tbfleming
Copy link
Member

They were temporary until the eosio namespace got versions of those. They can be removed now. See eosio/authority.hpp.

@James-Mart James-Mart changed the title tester: move permission/authority structs to accessible area tester: use eosio/authority rather than tester equivalent Feb 1, 2022
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 a pull request may close this issue.

2 participants