lesson 7 : too many arguments: passed to contract (count=1, expectedCount=0, code=UNEXPECTED_ARGUMENT, version=contracts/5.7.0)

[tiwariji99]

i am at 12:10:00 but after typing yarn hardhat test in terminal i am getting error :

yarn run v1.22.19
$ C:\Users\HP\hardhat-fundme-fcc\node_modules\.bin\hardhat test

  Fundme
    constructor
      √ sets the aggregator address correctly
    fund
      √ fails if you dont send enough ETH
      √ update the amount funded data structure

      1) Adds funder to array of getpriceFeed
    withdraw
      √ withdraw ETH from a single funder

      2) allows us to withdraw with multiple getpriceFeed
      √ only allow the owner to withdraw

      3) cheaperwithdraw testing...


  5 passing (897ms)
  3 failing

  1) Fundme
       fund
         Adds funder to array of getpriceFeed:
     Error: too many arguments: passed to contract (count=1, expectedCount=0, code=UNEXPECTED_ARGUMENT, version=contracts/5.7.0)
      at Logger.makeError (node_modules\@ethersproject\logger\src.ts\index.ts:269:28)
      at Logger.throwError (node_modules\@ethersproject\logger\src.ts\index.ts:281:20)        
      at Logger.checkArgumentCount (node_modules\@ethersproject\logger\src.ts\index.ts:347:18)
      at C:\Users\HP\hardhat-fundme-fcc\node_modules\@ethersproject\contracts\src.ts\index.ts:187:12
      at step (node_modules\@ethersproject\contracts\lib\index.js:48:23)
      at Object.next (node_modules\@ethersproject\contracts\lib\index.js:29:53)
      at C:\Users\HP\hardhat-fundme-fcc\node_modules\@ethersproject\contracts\lib\index.js:23:71
      at new Promise (<anonymous>)
      at __awaiter (node_modules\@ethersproject\contracts\lib\index.js:19:12)
      at populateTransaction (node_modules\@ethersproject\contracts\lib\index.js:152:12)      

  2) Fundme
       withdraw
         allows us to withdraw with multiple getpriceFeed:
     Error: too many arguments: passed to contract (count=1, expectedCount=0, code=UNEXPECTED_ARGUMENT, version=contracts/5.7.0)
      at Logger.makeError (node_modules\@ethersproject\logger\src.ts\index.ts:269:28)
      at Logger.throwError (node_modules\@ethersproject\logger\src.ts\index.ts:281:20)        
      at Logger.checkArgumentCount (node_modules\@ethersproject\logger\src.ts\index.ts:347:18)
      at C:\Users\HP\hardhat-fundme-fcc\node_modules\@ethersproject\contracts\src.ts\index.ts:187:12
      at step (node_modules\@ethersproject\contracts\lib\index.js:48:23)
      at Object.next (node_modules\@ethersproject\contracts\lib\index.js:29:53)
      at C:\Users\HP\hardhat-fundme-fcc\node_modules\@ethersproject\contracts\lib\index.js:23:71
      at new Promise (<anonymous>)
      at __awaiter (node_modules\@ethersproject\contracts\lib\index.js:19:12)
      at populateTransaction (node_modules\@ethersproject\contracts\lib\index.js:152:12)      

  3) Fundme
       withdraw
         cheaperwithdraw testing...:
     Error: too many arguments: passed to contract (count=1, expectedCount=0, code=UNEXPECTED_ARGUMENT, version=contracts/5.7.0)
      at Logger.makeError (node_modules\@ethersproject\logger\src.ts\index.ts:269:28)
      at Logger.throwError (node_modules\@ethersproject\logger\src.ts\index.ts:281:20)        
      at Logger.checkArgumentCount (node_modules\@ethersproject\logger\src.ts\index.ts:347:18)
      at C:\Users\HP\hardhat-fundme-fcc\node_modules\@ethersproject\contracts\src.ts\index.ts:187:12
      at step (node_modules\@ethersproject\contracts\lib\index.js:48:23)
      at Object.next (node_modules\@ethersproject\contracts\lib\index.js:29:53)
      at C:\Users\HP\hardhat-fundme-fcc\node_modules\@ethersproject\contracts\lib\index.js:23:71
      at new Promise (<anonymous>)
      at __awaiter (node_modules\@ethersproject\contracts\lib\index.js:19:12)
      at populateTransaction (node_modules\@ethersproject\contracts\lib\index.js:152:12)      



error Command failed with exit code 3.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

fundme.test.js

const {deployments,ethers,getNamedAccounts}=require("hardhat")
const {assert,expect}=require("chai")
    
//import("hardhat/console.sol") ths method can also be used to debug the file using console.log
 
describe("Fundme",function (){
    let fundme
    let deployer
    let mockV3Aggregator
    const sendvalue=ethers.utils.parseEther("1")//1eth
    beforeEach(async ()=> {
        //deploy our fundme contract using hardhat-deploy
        // const accounts=await ethers.getSigners()
        // const accountzero=accounts[0]
        deployer=(await getNamedAccounts()).deployer
        await deployments.fixture(["all"])
        fundme=await ethers.getContract("Fundme",deployer)
        mockV3Aggregator=await ethers.getContract("MockV3Aggregator",deployer)
    })

    describe("constructor", function (){
        
        it("sets the aggregator address correctly",async ()=> {
           const response=await fundme.getpriceFeed()
           assert.equal(response,mockV3Aggregator.address)
        })
    })
    describe("fund", function(){
        it("fails if you dont send enough ETH",async ()=> {
           await expect(fundme.fund()).to.be.revertedWith(
            "you need to spend more ETH!"
           )
        })
        it("update the amount funded data structure",async ()=>{
            await fundme.fund({value:sendvalue})
            const response=await fundme.getaddresstoamountfunded(deployer)
            assert.equal(response.toString(),sendvalue.toString())
        })
        it("Adds funder to array of getpriceFeed",async ()=>{
            await fundme.fund({value:sendvalue})
            const response=await fundme.getpriceFeed(0)
            assert.equal(response,deployer)
        })
    })
    describe("withdraw",function (){
       beforeEach(async ()=>{
        await fundme.fund({value:sendvalue})
       })
       it("withdraw ETH from a single funder",async ()=>{
        //arrange

        const startingfundmebalance=
             await fundme.provider.getBalance(fundme.address)
        const startingdeployerbalance=
            await fundme.provider.getBalance(deployer)

        //act
        const transactionResponse=await fundme.withdraw()
        const transactionReceipt=await transactionResponse.wait(1)
        const {gasUsed,effectiveGasPrice}=transactionReceipt
        const gascost=gasUsed.mul(effectiveGasPrice)


        const endingfundmebalance=
        await fundme.provider.getBalance(fundme.address)
        const endingdeployerbalance=
        await fundme.provider.getBalance(deployer)

        //assert
        assert.equal(endingfundmebalance,0)
        assert.equal(startingfundmebalance.add(startingdeployerbalance).toString(),
        endingdeployerbalance.add(gascost).toString())
        })
        it("allows us to withdraw with multiple getpriceFeed",async()=>{
            //arrange
            
            const accounts=await ethers.getSigners()
            for(let i=1;i<6;i++){
                const fundmeconnectedcontract=await fundme.connect(
                    accounts[i]
                )
                await fundmeconnectedcontract.fund({value:sendvalue})    
            }
            const startingfundmebalance=
             await fundme.provider.getBalance(fundme.address)
            const startingdeployerbalance=
             await fundme.provider.getBalance(deployer)
             
             //act
             
             const transactionResponse=await fundme.withdraw()
             const transactionReceipt=await transactionResponse.wait(1)
             const {gasUsed,effectiveGasPrice}=transactionReceipt
             const gascost=gasUsed.mul(effectiveGasPrice)

             const endingfundmebalance=
             await fundme.provider.getBalance(fundme.address)
             const endingdeployerbalance=
             await fundme.provider.getBalance(deployer)

             //assert
             
             assert.equal(endingfundmebalance,0)
             assert.equal(startingfundmebalance.add(startingdeployerbalance).toString(),
             endingdeployerbalance.add(gascost).toString()
             )
             //make sure that getpriceFeed array are reset properly
             await expect(fundme.getpriceFeed(0)).to.be.reverted
             for(i=1;i<6;i++){
                assert.equal(
                    await fundme.getaddresstoamountfunded(accounts[i].address),
                    0
                )
             }
        })
        it("only allow the owner to withdraw",async()=>{
            const accounts= await ethers.getSigners()
            const attacker=accounts[1]
            const attackerconnectedcontract=await fundme.connect(attacker)
            await expect(attackerconnectedcontract.withdraw()).to.be.revertedWithCustomError(
                fundme,"Fundme_notowner"
            )
        })
        it("cheaperwithdraw testing...",async()=>{
            //arrange
            
            const accounts=await ethers.getSigners()
            for(let i=1;i<6;i++){
                const fundmeconnectedcontract=await fundme.connect(
                    accounts[i]
                )
                await fundmeconnectedcontract.fund({value:sendvalue})    
            }
            const startingfundmebalance=
             await fundme.provider.getBalance(fundme.address)
            const startingdeployerbalance=
             await fundme.provider.getBalance(deployer)
             
             //act
             
             const transactionResponse=await fundme.cheaperwithdraw()
             const transactionReceipt=await transactionResponse.wait(1)
             const {gasUsed,effectiveGasPrice}=transactionReceipt
             const gascost=gasUsed.mul(effectiveGasPrice)

             const endingfundmebalance=
             await fundme.provider.getBalance(fundme.address)
             const endingdeployerbalance=
             await fundme.provider.getBalance(deployer)

             //assert
             
             assert.equal(endingfundmebalance,0)
             assert.equal(startingfundmebalance.add(startingdeployerbalance).toString(),
             endingdeployerbalance.add(gascost).toString()
             )
             //make sure that getpriceFeed array are reset properly
             await expect(fundme.getpriceFeed(0)).to.be.reverted
             for(i=1;i<6;i++){
                assert.equal(
                    await fundme.getaddresstoamountfunded(accounts[i].address),
                    0
                )
             }
        })
        
    })
})

fundme.sol

//SPDX-License-Identifier: MIT
//pragma
 pragma solidity ^0.8.8;
 //import
import "./priceconvertor.sol";
//error codes
 error Fundme_notowner();
 //interfaces,library,contract

/**
 @title a contract for crowd funding 
 @author tiwariji 
 @notice this contract is to demo a sample funding contract
 @dev this implements price feeds as our library 
 */

contract Fundme{
    //type declarations
    using priceconvertor for uint256;
     // constant and immutable are better way to save gas 
    uint256 public constant MINIMUM_USD=50* 10**18;

//  uint256 public number;
// ask about error on stack exchange ETH
                    
       address[] private s_funders;
//state variables
mapping(address=> uint256) private  s_addresstoamountfunded;

address private immutable  i_owner;
AggregatorV3Interface private s_priceFeed;

//modifier
modifier onlyowner{
      //  require(msg.sender == i_owner ,"sender is not owner");
      if(msg.sender!=i_owner){revert Fundme_notowner();}
        _;
}
//FUnction Order 
//constructor
//receive
//fallback
//external
//public
//internal
//private
//view/pure


        constructor(address s_priceFeedAddress){
            i_owner=msg.sender;
            s_priceFeed=AggregatorV3Interface(s_priceFeedAddress);

        }
      receive() external payable {
        fund();
      }
      fallback() external payable {
        fund();
      }
/**
 @notice this function funds this contract
 @dev this implements price feed as our library
 */
      function fund()public payable {
      require(
        msg.value.getconversionrate(s_priceFeed)>=MINIMUM_USD,
        "you need to spend more ETH!"
      );
        
      s_addresstoamountfunded[msg.sender]+=msg.value;
      s_funders.push(msg.sender);
        
          //reverting returns the total gas spent when particular amount of transaction does not meet the conditions
          
        //   number=5;
        // msg.value.getconversionrate();
         
          // msg.value has 18 decimal places 


          // getting an input from the real world is a genuine concern and to do thiswe also need to take care that there
          //sholud not be a single data provider we are provided with
          // chainlink which is a decentralized oracle network which help us to build hybrid smart contracts 
          //which is also able to extract data from the real world
          
      }
      

      function withdraw() public onlyowner{
        //    require(msg.sender==owner,"sender is not owner!");

          for(uint256 funderindex=0;funderindex<s_funders.length;funderindex++){
              address funder= s_funders[funderindex];
              s_addresstoamountfunded[funder]=0;
          }
          //reset the array 
          s_funders = new address[](0);
          //transfer 
          //msg.sender is of type address

         payable( msg.sender).transfer(address(this).balance);
         //send 
         bool sendsuccess= payable(msg.sender).send(address(this).balance);
         require(sendsuccess ,"send failed");
         //call
         (bool callsuccess ,)=payable(msg.sender).call{value : address(this).balance}("");
         require(callsuccess,"call failed");
         
          
      }
      function cheaperwithdraw() public payable onlyowner{
          address[] memory funders=s_funders;
          for(
            uint256 funderindex=0; 
            funderindex<funders.length; 
            funderindex++){
             
            address funder=funders[funderindex];
            s_addresstoamountfunded[funder]=0;

          }
          s_funders=new address[](0);
          (bool success,)=i_owner.call{value:address(this).balance}("");
          require(success);
      }

      function getfunder(uint256 index) public view returns(address){
        return s_funders[index];
      }
      function getowner()public view returns(address){
        return i_owner;
      }
      function getaddresstoamountfunded(address funder) public view returns(uint256){
        return s_addresstoamountfunded[funder];
      }
      function getpriceFeed() public view returns (AggregatorV3Interface){
        return s_priceFeed;
      }

   //modifier (could be used when several function need some change)
   //
      //receive and fallback basically takes large amount of gas
      
    
}
   //name of constant is just a pointer to data pointed from it
   //constant and immutable variable does not take part in storage 
   //dynamic variables do take some spaces of any storage box but its length is not confirmed  
   //global variable do take spot in storage
   //mapping only creates a gap in storage
   //practise to create a function which gives several outputs as memory locations
  // gas prices are calculated on basis of opcodes written inside 

[alymurtazamemon]

@tiwariji99 Please leave your repository link.