centrifuge / centrifuge-chain

Centrifuge Chain: the layer-1 blockchain for real-world assets, built on Substrate.

https://centrifuge.io

GNU Lesser General Public License v3.0

182 stars 78 forks source link

Mock builder update. Improved FI coverage #1945

Closed lemunozm closed 1 month ago

lemunozm commented 1 month ago

Description

Added new version of mock builder with method handlers, supporting check how many times a mock was called.
Fixed some wrong configured default-features (I do not know why Cargo did not complained)
Improved FI tests checking the times the hooks are called.
~Cargo update~ Something is really off with the cargo update. Just doing cargo update on main makes the compilation of runtime fail due to std issues. I was not able to find the guilty one.

lemunozm commented 1 month ago

I've needed to also update all inner dependencies to get rid of a cargo bug 🫤

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 47.73%. Comparing base (fc2b451) to head (f858bbd).

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #1945 +/- ## ======================================= Coverage 47.72% 47.73% ======================================= Files 184 184 Lines 12969 12969 ======================================= + Hits 6190 6191 +1 + Misses 6779 6778 -1 ```

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

lemunozm commented 1 month ago

I've touched some file that requires also the approval of any of you @wischli @hieronx 🙏🏻

lemunozm commented 1 month ago

Could you force @hieronx ? It seems like we also need the Frederik approval