crypto-com / defi-wallet-core-rs

Cross-platform, cross-blockchain wallet library in Rust for DeFi Wallet and other applications.
Other
48 stars 16 forks source link

Problem: abi Array is encoed as FixedArray (fix #930) #931

Closed leejw51crypto closed 1 year ago

leejw51crypto commented 1 year ago

Solution: fix array encoded as array

codecov-commenter commented 1 year ago

Codecov Report

Patch coverage has no change and project coverage change: -0.04% :warning:

Comparison is base (e452233) 42.15% compared to head (02c8fc1) 42.12%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #931 +/- ## ========================================== - Coverage 42.15% 42.12% -0.04% ========================================== Files 48 46 -2 Lines 9346 9342 -4 ========================================== - Hits 3940 3935 -5 - Misses 5406 5407 +1 ``` | [Files Changed](https://app.codecov.io/gh/crypto-com/defi-wallet-core-rs/pull/931?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=crypto-com) | Coverage Δ | | |---|---|---| | [common/src/node/ethereum/abi.rs](https://app.codecov.io/gh/crypto-com/defi-wallet-core-rs/pull/931?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=crypto-com#diff-Y29tbW9uL3NyYy9ub2RlL2V0aGVyZXVtL2FiaS5ycw==) | `70.58% <0.00%> (ø)` | | | [common/src/transaction/ethereum/abi\_contract.rs](https://app.codecov.io/gh/crypto-com/defi-wallet-core-rs/pull/931?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=crypto-com#diff-Y29tbW9uL3NyYy90cmFuc2FjdGlvbi9ldGhlcmV1bS9hYmlfY29udHJhY3QucnM=) | `93.25% <0.00%> (ø)` | | ... and [5 files with indirect coverage changes](https://app.codecov.io/gh/crypto-com/defi-wallet-core-rs/pull/931/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=crypto-com)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.