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

Add full MsgServerV2 test with mock application #7372

Closed
Tracked by #7345
chatton opened this issue Oct 1, 2024 · 2 comments
Closed
Tracked by #7345

Add full MsgServerV2 test with mock application #7372

chatton opened this issue Oct 1, 2024 · 2 comments
Labels
feat: ibc-eureka type: test coverage Increase code and use case coverage by means of unit, integration, e2e or manual tests

Comments

@chatton
Copy link
Contributor

chatton commented Oct 1, 2024

We can write a full MsgServerV2 test which uses mock applications, the same as TestMsgServerV2PacketFlow in the POC branch.

This will require mock applications wired up.

@gjermundgaraba
Copy link
Contributor

@chatton @bznein is this covered in #7524?

@chatton
Copy link
Contributor Author

chatton commented Nov 20, 2024

Ah yes I think it is, we can close the issue!

@chatton chatton closed this as completed Nov 20, 2024
@womensrights womensrights added the type: test coverage Increase code and use case coverage by means of unit, integration, e2e or manual tests label Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat: ibc-eureka type: test coverage Increase code and use case coverage by means of unit, integration, e2e or manual tests
Projects
None yet
Development

No branches or pull requests

3 participants