fix(forge): do not revert if event with count 0 not emitted (#10534)

Author: grandizzy

crates/cheatcodes/src/inspector.rs

@@ -1656,8 +1656,9 @@ impl Inspector<&mut dyn DatabaseExt> for Cheatcodes {
                 })
                 .collect::<Vec<_>>();
 
-            // Not all emits were matched.
-            if self.expected_emits.iter().any(|(expected, _)| !expected.found) {
+            // Revert if not all emits expected were matched.
+            if self.expected_emits.iter().any(|(expected, _)| !expected.found && expected.count > 0)
+            {
                 outcome.result.result = InstructionResult::Revert;
                 outcome.result.output = "log != expected log".abi_encode().into();
                 return outcome;

crates/forge/tests/it/repros.rs

@@ -404,3 +404,6 @@ test_repro!(10302);
 
 // https://github.com/foundry-rs/foundry/issues/10477
 test_repro!(10477);
+
+// https://github.com/foundry-rs/foundry/issues/10527
+test_repro!(10527);

testdata/default/repros/Issue10527.t.sol

@@ -0,0 +1,42 @@
+// SPDX-License-Identifier: MIT OR Apache-2.0
+pragma solidity ^0.8.18;
+
+import "ds-test/test.sol";
+import "cheats/Vm.sol";
+
+contract A {
+    event Event1();
+    event Event2();
+
+    function foo() public {
+        emit Event1();
+    }
+
+    function bar() public {
+        emit Event2();
+    }
+}
+
+contract Issue10527Test is DSTest {
+    Vm constant vm = Vm(HEVM_ADDRESS);
+
+    A a;
+
+    function setUp() public {
+        a = new A();
+    }
+
+    function test_foo_Event1() public {
+        vm.expectEmit(address(a));
+        emit A.Event1();
+
+        a.foo();
+    }
+
+    function test_foo_Event2() public {
+        vm.expectEmit({emitter: address(a), count: 0});
+        emit A.Event2();
+
+        a.foo();
+    }
+}