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

contract_wrapper: tests refactoring #806

Open
sept-en opened this issue Sep 18, 2022 · 0 comments
Open

contract_wrapper: tests refactoring #806

sept-en opened this issue Sep 18, 2022 · 0 comments
Assignees
Labels
relayers rust Pull requests that update Rust code testing

Comments

@sept-en
Copy link
Contributor

sept-en commented Sep 18, 2022

contract_wrapper crate contains tests for dao_contract module, having MockContractWrapper struct, which contains an implementation of ContractWrapper trait there. The test data contains just some responses for some calls as a binary representation. To make tests more clean and improve it's readability, it's better to use some structs with pre-defined values and then serialize them to the expected format.

@sept-en sept-en added testing relayers rust Pull requests that update Rust code labels Sep 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
relayers rust Pull requests that update Rust code testing
Projects
None yet
Development

No branches or pull requests

2 participants