Contract event listener is not firing for indexed value and for multiple events

[6be709c0]

Version of Hardhat

2.16.1

What happened?

Update of #4044. After the fix, new issues are:

• Doesn't trigger when you have an indexed value
• If you trigger multiple events, the first one is emitted.

Contact

contract ContractA {
    event EventNumber(uint amount);
    event EventAddress(address player);
    event EventAddressIndexed(address indexed player);

    function testWithNumber(uint amount) public {
        emit EventNumber(amount);
    }

    function testWithEventAddress() public {
        emit EventAddress(msg.sender);
    }

    function testWithEventAddressIndexed() public {
        emit EventAddressIndexed(msg.sender);
    }

    function testWithMultipleEvents() public {
        emit EventNumber(1337);
        emit EventNumber(5);
        emit EventAddress(msg.sender);
    }
}

Using the following test:

await new Promise(async (resolve) => {
      contractA.on("EventNumber", (amount: BigNumber) => {
        console.log(`- Received events testWithNumber with amount: ${amount}`);
      });
      contractA.on("EventAddress", (address) => {
        console.log(
          `- Received events testWithEventAddress with value: ${address}`
        );
      });
      contractA.on("EventAddressIndexed", (address) => {
        console.log(
          `- Received events testWithEventAddressIndexed with value: ${address}`
        );
      });

      console.log("\n## Triggering test event with a number...");
      await contractA.testWithNumber(123);
      await wait(1000);
      console.log("\n## Triggering test event with an address...");
      await contractA.testWithEventAddress();
      await wait(1000);
      console.log("\n## Triggering test event with an address indexed...");
      await contractA.testWithEventAddressIndexed();
      await wait(1000);
      console.log("\n## Triggering test with 2 events...");
      await contractA.testWithMultipleEvents();
      await wait(1000);
    });

Actual behavior:

## Triggering test event with a number...
- Received events testWithNumber with amount: 123

## Triggering test event with an address...
- Received events testWithEventAddress with value: 0xf39Fd6e51aad88F6F4ce6aB8827279cffFb92266

## Triggering test event with an address indexed...

## Triggering test with 2 events...
- Received events testWithNumber with amount: 1337

Expected Behavior:

## Triggering test event with a number...
- Received events testWithNumber with amount: 123

## Triggering test event with an address...
- Received events testWithEventAddress with value: 0xf39Fd6e51aad88F6F4ce6aB8827279cffFb92266

## Triggering test event with an address indexed...
- Received events testWithEventAddressIndexed with value: 0xf39Fd6e51aad88F6F4ce6aB8827279cffFb92266

## Triggering test with 2 events...
- Received events testWithNumber with amount: 1337
- Received events testWithNumber with amount: 5
- Received events testWithEventAddress with value: 0xf39Fd6e51aad88F6F4ce6aB8827279cffFb92266

Minimal reproduction steps

git clone -b issue-2 https://github.com/lescaudr/ethers-event-issue

• pnpm install
• npx hardhat compile
• npx hardhat test

Search terms

No response

[fvictorio]

Hi @lescaudr, thanks for reporting this. I can confirm the bug is on our side, I'll work on it asap.

[fvictorio]

Fixed in the latest version of hardhat-ethers