Error : Missing argument

[Benrockiee]

This statement checks if voters's weight property is 0 or not, if it does, set the weight to 1..

require(voters[voter].weight == 0);
    voters[voter].weight = 1;

so i wrote this test for it

  it("ensure the elibility of the voter", async function () {
    const currentVoterEligibility = await voting.giveRightToVote()
    const expectedEligibilityString = "1"
    assert.equal(
      expectedEligibilityString,
      toString(),
      expectedEligibilityString
    )
  })
})

but it looks like the error is coming from here as its looking for an argument, i already assigned 0 to it but didnt work..

await voting.giveRightToVote()

and this is the error from terminal

    ensure the elibility of the voter:
     Error: missing argument: passed to contract (count=0, expectedCount=1, code=MISSING_ARGUMENT, version=contracts/5.6.2)

[yogesh0509]

Can you post the complete contract?
Also
assert.equal( expectedEligibilityString, toString(), expectedEligibilityString )
should be
assert.equal( currentVoterEligibility.toString(), expectedEligibilityString )

[Benrockiee]

@yogesh0509 it needs an Argument

[Benrockiee]

Here is the contract

//SPDX-License-Identifier: MIT

pragma solidity ^0.8.7;

contract voting {
  struct Voter {
    uint256 vote;
    uint256 weight;
    bool voted;
    address delegate;
  }

  struct Proposal {
    bytes32 name;
    uint256 voteCount;
  }

  address public chairperson;

  mapping(address => Voter) public voters;

  Proposal[] public proposals;

  constructor(bytes32[] memory proposalNames) {
    chairperson = msg.sender;
    voters[chairperson].weight = 1;
    for (uint256 i = 0; i < proposalNames.length; i++) {
      proposals.push(Proposal({name: proposalNames[i], voteCount: 0}));
    }
  }

  function giveRightToVote(address voter) external {
    require(
      msg.sender == chairperson,
      "Only the chairperson has the right to assign voters"
    );
    require(voters[voter].voted, "User already voted");
    require(voters[voter].weight == 0);
    voters[voter].weight = 1;
  }

  function delegate(address to) external {
    Voter storage sender = voters[msg.sender];
    require(sender.weight != 0, "User is not eligible to vote");
    require(!sender.voted, "This user already voted");
    require(to != msg.sender, "Self delegation is not allowed");

    while (voters[to].delegate != address(0)) to = (voters[to].delegate);

    require(to != msg.sender, "found loop in delegation");

    Voter storage delegate_ = voters[to];
    require(delegate_.weight >= 1);

    sender.voted = true;
    sender.delegate = to;

    if (delegate_.voted) {
      proposals[delegate_.vote].voteCount += sender.weight;
    } else {
      delegate_.weight += sender.weight;
    }
  }

  function vote(uint256 proposal) external {
    Voter storage sender = voters[msg.sender];
    require(sender.weight != 0, "User is not eligible to vote");
    require(!sender.voted, "User already voted");

    sender.voted = true;
    sender.vote = proposal;
    proposals[proposal].voteCount += sender.weight;
  }

  function winningProposal() public view returns (uint256 winningProposal_) {
    uint256 winningVoteCount = 0;
    for (uint256 p = 0; p < proposals.length; p++) {
      if (proposals[p].voteCount > winningVoteCount) {
        winningVoteCount = proposals[p].voteCount;
        winningProposal_ = p;
      }
    }
  }

  function winnerName() external view returns (bytes32 winnerName_) {
    winnerName_ = proposals[winningProposal()].name;
  }
}

[yogesh0509]

It is throwing an error because the function giveRightToVote(address voter) is expecting an argument voter of address type whereas while calling the function const currentVoterEligibility = await voting.giveRightToVote() in your test, you are not giving any argument.

[Benrockiee]

@yogesh0509 So what argument should i give it?

[yogesh0509]

if you have used,
accounts = await ethers.getSigners() in your test earlier, just pluck any account from the accounts array, e.g. accounts[0].address as the argument

[Benrockiee]

No I havent used that..

[yogesh0509]

Then, use any account address

[Benrockiee]

can you write it out

[technophile-04]

  it("ensure the elibility of the voter", async function () {
    const accounts = await ethers.getSigners()
    const currentVoterEligibility = await voting.giveRightToVote(accounts[0].address)
    const expectedEligibilityString = "1"
    assert.equal(
      expectedEligibilityString.toString(),
      expectedEligibilityString
    )
  })
})

[Benrockiee]

@technophile-04 It looks like its been reverted by the previous test, this is the error it gives

 ensure the elibility of the voter and equals it to 1:
     Error: VM Exception while processing transaction: reverted with reason string 'User already voted'
    at voting.giveRightToVote (contracts/voting.sol:37)
    at HardhatNode._mineBlockWithPendingTxs (node_modules/hardhat/src/internal/hardhat-network/provider/node.ts:1773:23)
    at HardhatNode.mineBlock (node_modules/hardhat/src/internal/hardhat-network/provider/node.ts:466:16)
    at EthModule._sendTransactionAndReturnHash (node_modules/hardhat/src/internal/hardhat-network/provider/modules/eth.ts:1504:18)
    at HardhatNetworkProvider.request (node_modules/hardhat/src/internal/hardhat-network/provider/provider.ts:118:18)
    at EthersProviderWrapper.send (node_modules/@nomiclabs/hardhat-ethers/src/internal/ethers-provider-wrapper.ts:13:20)
  

[Benrockiee]

This is the previous test

  it("checks to see if users already voted", async function () {
    const checkVoter = "User already voted"
    assert.equal(checkVoter.toString(), checkVoter)
  })

[technophile-04]

Try this instead use accounts[2].address

const currentVoterEligibility = await voting.giveRightToVote(accounts[2].address)

[Benrockiee]

Its saying same error bro.. The last/previous test seems to be affecting it and i dont know why

[Benrockiee]

Its reverting