Skip to content
New issue

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

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

Already on GitHub? # to your account

[ICS02] Cover client frozen/expired conditions with unit tests #538

Closed
Tracked by #554
Farhad-Shabani opened this issue Mar 15, 2023 · 0 comments · Fixed by #921
Closed
Tracked by #554

[ICS02] Cover client frozen/expired conditions with unit tests #538

Farhad-Shabani opened this issue Mar 15, 2023 · 0 comments · Fixed by #921
Assignees
Labels
O: testing Objective: aims to improve testing coverage

Comments

@Farhad-Shabani
Copy link
Member

Farhad-Shabani commented Mar 15, 2023

Problem Statement

Make sure that we covered failure scenarios everywhere we check a status of a client (like here) by introducing frozen and expired conditions.
Adding required fields to the MockCotext and writing unit tests should accomplish this.

Should come after fixing #536

@Farhad-Shabani Farhad-Shabani added the O: testing Objective: aims to improve testing coverage label Mar 15, 2023
@Farhad-Shabani Farhad-Shabani moved this to 📥 To Do in ibc-rs Mar 15, 2023
@Farhad-Shabani Farhad-Shabani added the A: blocked Admin: blocked by another (internal/external) issue or PR label Jul 1, 2023
@Farhad-Shabani Farhad-Shabani removed the A: blocked Admin: blocked by another (internal/external) issue or PR label Sep 7, 2023
@github-project-automation github-project-automation bot moved this from 📥 To Do to ✅ Done in ibc-rs Oct 18, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
O: testing Objective: aims to improve testing coverage
Projects
None yet
2 participants