Lecture 7 hardhat fund me

[Atharv-02]

Why did I get the following error
fund me contract -

// SPDX-License-Identifier: MIT
// Pragma
pragma solidity ^0.8.8;
//Imports
import "@chainlink/contracts/src/v0.8/interfaces/AggregatorV3Interface.sol";
import "./PriceConverter.sol";
//Error Codes
error FundMe__NotOwner();

/// @title A contract for crowd funding
/// @author Atharv Gulati
/// @notice This contract is to demo a sample funding contract
/// @dev This impliments priceFeeds as a library
contract FundMe {
    //Type Declarations
    using PriceConverter for uint256;
    //State Variables
    mapping(address => uint256) public addressToAmountFunded;
    address[] public funders;

    // Could we make this constant?  /* hint: no! We should make it immutable! */
    address public /*immutable*/ i_owner;
    uint256 public constant MINIMUM_USD = 50 * 10**18;
    AggregatorV3Interface public priceFeed;

    //modifier
    modifier onlyOwner() {
        // require(msg.sender == owner);
        if (msg.sender != i_owner) revert FundMe__NotOwner();
        _;
    }

    //functions
    //constructor
    constructor(address priceFeedAdress) {
        i_owner = msg.sender;
        priceFeed = AggregatorV3Interface(priceFeedAdress);
    }

    //recieve
    // receive() external payable {
    //     fund();
    // }

    //fallback
    // fallback() external payable {
    //     fund();
    // }

    //public functions
    function fund() public payable {
        require(
            msg.value.getConversionRate(priceFeed) >= MINIMUM_USD,
            "You need to spend more ETH!"
        );
        // require(PriceConverter.getConversionRate(msg.value) >= MINIMUM_USD, "You need to spend more ETH!");
        addressToAmountFunded[msg.sender] += msg.value;
        funders.push(msg.sender);
    }

    function withdraw() public onlyOwner {
        for (
            uint256 funderIndex = 0;
            funderIndex < funders.length;
            funderIndex++
        ) {
            address funder = funders[funderIndex];
            addressToAmountFunded[funder] = 0;
        }
        funders = new address[](0);
        // // transfer
        // 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");
    }
}

FundMe.test.js-

const { assert, expect } = require('chai');
const { deployments, ethers, getNamedAccounts } = require('hardhat');

describe('FundMe', () => {
  let fundMe;
  let deployer;
  let mockV3Aggregator;
  beforeEach(async () => {
    //deploy our fund me contract
    //using hardhat deploy

    deployer = (await getNamedAccounts()).deployer;
    await deployments.fixture(['all']);
    fundMe = await ethers.getContract('FundMe', deployer);
    mockV3Aggregator = await ethers.getContract('MockV3Aggregator', deployer);
  });

  describe('constructor', () => {
    it('Sets the aggregator addresses correctly', async () => {
      const response = await fundMe.priceFeed();
      assert.equal(response, mockV3Aggregator.address);
    });
    it('Fails if you donot send enough ETH', async () => {
      await expect(fundMe.fund()).to.be.revertedWith(
        'You need to spend more ETH!'
      );
    });
  });
});

[alymurtazamemon]

@Atharv-02 Your code looks correct, I need to see other things, it would be better if you leave your repo link so I can also test it locally.

[Atharv-02]

https://github.com/Atharv-02/test

[Atharv-02]

Hi @alymurtazamemon
did you figure out what the issue was