VenusProtocol / protocol-reserve

Contracts for Protocol Reserve
7 stars 6 forks source link

[VEN-2718]: Add access controls and receiver to sweepTokenFromPool #100

Closed kkirka closed 2 months ago

kkirka commented 3 months ago

Problem: Venus still uses Governance to cover some shortfall positions, especially in the core pool. With the current implemetation of the risk fund, it is not possible to use custom flows for covering the debts – it requires a normal VIP and transfers funds directly to the pool's Comptroller. It is not always sufficient, e.g. we may need to repay a borrow in a specific vToken rather than fund the pool's Comptroller. We may also need to use a critical or a fast-track timelock to perform this operation as this might be time-sensitive.

Solution:

github-actions[bot] commented 2 months ago

Code Coverage

Package Line Rate Branch Rate Health
Interfaces 100% 100%
ProtocolReserve 86% 65%
Test 100% 100%
Test.Mocks 61% 29%
TokenConverter 89% 74%
Utils 100% 100%
Summary 86% (614 / 718) 70% (205 / 292)