@gooddollar/goodprotocol 1.0.29-beta.4 → 1.0.29-beta.5

package/test/invite/InvitesV2.test.ts

@@ -0,0 +1,355 @@
+import hre, { ethers, upgrades } from "hardhat";
+import { loadFixture } from "@nomicfoundation/hardhat-network-helpers";
+import { expect } from "chai";
+import { InvitesV2, IGoodDollar, IIdentity } from "../../types";
+import { SignerWithAddress } from "@nomiclabs/hardhat-ethers/dist/src/signer-with-address";
+import { createDAO } from "../helpers";
+
+const BN = ethers.BigNumber;
+
+describe("InvitesV2", () => {
+  let invites: InvitesV2, founder: SignerWithAddress;
+  let inviter1,
+    inviter2,
+    invitee1,
+    invitee2,
+    invitee3,
+    invitee4,
+    invitee5,
+    invitee6,
+    invitee7,
+    invitee8;
+
+  let avatar, gd: IGoodDollar, Controller, id: IIdentity;
+
+  before(async () => {
+    [
+      founder,
+      inviter1,
+      inviter2,
+      invitee1,
+      invitee2,
+      invitee3,
+      invitee4,
+      invitee5,
+      invitee6,
+      invitee7,
+      invitee8
+    ] = await ethers.getSigners();
+
+    const InvitesV2 = await ethers.getContractFactory("InvitesV2");
+
+    let {
+      daoCreator,
+      controller,
+      avatar: av,
+      gd: gooddollar,
+      identity,
+      nameService
+    } = await loadFixture(createDAO);
+
+    Controller = controller;
+    avatar = av;
+
+    invites = (await upgrades.deployProxy(
+      InvitesV2,
+      [nameService.address, gooddollar, 500, founder.address],
+      {
+        kind: "uups"
+      }
+    )) as InvitesV2;
+
+    gd = (await ethers.getContractAt(
+      "IGoodDollar",
+      gooddollar,
+      founder
+    )) as IGoodDollar;
+    id = (await ethers.getContractAt(
+      "IIdentity",
+      identity,
+      founder
+    )) as IIdentity;
+
+    await gd["mint(address,uint256)"](invites.address, BN.from(5000));
+    // await gd.transfer(invites.address, BN.from(5000));
+  });
+
+  it("should have balance", async () => {
+    const balance = await gd.balanceOf(invites.address);
+    expect(balance).to.equal(5000);
+  });
+
+  it("should have version", async () => {
+    expect(await invites.active()).to.be.true;
+    const version = await invites.version();
+    expect(version).to.be.equal("2.0");
+  });
+
+  it("should let anyone join", async () => {
+    await invites
+      .connect(inviter1)
+      .join(ethers.utils.hexZeroPad("0xfa", 32), ethers.constants.HashZero);
+    let inviter = await invites.users(inviter1.address);
+    expect(inviter.inviteCode).to.equal(ethers.utils.hexZeroPad("0xfa", 32));
+  });
+
+  it("should allow to join only once", async () => {
+    await expect(
+      invites
+        .connect(inviter1)
+        .join(
+          ethers.utils.hexZeroPad("0xfa", 32),
+          ethers.utils.hexZeroPad("0x01", 32)
+        )
+    ).to.revertedWith("user already joined");
+  });
+
+  it("should not allow code reuse", async () => {
+    // const invites = await Invites.deployed();
+    await expect(
+      invites
+        .connect(inviter2)
+        .join(ethers.utils.hexZeroPad("0xfa", 32), ethers.constants.HashZero)
+    ).to.revertedWith("invite code already in use");
+  });
+
+  it("should mark inviter", async () => {
+    await invites
+      .connect(invitee1)
+      .join(
+        ethers.utils.hexZeroPad("0xaa", 32),
+        ethers.utils.hexZeroPad("0xfa", 32)
+      );
+    let invitee = await invites.users(invitee1.address);
+    let inviterInvitees = await invites.getInvitees(inviter1.address);
+    expect(invitee.invitedBy).to.be.equal(inviter1.address);
+    expect(inviterInvitees).to.include(invitee1.address);
+  });
+
+  it("should not pay bounty for non whitelisted invitee", async () => {
+    await expect(
+      invites.connect(inviter1).bountyFor(invitee1.address)
+    ).to.revertedWith("user not elligble for bounty yet");
+  });
+
+  it("should not pay bounty for non whitelisted inviter", async () => {
+    await id.addWhitelistedWithDID(invitee1.address, Math.random() + "");
+    expect(await id.isWhitelisted(invitee1.address)).to.be.true;
+    expect(await invites.canCollectBountyFor(invitee1.address)).to.be.false;
+    await expect(
+      invites.connect(inviter1).bountyFor(invitee1.address)
+    ).to.revertedWith("user not elligble for bounty yet");
+  });
+
+  it("should pay bounty for whitelisted invitee and inviter", async () => {
+    const bounty = (await invites.levels(0)).bounty.toNumber();
+    await id
+      .addWhitelistedWithDID(inviter1.address, Math.random() + "")
+      .catch(e => e);
+    const startBalance = await gd
+      .balanceOf(inviter1.address)
+      .then(_ => _.toNumber());
+    expect(await id.isWhitelisted(inviter1.address)).to.be.true;
+    let pending = await invites.getPendingInvitees(inviter1.address);
+    expect(pending.length, "pending").to.be.equal(1);
+    const inviteeBalance = await gd
+      .balanceOf(invitee1.address)
+      .then(_ => _.toNumber());
+    await invites.connect(inviter1).bountyFor(invitee1.address);
+
+    let invitee = await invites.users(invitee1.address);
+    let inviter = await invites.users(inviter1.address);
+    const endBalance = await gd
+      .balanceOf(inviter1.address)
+      .then(_ => _.toNumber());
+
+    pending = await invites.getPendingInvitees(inviter1.address);
+    const txFee = await gd.getFees(bounty).then(_ => _["0"].toNumber()); //gd might have a tx fee
+    const txFee2 = await gd.getFees(bounty / 2).then(_ => _["0"].toNumber()); //gd might have a tx fee
+
+    expect(pending.length, "pending").to.be.equal(0);
+    expect(invitee.bountyPaid).to.be.true;
+    expect(inviter.totalApprovedInvites.toNumber()).to.be.equal(1);
+    expect(inviter.totalEarned.toNumber()).to.be.equal(bounty);
+    expect(
+      endBalance - startBalance + txFee,
+      "inviter rewards not matching bounty"
+    ).to.be.equal(bounty);
+    expect(
+      (await gd.balanceOf(invitee1.address).then(_ => _.toNumber())) -
+        inviteeBalance,
+      "invitee rewrad should be bounty/2"
+    ).to.be.equal(bounty / 2 - txFee2); //test that invitee  got half bonus
+  });
+
+  it("should update global stats", async () => {
+    const bounty = (await invites.levels(0)).bounty.toNumber();
+    const stats = await invites.stats();
+    expect(stats.totalApprovedInvites.toNumber()).to.be.equal(
+      1,
+      "approved invites"
+    );
+    expect(stats.totalInvited.toNumber()).to.be.equal(1, "total  invited");
+    expect(stats.totalBountiesPaid.toNumber()).to.be.equal(bounty);
+  });
+
+  it("should not pay bounty twice", async () => {
+    await expect(
+      invites.connect(inviter2).bountyFor(invitee1.address)
+    ).to.revertedWith("user not elligble for bounty yet");
+  });
+
+  it("should not fail in collectBounties for invalid invitees", async () => {
+    await invites
+      .connect(invitee7)
+      .join(
+        ethers.utils.hexZeroPad("0x01", 32),
+        ethers.utils.hexZeroPad("0xfa", 32)
+      );
+    await invites
+      .connect(invitee8)
+      .join(
+        ethers.utils.hexZeroPad("0x02", 32),
+        ethers.utils.hexZeroPad("0xfa", 32)
+      );
+
+    let pending = await invites.getPendingInvitees(inviter1.address);
+    expect(pending.length, "pending").to.be.equal(2);
+    await expect(invites.connect(inviter1).collectBounties()).to.not.reverted;
+    let user1 = await invites.users(invitee7.address);
+    let user2 = await invites.users(invitee8.address);
+    pending = await invites.getPendingInvitees(inviter1.address);
+    expect(
+      await invites.getPendingBounties(inviter1.address).then(_ => _.toNumber())
+    ).to.be.equal(0);
+    expect(user1.bountyPaid).to.be.false;
+    expect(user2.bountyPaid).to.be.false;
+    expect(pending.length, "pending").to.be.equal(2);
+  });
+
+  it("should collectBounties for inviter", async () => {
+    await id
+      .addWhitelistedWithDID(invitee7.address, Math.random() + "")
+      .catch(e => e);
+    await id
+      .addWhitelistedWithDID(invitee8.address, Math.random() + "")
+      .catch(e => e);
+    expect(
+      await invites.getPendingBounties(inviter1.address).then(_ => _.toNumber())
+    ).to.be.equal(2);
+    const res = await invites
+      .connect(inviter1)
+      .collectBounties()
+      .catch(e => e);
+
+    let user1 = await invites.users(invitee7.address);
+    let user2 = await invites.users(invitee8.address);
+    let pending = await invites.getPendingInvitees(inviter1.address);
+    expect(
+      await invites.getPendingBounties(inviter1.address).then(_ => _.toNumber())
+    ).to.be.equal(0);
+    expect(pending.length, "pending").to.be.equal(0);
+    expect(user1.bountyPaid, "user1").to.be.true;
+    expect(user2.bountyPaid, "user2").to.be.true;
+  });
+
+  it("should not set level not by owner", async () => {
+    await expect(
+      invites.connect(inviter1).setLevel(0, 1, 5, 1)
+    ).to.revertedWith("Only owner or avatar can perform this action");
+  });
+
+  it("should set level by owner", async () => {
+    await invites.setLevel(0, 1, 5, 1);
+    let lvl = await invites.levels(0);
+    expect(lvl.toNext.toNumber()).to.be.equal(1);
+    expect(lvl.daysToComplete.toNumber()).to.be.equal(1);
+    await invites.setLevel(1, 0, 10, 2);
+    lvl = await invites.levels(1);
+    expect(lvl.toNext.toNumber()).to.be.equal(0);
+    expect(lvl.daysToComplete.toNumber()).to.be.equal(2);
+    expect(lvl.bounty.toNumber()).to.be.equal(10);
+  });
+
+  it("should update inviter level", async () => {
+    await invites
+      .connect(inviter1)
+      .join(ethers.utils.hexZeroPad("0xfa", 32), ethers.constants.HashZero)
+      .then(_ => _.wait())
+      .catch(e => e);
+
+    await id
+      .addWhitelistedWithDID(inviter1.address, Math.random() + "")
+      .catch(e => e);
+    await invites.setLevel(0, 1, 5, 1); //1 inviter to level up
+    await invites.setLevel(1, 0, 10, 2); // 10 bounty for second level
+
+    await invites
+      .connect(invitee4)
+      .join(
+        ethers.utils.hexZeroPad("0x03", 32),
+        ethers.utils.hexZeroPad("0xfa", 32)
+      );
+    await invites
+      .connect(invitee5)
+      .join(
+        ethers.utils.hexZeroPad("0x04", 32),
+        ethers.utils.hexZeroPad("0xfa", 32)
+      );
+    await id
+      .addWhitelistedWithDID(invitee4.address, Math.random() + "")
+      .catch(e => e);
+    await id
+      .addWhitelistedWithDID(invitee5.address, Math.random() + "")
+      .catch(e => e);
+    const res1 = await (await invites.bountyFor(invitee4.address)).wait();
+
+    const log1 = res1.events.find(_ => _.event === "InviterBounty");
+    expect(log1.event).to.be.equal("InviterBounty");
+    expect(log1.args.inviterLevel.toNumber()).to.be.equal(1);
+    expect(log1.args.earnedLevel).to.be.equal(true);
+    expect(log1.args.bountyPaid.toNumber()).to.be.equal(5);
+
+    let inviter = await invites.users(inviter1.address);
+    expect(inviter.level.toNumber()).to.be.equal(1);
+    const res2 = await (
+      await invites.connect(inviter1).collectBounties()
+    ).wait();
+    const log2 = res2.events.find(_ => _.event === "InviterBounty");
+    expect(log2.event).to.be.equal("InviterBounty");
+    expect(log2.args.inviterLevel.toNumber()).to.be.equal(1);
+    expect(log2.args.earnedLevel).to.be.equal(false);
+    expect(log2.args.bountyPaid.toNumber()).to.be.equal(10);
+  });
+
+  it("should allow to set inviter later and pay bounty", async () => {
+    await invites
+      .connect(invitee6)
+      .join(ethers.utils.hexZeroPad("0xfd", 32), ethers.constants.HashZero);
+    await invites
+      .connect(invitee6)
+      .join(
+        ethers.utils.hexZeroPad("0xfd", 32),
+        ethers.utils.hexZeroPad("0xfa", 32)
+      );
+    const invitee = await invites.users(invitee6.address);
+    expect(invitee.invitedBy).to.equal(inviter1.address);
+    await id
+      .addWhitelistedWithDID(invitee6.address, Math.random() + "")
+      .catch(e => e);
+    await expect(invites.bountyFor(invitee6.address)).to.emit(
+      invites,
+      "InviterBounty"
+    );
+  });
+
+  it("should end contract by owner", async () => {
+    expect(
+      await gd.balanceOf(invites.address).then(_ => _.toNumber())
+    ).to.be.gt(0);
+    await invites.end();
+    expect(
+      await gd.balanceOf(invites.address).then(_ => _.toNumber())
+    ).to.be.eq(0);
+  });
+});

package/test/reserve/DistributionHelper.test.ts

@@ -1,4 +1,5 @@
 import { ethers, waffle, upgrades } from "hardhat";
+import { loadFixture } from "@nomicfoundation/hardhat-network-helpers";
 import { expect } from "chai";
 import {
   GoodReserveCDai,
@@ -40,7 +41,7 @@ describe("DistributionHelper", () => {
       cdaiAddress,
       genericCall: gc,
       nameService: ns
-    } = await createDAO();
+    } = await loadFixture(createDAO);
 
     nameService = ns;
     genericCall = gc;

package/test/reserve/GoodMarketMaker.test.ts

@@ -1,4 +1,5 @@
 import { default as hre, ethers, upgrades } from "hardhat";
+import { loadFixture } from "@nomicfoundation/hardhat-network-helpers";
 import { deployMockContract, MockContract } from "ethereum-waffle";
 import { expect } from "chai";
 import { GoodMarketMaker, CERC20 } from "../../types";
@@ -42,8 +43,6 @@ describe("GoodMarketMaker - calculate gd value at reserve", () => {
 
   before(async () => {
     [founder, staker, fakeReserve, ...signers] = await ethers.getSigners();
-    cdai = await deploycDAIMock();
-    dai = await deployDAIMock();
 
     let {
       controller: ctrl,
@@ -56,7 +55,7 @@ describe("GoodMarketMaker - calculate gd value at reserve", () => {
       setSchemes,
       setReserveToken,
       setDAOAddress
-    } = await createDAO();
+    } = await loadFixture(createDAO);
     avatar = av;
     marketMaker = mm.connect(fakeReserve);
     controller = ctrl;
@@ -64,9 +63,12 @@ describe("GoodMarketMaker - calculate gd value at reserve", () => {
 
     console.log("deployed dao", { goodDollar, identity, controller, avatar });
 
+    cdai = await deploycDAIMock();
+    dai = await deployDAIMock();
+
     //give founder generic call permission
     await setSchemes([founder.address]);
-    setDAOAddress("RESERVE", fakeReserve.address);
+    await setDAOAddress("RESERVE", fakeReserve.address);
     console.log("starting tests...");
   });
 
@@ -600,12 +602,17 @@ describe("GoodMarketMaker - calculate gd value at reserve", () => {
 
   it("should not set reserve ratio daily expansion with illigal values", async () => {
     const invalidZeroDenominator = 0;
-    await expect(marketMaker.setReserveRatioDailyExpansion(1, invalidZeroDenominator)).to.be.
-      revertedWith("denominator must be above 0");
+    await expect(
+      marketMaker.setReserveRatioDailyExpansion(1, invalidZeroDenominator)
+    ).to.be.revertedWith("denominator must be above 0");
 
     const denominator = 1;
     const nominatorHigherThanDenom = 2;
-    await expect(marketMaker.setReserveRatioDailyExpansion(nominatorHigherThanDenom, denominator)).to.be.
-      revertedWith("Invalid nom or denom value");
+    await expect(
+      marketMaker.setReserveRatioDailyExpansion(
+        nominatorHigherThanDenom,
+        denominator
+      )
+    ).to.be.revertedWith("Invalid nom or denom value");
   });
 });

package/test/reserve/GoodReserveCDai.cap.test.ts

@@ -1,4 +1,5 @@
 import { default as hre, ethers, upgrades } from "hardhat";
+import { loadFixture } from "@nomicfoundation/hardhat-network-helpers";
 import { expect } from "chai";
 import { GoodMarketMaker, CERC20, GoodReserveCDai } from "../../types";
 import { createDAO, increaseTime, advanceBlocks } from "../helpers";
@@ -36,7 +37,7 @@ describe("GoodReserve - Enforce token cap", () => {
       marketMaker: mm,
       reserve,
       cdaiAddress
-    } = await createDAO();
+    } = await loadFixture(createDAO);
 
     cDai = cdaiAddress;
     avatar = av;
@@ -111,6 +112,12 @@ describe("GoodReserve - Enforce token cap", () => {
       .connect(founder)
       .mintRewardFromRR(cDai, founder.address, 10);
     expect(await goodDollar.balanceOf(founder.address)).to.equal(1020);
+
+    encodedCall = goodDollar.interface.encodeFunctionData("addMinter", [
+      avatar
+    ]);
+
+    await ictrl.genericCall(goodDollar.address, encodedCall, avatar, 0);
   });
 
   it("should not be able to mint if not core contract and GoodReserve is minter", async () => {

package/test/reserve/GoodReserveCDai.distribution.test.ts

@@ -1,4 +1,5 @@
 import { ethers, waffle } from "hardhat";
+import { loadFixture } from "@nomicfoundation/hardhat-network-helpers";
 import { expect } from "chai";
 import { GoodReserveCDai, DistributionHelper } from "../../types";
 import { createDAO, increaseTime } from "../helpers";
@@ -34,7 +35,7 @@ describe("GoodReserve - Distribution Helper", () => {
       cdaiAddress,
       genericCall: gc,
       nameService: ns
-    } = await createDAO();
+    } = await loadFixture(createDAO);
 
     nameService = ns;
     genericCall = gc;

package/test/reserve/GoodReserveCDai.gdx.test.ts

@@ -1,4 +1,5 @@
 import { default as hre, ethers, upgrades } from "hardhat";
+import { loadFixture } from "@nomicfoundation/hardhat-network-helpers";
 import { BigNumber, Signer } from "ethers";
 import { deployMockContract, MockContract } from "ethereum-waffle";
 import { expect } from "chai";
@@ -50,7 +51,7 @@ describe("GDX Token", () => {
       cdaiAddress,
       genericCall: gn,
       runAsAvatarOnly: raao
-    } = await createDAO();
+    } = await loadFixture(createDAO);
 
     dai = await ethers.getContractAt("DAIMock", daiAddress);
     cDAI = await ethers.getContractAt("cDAIMock", cdaiAddress);

package/test/reserve/GoodReserveCDai.pause.test.ts

@@ -1,4 +1,5 @@
 import { ethers, waffle } from "hardhat";
+import { loadFixture } from "@nomicfoundation/hardhat-network-helpers";
 import { expect } from "chai";
 import { GoodReserveCDai, DistributionHelper } from "../../types";
 import { createDAO, increaseTime } from "../helpers";
@@ -34,7 +35,7 @@ describe("GoodReserve - Distribution Helper", () => {
       daiAddress,
       cdaiAddress,
       genericCall: gc
-    } = await createDAO();
+    } = await loadFixture(createDAO);
 
     dai = await ethers.getContractAt("DAIMock", daiAddress);
     cDAI = await ethers.getContractAt("cDAIMock", cdaiAddress);

package/test/reserve/GoodReserveCDai.test.ts

@@ -1,4 +1,5 @@
 import { default as hre, ethers, upgrades } from "hardhat";
+import { loadFixture } from "@nomicfoundation/hardhat-network-helpers";
 import { BigNumber, Contract, Signer } from "ethers";
 import { deployMockContract, MockContract } from "ethereum-waffle";
 import { expect } from "chai";
@@ -48,8 +49,8 @@ describe("GoodReserve - staking with cDAI mocks", () => {
       daiAddress,
       cdaiAddress,
       reserve,
-      runAsAvatarOnly: raao,
-    } = await createDAO();
+      runAsAvatarOnly: raao
+    } = await loadFixture(createDAO);
 
     dai = await ethers.getContractAt("DAIMock", daiAddress);
     cDAI = await ethers.getContractAt("cDAIMock", cdaiAddress);
@@ -86,7 +87,6 @@ describe("GoodReserve - staking with cDAI mocks", () => {
 
     marketMaker = mm;
 
-    const reserveFactory = await ethers.getContractFactory("GoodReserveCDai");
     console.log("deployed contribution, deploying reserve...", {
       founder: founder.address
     });
@@ -117,26 +117,8 @@ describe("GoodReserve - staking with cDAI mocks", () => {
 
   it("should set fundManager in the reserve by avatar", async () => {
     await setDAOAddress("FUND_MANAGER", founder.address);
-    // const rFactory = await ethers.getContractFactory("GoodReserveCDai");
-    // const ctrl = await ethers.getContractAt(
-    //   "Controller",
-    //   controller,
-    //   schemeMock
-    // );
-    // const encodedCall = rFactory.interface.encodeFunctionData(
-    //   "setFundManager",
-    //   [founder.address]
-    // );
-    // await ctrl.genericCall(goodReserve.address, encodedCall, avatar, 0);
-    // const newFM = await goodReserve.fundManager();
-    // expect(newFM.toString()).to.be.equal(founder.address);
   });
 
-  // it("should returned true for isActive", async () => {
-  //   const isActive = await goodReserve.isActive();
-  //   expect(isActive.toString()).to.be.equal("true");
-  // });
-
   it("should returned fixed 0.0001 market price", async () => {
     const gdPrice = await goodReserve["currentPrice()"]();
     const cdaiWorthInGD = gdPrice.mul(BN.from("100000000"));
@@ -1139,22 +1121,23 @@ describe("GoodReserve - staking with cDAI mocks", () => {
 
     await runAsAvatarOnly(
       goodReserve,
-      "setReserveRatioDailyExpansion(uint256,uint256)", 
-      1, 
+      "setReserveRatioDailyExpansion(uint256,uint256)",
+      1,
       1e15
     );
 
     let newReserveRatioDailyExpansion =
       await marketMaker.reserveRatioDailyExpansion();
     console.log(newReserveRatioDailyExpansion.toString());
-    
-    expect(newReserveRatioDailyExpansion).to.not.equal(currentReserveRatioDailyExpansion);
+
+    expect(newReserveRatioDailyExpansion).to.not.equal(
+      currentReserveRatioDailyExpansion
+    );
     expect(newReserveRatioDailyExpansion).to.be.equal(BN.from("1000000000000"));
 
     const encodedCall = goodReserve.interface.encodeFunctionData(
-      "setReserveRatioDailyExpansion", 
-      [BN.from(currentReserveRatioDailyExpansion).div(1e12),
-      1e15]
+      "setReserveRatioDailyExpansion",
+      [BN.from(currentReserveRatioDailyExpansion).div(1e12), 1e15]
     );
     const ctrl = await ethers.getContractAt(
       "Controller",

package/test/reserve/GoodReserveCDai.uniswap.test.ts

@@ -1,4 +1,5 @@
 import { ethers, upgrades } from "hardhat";
+import { loadFixture } from "@nomicfoundation/hardhat-network-helpers";
 import { BigNumber, Contract } from "ethers";
 import { expect } from "chai";
 import { GoodMarketMaker, GoodReserveCDai } from "../../types";
@@ -43,6 +44,24 @@ describe("GoodReserve - buy/sell with any token through uniswap", () => {
   before(async () => {
     [founder, staker, ...signers] = await ethers.getSigners();
     schemeMock = signers.pop();
+
+    let {
+      controller: ctrl,
+      avatar: av,
+      gd,
+      identity,
+      daoCreator,
+      nameService: ns,
+      setDAOAddress: sda,
+      setSchemes,
+      marketMaker: mm,
+      daiAddress,
+      cdaiAddress,
+      reserve,
+      setReserveToken,
+      COMP
+    } = await loadFixture(createDAO);
+
     const cdaiFactory = await ethers.getContractFactory("cDAIMock");
     const daiFactory = await ethers.getContractFactory("DAIMock");
     const routerFactory = new ethers.ContractFactory(
@@ -69,23 +88,6 @@ describe("GoodReserve - buy/sell with any token through uniswap", () => {
 
     tokenB = await daiFactory.deploy(); // another erc20 for uniswap router test
 
-    let {
-      controller: ctrl,
-      avatar: av,
-      gd,
-      identity,
-      daoCreator,
-      nameService: ns,
-      setDAOAddress: sda,
-      setSchemes,
-      marketMaker: mm,
-      daiAddress,
-      cdaiAddress,
-      reserve,
-      setReserveToken,
-      COMP
-    } = await createDAO();
-
     dai = await ethers.getContractAt("DAIMock", daiAddress);
     cDAI = await ethers.getContractAt("cDAIMock", cdaiAddress);
     comp = COMP;
@@ -118,7 +120,7 @@ describe("GoodReserve - buy/sell with any token through uniswap", () => {
     });
     goodReserve = reserve;
 
-    setDAOAddress("UNISWAP_ROUTER", uniswapRouter.address);
+    await setDAOAddress("UNISWAP_ROUTER", uniswapRouter.address);
     const exchangeHelperFactory = await ethers.getContractFactory(
       "ExchangeHelper"
     );

package/test/staking/CompoundStakingFactory.test.ts

@@ -1,4 +1,5 @@
 import { default as hre, ethers, upgrades } from "hardhat";
+import { loadFixture } from "@nomicfoundation/hardhat-network-helpers";
 import { deployMockContract, MockContract } from "ethereum-waffle";
 import { expect } from "chai";
 import {
@@ -27,7 +28,7 @@ describe("CompoundStakingFactory", () => {
 
   before(async () => {
     [founder, ...signers] = await ethers.getSigners();
-    dao = await createDAO();
+    dao = await loadFixture(createDAO);
 
     const compUsdOracleFactory = await ethers.getContractFactory(
       "CompUSDMockOracle"
@@ -128,13 +129,16 @@ describe("CompoundStakingFactory", () => {
 
     const INITIAL_COLLECT_INTEREST_GAS_COST = 250000;
     const INITIAL_COLLECT_COMP_GAS_COST = 150000;
-    const gasCostForInterestTransfer = await goodCompoundStakingV2.getGasCostForInterestTransfer();
-    expect(gasCostForInterestTransfer).to.equal(INITIAL_COLLECT_INTEREST_GAS_COST);
+    const gasCostForInterestTransfer =
+      await goodCompoundStakingV2.getGasCostForInterestTransfer();
+    expect(gasCostForInterestTransfer).to.equal(
+      INITIAL_COLLECT_INTEREST_GAS_COST
+    );
 
     const settings = await goodCompoundStakingV2.getSettings();
     const collectInteresetGasCost = settings[0];
     const compCollectGasCost = settings[1];
     expect(collectInteresetGasCost).to.equal(INITIAL_COLLECT_INTEREST_GAS_COST);
     expect(compCollectGasCost).to.equal(INITIAL_COLLECT_COMP_GAS_COST);
-    });
+  });
 });