From 553801661b64f5ff48011ffa524c8684f7face98 Mon Sep 17 00:00:00 2001
From: Andrey <SovereignAndrey>
Date: Fri, 21 Jul 2023 09:11:05 +0800
Subject: [PATCH] use hardhat helpers to impersonate Accounts Contract to send
 msgs

---
 test/core/IndexFund.ts | 19 ++++++++-----------
 1 file changed, 8 insertions(+), 11 deletions(-)

diff --git a/test/core/IndexFund.ts b/test/core/IndexFund.ts
index 6809700cb..e16340ff7 100644
--- a/test/core/IndexFund.ts
+++ b/test/core/IndexFund.ts
@@ -1,6 +1,7 @@
 import {expect} from "chai";
 import hre from "hardhat";
 import {SignerWithAddress} from "@nomiclabs/hardhat-ethers/signers";
+import {impersonateAccount, setBalance} from "@nomicfoundation/hardhat-network-helpers";
 import {
   AccountsDepositWithdrawEndowments,
   AccountsDepositWithdrawEndowments__factory,
@@ -317,16 +318,6 @@ describe("IndexFund", function () {
       await indexFund.createIndexFund("Test Fund #1", "Test fund", [2, 3], true, 50, 0);
       await indexFund.createIndexFund("Test Fund #2", "Test fund", [3], false, 50, 0);
 
-      // registrar config has the accounts contract set as owner (for ease of testing)
-      let {splitToLiquid, ...curConfig} = await registrar.queryConfig();
-      await registrar.updateConfig({
-        ...curConfig,
-        accountsContract: owner.address,
-        splitMax: 100,
-        splitMin: 0,
-        splitDefault: 50,
-        collectorShare: 0,
-      });
     });
 
     it("reverts when the message sender is not the accounts contract", async function () {
@@ -334,12 +325,18 @@ describe("IndexFund", function () {
     });
 
     it("passes with correct sender", async function () {
+      // create a signer for the Accounts contract to send msgs from
+      let regConfig = await registrar.queryConfig();
+      await impersonateAccount(regConfig.accountsContract);
+      await setBalance(regConfig.accountsContract, 1000000000000000000); // give it some gas money, as impersonateAccount does not
+      const acctSigner = await ethers.getSigner(regConfig.accountsContract);
+
       // Endowment #1 should be invloved with two funds
       let funds = await indexFund.queryInvolvedFunds(3);
       expect(funds.length).to.equal(2);
 
       // remove Endowment #1 from all funds
-      expect(await indexFund.connect(owner).removeMember(3)).to.emit(indexFund, "MemberRemoved");
+      expect(await indexFund.connect(acctSigner).removeMember(3)).to.emit(indexFund, "MemberRemoved");
 
       // Endowment #1 should not be invloved with any funds now
       funds = await indexFund.queryInvolvedFunds(3);