refactor: apply rector rules for PHPUnit 10

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

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

apps/contactsinteraction/tests/Db/RecentContactMapperTest.php

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

apps/dav/lib/Connector/Sabre/Directory.php

@@ -211,7 +211,7 @@ public function getChild($name, $info = null, ?IRequest $request = null, ?IL10N
 		}
 
 		if ($info->getMimeType() === FileInfo::MIMETYPE_FOLDER) {
-			$node = new \OCA\DAV\Connector\Sabre\Directory($this->fileView, $info, $this->tree, $this->shareManager);
+			$node = new Directory($this->fileView, $info, $this->tree, $this->shareManager);
 		} else {
 			// In case reading a directory was allowed but it turns out the node was a not a directory, reject it now.
 			if (!$this->info->isReadable()) {

apps/dav/lib/Server.php

@@ -92,6 +92,7 @@
 use OCP\ITagManager;
 use OCP\IURLGenerator;
 use OCP\IUserSession;
+use OCP\Mail\IEmailValidator;
 use OCP\Mail\IMailer;
 use OCP\Profiler\IProfiler;
 use OCP\SabrePluginEvent;
@@ -351,7 +352,7 @@ public function __construct(
 						\OCP\Server::get(IMipService::class),
 						\OCP\Server::get(EventComparisonService::class),
 						\OCP\Server::get(\OCP\Mail\Provider\IManager::class),
-						\OCP\Server::get(\OCP\Mail\IEmailValidator::class),
+						\OCP\Server::get(IEmailValidator::class),
 					));
 				}
 				$this->server->addPlugin(new \OCA\DAV\CalDAV\Search\SearchPlugin());

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

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

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

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

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

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

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

@@ -22,9 +22,7 @@
 use Test\TestCase;
 use function scandir;
 
-/**
- * @group DB
- */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class CalendarMigratorTest extends TestCase {
 
 	private IUserManager $userManager;

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

@@ -22,9 +22,7 @@
 use Test\TestCase;
 use function scandir;
 
-/**
- * @group DB
- */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class ContactsMigratorTest extends TestCase {
 
 	private IUserManager $userManager;

apps/dav/tests/unit/AppInfo/ApplicationTest.php

@@ -16,10 +16,10 @@
 /**
  * Class ApplicationTest
  *
- * @group DB
  *
  * @package OCA\DAV\Tests\Unit\AppInfo
  */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class ApplicationTest extends TestCase {
 	public function test(): void {
 		$app = new Application();

apps/dav/tests/unit/BackgroundJob/UserStatusAutomationTest.php

@@ -25,9 +25,7 @@
 use Psr\Log\LoggerInterface;
 use Test\TestCase;
 
-/**
- * @group DB
- */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class UserStatusAutomationTest extends TestCase {
 	protected ITimeFactory&MockObject $time;
 	protected IJobList&MockObject $jobList;

apps/dav/tests/unit/CalDAV/AbstractCalDavBackend.php

@@ -39,10 +39,10 @@
 /**
  * Class CalDavBackendTest
  *
- * @group DB
  *
  * @package OCA\DAV\Tests\unit\CalDAV
  */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 abstract class AbstractCalDavBackend extends TestCase {
 
 

apps/dav/tests/unit/CalDAV/Activity/Filter/GenericTest.php

@@ -13,9 +13,7 @@
 use OCP\Server;
 use Test\TestCase;
 
-/**
- * @group DB
- */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class GenericTest extends TestCase {
 	public static function dataFilters(): array {
 		return [

apps/dav/tests/unit/CalDAV/CalDavBackendTest.php

@@ -28,9 +28,8 @@
 
 /**
  * Class CalDavBackendTest
- *
- * @group DB
  */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class CalDavBackendTest extends AbstractCalDavBackend {
 	public function testCalendarOperations(): void {
 		$calendarId = $this->createTestCalendar();

apps/dav/tests/unit/CalDAV/PublicCalendarRootTest.php

@@ -29,10 +29,10 @@
 /**
  * Class PublicCalendarRootTest
  *
- * @group DB
  *
  * @package OCA\DAV\Tests\unit\CalDAV
  */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class PublicCalendarRootTest extends TestCase {
 	public const UNIT_TEST_USER = '';
 	private CalDavBackend $backend;

apps/dav/tests/unit/CalDAV/Reminder/NotificationProviderManagerTest.php

@@ -17,9 +17,7 @@
 use OCP\AppFramework\QueryException;
 use Test\TestCase;
 
-/**
- * @group DB
- */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class NotificationProviderManagerTest extends TestCase {
 	private NotificationProviderManager $providerManager;
 

apps/dav/tests/unit/CardDAV/CardDavBackendTest.php

@@ -43,10 +43,10 @@
 /**
  * Class CardDavBackendTest
  *
- * @group DB
  *
  * @package OCA\DAV\Tests\unit\CardDAV
  */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class CardDavBackendTest extends TestCase {
 	private Principal&MockObject $principal;
 	private IUserManager&MockObject $userManager;

apps/dav/tests/unit/Command/RemoveInvalidSharesTest.php

@@ -23,8 +23,8 @@
  * Class RemoveInvalidSharesTest
  *
  * @package OCA\DAV\Tests\Unit\Repair
- * @group DB
  */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class RemoveInvalidSharesTest extends TestCase {
 	private RemoveInvalidShares $command;
 

apps/dav/tests/unit/Connector/LegacyPublicAuthTest.php

@@ -20,10 +20,10 @@
 /**
  * Class LegacyPublicAuthTest
  *
- * @group DB
  *
  * @package OCA\DAV\Tests\unit\Connector
  */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class LegacyPublicAuthTest extends \Test\TestCase {
 	private ISession&MockObject $session;
 	private IRequest&MockObject $request;

apps/dav/tests/unit/Connector/Sabre/AuthTest.php

@@ -27,8 +27,8 @@
  * Class AuthTest
  *
  * @package OCA\DAV\Tests\unit\Connector\Sabre
- * @group DB
  */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class AuthTest extends TestCase {
 	private ISession&MockObject $session;
 	private Session&MockObject $userSession;

apps/dav/tests/unit/Connector/Sabre/BearerAuthTest.php

@@ -19,9 +19,7 @@
 use Sabre\HTTP\ResponseInterface;
 use Test\TestCase;
 
-/**
- * @group DB
- */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class BearerAuthTest extends TestCase {
 	private IUserSession&MockObject $userSession;
 	private ISession&MockObject $session;

apps/dav/tests/unit/Connector/Sabre/CustomPropertiesBackendTest.php

@@ -22,10 +22,10 @@
 /**
  * Class CustomPropertiesBackend
  *
- * @group DB
  *
  * @package OCA\DAV\Tests\unit\Connector\Sabre
  */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class CustomPropertiesBackendTest extends \Test\TestCase {
 	private \Sabre\DAV\Server $server;
 	private \Sabre\DAV\Tree&MockObject $tree;

apps/dav/tests/unit/Connector/Sabre/DirectoryTest.php

@@ -55,9 +55,7 @@ public function getRelativePath($path): ?string {
 }
 
 
-/**
- * @group DB
- */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class DirectoryTest extends \Test\TestCase {
 	use UserTrait;
 

apps/dav/tests/unit/Connector/Sabre/FileTest.php

@@ -46,10 +46,10 @@
 /**
  * Class File
  *
- * @group DB
  *
  * @package OCA\DAV\Tests\unit\Connector\Sabre
  */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class FileTest extends TestCase {
 	use MountProviderTrait;
 	use UserTrait;

apps/dav/tests/unit/Connector/Sabre/FilesPluginTest.php

@@ -35,9 +35,7 @@
 use Sabre\Xml\Service;
 use Test\TestCase;
 
-/**
- * @group DB
- */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class FilesPluginTest extends TestCase {
 
 	private Tree&MockObject $tree;

apps/dav/tests/unit/Connector/Sabre/NodeTest.php

@@ -29,9 +29,9 @@
 /**
  * Class NodeTest
  *
- * @group DB
  * @package OCA\DAV\Tests\unit\Connector\Sabre
  */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class NodeTest extends \Test\TestCase {
 	public static function davPermissionsProvider(): array {
 		return [

apps/dav/tests/unit/Connector/Sabre/ObjectTreeTest.php

@@ -23,10 +23,10 @@
 /**
  * Class ObjectTreeTest
  *
- * @group DB
  *
  * @package OCA\DAV\Tests\Unit\Connector\Sabre
  */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class ObjectTreeTest extends \Test\TestCase {
 	public static function copyDataProvider(): array {
 		return [

apps/dav/tests/unit/Connector/Sabre/PublicAuthTest.php

@@ -22,10 +22,10 @@
 /**
  * Class PublicAuthTest
  *
- * @group DB
  *
  * @package OCA\DAV\Tests\unit\Connector
  */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class PublicAuthTest extends \Test\TestCase {
 
 	private ISession&MockObject $session;

apps/dav/tests/unit/Connector/Sabre/RequestTest/DeleteTest.php

@@ -14,10 +14,10 @@
 /**
  * Class DeleteTest
  *
- * @group DB
  *
  * @package OCA\DAV\Tests\unit\Connector\Sabre\RequestTest
  */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class DeleteTest extends RequestTestCase {
 	public function testBasicUpload(): void {
 		$user = self::getUniqueID();

apps/dav/tests/unit/Connector/Sabre/RequestTest/DownloadTest.php

@@ -14,10 +14,10 @@
 /**
  * Class DownloadTest
  *
- * @group DB
  *
  * @package OCA\DAV\Tests\unit\Connector\Sabre\RequestTest
  */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class DownloadTest extends RequestTestCase {
 	public function testDownload(): void {
 		$user = self::getUniqueID();

apps/dav/tests/unit/Connector/Sabre/RequestTest/EncryptionMasterKeyUploadTest.php

@@ -17,10 +17,10 @@
 /**
  * Class EncryptionMasterKeyUploadTest
  *
- * @group DB
  *
  * @package OCA\DAV\Tests\Unit\Connector\Sabre\RequestTest
  */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class EncryptionMasterKeyUploadTest extends UploadTest {
 	use EncryptionTrait;
 

apps/dav/tests/unit/Connector/Sabre/RequestTest/EncryptionUploadTest.php

@@ -17,10 +17,10 @@
 /**
  * Class EncryptionUploadTest
  *
- * @group DB
  *
  * @package OCA\DAV\Tests\Unit\Connector\Sabre\RequestTest
  */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class EncryptionUploadTest extends UploadTest {
 	use EncryptionTrait;
 

apps/dav/tests/unit/Connector/Sabre/RequestTest/PartFileInRootUploadTest.php

@@ -14,10 +14,10 @@
 /**
  * Class PartFileInRootUploadTest
  *
- * @group DB
  *
  * @package OCA\DAV\Tests\unit\Connector\Sabre\RequestTest
  */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class PartFileInRootUploadTest extends UploadTest {
 	protected function setUp(): void {
 		$config = Server::get(IConfig::class);

apps/dav/tests/unit/Connector/Sabre/RequestTest/UploadTest.php

@@ -16,10 +16,10 @@
 /**
  * Class UploadTest
  *
- * @group DB
  *
  * @package OCA\DAV\Tests\unit\Connector\Sabre\RequestTest
  */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class UploadTest extends RequestTestCase {
 	public function testBasicUpload(): void {
 		$user = self::getUniqueID();