test: replace addToAssertionCount checkbox tests with DoesNotPerformAssertions in Hooks and DI tests

Signed-off-by: Anna Larch <anna@nextcloud.com>
AI-Assisted-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: miaulalala

tests/lib/AppFramework/Bootstrap/FunctionInjectorTest.php

@@ -34,12 +34,10 @@ public function testInjectFnNotRegistered(): void {
 		});
 	}
 
+	#[\PHPUnit\Framework\Attributes\DoesNotPerformAssertions]
 	public function testInjectFnNotRegisteredButNullable(): void {
 		(new FunctionInjector($this->container))->injectFn(static function (?Foo $p1): void {
 		});
-
-		// Nothing to assert. No errors means everything is fine.
-		$this->addToAssertionCount(1);
 	}
 
 	public function testInjectFnByType(): void {
@@ -51,18 +49,13 @@ public function testInjectFnByType(): void {
 
 		(new FunctionInjector($this->container))->injectFn(static function (Foo $p1): void {
 		});
-
-		// Nothing to assert. No errors means everything is fine.
-		$this->addToAssertionCount(1);
 	}
 
+	#[\PHPUnit\Framework\Attributes\DoesNotPerformAssertions]
 	public function testInjectFnByName(): void {
 		$this->container->registerParameter('test', 'abc');
 
 		(new FunctionInjector($this->container))->injectFn(static function ($test): void {
 		});
-
-		// Nothing to assert. No errors means everything is fine.
-		$this->addToAssertionCount(1);
 	}
 }

tests/lib/Hooks/BasicEmitterTest.php

@@ -148,17 +148,17 @@ public function testNamedArguments(): void {
 		$this->emitter->emitEvent('Test', 'test', ['foo' => 'foo', 'bar' => 'bar']);
 	}
 
+	#[\PHPUnit\Framework\Attributes\DoesNotPerformAssertions]
 	public function testRemoveAllSpecified(): void {
 		$listener = function (): void {
 			throw new EmittedException;
 		};
 		$this->emitter->listen('Test', 'test', $listener);
 		$this->emitter->removeListener('Test', 'test', $listener);
 		$this->emitter->emitEvent('Test', 'test');
-
-		$this->addToAssertionCount(1);
 	}
 
+	#[\PHPUnit\Framework\Attributes\DoesNotPerformAssertions]
 	public function testRemoveWildcardListener(): void {
 		$listener1 = function (): void {
 			throw new EmittedException;
@@ -170,10 +170,9 @@ public function testRemoveWildcardListener(): void {
 		$this->emitter->listen('Test', 'test', $listener2);
 		$this->emitter->removeListener('Test', 'test');
 		$this->emitter->emitEvent('Test', 'test');
-
-		$this->addToAssertionCount(1);
 	}
 
+	#[\PHPUnit\Framework\Attributes\DoesNotPerformAssertions]
 	public function testRemoveWildcardMethod(): void {
 		$listener = function (): void {
 			throw new EmittedException;
@@ -183,10 +182,9 @@ public function testRemoveWildcardMethod(): void {
 		$this->emitter->removeListener('Test', null, $listener);
 		$this->emitter->emitEvent('Test', 'test');
 		$this->emitter->emitEvent('Test', 'foo');
-
-		$this->addToAssertionCount(1);
 	}
 
+	#[\PHPUnit\Framework\Attributes\DoesNotPerformAssertions]
 	public function testRemoveWildcardScope(): void {
 		$listener = function (): void {
 			throw new EmittedException;
@@ -196,10 +194,9 @@ public function testRemoveWildcardScope(): void {
 		$this->emitter->removeListener(null, 'test', $listener);
 		$this->emitter->emitEvent('Test', 'test');
 		$this->emitter->emitEvent('Bar', 'test');
-
-		$this->addToAssertionCount(1);
 	}
 
+	#[\PHPUnit\Framework\Attributes\DoesNotPerformAssertions]
 	public function testRemoveWildcardScopeAndMethod(): void {
 		$listener = function (): void {
 			throw new EmittedException;
@@ -211,8 +208,6 @@ public function testRemoveWildcardScopeAndMethod(): void {
 		$this->emitter->emitEvent('Test', 'test');
 		$this->emitter->emitEvent('Test', 'foo');
 		$this->emitter->emitEvent('Bar', 'foo');
-
-		$this->addToAssertionCount(1);
 	}
 
 	public function testRemoveKeepOtherCallback(): void {
@@ -228,8 +223,6 @@ public function testRemoveKeepOtherCallback(): void {
 		$this->emitter->listen('Test', 'test', $listener2);
 		$this->emitter->removeListener('Test', 'test', $listener1);
 		$this->emitter->emitEvent('Test', 'test');
-
-		$this->addToAssertionCount(1);
 	}
 
 	public function testRemoveKeepOtherMethod(): void {
@@ -242,8 +235,6 @@ public function testRemoveKeepOtherMethod(): void {
 		$this->emitter->listen('Test', 'foo', $listener);
 		$this->emitter->removeListener('Test', 'foo', $listener);
 		$this->emitter->emitEvent('Test', 'test');
-
-		$this->addToAssertionCount(1);
 	}
 
 	public function testRemoveKeepOtherScope(): void {
@@ -256,8 +247,6 @@ public function testRemoveKeepOtherScope(): void {
 		$this->emitter->listen('Bar', 'test', $listener);
 		$this->emitter->removeListener('Bar', 'test', $listener);
 		$this->emitter->emitEvent('Test', 'test');
-
-		$this->addToAssertionCount(1);
 	}
 
 	public function testRemoveNonExistingName(): void {
@@ -269,7 +258,5 @@ public function testRemoveNonExistingName(): void {
 		$this->emitter->listen('Test', 'test', $listener);
 		$this->emitter->removeListener('Bar', 'test', $listener);
 		$this->emitter->emitEvent('Test', 'test');
-
-		$this->addToAssertionCount(1);
 	}
 }