-
Notifications
You must be signed in to change notification settings - Fork 2
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
[VEN-1709]: simulations for adding a new market in DeFi pool #41
Conversation
|
||
it("should set vankrBNB_DeFi collateral factor to 0", async () => { | ||
const market = await comptroller.markets(vankrBNB_DeFi); | ||
expect(market.collateralFactorMantissa).to.equal(parseUnits("0", 18)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
expect(market.collateralFactorMantissa).to.equal(parseUnits("0", 18)); | |
expect(market.collateralFactorMantissa).to.equal(0); |
|
||
it("should set vankrBNB_DeFi liquidation threshold to 0", async () => { | ||
const market = await comptroller.markets(vankrBNB_DeFi); | ||
expect(market.liquidationThresholdMantissa).to.equal(parseUnits("0", 18)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
expect(market.liquidationThresholdMantissa).to.equal(parseUnits("0", 18)); | |
expect(market.liquidationThresholdMantissa).to.equal(0); |
const comptroller = await ethers.getContractAt(COMPTROLLER_ABI, COMPTROLLER_DeFi); | ||
expect(await comptroller.getRewardDistributors()).to.have.lengthOf(2); | ||
}); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it("should be added to DeFi pool", async () => { | |
expect(await comptroller.getRewardDistributors()).to.include(REWARD_DISTRIBUTOR); | |
}); | |
|
||
describe("Reward Distributor", () => { | ||
it("should have 2 rewards distributor in DeFi pool", async () => { | ||
const comptroller = await ethers.getContractAt(COMPTROLLER_ABI, COMPTROLLER_DeFi); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have comptroller
defined already, so no need to redefine AFAIU
const comptroller = await ethers.getContractAt(COMPTROLLER_ABI, COMPTROLLER_DeFi); |
Description
Resolves VEN-1709