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

chore: update test coverage for ibc client #846

Merged
merged 1 commit into from
Mar 15, 2024
Merged

Conversation

bishalbikram
Copy link
Collaborator

Improve test cases for ics-02 client

  • updated existing test cases
  • added new test cases to cover edge cases

@bishalbikram bishalbikram added test Testing update for Unit test, Integration test and E2E testing Cosmwasm rust Pull requests that update Rust code labels Mar 14, 2024
@bishalbikram bishalbikram self-assigned this Mar 14, 2024
@github-actions github-actions bot removed the test Testing update for Unit test, Integration test and E2E testing label Mar 14, 2024
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.54%. Comparing base (9497615) to head (506aa1f).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #846      +/-   ##
============================================
+ Coverage     70.28%   70.54%   +0.26%     
+ Complexity      418      414       -4     
============================================
  Files           155      155              
  Lines         14029    14026       -3     
  Branches        292      290       -2     
============================================
+ Hits           9860     9895      +35     
+ Misses         4022     3982      -40     
- Partials        147      149       +2     
Flag Coverage Δ
rust 68.71% <ø> (+0.33%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 3 files with indirect coverage changes

Copy link
Collaborator

@ibrizsabin ibrizsabin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@ibrizsabin ibrizsabin merged commit 9a7876b into main Mar 15, 2024
9 checks passed
@ibrizsabin ibrizsabin deleted the coverage/ics03-client branch March 15, 2024 09:26
ibrizsabin added a commit that referenced this pull request May 15, 2024
* feat: add wasm client project

* fix: add wasm proto

* fix: import ics08wasm

* fix: import wasm-08

* fix: generalize context

* fix: build deps

* fix: fix deps issue

* fix: split query handler

* fix: wrap result in response

* feat: add verify header

* feat: add genesis export

* fix: impl client status

* fix: fix unit test

* fix: add debug logs

* fix: cleanup

* chore: pass build

* fix: proto changes

* fix: split traits

* chore: pass build

* fix: remove inner

* fix: query handler

* fix: add tests

* fix: add update test

* fix: downgrade

* fix: downgrade cosmwasm

* fix: update test utils

* fix: point to downgraded xcall

* fix: lower multi test

* fix: lock on 1.6

* fix: revert downgrade

* fix: try disable features

* fix: upgrade cw-storage-plus

* fix: lock to 1.4

* fix: move cw-multi test

* chore: pass build

* fix: fix branch

* fix: add height

* fix: cleanup

* chore: pass build

* fix: remove toolchain.toml

* fix: fix verify membership

* fix: ack hashing

* doc: add build note

* chore: pass build

* fix: do sha256

* fix: check misbehaviour

* chore: pass build

* fix:resolve comments

* chore: pass build

* fix: remove unnecessary hash

* chore: pass build

* fix: rename client id

* fix: Fix reverse sign on timestamp check for request timeout

* chore: update test coverage for ibc client (#846)

* chore: update test coverage for ics03-connection (#848)

* chore: update test coverage for ibc client

* chore: improve test coverage for ibc connection

* chore: add test coverage for cosmwasm ibc core (#850)

* docs: Add explanation on how a connection can be manually verified (#838)

doc: Add explanation on how a connection can be manually verified

* fix: add reference to CI workflow to fix error (#839)

Co-authored-by: ForwardSlashBack <[email protected]>

* chore: add unit test coverage for icon light client (#851)

* ci: add injectived, and neutornd binary (#858)

* ci: add injectived, and neutornd binary

* ci: update archwayd to v6.0.0

* ci: update archwayd to v6.0.0

* chore: add test coverage for xcall-ibc-connection (#853)

* fix: fix test

* fix: resolve test issues

* fix: cargo fmt

---------

Co-authored-by: AntonAndell <[email protected]>
Co-authored-by: undefined <[email protected]>
Co-authored-by: ForwardSlashBack <[email protected]>
Co-authored-by: ForwardSlashBack <[email protected]>
Co-authored-by: DeepakBomjan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Cosmwasm rust Pull requests that update Rust code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants