cairo_native icon indicating copy to clipboard operation
cairo_native copied to clipboard

Add trace dump contracts 2

Open JulianGCalderon opened this issue 1 year ago • 1 comments

Implements more types

Checklist

  • [ ] Linked to Github Issue
  • [ ] Unit tests added
  • [ ] Integration tests added.
  • [ ] This change requires new documentation.
    • [ ] Documentation has been added/updated.

JulianGCalderon avatar Oct 08 '24 15:10 JulianGCalderon

Codecov Report

Attention: Patch coverage is 79.51743% with 382 lines in your changes missing coverage. Please review.

Project coverage is 82.74%. Comparing base (cff7a11) to head (07bf052). Report is 12 commits behind head on add-trace-dump-contracts.

Files with missing lines Patch % Lines
src/starknet.rs 16.23% 98 Missing :warning:
src/types/starknet.rs 1.96% 50 Missing :warning:
src/starknet_stub.rs 73.48% 48 Missing :warning:
src/libfuncs/starknet.rs 0.00% 46 Missing :warning:
src/libfuncs/array.rs 86.21% 43 Missing :warning:
src/executor/contract.rs 70.83% 14 Missing :warning:
src/ffi.rs 33.33% 14 Missing :warning:
src/libfuncs/gas.rs 85.10% 14 Missing :warning:
src/compiler.rs 89.89% 10 Missing :warning:
src/metadata/runtime_bindings.rs 94.18% 10 Missing :warning:
... and 16 more

:exclamation: There is a different number of reports uploaded between BASE (cff7a11) and HEAD (07bf052). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (cff7a11) HEAD (07bf052)
2 1
Additional details and impacted files
@@                     Coverage Diff                      @@
##           add-trace-dump-contracts     #838      +/-   ##
============================================================
- Coverage                     88.11%   82.74%   -5.38%     
============================================================
  Files                           123      120       -3     
  Lines                         34973    34954      -19     
============================================================
- Hits                          30818    28922    -1896     
- Misses                         4155     6032    +1877     

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

codecov-commenter avatar Oct 09 '24 16:10 codecov-commenter