bosagora / agora

POC Node implementation for CoinNet
https://bosagora.io
MIT License
37 stars 22 forks source link

Ledger: Dont validate UTXO still exists for nodes to be slashed #3148

Closed omerfirmak closed 2 years ago

omerfirmak commented 2 years ago

Stakes are locked for the duration of their enrollment, there is no way they are spent.

codecov[bot] commented 2 years ago

Codecov Report

Merging #3148 (057c9de) into v0.x.x (333a126) will increase coverage by 60.18%. The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           v0.x.x    #3148       +/-   ##
===========================================
+ Coverage   18.81%   78.99%   +60.18%     
===========================================
  Files          54      209      +155     
  Lines        2355    18775    +16420     
===========================================
+ Hits          443    14831    +14388     
- Misses       1912     3944     +2032     
Flag Coverage Δ
integration 16.26% <ø> (-2.55%) :arrow_down:
unittests 87.33% <100.00%> (?)

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

Impacted Files Coverage Δ
source/agora/consensus/Ledger.d 97.16% <100.00%> (ø)
source/scpd/scp/SCPDriver.d 60.00% <0.00%> (-15.00%) :arrow_down:
source/scpd/scp/SCP.d 100.00% <0.00%> (ø)
source/agora/common/Set.d 96.77% <0.00%> (ø)
source/agora/network/Client.d 81.67% <0.00%> (ø)
source/agora/cli/client/GenTxProcess.d 0.00% <0.00%> (ø)
source/agora/consensus/data/Block.d 99.28% <0.00%> (ø)
source/agora/test/PreImageNoFlood.d 93.33% <0.00%> (ø)
source/agora/test/RestoreSlashingInfo.d 93.33% <0.00%> (ø)
source/agora/node/FullNode.d 74.69% <0.00%> (ø)
... and 151 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 333a126...057c9de. Read the comment docs.