filecoin-project / specs-actors

DEPRECATED Specification of builtin actors, in the form of executable code.

Other

86 stars 102 forks source link

added logs for when miner state changes #1518

Closed laudiacay closed 2 years ago

laudiacay commented 3 years ago

This PR adds logs for when miner state balances change as requested in #1500. There were some spots where i wasn’t sure if the functions in miner_state.go were returning the right value, so i flagged those, and also found one spot where the wrong thing was being logged and fixed it.

This PR came from my IPad so sorry in advance if there’s anything wonky... tried to check diffs over carefully

codecov-commenter commented 2 years ago

Codecov Report

Merging #1518 (26ac28a) into next (b91ffa9) will increase coverage by 0.0%. The diff coverage is 85.0%.

@@          Coverage Diff          @@
##            next   #1518   +/-   ##
=====================================
  Coverage   71.5%   71.5%           
=====================================
  Files         73      73           
  Lines       8566    8585   +19     
=====================================
+ Hits        6127    6144   +17     
- Misses      1554    1555    +1     
- Partials     885     886    +1

laudiacay commented 2 years ago

Closes #1500