Revert "Treat InvokedCount like any other mock object expectation and…

… only verify it after the test method has finished"

This reverts commit ec59ed6.

src/Framework/MockObject/Rule/InvocationOrder.php

@@ -37,7 +37,13 @@ public function hasBeenInvoked(): bool
     final public function invoked(BaseInvocation $invocation): void
     {
         $this->invocations[] = $invocation;
+
+        $this->invokedDo($invocation);
     }
 
     abstract public function matches(BaseInvocation $invocation): bool;
+
+    protected function invokedDo(BaseInvocation $invocation): void
+    {
+    }
 }

src/Framework/MockObject/Rule/InvokedCount.php

@@ -61,4 +61,27 @@ public function verify(): void
             );
         }
     }
+
+    /**
+     * @throws ExpectationFailedException
+     */
+    protected function invokedDo(BaseInvocation $invocation): void
+    {
+        $count = $this->numberOfInvocations();
+
+        if ($count > $this->expectedCount) {
+            $message = $invocation->toString() . ' ';
+
+            $message .= match ($this->expectedCount) {
+                0       => 'was not expected to be called.',
+                1       => 'was not expected to be called more than once.',
+                default => sprintf(
+                    'was not expected to be called more than %d times.',
+                    $this->expectedCount,
+                ),
+            };
+
+            throw new ExpectationFailedException($message);
+        }
+    }
 }

tests/unit/Framework/MockObject/OldTestDoubleTestsTest.php

@@ -759,6 +759,58 @@ public function testVerificationOfMethodNameFailsWithWrongParameters(): void
         $this->resetMockObjects();
     }
 
+    public function testVerificationOfNeverFailsWithEmptyParameters(): void
+    {
+        $mock = $this->getMockBuilder(SomeClass::class)
+            ->addMethods(['right', 'wrong'])
+            ->getMock();
+
+        $mock->expects($this->never())
+            ->method('right')
+            ->with();
+
+        try {
+            $mock->right();
+            $this->fail('Expected exception');
+        } catch (ExpectationFailedException $e) {
+            $this->assertSame(
+                sprintf(
+                    '%s::right() was not expected to be called.',
+                    SomeClass::class,
+                ),
+                $e->getMessage(),
+            );
+        }
+
+        $this->resetMockObjects();
+    }
+
+    public function testVerificationOfNeverFailsWithAnyParameters(): void
+    {
+        $mock = $this->getMockBuilder(SomeClass::class)
+            ->addMethods(['right', 'wrong'])
+            ->getMock();
+
+        $mock->expects($this->never())
+            ->method('right')
+            ->withAnyParameters();
+
+        try {
+            $mock->right();
+            $this->fail('Expected exception');
+        } catch (ExpectationFailedException $e) {
+            $this->assertSame(
+                sprintf(
+                    '%s::right() was not expected to be called.',
+                    SomeClass::class,
+                ),
+                $e->getMessage(),
+            );
+        }
+
+        $this->resetMockObjects();
+    }
+
     public function testWithAnythingInsteadOfWithAnyParameters(): void
     {
         $mock = $this->getMockBuilder(SomeClass::class)