Merge pull request #60759 from nextcloud/backport/60748/stable34

[stable34] test: replace checkbox assertions with DoesNotPerformAssertions (middleware)

Author: AndyScherzinger

tests/lib/AppFramework/Middleware/Security/SameSiteCookieMiddlewareTest.php

@@ -50,12 +50,12 @@ protected function setUp(): void {
 		$this->middleware = new SameSiteCookieMiddleware($this->request, new MiddlewareUtils($this->reflector, $this->logger));
 	}
 
+	#[\PHPUnit\Framework\Attributes\DoesNotPerformAssertions]
 	public function testBeforeControllerNoIndex(): void {
 		$this->request->method('getScriptName')
 			->willReturn('/ocs/v2.php');
 
 		$this->middleware->beforeController(new NoAnnotationController('foo', $this->request), 'foo');
-		$this->addToAssertionCount(1);
 	}
 
 	public function testBeforeControllerIndexHasAnnotation(): void {
@@ -68,7 +68,6 @@ public function testBeforeControllerIndexHasAnnotation(): void {
 			->willReturn(true);
 
 		$this->middleware->beforeController(new HasAnnotationController('foo', $this->request), 'foo');
-		$this->addToAssertionCount(1);
 	}
 
 	public function testBeforeControllerIndexNoAnnotationPassingCheck(): void {
@@ -84,7 +83,6 @@ public function testBeforeControllerIndexNoAnnotationPassingCheck(): void {
 			->willReturn(true);
 
 		$this->middleware->beforeController(new NoAnnotationController('foo', $this->request), 'foo');
-		$this->addToAssertionCount(1);
 	}
 
 	public function testBeforeControllerIndexNoAnnotationFailingCheck(): void {

tests/lib/AppFramework/Middleware/Security/SecurityMiddlewareTest.php

@@ -470,21 +470,21 @@ public function testIsNotSubAdminCheck(string $method): void {
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('dataNoCSRFRequiredSubAdminRequired')]
+	#[\PHPUnit\Framework\Attributes\DoesNotPerformAssertions]
 	public function testIsSubAdminCheck(string $method): void {
 		$this->reader->reflect($this->controller, $method);
 		$sec = $this->getMiddleware(true, false, true);
 
 		$sec->beforeController($this->controller, $method);
-		$this->addToAssertionCount(1);
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('dataNoCSRFRequiredSubAdminRequired')]
+	#[\PHPUnit\Framework\Attributes\DoesNotPerformAssertions]
 	public function testIsSubAdminAndAdminCheck(string $method): void {
 		$this->reader->reflect($this->controller, $method);
 		$sec = $this->getMiddleware(true, true, true);
 
 		$sec->beforeController($this->controller, $method);
-		$this->addToAssertionCount(1);
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('dataNoCSRFRequired')]
@@ -506,7 +506,6 @@ public function testRestrictedAppLoggedInPublicPage(string $method): void {
 			->willReturn(false);
 
 		$middleware->beforeController($this->controller, $method);
-		$this->addToAssertionCount(1);
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('dataNoAdminRequiredNoCSRFRequiredPublicPage')]
@@ -523,7 +522,6 @@ public function testRestrictedAppNotLoggedInPublicPage(string $method): void {
 			->willReturn(false);
 
 		$middleware->beforeController($this->controller, $method);
-		$this->addToAssertionCount(1);
 	}
 
 	#[\PHPUnit\Framework\Attributes\DataProvider('dataNoAdminRequiredNoCSRFRequired')]