Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release v5.0.2 #5

Merged
merged 48 commits into from
Dec 20, 2023
Merged

Release v5.0.2 #5

merged 48 commits into from
Dec 20, 2023

Conversation

JGcarv
Copy link
Contributor

@JGcarv JGcarv commented Dec 19, 2023

No description provided.

vbidin and others added 30 commits January 13, 2023 17:32
…34) (#265)

* feat: add isCollateralMaintained check in makePayment()

* feat: Add test for collateralNotMaintained for makePayment

* feat: Update test based on JGs suggestion

- Expose collateral not maintained check when using drawable funds only

* fix: Update based on PR comments

* feat: Update test to include collateralAsset and calc for collateral requirement

* refactor: Move mint of collateral asset
* feat: set lender during initialization

* fix: update globals mock

* fix: update mocks and add missing tests

* fix: address all pr comments

* fix: update spacing

Co-authored-by: lucas-manuel <[email protected]>
* fix: update to not accrue interest into the next payment interval

when the loan is late on a refinance

* feat: Test to ensure late loan refi interest calculated correctly

* feat: Add literal assertions for interest amounts

* fix: Update based on PR comments

* fix: update params

Co-authored-by: Lucas Manuel <[email protected]>
* feat: enable borrower to perform upgrades

* fix: update error message

* fix: address PR comments

* fix: update failing upgrade pause test
* feat: rename lateInterest var

* fix: name
* fix: internal audit findings

* ci: update coverage threshold

---------

Co-authored-by: lucas-manuel <[email protected]>
* feat: update to use isInstanceOf in leiu of isFactory

* feat: update key

* feat: update key

* chore: update var name
* feat: add V5 migrator and tests

* fix: missing word on comment

* test: remove loan v4 mock

* fix: update rates in tests as well as use scaled precision in payments calc

* fix: tests after rate change

* test: added test for initializer

* fix: adjust PR comments

---------

Co-authored-by: lucas-manuel <[email protected]>
* feat: globals `isFunctionPaused` (sc-11842)

* feat: remove `isFactory`

* feat: add pause on migrate and setImplementation

* refactor: undo function reordering

---------

Co-authored-by: 0xfarhaan <[email protected]>
* fix: underscore syntax [3S #9] (SC-12033)

* refactor: cleanup

---------

Co-authored-by: Lucas Manuel <[email protected]>
* fix: make change and fix tests

* fix: account origination fees into assertions

---------

Co-authored-by: lucas-manuel <[email protected]>
* refactor: reorder

* refactor: interfaces cleanup

* fix: update formatting

---------

Co-authored-by: lucas-manuel <[email protected]>
* feat: only securityAdmin can upgrade loan

* style: remove extra _

* fix: update test name

Co-authored-by: Michael De Luca <[email protected]>

---------

Co-authored-by: Lucas Manuel <[email protected]>
Co-authored-by: Michael De Luca <[email protected]>
* fix: make hundred percent public

* test: add test for getter
* refactor: rename to lateInterestPremiumRate

* refactor: update var name in tests

* fix: pr review

---------

Co-authored-by: Michael De Luca <[email protected]>
Co-authored-by: Lucas Manuel <[email protected]>
… (#296)

* feat: clean up impairment removal (SC-12191) (SB #22)

* fix: update events

---------

Co-authored-by: JG Carvalho <[email protected]>
Co-authored-by: Lucas Manuel <[email protected]>
Co-authored-by: lucas-manuel <[email protected]>
* fix: delete refinance commitment on clear accounting

* fix: also clear refinance interest

* test: add assertion on clear loan
vbidin and others added 16 commits May 16, 2023 13:54
* chore: cleanup and bytecode size check (SC-12382)

* fix: coverage report and size check script
* feat: update factory to include canDeploy (sc-13704)

* fix: add err msg

---------

Co-authored-by: 0xfarhaan <[email protected]>
* feat: create loan migrator to update factory on loan (SC-13705)

* feat: add isLoan update function (SC-13705)

* feat: unit tests (SC-13705)

* fix: overload isLoan (SC-13705)

* fix: test file name (SC-13705)

* fix: comments (SC-13705)

* refactor: update tests and formatting on contracts

* fix: comment

* chore: address PR comments

---------

Co-authored-by: 0xfarhaan <[email protected]>
@JGcarv JGcarv self-assigned this Dec 19, 2023
@JGcarv JGcarv requested a review from a team as a code owner December 19, 2023 21:44
Copy link

LCOV of commit e491d23 during Forge Tests (PR) #9

Summary coverage rate:
  lines......: 96.9% (343 of 354 lines)
  functions..: 91.0% (101 of 111 functions)
  branches...: no data found

Files changed coverage rate:
                                     |Lines       |Functions  |Branches    
  Filename                           |Rate     Num|Rate    Num|Rate     Num
  =========================================================================
  contracts/MapleLoanFactory.sol     | 100%      3| 100%     2|    -      0
  contracts/MapleLoanFeeManager.sol  |83.7%     49|72.2%    18|    -      0
  contracts/MapleLoanV502Migrator.sol| 100%      5| 100%     1|    -      0
  contracts/MapleRefinancer.sol      | 100%     19| 100%    12|    -      0

Copy link

LCOV of commit 544131b during Forge Tests (PR) #10

Summary coverage rate:
  lines......: 96.9% (343 of 354 lines)
  functions..: 91.0% (101 of 111 functions)
  branches...: no data found

Files changed coverage rate:
                                     |Lines       |Functions  |Branches    
  Filename                           |Rate     Num|Rate    Num|Rate     Num
  =========================================================================
  contracts/MapleLoanFactory.sol     | 100%      3| 100%     2|    -      0
  contracts/MapleLoanFeeManager.sol  |83.7%     49|72.2%    18|    -      0
  contracts/MapleLoanV502Migrator.sol| 100%      5| 100%     1|    -      0
  contracts/MapleRefinancer.sol      | 100%     19| 100%    12|    -      0

@JGcarv JGcarv merged commit eb0ebf3 into main Dec 20, 2023
3 checks passed
@JGcarv JGcarv deleted the release branch December 20, 2023 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants