Skip to content

test: add coverage for assigned issues #902 and #903 - #916

Merged
ogazboiz merged 1 commit into
LabsCrypt:mainfrom
edehvictor:fix/edehvictor-902-903
May 27, 2026
Merged

test: add coverage for assigned issues #902 and #903#916
ogazboiz merged 1 commit into
LabsCrypt:mainfrom
edehvictor:fix/edehvictor-902-903

Conversation

@edehvictor

@edehvictor edehvictor commented May 27, 2026

Copy link
Copy Markdown
Contributor

Closes #902
Closes #903
Closes #904
Closes #901

Assigned issues

Scope of this PR

This branch implements the two simplest assigned issues from the set above:

The remaining assigned issues are acknowledged here for tracking, but are not included in this branch:

Changes

  • Added additional LoanManager liquidation tests covering collateral shortfall, repaid-loan rejection, and LoanLiquidated event assertions.
  • Added an integration test for borrower contest flow and admin dispute resolution for both confirm and reverse actions.
  • Asserted borrower/admin notification behavior in the dispute flow test.

Testing

  • node --experimental-vm-modules node_modules/jest/bin/jest.js src/__tests__/integration/loanDisputeFlow.test.ts
  • Rust contract tests were not run in this environment because cargo is not installed on this machine.

@ogazboiz
ogazboiz merged commit cf8744c into LabsCrypt:main May 27, 2026
3 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants