Skip to content

Commit

Permalink
fix: use correct address for strategist at deployment.
Browse files Browse the repository at this point in the history
  • Loading branch information
clement-ux committed Jan 6, 2025
1 parent 8d2de44 commit 543378d
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 4 deletions.
6 changes: 4 additions & 2 deletions contracts/deploy/mainnet/114_simple_harvester.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,13 +11,15 @@ module.exports = deploymentWithGovernanceProposal(
// proposalId: "",
},
async ({ deployWithConfirmation }) => {
const { deployerAddr } = await getNamedAccounts();
const { strategistAddr } = await getNamedAccounts();

// 1. Deploy contract
const dOETHHarvesterSimple = await deployWithConfirmation(
"OETHHarvesterSimple",
[addresses.mainnet.Timelock, deployerAddr] // Need to adjust governor and strategist
[addresses.mainnet.Timelock, strategistAddr]
);

console.log("strategistAddr: ", strategistAddr);
const cOETHHarvesterSimple = await ethers.getContractAt(
"OETHHarvesterSimple",
dOETHHarvesterSimple.address
Expand Down
6 changes: 4 additions & 2 deletions contracts/test/harvest/simple-harvester.mainnet.fork-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,13 +19,15 @@ describe("ForkTest: CurvePoolBooster", function () {

it("Should have correct parameters", async () => {
const { simpleOETHHarvester } = fixture;
const { deployerAddr } = await getNamedAccounts();
const { strategistAddr } = await getNamedAccounts();

expect(await simpleOETHHarvester.governor()).to.be.equal(
addresses.mainnet.Timelock
);

console.log("strategistAddr: ", strategistAddr);
expect(await simpleOETHHarvester.strategistAddr()).to.be.equal(
deployerAddr
strategistAddr
);
});

Expand Down

0 comments on commit 543378d

Please sign in to comment.