search

Rari-Capital / fuse-v1

Smart contracts for Fuse V1 by Rari Capital.
Other
3 stars 3 forks source link

Add accounting fix integration test #3

Closed zerosnacks closed 2 years ago

zerosnacks commented 2 years ago

Implements the accounting fix integration test

zerosnacks commented 2 years ago

Please note the conflicting file - development is now branching off of fuse-reactive-audit instead of fuse-fixed-accounting meaning the fix does not exist yet inside of this repo.

zerosnacks commented 2 years ago

Not exactly sure what has changed in the meantime as last week the test ran perfectly fine with the same settings (block number: 14684685).

Currently throws on line 71 of CEtherDelegateTempExploitAccounting.sol

[FAIL. Reason: Expect account #1 borrow balance to start at 0.] testShouldMergeAttackersSupplyAndBorrowBalances() (gas: 11018600)
zerosnacks commented 2 years ago

@sriyantra do we still need this PR?

sriyantra commented 2 years ago

@sriyantra do we still need this PR?

nope, closing