fix(psalm): Fix static analysis issues in apps/*/tests

There are still 1200 more to fix before we can enable static analysis
for the tests.

Signed-off-by: Carl Schwan <[email protected]>

Author: CarlSchwan

apps/comments/tests/Unit/AppInfo/ApplicationTest.php

@@ -26,7 +26,7 @@
  *
  * @package OCA\Comments\Tests\Unit\AppInfo
  */
-#[\PHPUnit\Framework\Attributes\Group('DB')]
+#[\PHPUnit\Framework\Attributes\Group(name: 'DB')]
 class ApplicationTest extends TestCase {
 	protected function setUp(): void {
 		parent::setUp();

apps/comments/tests/Unit/Collaboration/CommentersSorterTest.php

@@ -27,7 +27,7 @@ protected function setUp(): void {
 	/**
 	 * @param $data
 	 */
-	#[\PHPUnit\Framework\Attributes\DataProvider('sortDataProvider')]
+	#[\PHPUnit\Framework\Attributes\DataProvider(methodName: 'sortDataProvider')]
 	public function testSort($data): void {
 		$commentMocks = [];
 		foreach ($data['actors'] as $actorType => $actors) {

apps/comments/tests/Unit/EventHandlerTest.php

@@ -61,7 +61,7 @@ public static function handledProvider(): array {
 		];
 	}
 
-	#[\PHPUnit\Framework\Attributes\DataProvider('handledProvider')]
+	#[\PHPUnit\Framework\Attributes\DataProvider(methodName: 'handledProvider')]
 	public function testHandled(string $eventType): void {
 		/** @var IComment|MockObject $comment */
 		$comment = $this->createMock(IComment::class);

apps/comments/tests/Unit/Notification/ListenerTest.php

@@ -51,7 +51,7 @@ public static function eventProvider(): array {
 	 * @param string $eventType
 	 * @param string $notificationMethod
 	 */
-	#[\PHPUnit\Framework\Attributes\DataProvider('eventProvider')]
+	#[\PHPUnit\Framework\Attributes\DataProvider(methodName: 'eventProvider')]
 	public function testEvaluate(string $eventType, $notificationMethod): void {
 		/** @var IComment|MockObject $comment */
 		$comment = $this->createMock(IComment::class);
@@ -111,7 +111,7 @@ public function testEvaluate(string $eventType, $notificationMethod): void {
 		$this->listener->evaluate($event);
 	}
 
-	#[\PHPUnit\Framework\Attributes\DataProvider('eventProvider')]
+	#[\PHPUnit\Framework\Attributes\DataProvider(methodName: 'eventProvider')]
 	public function testEvaluateNoMentions(string $eventType): void {
 		/** @var IComment|MockObject $comment */
 		$comment = $this->createMock(IComment::class);

apps/contactsinteraction/tests/Db/RecentContactMapperTest.php

@@ -17,7 +17,7 @@
 use Sabre\VObject\UUIDUtil;
 use Test\TestCase;
 
-#[\PHPUnit\Framework\Attributes\Group('DB')]
+#[\PHPUnit\Framework\Attributes\Group(name: 'DB')]
 class RecentContactMapperTest extends TestCase {
 	private RecentContactMapper $recentContactMapper;
 	private ITimeFactory $time;

apps/dav/tests/integration/DAV/Sharing/CalDavSharingBackendTest.php

@@ -28,7 +28,7 @@
 use Psr\Log\LoggerInterface;
 use Test\TestCase;
 
-#[\PHPUnit\Framework\Attributes\Group('DB')]
+#[\PHPUnit\Framework\Attributes\Group(name: 'DB')]
 class CalDavSharingBackendTest extends TestCase {
 
 	private IDBConnection $db;

apps/dav/tests/integration/DAV/Sharing/SharingMapperTest.php

@@ -14,7 +14,7 @@
 use OCP\Server;
 use Test\TestCase;
 
-#[\PHPUnit\Framework\Attributes\Group('DB')]
+#[\PHPUnit\Framework\Attributes\Group(name: 'DB')]
 class SharingMapperTest extends TestCase {
 
 	private SharingMapper $mapper;

apps/dav/tests/integration/Db/PropertyMapperTest.php

@@ -13,7 +13,7 @@
 use OCP\Server;
 use Test\TestCase;
 
-#[\PHPUnit\Framework\Attributes\Group('DB')]
+#[\PHPUnit\Framework\Attributes\Group(name: 'DB')]
 class PropertyMapperTest extends TestCase {
 
 	/** @var PropertyMapper */

apps/dav/tests/integration/UserMigration/CalendarMigratorTest.php

@@ -22,7 +22,7 @@
 use Test\TestCase;
 use function scandir;
 
-#[\PHPUnit\Framework\Attributes\Group('DB')]
+#[\PHPUnit\Framework\Attributes\Group(name: 'DB')]
 class CalendarMigratorTest extends TestCase {
 
 	private IUserManager $userManager;
@@ -87,7 +87,7 @@ private function getSanitizedComponents(VCalendar $vCalendar): array {
 		);
 	}
 
-	#[\PHPUnit\Framework\Attributes\DataProvider('dataAssets')]
+	#[\PHPUnit\Framework\Attributes\DataProvider(methodName: 'dataAssets')]
 	public function testImportExportAsset(string $userId, string $filename, string $initialCalendarUri, VCalendar $importCalendar): void {
 		$user = $this->userManager->createUser($userId, 'topsecretpassword');
 

apps/dav/tests/integration/UserMigration/ContactsMigratorTest.php

@@ -22,7 +22,7 @@
 use Test\TestCase;
 use function scandir;
 
-#[\PHPUnit\Framework\Attributes\Group('DB')]
+#[\PHPUnit\Framework\Attributes\Group(name: 'DB')]
 class ContactsMigratorTest extends TestCase {
 
 	private IUserManager $userManager;
@@ -93,7 +93,7 @@ private function getProperties(VCard $vCard): array {
 	 * @param array{displayName: string, description?: string} $importMetadata
 	 * @param VCard[] $importCards
 	 */
-	#[\PHPUnit\Framework\Attributes\DataProvider('dataAssets')]
+	#[\PHPUnit\Framework\Attributes\DataProvider(methodName: 'dataAssets')]
 	public function testImportExportAsset(string $userId, string $filename, string $initialAddressBookUri, array $importMetadata, array $importCards): void {
 		$user = $this->userManager->createUser($userId, 'topsecretpassword');