From d6d6747a7332cc553c882c19b1369022a1a5599e Mon Sep 17 00:00:00 2001 From: Ferdinand Thiessen Date: Tue, 21 Oct 2025 01:52:40 +0200 Subject: [PATCH] refactor: apply rector rules for PHPUnit 10 Signed-off-by: Ferdinand Thiessen --- .../tests/Unit/AppInfo/ApplicationTest.php | 3 +- .../tests/Db/RecentContactMapperTest.php | 4 +- apps/dav/lib/Connector/Sabre/Directory.php | 2 +- apps/dav/lib/Server.php | 3 +- .../DAV/Sharing/CalDavSharingBackendTest.php | 4 +- .../DAV/Sharing/SharingMapperTest.php | 4 +- .../integration/Db/PropertyMapperTest.php | 4 +- .../UserMigration/CalendarMigratorTest.php | 4 +- .../UserMigration/ContactsMigratorTest.php | 4 +- .../tests/unit/AppInfo/ApplicationTest.php | 2 +- .../UserStatusAutomationTest.php | 4 +- .../unit/CalDAV/AbstractCalDavBackend.php | 2 +- .../CalDAV/Activity/Filter/GenericTest.php | 4 +- .../tests/unit/CalDAV/CalDavBackendTest.php | 3 +- .../unit/CalDAV/PublicCalendarRootTest.php | 2 +- .../NotificationProviderManagerTest.php | 4 +- .../tests/unit/CardDAV/CardDavBackendTest.php | 2 +- .../unit/Command/RemoveInvalidSharesTest.php | 2 +- .../unit/Connector/LegacyPublicAuthTest.php | 2 +- .../tests/unit/Connector/Sabre/AuthTest.php | 2 +- .../unit/Connector/Sabre/BearerAuthTest.php | 4 +- .../Sabre/CustomPropertiesBackendTest.php | 2 +- .../unit/Connector/Sabre/DirectoryTest.php | 4 +- .../tests/unit/Connector/Sabre/FileTest.php | 2 +- .../unit/Connector/Sabre/FilesPluginTest.php | 4 +- .../tests/unit/Connector/Sabre/NodeTest.php | 2 +- .../unit/Connector/Sabre/ObjectTreeTest.php | 2 +- .../unit/Connector/Sabre/PublicAuthTest.php | 2 +- .../Sabre/RequestTest/DeleteTest.php | 2 +- .../Sabre/RequestTest/DownloadTest.php | 2 +- .../EncryptionMasterKeyUploadTest.php | 2 +- .../RequestTest/EncryptionUploadTest.php | 2 +- .../RequestTest/PartFileInRootUploadTest.php | 2 +- .../Sabre/RequestTest/UploadTest.php | 2 +- .../unit/DAV/CustomPropertiesBackendTest.php | 4 +- .../Migration/CalDAVRemoveEmptyValueTest.php | 2 +- apps/dav/tests/unit/ServerTest.php | 2 +- .../tests/Command/FixEncryptedVersionTest.php | 2 +- .../tests/Command/TestEnableMasterKey.php | 3 ++ apps/encryption/tests/Crypto/CryptTest.php | 3 +- .../encryption/tests/EncryptedStorageTest.php | 4 +- .../Listeners/UserEventsListenersTest.php | 4 +- .../tests/PassphraseServiceTest.php | 4 +- apps/encryption/tests/SessionTest.php | 8 +--- .../RequestHandlerControllerTest.php | 2 +- .../tests/FederatedShareProviderTest.php | 2 +- apps/federatedfilesharing/tests/TestCase.php | 5 +-- apps/federation/lib/DbHandler.php | 2 - .../BackgroundJob/GetSharedSecretTest.php | 2 +- apps/federation/tests/DbHandlerTest.php | 4 +- .../tests/Activity/Filter/GenericTest.php | 2 +- .../DeleteOrphanedItemsJobTest.php | 2 +- .../tests/BackgroundJob/ScanFilesTest.php | 2 +- .../tests/Command/DeleteOrphanedFilesTest.php | 2 +- .../tests/Controller/ViewControllerTest.php | 2 +- apps/files/tests/Service/TagServiceTest.php | 2 +- .../Listener/StorePasswordListenerTest.php | 4 +- .../tests/OwnCloudFunctionsTest.php | 2 +- .../tests/Service/DBConfigServiceTest.php | 4 +- .../Service/GlobalStoragesServiceTest.php | 4 +- .../tests/Service/StoragesServiceTestCase.php | 4 +- .../Service/UserGlobalStoragesServiceTest.php | 4 +- .../tests/Service/UserStoragesServiceTest.php | 4 +- .../tests/Storage/Amazons3MultiPartTest.php | 4 +- .../tests/Storage/Amazons3Test.php | 4 +- apps/files_external/tests/Storage/FtpTest.php | 2 +- .../tests/Storage/OwncloudTest.php | 2 +- .../tests/Storage/SFTP_KeyTest.php | 2 +- .../files_external/tests/Storage/SftpTest.php | 2 +- apps/files_external/tests/Storage/SmbTest.php | 2 +- .../tests/Storage/SwiftTest.php | 2 +- .../tests/Storage/VersionedAmazonS3Test.php | 6 +-- .../tests/Storage/WebdavTest.php | 2 +- apps/files_sharing/tests/ApiTest.php | 38 ++++++++----------- apps/files_sharing/tests/CacheTest.php | 3 +- apps/files_sharing/tests/CapabilitiesTest.php | 3 +- .../Command/CleanupRemoteStoragesTest.php | 2 +- .../Controller/ShareAPIControllerTest.php | 2 +- .../tests/Controller/ShareControllerTest.php | 3 +- .../Controller/ShareesAPIControllerTest.php | 2 +- .../tests/DeleteOrphanedSharesJobTest.php | 2 +- .../tests/EncryptedSizePropagationTest.php | 4 +- .../tests/EtagPropagationTest.php | 2 +- .../tests/ExpireSharesJobTest.php | 2 +- .../tests/External/CacheTest.php | 2 +- .../tests/External/ManagerTest.php | 2 +- .../tests/External/ScannerTest.php | 4 +- .../tests/ExternalStorageTest.php | 3 +- .../tests/GroupEtagPropagationTest.php | 3 +- apps/files_sharing/tests/HelperTest.php | 3 +- apps/files_sharing/tests/LockingTest.php | 2 +- .../tests/Migration/SetPasswordColumnTest.php | 3 +- .../files_sharing/tests/MountProviderTest.php | 4 +- apps/files_sharing/tests/ShareTest.php | 3 +- apps/files_sharing/tests/SharedMountTest.php | 3 +- .../files_sharing/tests/SharedStorageTest.php | 3 +- .../tests/SharesReminderJobTest.php | 2 +- .../tests/SizePropagationTest.php | 2 +- apps/files_sharing/tests/TestCase.php | 5 +-- .../tests/UnshareChildrenTest.php | 2 +- apps/files_sharing/tests/UpdaterTest.php | 3 +- apps/files_sharing/tests/WatcherTest.php | 3 +- .../tests/Command/CleanUpTest.php | 2 +- .../tests/Command/ExpireTest.php | 2 +- .../tests/Command/ExpireTrashTest.php | 2 +- apps/files_trashbin/tests/StorageTest.php | 2 +- apps/files_trashbin/tests/TrashbinTest.php | 3 +- .../tests/Command/CleanupTest.php | 2 +- .../tests/Command/ExpireTest.php | 2 +- apps/files_versions/tests/StorageTest.php | 4 +- apps/files_versions/tests/VersioningTest.php | 3 +- .../LoginRedirectorControllerTest.php | 4 +- .../Controller/SettingsControllerTest.php | 4 +- .../oauth2/tests/Db/AccessTokenMapperTest.php | 4 +- apps/oauth2/tests/Db/ClientMapperTest.php | 4 +- .../tests/CapabilitiesTest.php | 2 +- .../tests/Controller/AppsControllerTest.php | 2 +- .../tests/AppInfo/ApplicationTest.php | 2 +- .../AdminSettingsControllerTest.php | 2 +- .../Controller/AppSettingsControllerTest.php | 3 +- .../Controller/CheckSetupControllerTest.php | 2 +- .../tests/Controller/UsersControllerTest.php | 3 +- .../tests/Settings/Admin/ServerTest.php | 4 +- .../SetupChecks/SupportedDatabaseTest.php | 4 +- .../UserMigration/AccountMigratorTest.php | 4 +- .../tests/ShareByMailProviderTest.php | 2 +- .../Version2006Date20240905111627Test.php | 10 ++--- apps/theming/tests/ServicesTest.php | 2 +- .../tests/Db/BackupCodeMapperTest.php | 4 +- .../tests/Service/BackupCodeStorageTest.php | 4 +- apps/user_ldap/tests/AccessTest.php | 2 +- apps/user_ldap/tests/ConnectionTest.php | 2 +- apps/user_ldap/tests/Group_LDAPTest.php | 2 +- apps/user_ldap/tests/HelperTest.php | 4 +- apps/user_ldap/tests/Jobs/SyncTest.php | 4 +- apps/user_ldap/tests/LDAPProviderTest.php | 2 +- .../tests/Mapping/GroupMappingTest.php | 2 +- .../tests/Mapping/UserMappingTest.php | 2 +- .../tests/Migration/UUIDFixGroupTest.php | 2 +- .../tests/Migration/UUIDFixUserTest.php | 2 +- apps/user_ldap/tests/Settings/AdminTest.php | 2 +- .../tests/User/DeletedUsersIndexTest.php | 2 +- apps/user_ldap/tests/User/ManagerTest.php | 2 +- apps/user_ldap/tests/User/UserTest.php | 2 +- apps/user_ldap/tests/User_LDAPTest.php | 6 +-- apps/user_ldap/tests/WizardTest.php | 2 +- .../Service/StatusServiceIntegrationTest.php | 4 +- .../tests/Db/WebhookListenerMapperTest.php | 4 +- .../tests/Check/FileMimeTypeTest.php | 4 +- apps/workflowengine/tests/ManagerTest.php | 2 +- build/rector.php | 10 +---- core/Controller/WhatsNewController.php | 3 +- cron.php | 2 +- tests/Core/Command/Apps/AppsDisableTest.php | 3 +- tests/Core/Command/Apps/AppsEnableTest.php | 3 +- tests/lib/Accounts/AccountManagerTest.php | 2 +- tests/lib/Accounts/HooksTest.php | 2 +- tests/lib/AllConfigTest.php | 13 +++---- tests/lib/AppConfigIntegrationTest.php | 3 +- tests/lib/AppFramework/Db/QBMapperDBTest.php | 2 +- .../DependencyInjection/DIContainerTest.php | 4 +- .../DIIntergrationTests.php | 3 ++ .../lib/AppFramework/Http/DispatcherTest.php | 2 +- .../Security/RateLimitingMiddlewareTest.php | 4 +- .../Utility/ControllerMethodReflectorTest.php | 3 -- tests/lib/AppScriptSortTest.php | 2 +- tests/lib/AppTest.php | 3 +- .../Token/PublicKeyTokenMapperTest.php | 4 +- .../Authentication/Token/RemoteWipeTest.php | 3 +- .../Db/ProviderUserAssignmentDaoTest.php | 4 +- tests/lib/BackgroundJob/JobListTest.php | 2 +- tests/lib/Cache/FileCacheTest.php | 2 +- tests/lib/Command/BackgroundModeTest.php | 4 +- tests/lib/Command/CronBusTest.php | 4 +- tests/lib/Comments/ManagerTest.php | 3 +- tests/lib/Config/LexiconTest.php | 2 +- tests/lib/Config/UserConfigTest.php | 2 +- tests/lib/DB/ConnectionTest.php | 4 +- tests/lib/DB/MigratorTest.php | 2 +- tests/lib/DB/OCPostgreSqlPlatformTest.php | 2 +- .../QueryBuilder/ExpressionBuilderDBTest.php | 4 +- .../DB/QueryBuilder/ExpressionBuilderTest.php | 2 +- .../DB/QueryBuilder/FunctionBuilderTest.php | 2 +- .../PartitionedQueryBuilderTest.php | 4 +- .../lib/DB/QueryBuilder/QueryBuilderTest.php | 2 +- .../Sharded/SharedQueryBuilderTest.php | 4 +- tests/lib/DirectEditing/ManagerTest.php | 2 +- tests/lib/Encryption/DecryptAllTest.php | 2 +- tests/lib/Encryption/ManagerTest.php | 4 +- tests/lib/Federation/CloudIdManagerTest.php | 4 +- tests/lib/Federation/CloudIdTest.php | 4 +- tests/lib/Files/Cache/CacheTest.php | 2 +- tests/lib/Files/Cache/FileAccessTest.php | 4 +- tests/lib/Files/Cache/HomeCacheTest.php | 2 +- .../lib/Files/Cache/LocalRootScannerTest.php | 4 +- .../Files/Cache/MoveFromCacheTraitTest.php | 3 +- tests/lib/Files/Cache/PropagatorTest.php | 4 +- tests/lib/Files/Cache/ScannerTest.php | 2 +- tests/lib/Files/Cache/SearchBuilderTest.php | 4 +- tests/lib/Files/Cache/UpdaterLegacyTest.php | 2 +- tests/lib/Files/Cache/UpdaterTest.php | 2 +- tests/lib/Files/Cache/WatcherTest.php | 2 +- .../lib/Files/Cache/Wrapper/CacheJailTest.php | 2 +- .../Wrapper/CachePermissionsMaskTest.php | 2 +- tests/lib/Files/Config/UserMountCacheTest.php | 4 +- tests/lib/Files/EtagTest.php | 2 +- tests/lib/Files/FilesystemTest.php | 2 +- .../lib/Files/Mount/RootMountProviderTest.php | 4 +- tests/lib/Files/Node/FileTest.php | 2 +- tests/lib/Files/Node/FolderTest.php | 2 +- tests/lib/Files/Node/HookConnectorTest.php | 2 +- tests/lib/Files/Node/IntegrationTest.php | 2 +- tests/lib/Files/ObjectStore/AzureTest.php | 4 +- .../ObjectStore/ObjectStoreScannerTest.php | 4 +- .../ObjectStore/ObjectStoreStorageTest.php | 4 +- ...ObjectStoreStoragesDifferentBucketTest.php | 4 +- .../ObjectStoreStoragesSameBucketTest.php | 4 +- tests/lib/Files/ObjectStore/S3Test.php | 4 +- tests/lib/Files/ObjectStore/SwiftTest.php | 4 +- tests/lib/Files/PathVerificationTest.php | 2 +- .../Search/QueryOptimizer/CombinedTests.php | 3 ++ .../Files/Search/SearchIntegrationTest.php | 4 +- tests/lib/Files/SimpleFS/SimpleFolderTest.php | 4 +- tests/lib/Files/Storage/CommonTest.php | 2 +- tests/lib/Files/Storage/CopyDirectoryTest.php | 2 +- tests/lib/Files/Storage/HomeTest.php | 2 +- tests/lib/Files/Storage/LocalTest.php | 2 +- .../Files/Storage/Wrapper/KnownMtimeTest.php | 4 +- .../Storage/Wrapper/PermissionsMaskTest.php | 4 +- tests/lib/Files/Storage/Wrapper/QuotaTest.php | 2 +- tests/lib/Files/Utils/ScannerTest.php | 2 +- tests/lib/Files/ViewTest.php | 2 +- .../FilesMetadataManagerTest.php | 6 +-- tests/lib/Group/Backend.php | 3 +- tests/lib/Group/DatabaseTest.php | 3 +- tests/lib/Group/Dummy.php | 6 ++- tests/lib/HelperStorageTest.php | 3 +- tests/lib/ImageTest.php | 4 +- tests/lib/InfoXmlTest.php | 2 +- tests/lib/InstallerTest.php | 2 +- tests/lib/Lock/DBLockingProviderTest.php | 2 +- .../Lock/NonCachingDBLockingProviderTest.php | 3 +- tests/lib/Lockdown/Filesystem/NoFSTest.php | 4 +- tests/lib/Memcache/APCuTest.php | 6 +-- tests/lib/Memcache/ArrayCacheTest.php | 4 +- tests/lib/Memcache/CasTraitTest.php | 4 +- tests/lib/Memcache/FactoryTest.php | 4 +- tests/lib/Memcache/MemcachedTest.php | 6 +-- tests/lib/Memcache/RedisTest.php | 6 +-- .../lib/Preview/BackgroundCleanupJobTest.php | 2 +- tests/lib/Preview/BitmapTest.php | 2 +- tests/lib/Preview/HEICTest.php | 2 +- tests/lib/Preview/ImageTest.php | 2 +- tests/lib/Preview/MP3Test.php | 2 +- tests/lib/Preview/MovePreviewJobTest.php | 4 +- .../Preview/MovieBrokenStuckFfmpegTest.php | 2 +- tests/lib/Preview/MovieTest.php | 2 +- tests/lib/Preview/MovieTestRemoteFile.php | 5 ++- tests/lib/Preview/OfficeTest.php | 2 +- tests/lib/Preview/PreviewMapperTest.php | 4 +- tests/lib/Preview/PreviewServiceTest.php | 4 +- tests/lib/Preview/Provider.php | 2 +- tests/lib/Preview/SVGTest.php | 6 +-- tests/lib/Preview/TXTTest.php | 2 +- tests/lib/Repair/CleanTagsTest.php | 2 +- .../Repair/OldGroupMembershipSharesTest.php | 2 +- .../Owncloud/UpdateLanguageCodesTest.php | 2 +- tests/lib/Repair/RepairCollationTest.php | 2 +- tests/lib/Repair/RepairInvalidSharesTest.php | 2 +- tests/lib/Repair/RepairMimeTypesTest.php | 2 +- tests/lib/Route/RouterTest.php | 2 +- tests/lib/Security/CertificateManagerTest.php | 3 +- tests/lib/Security/CredentialsManagerTest.php | 4 +- tests/lib/ServerTest.php | 2 +- tests/lib/Share/HelperTest.php | 5 +-- tests/lib/Share/ShareTest.php | 3 +- .../lib/Share20/DefaultShareProviderTest.php | 2 +- tests/lib/Share20/ManagerTest.php | 2 +- tests/lib/Share20/ShareByMailProviderTest.php | 2 +- tests/lib/SubAdminTest.php | 4 +- tests/lib/SystemTag/SystemTagManagerTest.php | 2 +- .../SystemTag/SystemTagObjectMapperTest.php | 2 +- tests/lib/TagsTest.php | 7 +--- .../lib/TaskProcessing/TaskProcessingTest.php | 4 +- tests/lib/TestCase.php | 12 ++++-- .../lib/TextProcessing/TextProcessingTest.php | 4 +- tests/lib/User/DatabaseTest.php | 3 +- tests/lib/User/Dummy.php | 3 ++ tests/lib/User/ManagerTest.php | 8 ++-- tests/lib/User/SessionTest.php | 2 +- tests/lib/User/UserTest.php | 2 +- tests/lib/UtilCheckServerTest.php | 3 +- tests/lib/UtilTest.php | 2 +- 293 files changed, 350 insertions(+), 581 deletions(-) diff --git a/apps/comments/tests/Unit/AppInfo/ApplicationTest.php b/apps/comments/tests/Unit/AppInfo/ApplicationTest.php index 119db5333b5..ac34c97778c 100644 --- a/apps/comments/tests/Unit/AppInfo/ApplicationTest.php +++ b/apps/comments/tests/Unit/AppInfo/ApplicationTest.php @@ -24,10 +24,9 @@ use Test\TestCase; /** * 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(); diff --git a/apps/contactsinteraction/tests/Db/RecentContactMapperTest.php b/apps/contactsinteraction/tests/Db/RecentContactMapperTest.php index f5df27820d7..e3b844e6638 100644 --- a/apps/contactsinteraction/tests/Db/RecentContactMapperTest.php +++ b/apps/contactsinteraction/tests/Db/RecentContactMapperTest.php @@ -17,9 +17,7 @@ use Sabre\VObject\Component\VCard; use Sabre\VObject\UUIDUtil; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class RecentContactMapperTest extends TestCase { private RecentContactMapper $recentContactMapper; private ITimeFactory $time; diff --git a/apps/dav/lib/Connector/Sabre/Directory.php b/apps/dav/lib/Connector/Sabre/Directory.php index fe09c3f423f..a5333fbc9d9 100644 --- a/apps/dav/lib/Connector/Sabre/Directory.php +++ b/apps/dav/lib/Connector/Sabre/Directory.php @@ -211,7 +211,7 @@ class Directory extends Node implements \Sabre\DAV\ICollection, \Sabre\DAV\IQuot } 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()) { diff --git a/apps/dav/lib/Server.php b/apps/dav/lib/Server.php index 9e943484465..7398af83377 100644 --- a/apps/dav/lib/Server.php +++ b/apps/dav/lib/Server.php @@ -92,6 +92,7 @@ use OCP\ISession; 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 @@ class Server { \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()); diff --git a/apps/dav/tests/integration/DAV/Sharing/CalDavSharingBackendTest.php b/apps/dav/tests/integration/DAV/Sharing/CalDavSharingBackendTest.php index ef60817b84c..c8d0b041fcd 100644 --- a/apps/dav/tests/integration/DAV/Sharing/CalDavSharingBackendTest.php +++ b/apps/dav/tests/integration/DAV/Sharing/CalDavSharingBackendTest.php @@ -28,9 +28,7 @@ use PHPUnit\Framework\MockObject\MockObject; use Psr\Log\LoggerInterface; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CalDavSharingBackendTest extends TestCase { private IDBConnection $db; diff --git a/apps/dav/tests/integration/DAV/Sharing/SharingMapperTest.php b/apps/dav/tests/integration/DAV/Sharing/SharingMapperTest.php index bcf84254034..68dec89aa3e 100644 --- a/apps/dav/tests/integration/DAV/Sharing/SharingMapperTest.php +++ b/apps/dav/tests/integration/DAV/Sharing/SharingMapperTest.php @@ -14,9 +14,7 @@ use OCP\IDBConnection; use OCP\Server; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class SharingMapperTest extends TestCase { private SharingMapper $mapper; diff --git a/apps/dav/tests/integration/Db/PropertyMapperTest.php b/apps/dav/tests/integration/Db/PropertyMapperTest.php index 9bd47a82d35..22ddbdf3bb4 100644 --- a/apps/dav/tests/integration/Db/PropertyMapperTest.php +++ b/apps/dav/tests/integration/Db/PropertyMapperTest.php @@ -13,9 +13,7 @@ use OCA\DAV\Db\PropertyMapper; use OCP\Server; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class PropertyMapperTest extends TestCase { /** @var PropertyMapper */ diff --git a/apps/dav/tests/integration/UserMigration/CalendarMigratorTest.php b/apps/dav/tests/integration/UserMigration/CalendarMigratorTest.php index 0682733a103..ac287933037 100644 --- a/apps/dav/tests/integration/UserMigration/CalendarMigratorTest.php +++ b/apps/dav/tests/integration/UserMigration/CalendarMigratorTest.php @@ -22,9 +22,7 @@ use Symfony\Component\Console\Output\OutputInterface; use Test\TestCase; use function scandir; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CalendarMigratorTest extends TestCase { private IUserManager $userManager; diff --git a/apps/dav/tests/integration/UserMigration/ContactsMigratorTest.php b/apps/dav/tests/integration/UserMigration/ContactsMigratorTest.php index 9ae1ca28501..bc7d42d1070 100644 --- a/apps/dav/tests/integration/UserMigration/ContactsMigratorTest.php +++ b/apps/dav/tests/integration/UserMigration/ContactsMigratorTest.php @@ -22,9 +22,7 @@ use Symfony\Component\Console\Output\OutputInterface; use Test\TestCase; use function scandir; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ContactsMigratorTest extends TestCase { private IUserManager $userManager; diff --git a/apps/dav/tests/unit/AppInfo/ApplicationTest.php b/apps/dav/tests/unit/AppInfo/ApplicationTest.php index 336f487e0b8..5af3836741e 100644 --- a/apps/dav/tests/unit/AppInfo/ApplicationTest.php +++ b/apps/dav/tests/unit/AppInfo/ApplicationTest.php @@ -16,10 +16,10 @@ use Test\TestCase; /** * 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(); diff --git a/apps/dav/tests/unit/BackgroundJob/UserStatusAutomationTest.php b/apps/dav/tests/unit/BackgroundJob/UserStatusAutomationTest.php index d49d20180d9..41e13df975b 100644 --- a/apps/dav/tests/unit/BackgroundJob/UserStatusAutomationTest.php +++ b/apps/dav/tests/unit/BackgroundJob/UserStatusAutomationTest.php @@ -25,9 +25,7 @@ use PHPUnit\Framework\MockObject\MockObject; 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; diff --git a/apps/dav/tests/unit/CalDAV/AbstractCalDavBackend.php b/apps/dav/tests/unit/CalDAV/AbstractCalDavBackend.php index 86fc6d06b3b..3e419e086ac 100644 --- a/apps/dav/tests/unit/CalDAV/AbstractCalDavBackend.php +++ b/apps/dav/tests/unit/CalDAV/AbstractCalDavBackend.php @@ -39,10 +39,10 @@ use Test\TestCase; /** * Class CalDavBackendTest * - * @group DB * * @package OCA\DAV\Tests\unit\CalDAV */ +#[\PHPUnit\Framework\Attributes\Group('DB')] abstract class AbstractCalDavBackend extends TestCase { diff --git a/apps/dav/tests/unit/CalDAV/Activity/Filter/GenericTest.php b/apps/dav/tests/unit/CalDAV/Activity/Filter/GenericTest.php index 87b55f14bcc..ca8f5d5669f 100644 --- a/apps/dav/tests/unit/CalDAV/Activity/Filter/GenericTest.php +++ b/apps/dav/tests/unit/CalDAV/Activity/Filter/GenericTest.php @@ -13,9 +13,7 @@ use OCP\Activity\IFilter; use OCP\Server; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class GenericTest extends TestCase { public static function dataFilters(): array { return [ diff --git a/apps/dav/tests/unit/CalDAV/CalDavBackendTest.php b/apps/dav/tests/unit/CalDAV/CalDavBackendTest.php index f9205d5d322..abb01551037 100644 --- a/apps/dav/tests/unit/CalDAV/CalDavBackendTest.php +++ b/apps/dav/tests/unit/CalDAV/CalDavBackendTest.php @@ -28,9 +28,8 @@ use function time; /** * Class CalDavBackendTest - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CalDavBackendTest extends AbstractCalDavBackend { public function testCalendarOperations(): void { $calendarId = $this->createTestCalendar(); diff --git a/apps/dav/tests/unit/CalDAV/PublicCalendarRootTest.php b/apps/dav/tests/unit/CalDAV/PublicCalendarRootTest.php index 39b8427ba6d..ee3be9fb16c 100644 --- a/apps/dav/tests/unit/CalDAV/PublicCalendarRootTest.php +++ b/apps/dav/tests/unit/CalDAV/PublicCalendarRootTest.php @@ -29,10 +29,10 @@ use Test\TestCase; /** * 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; diff --git a/apps/dav/tests/unit/CalDAV/Reminder/NotificationProviderManagerTest.php b/apps/dav/tests/unit/CalDAV/Reminder/NotificationProviderManagerTest.php index 6b813ed0228..4bea30dbfc9 100644 --- a/apps/dav/tests/unit/CalDAV/Reminder/NotificationProviderManagerTest.php +++ b/apps/dav/tests/unit/CalDAV/Reminder/NotificationProviderManagerTest.php @@ -17,9 +17,7 @@ use OCA\DAV\Capabilities; use OCP\AppFramework\QueryException; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class NotificationProviderManagerTest extends TestCase { private NotificationProviderManager $providerManager; diff --git a/apps/dav/tests/unit/CardDAV/CardDavBackendTest.php b/apps/dav/tests/unit/CardDAV/CardDavBackendTest.php index 4e360ed31a3..00ab531dc90 100644 --- a/apps/dav/tests/unit/CardDAV/CardDavBackendTest.php +++ b/apps/dav/tests/unit/CardDAV/CardDavBackendTest.php @@ -43,10 +43,10 @@ use function time; /** * 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; diff --git a/apps/dav/tests/unit/Command/RemoveInvalidSharesTest.php b/apps/dav/tests/unit/Command/RemoveInvalidSharesTest.php index 7a8518ac70c..6caec926636 100644 --- a/apps/dav/tests/unit/Command/RemoveInvalidSharesTest.php +++ b/apps/dav/tests/unit/Command/RemoveInvalidSharesTest.php @@ -23,8 +23,8 @@ use Test\TestCase; * Class RemoveInvalidSharesTest * * @package OCA\DAV\Tests\Unit\Repair - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class RemoveInvalidSharesTest extends TestCase { private RemoveInvalidShares $command; diff --git a/apps/dav/tests/unit/Connector/LegacyPublicAuthTest.php b/apps/dav/tests/unit/Connector/LegacyPublicAuthTest.php index 8b8c775c8ec..2c25b32fa02 100644 --- a/apps/dav/tests/unit/Connector/LegacyPublicAuthTest.php +++ b/apps/dav/tests/unit/Connector/LegacyPublicAuthTest.php @@ -20,10 +20,10 @@ use PHPUnit\Framework\MockObject\MockObject; /** * 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; diff --git a/apps/dav/tests/unit/Connector/Sabre/AuthTest.php b/apps/dav/tests/unit/Connector/Sabre/AuthTest.php index 4b42a815708..cf6efea0c5b 100644 --- a/apps/dav/tests/unit/Connector/Sabre/AuthTest.php +++ b/apps/dav/tests/unit/Connector/Sabre/AuthTest.php @@ -27,8 +27,8 @@ use Test\TestCase; * 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; diff --git a/apps/dav/tests/unit/Connector/Sabre/BearerAuthTest.php b/apps/dav/tests/unit/Connector/Sabre/BearerAuthTest.php index 1e6267d4cbb..01038747bde 100644 --- a/apps/dav/tests/unit/Connector/Sabre/BearerAuthTest.php +++ b/apps/dav/tests/unit/Connector/Sabre/BearerAuthTest.php @@ -19,9 +19,7 @@ use Sabre\HTTP\RequestInterface; 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; diff --git a/apps/dav/tests/unit/Connector/Sabre/CustomPropertiesBackendTest.php b/apps/dav/tests/unit/Connector/Sabre/CustomPropertiesBackendTest.php index cafbdd3ca40..1ffe47f70d8 100644 --- a/apps/dav/tests/unit/Connector/Sabre/CustomPropertiesBackendTest.php +++ b/apps/dav/tests/unit/Connector/Sabre/CustomPropertiesBackendTest.php @@ -22,10 +22,10 @@ use Sabre\DAV\Tree; /** * 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; diff --git a/apps/dav/tests/unit/Connector/Sabre/DirectoryTest.php b/apps/dav/tests/unit/Connector/Sabre/DirectoryTest.php index 421ee1bdc12..9ebe921d5b2 100644 --- a/apps/dav/tests/unit/Connector/Sabre/DirectoryTest.php +++ b/apps/dav/tests/unit/Connector/Sabre/DirectoryTest.php @@ -55,9 +55,7 @@ class TestViewDirectory extends View { } -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class DirectoryTest extends \Test\TestCase { use UserTrait; diff --git a/apps/dav/tests/unit/Connector/Sabre/FileTest.php b/apps/dav/tests/unit/Connector/Sabre/FileTest.php index 60c8382e131..64ee2df0253 100644 --- a/apps/dav/tests/unit/Connector/Sabre/FileTest.php +++ b/apps/dav/tests/unit/Connector/Sabre/FileTest.php @@ -46,10 +46,10 @@ use Test\Traits\UserTrait; /** * Class File * - * @group DB * * @package OCA\DAV\Tests\unit\Connector\Sabre */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class FileTest extends TestCase { use MountProviderTrait; use UserTrait; diff --git a/apps/dav/tests/unit/Connector/Sabre/FilesPluginTest.php b/apps/dav/tests/unit/Connector/Sabre/FilesPluginTest.php index 4df3accfda9..8d27bc337e4 100644 --- a/apps/dav/tests/unit/Connector/Sabre/FilesPluginTest.php +++ b/apps/dav/tests/unit/Connector/Sabre/FilesPluginTest.php @@ -35,9 +35,7 @@ use Sabre\HTTP\ResponseInterface; use Sabre\Xml\Service; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class FilesPluginTest extends TestCase { private Tree&MockObject $tree; diff --git a/apps/dav/tests/unit/Connector/Sabre/NodeTest.php b/apps/dav/tests/unit/Connector/Sabre/NodeTest.php index 11970769a1e..ddea281ea1d 100644 --- a/apps/dav/tests/unit/Connector/Sabre/NodeTest.php +++ b/apps/dav/tests/unit/Connector/Sabre/NodeTest.php @@ -29,9 +29,9 @@ use PHPUnit\Framework\MockObject\MockObject; /** * 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 [ diff --git a/apps/dav/tests/unit/Connector/Sabre/ObjectTreeTest.php b/apps/dav/tests/unit/Connector/Sabre/ObjectTreeTest.php index b07778e4fbd..592d6c958d0 100644 --- a/apps/dav/tests/unit/Connector/Sabre/ObjectTreeTest.php +++ b/apps/dav/tests/unit/Connector/Sabre/ObjectTreeTest.php @@ -23,10 +23,10 @@ use OCP\Files\Mount\IMountManager; /** * 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 [ diff --git a/apps/dav/tests/unit/Connector/Sabre/PublicAuthTest.php b/apps/dav/tests/unit/Connector/Sabre/PublicAuthTest.php index 26d0ef541f2..7fb2339c6f4 100644 --- a/apps/dav/tests/unit/Connector/Sabre/PublicAuthTest.php +++ b/apps/dav/tests/unit/Connector/Sabre/PublicAuthTest.php @@ -22,10 +22,10 @@ use Psr\Log\LoggerInterface; /** * Class PublicAuthTest * - * @group DB * * @package OCA\DAV\Tests\unit\Connector */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class PublicAuthTest extends \Test\TestCase { private ISession&MockObject $session; diff --git a/apps/dav/tests/unit/Connector/Sabre/RequestTest/DeleteTest.php b/apps/dav/tests/unit/Connector/Sabre/RequestTest/DeleteTest.php index 7d3488e6b5a..34cc0cd5fb3 100644 --- a/apps/dav/tests/unit/Connector/Sabre/RequestTest/DeleteTest.php +++ b/apps/dav/tests/unit/Connector/Sabre/RequestTest/DeleteTest.php @@ -14,10 +14,10 @@ use OCP\Files\FileInfo; /** * 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(); diff --git a/apps/dav/tests/unit/Connector/Sabre/RequestTest/DownloadTest.php b/apps/dav/tests/unit/Connector/Sabre/RequestTest/DownloadTest.php index 34171963ef0..490833b911d 100644 --- a/apps/dav/tests/unit/Connector/Sabre/RequestTest/DownloadTest.php +++ b/apps/dav/tests/unit/Connector/Sabre/RequestTest/DownloadTest.php @@ -14,10 +14,10 @@ use OCP\Lock\ILockingProvider; /** * 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(); diff --git a/apps/dav/tests/unit/Connector/Sabre/RequestTest/EncryptionMasterKeyUploadTest.php b/apps/dav/tests/unit/Connector/Sabre/RequestTest/EncryptionMasterKeyUploadTest.php index 615490ddc92..cf552e4d4c8 100644 --- a/apps/dav/tests/unit/Connector/Sabre/RequestTest/EncryptionMasterKeyUploadTest.php +++ b/apps/dav/tests/unit/Connector/Sabre/RequestTest/EncryptionMasterKeyUploadTest.php @@ -17,10 +17,10 @@ use Test\Traits\EncryptionTrait; /** * Class EncryptionMasterKeyUploadTest * - * @group DB * * @package OCA\DAV\Tests\Unit\Connector\Sabre\RequestTest */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class EncryptionMasterKeyUploadTest extends UploadTest { use EncryptionTrait; diff --git a/apps/dav/tests/unit/Connector/Sabre/RequestTest/EncryptionUploadTest.php b/apps/dav/tests/unit/Connector/Sabre/RequestTest/EncryptionUploadTest.php index efa7bb54cf8..19b2b170a9d 100644 --- a/apps/dav/tests/unit/Connector/Sabre/RequestTest/EncryptionUploadTest.php +++ b/apps/dav/tests/unit/Connector/Sabre/RequestTest/EncryptionUploadTest.php @@ -17,10 +17,10 @@ use Test\Traits\EncryptionTrait; /** * Class EncryptionUploadTest * - * @group DB * * @package OCA\DAV\Tests\Unit\Connector\Sabre\RequestTest */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class EncryptionUploadTest extends UploadTest { use EncryptionTrait; diff --git a/apps/dav/tests/unit/Connector/Sabre/RequestTest/PartFileInRootUploadTest.php b/apps/dav/tests/unit/Connector/Sabre/RequestTest/PartFileInRootUploadTest.php index e6fa489fb24..bd22f371858 100644 --- a/apps/dav/tests/unit/Connector/Sabre/RequestTest/PartFileInRootUploadTest.php +++ b/apps/dav/tests/unit/Connector/Sabre/RequestTest/PartFileInRootUploadTest.php @@ -14,10 +14,10 @@ use OCP\Server; /** * 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); diff --git a/apps/dav/tests/unit/Connector/Sabre/RequestTest/UploadTest.php b/apps/dav/tests/unit/Connector/Sabre/RequestTest/UploadTest.php index ec285363507..1bb1b1e42c4 100644 --- a/apps/dav/tests/unit/Connector/Sabre/RequestTest/UploadTest.php +++ b/apps/dav/tests/unit/Connector/Sabre/RequestTest/UploadTest.php @@ -16,10 +16,10 @@ use OCP\Lock\ILockingProvider; /** * 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(); diff --git a/apps/dav/tests/unit/DAV/CustomPropertiesBackendTest.php b/apps/dav/tests/unit/DAV/CustomPropertiesBackendTest.php index 08fdf288f9c..3e333bddabf 100644 --- a/apps/dav/tests/unit/DAV/CustomPropertiesBackendTest.php +++ b/apps/dav/tests/unit/DAV/CustomPropertiesBackendTest.php @@ -25,9 +25,7 @@ use Sabre\DAVACL\IACL; use Sabre\DAVACL\IPrincipal; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CustomPropertiesBackendTest extends TestCase { private const BASE_URI = '/remote.php/dav/'; diff --git a/apps/dav/tests/unit/Migration/CalDAVRemoveEmptyValueTest.php b/apps/dav/tests/unit/Migration/CalDAVRemoveEmptyValueTest.php index 1852d2709c1..18c4ebb020e 100644 --- a/apps/dav/tests/unit/Migration/CalDAVRemoveEmptyValueTest.php +++ b/apps/dav/tests/unit/Migration/CalDAVRemoveEmptyValueTest.php @@ -21,8 +21,8 @@ use Test\TestCase; * Class CalDAVRemoveEmptyValueTest * * @package OCA\DAV\Tests\Unit\DAV\Migration - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CalDAVRemoveEmptyValueTest extends TestCase { private LoggerInterface&MockObject $logger; private CalDavBackend&MockObject $backend; diff --git a/apps/dav/tests/unit/ServerTest.php b/apps/dav/tests/unit/ServerTest.php index 9ffe86d3053..96574643a8d 100644 --- a/apps/dav/tests/unit/ServerTest.php +++ b/apps/dav/tests/unit/ServerTest.php @@ -14,10 +14,10 @@ use OCP\IRequest; /** * Class ServerTest * - * @group DB * * @package OCA\DAV\Tests\Unit */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ServerTest extends \Test\TestCase { #[\PHPUnit\Framework\Attributes\DataProvider('providesUris')] diff --git a/apps/encryption/tests/Command/FixEncryptedVersionTest.php b/apps/encryption/tests/Command/FixEncryptedVersionTest.php index 6376e7293ab..b94d4400d5d 100644 --- a/apps/encryption/tests/Command/FixEncryptedVersionTest.php +++ b/apps/encryption/tests/Command/FixEncryptedVersionTest.php @@ -31,9 +31,9 @@ use Test\Traits\UserTrait; /** * Class FixEncryptedVersionTest * - * @group DB * @package OCA\Encryption\Tests\Command */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class FixEncryptedVersionTest extends TestCase { use MountProviderTrait; use EncryptionTrait; diff --git a/apps/encryption/tests/Command/TestEnableMasterKey.php b/apps/encryption/tests/Command/TestEnableMasterKey.php index fe1d635534b..aec1b1de719 100644 --- a/apps/encryption/tests/Command/TestEnableMasterKey.php +++ b/apps/encryption/tests/Command/TestEnableMasterKey.php @@ -19,6 +19,9 @@ use Symfony\Component\Console\Output\OutputInterface; use Test\TestCase; class TestEnableMasterKey extends TestCase { + public function __construct() { + parent::__construct(static::class); + } protected EnableMasterKey $enableMasterKey; protected Util&MockObject $util; protected IAppConfig&MockObject $config; diff --git a/apps/encryption/tests/Crypto/CryptTest.php b/apps/encryption/tests/Crypto/CryptTest.php index 1355e2c855d..cb817687b88 100644 --- a/apps/encryption/tests/Crypto/CryptTest.php +++ b/apps/encryption/tests/Crypto/CryptTest.php @@ -306,9 +306,8 @@ class CryptTest extends TestCase { /** * test decrypt() - * - * @depends testEncrypt */ + #[\PHPUnit\Framework\Attributes\Depends('testEncrypt')] public function testDecrypt($data): void { $result = self::invokePrivate( $this->crypt, diff --git a/apps/encryption/tests/EncryptedStorageTest.php b/apps/encryption/tests/EncryptedStorageTest.php index 59f419a7f7a..a47a71370eb 100644 --- a/apps/encryption/tests/EncryptedStorageTest.php +++ b/apps/encryption/tests/EncryptedStorageTest.php @@ -23,9 +23,7 @@ class TemporaryNoEncrypted extends Temporary implements IDisableEncryptionStorag } -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class EncryptedStorageTest extends TestCase { use MountProviderTrait; use EncryptionTrait; diff --git a/apps/encryption/tests/Listeners/UserEventsListenersTest.php b/apps/encryption/tests/Listeners/UserEventsListenersTest.php index 5c2cdb1e6dd..15b99c2e206 100644 --- a/apps/encryption/tests/Listeners/UserEventsListenersTest.php +++ b/apps/encryption/tests/Listeners/UserEventsListenersTest.php @@ -30,9 +30,7 @@ use OCP\User\Events\UserLoggedOutEvent; use PHPUnit\Framework\MockObject\MockObject; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class UserEventsListenersTest extends TestCase { protected Util&MockObject $util; diff --git a/apps/encryption/tests/PassphraseServiceTest.php b/apps/encryption/tests/PassphraseServiceTest.php index c2dc9d8173c..c275b6aadea 100644 --- a/apps/encryption/tests/PassphraseServiceTest.php +++ b/apps/encryption/tests/PassphraseServiceTest.php @@ -21,9 +21,7 @@ use PHPUnit\Framework\MockObject\MockObject; use Psr\Log\LoggerInterface; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class PassphraseServiceTest extends TestCase { protected Util&MockObject $util; diff --git a/apps/encryption/tests/SessionTest.php b/apps/encryption/tests/SessionTest.php index c24855e1ac4..b12ff495bff 100644 --- a/apps/encryption/tests/SessionTest.php +++ b/apps/encryption/tests/SessionTest.php @@ -28,17 +28,13 @@ class SessionTest extends TestCase { $this->instance->getPrivateKey(); } - /** - * @depends testThatGetPrivateKeyThrowsExceptionWhenNotSet - */ + #[\PHPUnit\Framework\Attributes\Depends('testThatGetPrivateKeyThrowsExceptionWhenNotSet')] public function testSetAndGetPrivateKey(): void { $this->instance->setPrivateKey('dummyPrivateKey'); $this->assertEquals('dummyPrivateKey', $this->instance->getPrivateKey()); } - /** - * @depends testSetAndGetPrivateKey - */ + #[\PHPUnit\Framework\Attributes\Depends('testSetAndGetPrivateKey')] public function testIsPrivateKeySet(): void { $this->instance->setPrivateKey('dummyPrivateKey'); $this->assertTrue($this->instance->isPrivateKeySet()); diff --git a/apps/federatedfilesharing/tests/Controller/RequestHandlerControllerTest.php b/apps/federatedfilesharing/tests/Controller/RequestHandlerControllerTest.php index 81c67a29254..e293f440fe3 100644 --- a/apps/federatedfilesharing/tests/Controller/RequestHandlerControllerTest.php +++ b/apps/federatedfilesharing/tests/Controller/RequestHandlerControllerTest.php @@ -31,8 +31,8 @@ use Psr\Log\LoggerInterface; * Class RequestHandlerTest * * @package OCA\FederatedFileSharing\Tests - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class RequestHandlerControllerTest extends \Test\TestCase { private string $owner = 'owner'; private string $user1 = 'user1'; diff --git a/apps/federatedfilesharing/tests/FederatedShareProviderTest.php b/apps/federatedfilesharing/tests/FederatedShareProviderTest.php index 7e97510af22..6ec0ca13852 100644 --- a/apps/federatedfilesharing/tests/FederatedShareProviderTest.php +++ b/apps/federatedfilesharing/tests/FederatedShareProviderTest.php @@ -36,8 +36,8 @@ use Psr\Log\LoggerInterface; * Class FederatedShareProviderTest * * @package OCA\FederatedFileSharing\Tests - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class FederatedShareProviderTest extends \Test\TestCase { protected IDBConnection $connection; protected AddressHandler&MockObject $addressHandler; diff --git a/apps/federatedfilesharing/tests/TestCase.php b/apps/federatedfilesharing/tests/TestCase.php index 499065f3b65..1cb1c064b94 100644 --- a/apps/federatedfilesharing/tests/TestCase.php +++ b/apps/federatedfilesharing/tests/TestCase.php @@ -17,12 +17,9 @@ use OCP\IUserSession; use OCP\Server; /** - * Class Test_Files_Sharing_Base - * - * @group DB - * * Base class for sharing tests. */ +#[\PHPUnit\Framework\Attributes\Group('DB')] abstract class TestCase extends \Test\TestCase { public const TEST_FILES_SHARING_API_USER1 = 'test-share-user1'; public const TEST_FILES_SHARING_API_USER2 = 'test-share-user2'; diff --git a/apps/federation/lib/DbHandler.php b/apps/federation/lib/DbHandler.php index 877663b058a..a56539bb30c 100644 --- a/apps/federation/lib/DbHandler.php +++ b/apps/federation/lib/DbHandler.php @@ -20,8 +20,6 @@ use OCP\IL10N; * Handles all database calls for the federation app * * @todo Port to QBMapper - * - * @group DB * @package OCA\Federation */ class DbHandler { diff --git a/apps/federation/tests/BackgroundJob/GetSharedSecretTest.php b/apps/federation/tests/BackgroundJob/GetSharedSecretTest.php index 943bdf352de..b847a4cfcf6 100644 --- a/apps/federation/tests/BackgroundJob/GetSharedSecretTest.php +++ b/apps/federation/tests/BackgroundJob/GetSharedSecretTest.php @@ -27,10 +27,10 @@ use Psr\Log\LoggerInterface; /** * Class GetSharedSecretTest * - * @group DB * * @package OCA\Federation\Tests\BackgroundJob */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class GetSharedSecretTest extends TestCase { private MockObject&IClient $httpClient; diff --git a/apps/federation/tests/DbHandlerTest.php b/apps/federation/tests/DbHandlerTest.php index a8295ad665a..4e9e1bd2c88 100644 --- a/apps/federation/tests/DbHandlerTest.php +++ b/apps/federation/tests/DbHandlerTest.php @@ -16,9 +16,7 @@ use OCP\Server; use PHPUnit\Framework\MockObject\MockObject; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class DbHandlerTest extends TestCase { private DbHandler $dbHandler; private IL10N&MockObject $il10n; diff --git a/apps/files/tests/Activity/Filter/GenericTest.php b/apps/files/tests/Activity/Filter/GenericTest.php index 40e2f9848b5..8d8a00ededa 100644 --- a/apps/files/tests/Activity/Filter/GenericTest.php +++ b/apps/files/tests/Activity/Filter/GenericTest.php @@ -17,8 +17,8 @@ use Test\TestCase; * Class GenericTest * * @package OCA\Files\Tests\Activity\Filter - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class GenericTest extends TestCase { public static function dataFilters(): array { return [ diff --git a/apps/files/tests/BackgroundJob/DeleteOrphanedItemsJobTest.php b/apps/files/tests/BackgroundJob/DeleteOrphanedItemsJobTest.php index 3f811fca407..17fd69a332c 100644 --- a/apps/files/tests/BackgroundJob/DeleteOrphanedItemsJobTest.php +++ b/apps/files/tests/BackgroundJob/DeleteOrphanedItemsJobTest.php @@ -18,10 +18,10 @@ use Psr\Log\LoggerInterface; /** * Class DeleteOrphanedItemsJobTest * - * @group DB * * @package Test\BackgroundJob */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class DeleteOrphanedItemsJobTest extends \Test\TestCase { protected IDBConnection $connection; protected LoggerInterface $logger; diff --git a/apps/files/tests/BackgroundJob/ScanFilesTest.php b/apps/files/tests/BackgroundJob/ScanFilesTest.php index 00d9ed823f9..22ec049741b 100644 --- a/apps/files/tests/BackgroundJob/ScanFilesTest.php +++ b/apps/files/tests/BackgroundJob/ScanFilesTest.php @@ -27,8 +27,8 @@ use Test\Traits\UserTrait; * Class ScanFilesTest * * @package OCA\Files\Tests\BackgroundJob - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ScanFilesTest extends TestCase { use UserTrait; use MountProviderTrait; diff --git a/apps/files/tests/Command/DeleteOrphanedFilesTest.php b/apps/files/tests/Command/DeleteOrphanedFilesTest.php index a488915e0cb..18b484123b9 100644 --- a/apps/files/tests/Command/DeleteOrphanedFilesTest.php +++ b/apps/files/tests/Command/DeleteOrphanedFilesTest.php @@ -22,10 +22,10 @@ use Test\TestCase; /** * Class DeleteOrphanedFilesTest * - * @group DB * * @package OCA\Files\Tests\Command */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class DeleteOrphanedFilesTest extends TestCase { private DeleteOrphanedFiles $command; diff --git a/apps/files/tests/Controller/ViewControllerTest.php b/apps/files/tests/Controller/ViewControllerTest.php index ede4c7f7fa3..cd466b3e97e 100644 --- a/apps/files/tests/Controller/ViewControllerTest.php +++ b/apps/files/tests/Controller/ViewControllerTest.php @@ -41,10 +41,10 @@ use Test\TestCase; /** * Class ViewControllerTest * - * @group RoutingWeirdness * * @package OCA\Files\Tests\Controller */ +#[\PHPUnit\Framework\Attributes\Group('RoutingWeirdness')] class ViewControllerTest extends TestCase { private ContainerInterface&MockObject $container; private IAppManager&MockObject $appManager; diff --git a/apps/files/tests/Service/TagServiceTest.php b/apps/files/tests/Service/TagServiceTest.php index 424e483102c..05a65180225 100644 --- a/apps/files/tests/Service/TagServiceTest.php +++ b/apps/files/tests/Service/TagServiceTest.php @@ -24,10 +24,10 @@ use PHPUnit\Framework\MockObject\MockObject; /** * Class TagServiceTest * - * @group DB * * @package OCA\Files */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class TagServiceTest extends \Test\TestCase { private string $user; private IUserSession&MockObject $userSession; diff --git a/apps/files_external/tests/Listener/StorePasswordListenerTest.php b/apps/files_external/tests/Listener/StorePasswordListenerTest.php index 04635b7dafd..848d8a35dac 100644 --- a/apps/files_external/tests/Listener/StorePasswordListenerTest.php +++ b/apps/files_external/tests/Listener/StorePasswordListenerTest.php @@ -18,9 +18,7 @@ use OCP\User\Events\UserLoggedInEvent; use PHPUnit\Framework\MockObject\MockObject; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class StorePasswordListenerTest extends TestCase { protected IUser&MockObject $mockedUser; diff --git a/apps/files_external/tests/OwnCloudFunctionsTest.php b/apps/files_external/tests/OwnCloudFunctionsTest.php index 2e052cb86b0..298fb746121 100644 --- a/apps/files_external/tests/OwnCloudFunctionsTest.php +++ b/apps/files_external/tests/OwnCloudFunctionsTest.php @@ -13,10 +13,10 @@ use OCA\Files_External\Lib\Storage\OwnCloud; /** * Class OwnCloudFunctions * - * @group DB * * @package OCA\Files_External\Tests */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class OwnCloudFunctionsTest extends \Test\TestCase { public static function configUrlProvider(): array { return [ diff --git a/apps/files_external/tests/Service/DBConfigServiceTest.php b/apps/files_external/tests/Service/DBConfigServiceTest.php index 85d8b70fda7..87c01dcc85f 100644 --- a/apps/files_external/tests/Service/DBConfigServiceTest.php +++ b/apps/files_external/tests/Service/DBConfigServiceTest.php @@ -14,9 +14,7 @@ use OCP\Security\ICrypto; use OCP\Server; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class DBConfigServiceTest extends TestCase { private IDBConnection $connection; private DBConfigService $dbConfig; diff --git a/apps/files_external/tests/Service/GlobalStoragesServiceTest.php b/apps/files_external/tests/Service/GlobalStoragesServiceTest.php index b4c8617830b..a76005718d3 100644 --- a/apps/files_external/tests/Service/GlobalStoragesServiceTest.php +++ b/apps/files_external/tests/Service/GlobalStoragesServiceTest.php @@ -13,9 +13,7 @@ use OCA\Files_External\MountConfig; use OCA\Files_External\Service\GlobalStoragesService; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class GlobalStoragesServiceTest extends StoragesServiceTestCase { protected function setUp(): void { parent::setUp(); diff --git a/apps/files_external/tests/Service/StoragesServiceTestCase.php b/apps/files_external/tests/Service/StoragesServiceTestCase.php index 052bc578a18..fdc086751af 100644 --- a/apps/files_external/tests/Service/StoragesServiceTestCase.php +++ b/apps/files_external/tests/Service/StoragesServiceTestCase.php @@ -53,9 +53,7 @@ class CleaningDBConfig extends DBConfigService { } } -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] abstract class StoragesServiceTestCase extends \Test\TestCase { protected StoragesService $service; protected BackendService&MockObject $backendService; diff --git a/apps/files_external/tests/Service/UserGlobalStoragesServiceTest.php b/apps/files_external/tests/Service/UserGlobalStoragesServiceTest.php index 2119872ea5b..e38835f2077 100644 --- a/apps/files_external/tests/Service/UserGlobalStoragesServiceTest.php +++ b/apps/files_external/tests/Service/UserGlobalStoragesServiceTest.php @@ -20,9 +20,7 @@ use OCP\Server; use PHPUnit\Framework\MockObject\MockObject; use Test\Traits\UserTrait; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class UserGlobalStoragesServiceTest extends GlobalStoragesServiceTest { use UserTrait; diff --git a/apps/files_external/tests/Service/UserStoragesServiceTest.php b/apps/files_external/tests/Service/UserStoragesServiceTest.php index 8c7c4d1f2db..99482a9cbbe 100644 --- a/apps/files_external/tests/Service/UserStoragesServiceTest.php +++ b/apps/files_external/tests/Service/UserStoragesServiceTest.php @@ -22,9 +22,7 @@ use OCP\Server; use PHPUnit\Framework\MockObject\MockObject; use Test\Traits\UserTrait; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class UserStoragesServiceTest extends StoragesServiceTestCase { use UserTrait; diff --git a/apps/files_external/tests/Storage/Amazons3MultiPartTest.php b/apps/files_external/tests/Storage/Amazons3MultiPartTest.php index b01f65b0b6e..2ba180dbbcf 100644 --- a/apps/files_external/tests/Storage/Amazons3MultiPartTest.php +++ b/apps/files_external/tests/Storage/Amazons3MultiPartTest.php @@ -12,11 +12,11 @@ use OCA\Files_External\Lib\Storage\AmazonS3; /** * Class Amazons3Test * - * @group DB - * @group S3 * * @package OCA\Files_External\Tests\Storage */ +#[\PHPUnit\Framework\Attributes\Group('DB')] +#[\PHPUnit\Framework\Attributes\Group('S3')] class Amazons3MultiPartTest extends \Test\Files\Storage\Storage { use ConfigurableStorageTrait; /** @var AmazonS3 */ diff --git a/apps/files_external/tests/Storage/Amazons3Test.php b/apps/files_external/tests/Storage/Amazons3Test.php index 349161cde25..864d6be03a8 100644 --- a/apps/files_external/tests/Storage/Amazons3Test.php +++ b/apps/files_external/tests/Storage/Amazons3Test.php @@ -13,11 +13,11 @@ use OCA\Files_External\Lib\Storage\AmazonS3; /** * Class Amazons3Test * - * @group DB - * @group S3 * * @package OCA\Files_External\Tests\Storage */ +#[\PHPUnit\Framework\Attributes\Group('DB')] +#[\PHPUnit\Framework\Attributes\Group('S3')] class Amazons3Test extends \Test\Files\Storage\Storage { use ConfigurableStorageTrait; /** @var AmazonS3 */ diff --git a/apps/files_external/tests/Storage/FtpTest.php b/apps/files_external/tests/Storage/FtpTest.php index 22a31d57da8..34a1393053a 100644 --- a/apps/files_external/tests/Storage/FtpTest.php +++ b/apps/files_external/tests/Storage/FtpTest.php @@ -13,10 +13,10 @@ use OCA\Files_External\Lib\Storage\FTP; /** * Class FtpTest * - * @group DB * * @package OCA\Files_External\Tests\Storage */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class FtpTest extends \Test\Files\Storage\Storage { use ConfigurableStorageTrait; diff --git a/apps/files_external/tests/Storage/OwncloudTest.php b/apps/files_external/tests/Storage/OwncloudTest.php index 205ff8ac4a1..0ab66713454 100644 --- a/apps/files_external/tests/Storage/OwncloudTest.php +++ b/apps/files_external/tests/Storage/OwncloudTest.php @@ -13,10 +13,10 @@ use OCA\Files_External\Lib\Storage\OwnCloud; /** * Class OwnCloudTest * - * @group DB * * @package OCA\Files_External\Tests\Storage */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class OwncloudTest extends \Test\Files\Storage\Storage { use ConfigurableStorageTrait; diff --git a/apps/files_external/tests/Storage/SFTP_KeyTest.php b/apps/files_external/tests/Storage/SFTP_KeyTest.php index 4e59685dc13..57ffea340fa 100644 --- a/apps/files_external/tests/Storage/SFTP_KeyTest.php +++ b/apps/files_external/tests/Storage/SFTP_KeyTest.php @@ -13,10 +13,10 @@ use OCA\Files_External\Lib\Storage\SFTP_Key; /** * Class SFTP_KeyTest * - * @group DB * * @package OCA\Files_External\Tests\Storage */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class SFTP_KeyTest extends \Test\Files\Storage\Storage { use ConfigurableStorageTrait; diff --git a/apps/files_external/tests/Storage/SftpTest.php b/apps/files_external/tests/Storage/SftpTest.php index 8861d5ce860..28426189655 100644 --- a/apps/files_external/tests/Storage/SftpTest.php +++ b/apps/files_external/tests/Storage/SftpTest.php @@ -13,10 +13,10 @@ use OCA\Files_External\Lib\Storage\SFTP; /** * Class SftpTest * - * @group DB * * @package OCA\Files_External\Tests\Storage */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class SftpTest extends \Test\Files\Storage\Storage { use ConfigurableStorageTrait; /** diff --git a/apps/files_external/tests/Storage/SmbTest.php b/apps/files_external/tests/Storage/SmbTest.php index 6bb941da4f8..dae402a1509 100644 --- a/apps/files_external/tests/Storage/SmbTest.php +++ b/apps/files_external/tests/Storage/SmbTest.php @@ -17,10 +17,10 @@ use PHPUnit\Framework\ExpectationFailedException; /** * Class SmbTest * - * @group DB * * @package OCA\Files_External\Tests\Storage */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class SmbTest extends \Test\Files\Storage\Storage { use ConfigurableStorageTrait; /** diff --git a/apps/files_external/tests/Storage/SwiftTest.php b/apps/files_external/tests/Storage/SwiftTest.php index c77f098bb5d..5f8ef462dc2 100644 --- a/apps/files_external/tests/Storage/SwiftTest.php +++ b/apps/files_external/tests/Storage/SwiftTest.php @@ -14,10 +14,10 @@ use OCA\Files_External\Lib\Storage\Swift; /** * Class SwiftTest * - * @group DB * * @package OCA\Files_External\Tests\Storage */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class SwiftTest extends \Test\Files\Storage\Storage { use ConfigurableStorageTrait; diff --git a/apps/files_external/tests/Storage/VersionedAmazonS3Test.php b/apps/files_external/tests/Storage/VersionedAmazonS3Test.php index 9d413620292..4a195e2ea5b 100644 --- a/apps/files_external/tests/Storage/VersionedAmazonS3Test.php +++ b/apps/files_external/tests/Storage/VersionedAmazonS3Test.php @@ -8,10 +8,8 @@ declare(strict_types=1); namespace OCA\Files_External\Tests\Storage; -/** - * @group DB - * @group S3 - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] +#[\PHPUnit\Framework\Attributes\Group('S3')] class VersionedAmazonS3Test extends Amazons3Test { protected function setUp(): void { parent::setUp(); diff --git a/apps/files_external/tests/Storage/WebdavTest.php b/apps/files_external/tests/Storage/WebdavTest.php index 43f392b1a25..e7abe92afab 100644 --- a/apps/files_external/tests/Storage/WebdavTest.php +++ b/apps/files_external/tests/Storage/WebdavTest.php @@ -16,10 +16,10 @@ use OCP\Server; /** * Class WebdavTest * - * @group DB * * @package OCA\Files_External\Tests\Storage */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class WebdavTest extends \Test\Files\Storage\Storage { use ConfigurableStorageTrait; diff --git a/apps/files_sharing/tests/ApiTest.php b/apps/files_sharing/tests/ApiTest.php index 759e2011c25..8813d46b17d 100644 --- a/apps/files_sharing/tests/ApiTest.php +++ b/apps/files_sharing/tests/ApiTest.php @@ -43,12 +43,11 @@ use Psr\Container\ContainerInterface; use Psr\Log\LoggerInterface; use Test\Traits\EmailValidatorTrait; +// TODO: convert to real integration tests /** * Class ApiTest - * - * @group DB - * TODO: convert to real integration tests */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ApiTest extends TestCase { use EmailValidatorTrait; @@ -217,9 +216,7 @@ class ApiTest extends TestCase { $ocs->cleanup(); } - /** - * @group RoutingWeirdness - */ + #[\PHPUnit\Framework\Attributes\Group('RoutingWeirdness')] public function testCreateShareLink(): void { $ocs = $this->createOCS(self::TEST_FILES_SHARING_API_USER1); $result = $ocs->createShare($this->folder, Constants::PERMISSION_ALL, IShare::TYPE_LINK); @@ -242,10 +239,8 @@ class ApiTest extends TestCase { $ocs->cleanup(); } - /** - * @group RoutingWeirdness - */ #[\PHPUnit\Framework\Attributes\DataProvider('dataAllowFederationOnPublicShares')] + #[\PHPUnit\Framework\Attributes\Group('RoutingWeirdness')] public function testCreateShareLinkPublicUpload(array $appConfig, int $permissions): void { $this->appConfig->method('getValueBool') ->willReturnMap([$appConfig]); @@ -441,8 +436,8 @@ class ApiTest extends TestCase { /** * @medium - * @group RoutingWeirdness */ + #[\PHPUnit\Framework\Attributes\Group('RoutingWeirdness')] public function testPublicLinkUrl(): void { $ocs = $this->createOCS(self::TEST_FILES_SHARING_API_USER1); $result = $ocs->createShare($this->folder, Constants::PERMISSION_ALL, IShare::TYPE_LINK); @@ -489,9 +484,9 @@ class ApiTest extends TestCase { /** * @medium - * @depends testCreateShareUserFile - * @depends testCreateShareLink */ + #[\PHPUnit\Framework\Attributes\Depends('testCreateShareUserFile')] + #[\PHPUnit\Framework\Attributes\Depends('testCreateShareLink')] public function testGetShareFromSource(): void { $node = $this->userFolder->get($this->filename); $share = $this->shareManager->newShare(); @@ -522,9 +517,9 @@ class ApiTest extends TestCase { /** * @medium - * @depends testCreateShareUserFile - * @depends testCreateShareLink */ + #[\PHPUnit\Framework\Attributes\Depends('testCreateShareUserFile')] + #[\PHPUnit\Framework\Attributes\Depends('testCreateShareLink')] public function testGetShareFromSourceWithReshares(): void { $node = $this->userFolder->get($this->filename); $share1 = $this->shareManager->newShare(); @@ -564,8 +559,8 @@ class ApiTest extends TestCase { /** * @medium - * @depends testCreateShareUserFile */ + #[\PHPUnit\Framework\Attributes\Depends('testCreateShareUserFile')] public function testGetShareFromId(): void { $node = $this->userFolder->get($this->filename); $share1 = $this->shareManager->newShare(); @@ -958,9 +953,9 @@ class ApiTest extends TestCase { /** * @medium - * @depends testCreateShareUserFile - * @depends testCreateShareLink */ + #[\PHPUnit\Framework\Attributes\Depends('testCreateShareUserFile')] + #[\PHPUnit\Framework\Attributes\Depends('testCreateShareLink')] public function testUpdateShare(): void { $password = md5(time()); @@ -1135,8 +1130,8 @@ class ApiTest extends TestCase { /** * @medium - * @depends testCreateShareUserFile */ + #[\PHPUnit\Framework\Attributes\Depends('testCreateShareUserFile')] public function testDeleteShare(): void { $node1 = $this->userFolder->get($this->filename); $share1 = $this->shareManager->newShare(); @@ -1323,10 +1318,9 @@ class ApiTest extends TestCase { /** * Make sure only ISO 8601 dates are accepted - * - * @group RoutingWeirdness */ #[\PHPUnit\Framework\Attributes\DataProvider('datesProvider')] + #[\PHPUnit\Framework\Attributes\Group('RoutingWeirdness')] public function testPublicLinkExpireDate($date, $valid): void { $ocs = $this->createOCS(self::TEST_FILES_SHARING_API_USER1); @@ -1356,9 +1350,7 @@ class ApiTest extends TestCase { $this->shareManager->deleteShare($share); } - /** - * @group RoutingWeirdness - */ + #[\PHPUnit\Framework\Attributes\Group('RoutingWeirdness')] public function testCreatePublicLinkExpireDateValid(): void { $config = Server::get(IConfig::class); diff --git a/apps/files_sharing/tests/CacheTest.php b/apps/files_sharing/tests/CacheTest.php index e95d3d4f91a..d451a9c2061 100644 --- a/apps/files_sharing/tests/CacheTest.php +++ b/apps/files_sharing/tests/CacheTest.php @@ -22,9 +22,8 @@ use OCP\Share\IShare; /** * Class CacheTest - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CacheTest extends TestCase { /** diff --git a/apps/files_sharing/tests/CapabilitiesTest.php b/apps/files_sharing/tests/CapabilitiesTest.php index 644cde444bd..914be0702c3 100644 --- a/apps/files_sharing/tests/CapabilitiesTest.php +++ b/apps/files_sharing/tests/CapabilitiesTest.php @@ -31,9 +31,8 @@ use Psr\Log\LoggerInterface; /** * Class CapabilitiesTest - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CapabilitiesTest extends \Test\TestCase { /** diff --git a/apps/files_sharing/tests/Command/CleanupRemoteStoragesTest.php b/apps/files_sharing/tests/Command/CleanupRemoteStoragesTest.php index 6f0960bf46c..d9c832a3185 100644 --- a/apps/files_sharing/tests/Command/CleanupRemoteStoragesTest.php +++ b/apps/files_sharing/tests/Command/CleanupRemoteStoragesTest.php @@ -20,10 +20,10 @@ use Test\TestCase; /** * Class CleanupRemoteStoragesTest * - * @group DB * * @package OCA\Files_Sharing\Tests\Command */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CleanupRemoteStoragesTest extends TestCase { protected IDBConnection $connection; diff --git a/apps/files_sharing/tests/Controller/ShareAPIControllerTest.php b/apps/files_sharing/tests/Controller/ShareAPIControllerTest.php index 34f06e1bcec..30a946c7847 100644 --- a/apps/files_sharing/tests/Controller/ShareAPIControllerTest.php +++ b/apps/files_sharing/tests/Controller/ShareAPIControllerTest.php @@ -63,8 +63,8 @@ use Test\Traits\EmailValidatorTrait; * Class ShareAPIControllerTest * * @package OCA\Files_Sharing\Tests\Controller - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ShareAPIControllerTest extends TestCase { use EmailValidatorTrait; diff --git a/apps/files_sharing/tests/Controller/ShareControllerTest.php b/apps/files_sharing/tests/Controller/ShareControllerTest.php index 28d26226673..a8381e70fd7 100644 --- a/apps/files_sharing/tests/Controller/ShareControllerTest.php +++ b/apps/files_sharing/tests/Controller/ShareControllerTest.php @@ -50,10 +50,9 @@ use OCP\Share\IShare; use PHPUnit\Framework\MockObject\MockObject; /** - * @group DB - * * @package OCA\Files_Sharing\Controllers */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ShareControllerTest extends \Test\TestCase { private string $user; diff --git a/apps/files_sharing/tests/Controller/ShareesAPIControllerTest.php b/apps/files_sharing/tests/Controller/ShareesAPIControllerTest.php index 18e1bf0347b..dcf0fcb962f 100644 --- a/apps/files_sharing/tests/Controller/ShareesAPIControllerTest.php +++ b/apps/files_sharing/tests/Controller/ShareesAPIControllerTest.php @@ -23,10 +23,10 @@ use PHPUnit\Framework\MockObject\MockObject; /** * Class ShareesTest * - * @group DB * * @package OCA\Files_Sharing\Tests\API */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ShareesAPIControllerTest extends TestCase { /** @var ShareesAPIController */ protected $sharees; diff --git a/apps/files_sharing/tests/DeleteOrphanedSharesJobTest.php b/apps/files_sharing/tests/DeleteOrphanedSharesJobTest.php index c245d157151..820c5e107f1 100644 --- a/apps/files_sharing/tests/DeleteOrphanedSharesJobTest.php +++ b/apps/files_sharing/tests/DeleteOrphanedSharesJobTest.php @@ -20,10 +20,10 @@ use OCP\Share\IShare; /** * Class DeleteOrphanedSharesJobTest * - * @group DB * * @package OCA\Files_Sharing\Tests */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class DeleteOrphanedSharesJobTest extends \Test\TestCase { /** * @var bool diff --git a/apps/files_sharing/tests/EncryptedSizePropagationTest.php b/apps/files_sharing/tests/EncryptedSizePropagationTest.php index 1be17df3957..284fe617212 100644 --- a/apps/files_sharing/tests/EncryptedSizePropagationTest.php +++ b/apps/files_sharing/tests/EncryptedSizePropagationTest.php @@ -12,9 +12,7 @@ use OCP\ITempManager; use OCP\Server; use Test\Traits\EncryptionTrait; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class EncryptedSizePropagationTest extends SizePropagationTest { use EncryptionTrait; diff --git a/apps/files_sharing/tests/EtagPropagationTest.php b/apps/files_sharing/tests/EtagPropagationTest.php index d8580ea92d5..11350082ca0 100644 --- a/apps/files_sharing/tests/EtagPropagationTest.php +++ b/apps/files_sharing/tests/EtagPropagationTest.php @@ -17,10 +17,10 @@ use OCP\Share\IShare; /** * Class EtagPropagationTest * - * @group SLOWDB * * @package OCA\Files_Sharing\Tests */ +#[\PHPUnit\Framework\Attributes\Group('SLOWDB')] class EtagPropagationTest extends PropagationTestCase { /** diff --git a/apps/files_sharing/tests/ExpireSharesJobTest.php b/apps/files_sharing/tests/ExpireSharesJobTest.php index 6a79d132007..e6db2424176 100644 --- a/apps/files_sharing/tests/ExpireSharesJobTest.php +++ b/apps/files_sharing/tests/ExpireSharesJobTest.php @@ -20,10 +20,10 @@ use OCP\Share\IShare; /** * Class ExpireSharesJobTest * - * @group DB * * @package OCA\Files_Sharing\Tests */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ExpireSharesJobTest extends \Test\TestCase { /** @var ExpireSharesJob */ diff --git a/apps/files_sharing/tests/External/CacheTest.php b/apps/files_sharing/tests/External/CacheTest.php index 39e2057a24c..c23101ea49e 100644 --- a/apps/files_sharing/tests/External/CacheTest.php +++ b/apps/files_sharing/tests/External/CacheTest.php @@ -22,10 +22,10 @@ use OCP\IUserManager; /** * Class Cache * - * @group DB * * @package OCA\Files_Sharing\Tests\External */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CacheTest extends TestCase { /** @var IManager|\PHPUnit\Framework\MockObject\MockObject */ protected $contactsManager; diff --git a/apps/files_sharing/tests/External/ManagerTest.php b/apps/files_sharing/tests/External/ManagerTest.php index 06c09fd5422..e1c40b18865 100644 --- a/apps/files_sharing/tests/External/ManagerTest.php +++ b/apps/files_sharing/tests/External/ManagerTest.php @@ -41,10 +41,10 @@ use Test\Traits\UserTrait; /** * Class ManagerTest * - * @group DB * * @package OCA\Files_Sharing\Tests\External */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ManagerTest extends TestCase { use UserTrait; diff --git a/apps/files_sharing/tests/External/ScannerTest.php b/apps/files_sharing/tests/External/ScannerTest.php index 8b44d47f2b1..e7d13755394 100644 --- a/apps/files_sharing/tests/External/ScannerTest.php +++ b/apps/files_sharing/tests/External/ScannerTest.php @@ -12,9 +12,7 @@ use OCA\Files_Sharing\External\Scanner; use OCA\Files_Sharing\External\Storage; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ScannerTest extends TestCase { protected Scanner $scanner; /** @var Storage|\PHPUnit\Framework\MockObject\MockObject */ diff --git a/apps/files_sharing/tests/ExternalStorageTest.php b/apps/files_sharing/tests/ExternalStorageTest.php index 1d9d2eed7bd..84013e3cb21 100644 --- a/apps/files_sharing/tests/ExternalStorageTest.php +++ b/apps/files_sharing/tests/ExternalStorageTest.php @@ -16,9 +16,8 @@ use OCP\Http\Client\IResponse; /** * Tests for the external Storage class for remote shares. - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ExternalStorageTest extends \Test\TestCase { public static function optionsProvider() { return [ diff --git a/apps/files_sharing/tests/GroupEtagPropagationTest.php b/apps/files_sharing/tests/GroupEtagPropagationTest.php index da9c7c6bd07..2a499ac8a4f 100644 --- a/apps/files_sharing/tests/GroupEtagPropagationTest.php +++ b/apps/files_sharing/tests/GroupEtagPropagationTest.php @@ -13,10 +13,9 @@ use OCP\Constants; use OCP\Share\IShare; /** - * @group SLOWDB - * * @package OCA\Files_Sharing\Tests */ +#[\PHPUnit\Framework\Attributes\Group('SLOWDB')] class GroupEtagPropagationTest extends PropagationTestCase { /** * "user1" creates /test, /test/sub and shares with group1 diff --git a/apps/files_sharing/tests/HelperTest.php b/apps/files_sharing/tests/HelperTest.php index 4d0d747b3e4..6df3d07bbf4 100644 --- a/apps/files_sharing/tests/HelperTest.php +++ b/apps/files_sharing/tests/HelperTest.php @@ -14,9 +14,8 @@ use OCP\Server; /** * Class HelperTest - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class HelperTest extends TestCase { /** diff --git a/apps/files_sharing/tests/LockingTest.php b/apps/files_sharing/tests/LockingTest.php index 280c364a136..f691d2ddfae 100644 --- a/apps/files_sharing/tests/LockingTest.php +++ b/apps/files_sharing/tests/LockingTest.php @@ -19,10 +19,10 @@ use OCP\Share\IShare; /** * Class LockingTest * - * @group DB * * @package OCA\Files_Sharing\Tests */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class LockingTest extends TestCase { /** * @var \Test\Util\User\Dummy diff --git a/apps/files_sharing/tests/Migration/SetPasswordColumnTest.php b/apps/files_sharing/tests/Migration/SetPasswordColumnTest.php index 5b5c906f144..120b0318e58 100644 --- a/apps/files_sharing/tests/Migration/SetPasswordColumnTest.php +++ b/apps/files_sharing/tests/Migration/SetPasswordColumnTest.php @@ -17,9 +17,8 @@ use OCP\Share\IShare; /** * Class SetPasswordColumnTest - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class SetPasswordColumnTest extends TestCase { /** @var IDBConnection */ diff --git a/apps/files_sharing/tests/MountProviderTest.php b/apps/files_sharing/tests/MountProviderTest.php index e043a1cb1ef..5a5ebae10a7 100644 --- a/apps/files_sharing/tests/MountProviderTest.php +++ b/apps/files_sharing/tests/MountProviderTest.php @@ -25,9 +25,7 @@ use OCP\Share\IShare; use PHPUnit\Framework\MockObject\MockObject; use Psr\Log\LoggerInterface; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class MountProviderTest extends \Test\TestCase { protected MountProvider $provider; diff --git a/apps/files_sharing/tests/ShareTest.php b/apps/files_sharing/tests/ShareTest.php index 737ad6dcb4e..472d9ecaa91 100644 --- a/apps/files_sharing/tests/ShareTest.php +++ b/apps/files_sharing/tests/ShareTest.php @@ -19,9 +19,8 @@ use OCP\Share\IShare; /** * Class ShareTest - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ShareTest extends TestCase { public const TEST_FOLDER_NAME = '/folder_share_api_test'; diff --git a/apps/files_sharing/tests/SharedMountTest.php b/apps/files_sharing/tests/SharedMountTest.php index cc9c70a241f..2741f51dd10 100644 --- a/apps/files_sharing/tests/SharedMountTest.php +++ b/apps/files_sharing/tests/SharedMountTest.php @@ -22,9 +22,8 @@ use OCP\Share\IShare; /** * Class SharedMountTest - * - * @group SLOWDB */ +#[\PHPUnit\Framework\Attributes\Group('SLOWDB')] class SharedMountTest extends TestCase { /** @var IGroupManager */ diff --git a/apps/files_sharing/tests/SharedStorageTest.php b/apps/files_sharing/tests/SharedStorageTest.php index 1c1f0a7b71d..8806721f6f6 100644 --- a/apps/files_sharing/tests/SharedStorageTest.php +++ b/apps/files_sharing/tests/SharedStorageTest.php @@ -25,9 +25,8 @@ use OCP\Share\IShare; /** * Class SharedStorageTest - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class SharedStorageTest extends TestCase { protected function setUp(): void { parent::setUp(); diff --git a/apps/files_sharing/tests/SharesReminderJobTest.php b/apps/files_sharing/tests/SharesReminderJobTest.php index ce468e279ec..dcf2fb1fae1 100644 --- a/apps/files_sharing/tests/SharesReminderJobTest.php +++ b/apps/files_sharing/tests/SharesReminderJobTest.php @@ -29,10 +29,10 @@ use Psr\Log\LoggerInterface; /** * Class SharesReminderJobTest * - * @group DB * * @package OCA\Files_Sharing\Tests */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class SharesReminderJobTest extends \Test\TestCase { private SharesReminderJob $job; private IDBConnection $db; diff --git a/apps/files_sharing/tests/SizePropagationTest.php b/apps/files_sharing/tests/SizePropagationTest.php index e1b67abca90..58429b23394 100644 --- a/apps/files_sharing/tests/SizePropagationTest.php +++ b/apps/files_sharing/tests/SizePropagationTest.php @@ -17,10 +17,10 @@ use Test\Traits\UserTrait; /** * Class SizePropagationTest * - * @group DB * * @package OCA\Files_Sharing\Tests */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class SizePropagationTest extends TestCase { use UserTrait; diff --git a/apps/files_sharing/tests/TestCase.php b/apps/files_sharing/tests/TestCase.php index 0d2022ed341..0e466e4aff2 100644 --- a/apps/files_sharing/tests/TestCase.php +++ b/apps/files_sharing/tests/TestCase.php @@ -27,12 +27,9 @@ use OCP\Share\IShare; use Test\Traits\MountProviderTrait; /** - * Class TestCase - * - * @group DB - * * Base class for sharing tests. */ +#[\PHPUnit\Framework\Attributes\Group('DB')] abstract class TestCase extends \Test\TestCase { use MountProviderTrait; diff --git a/apps/files_sharing/tests/UnshareChildrenTest.php b/apps/files_sharing/tests/UnshareChildrenTest.php index ac870212c99..0a12c02b9a8 100644 --- a/apps/files_sharing/tests/UnshareChildrenTest.php +++ b/apps/files_sharing/tests/UnshareChildrenTest.php @@ -15,10 +15,10 @@ use OCP\Util; /** * Class UnshareChildrenTest * - * @group DB * * @package OCA\Files_Sharing\Tests */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class UnshareChildrenTest extends TestCase { protected $subsubfolder; diff --git a/apps/files_sharing/tests/UpdaterTest.php b/apps/files_sharing/tests/UpdaterTest.php index 23044e0b2f3..c6eb0018842 100644 --- a/apps/files_sharing/tests/UpdaterTest.php +++ b/apps/files_sharing/tests/UpdaterTest.php @@ -21,9 +21,8 @@ use OCP\Share\IShare; /** * Class UpdaterTest - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class UpdaterTest extends TestCase { public const TEST_FOLDER_NAME = '/folder_share_updater_test'; diff --git a/apps/files_sharing/tests/WatcherTest.php b/apps/files_sharing/tests/WatcherTest.php index 15676836915..6b08b1e2371 100644 --- a/apps/files_sharing/tests/WatcherTest.php +++ b/apps/files_sharing/tests/WatcherTest.php @@ -15,9 +15,8 @@ use OCP\Share\IShare; /** * Class WatcherTest - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class WatcherTest extends TestCase { /** @var Storage */ diff --git a/apps/files_trashbin/tests/Command/CleanUpTest.php b/apps/files_trashbin/tests/Command/CleanUpTest.php index 41ed0e1e960..b8c98110da9 100644 --- a/apps/files_trashbin/tests/Command/CleanUpTest.php +++ b/apps/files_trashbin/tests/Command/CleanUpTest.php @@ -24,10 +24,10 @@ use Test\TestCase; /** * Class CleanUpTest * - * @group DB * * @package OCA\Files_Trashbin\Tests\Command */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CleanUpTest extends TestCase { protected IUserManager&MockObject $userManager; protected IRootFolder&MockObject $rootFolder; diff --git a/apps/files_trashbin/tests/Command/ExpireTest.php b/apps/files_trashbin/tests/Command/ExpireTest.php index 5a66dac8c6e..69804babcf5 100644 --- a/apps/files_trashbin/tests/Command/ExpireTest.php +++ b/apps/files_trashbin/tests/Command/ExpireTest.php @@ -14,10 +14,10 @@ use Test\TestCase; /** * Class ExpireTest * - * @group DB * * @package OCA\Files_Trashbin\Tests\Command */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ExpireTest extends TestCase { public function testExpireNonExistingUser(): void { $command = new Expire('test'); diff --git a/apps/files_trashbin/tests/Command/ExpireTrashTest.php b/apps/files_trashbin/tests/Command/ExpireTrashTest.php index edad8bfbd95..37d90bcccd9 100644 --- a/apps/files_trashbin/tests/Command/ExpireTrashTest.php +++ b/apps/files_trashbin/tests/Command/ExpireTrashTest.php @@ -24,10 +24,10 @@ use Test\TestCase; /** * Class ExpireTrashTest * - * @group DB * * @package OCA\Files_Trashbin\Tests\Command */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ExpireTrashTest extends TestCase { private Expiration $expiration; private Node $userFolder; diff --git a/apps/files_trashbin/tests/StorageTest.php b/apps/files_trashbin/tests/StorageTest.php index c58ddec97dd..3b8441a4124 100644 --- a/apps/files_trashbin/tests/StorageTest.php +++ b/apps/files_trashbin/tests/StorageTest.php @@ -46,10 +46,10 @@ class TemporaryNoCross extends Temporary { /** * Class Storage * - * @group DB * * @package OCA\Files_Trashbin\Tests */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class StorageTest extends \Test\TestCase { use MountProviderTrait; diff --git a/apps/files_trashbin/tests/TrashbinTest.php b/apps/files_trashbin/tests/TrashbinTest.php index 8dcd2769558..21c8bdb92f5 100644 --- a/apps/files_trashbin/tests/TrashbinTest.php +++ b/apps/files_trashbin/tests/TrashbinTest.php @@ -33,9 +33,8 @@ use OCP\Share\IShare; /** * Class Test_Encryption - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class TrashbinTest extends \Test\TestCase { public const TEST_TRASHBIN_USER1 = 'test-trashbin-user1'; public const TEST_TRASHBIN_USER2 = 'test-trashbin-user2'; diff --git a/apps/files_versions/tests/Command/CleanupTest.php b/apps/files_versions/tests/Command/CleanupTest.php index dd6665f5aef..99b17bc2981 100644 --- a/apps/files_versions/tests/Command/CleanupTest.php +++ b/apps/files_versions/tests/Command/CleanupTest.php @@ -22,10 +22,10 @@ use Test\TestCase; /** * Class CleanupTest * - * @group DB * * @package OCA\Files_Versions\Tests\Command */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CleanupTest extends TestCase { protected Manager&MockObject $userManager; protected IRootFolder&MockObject $rootFolder; diff --git a/apps/files_versions/tests/Command/ExpireTest.php b/apps/files_versions/tests/Command/ExpireTest.php index b74457a7fd6..9320e3e873f 100644 --- a/apps/files_versions/tests/Command/ExpireTest.php +++ b/apps/files_versions/tests/Command/ExpireTest.php @@ -14,10 +14,10 @@ use Test\TestCase; /** * Class ExpireTest * - * @group DB * * @package OCA\Files_Versions\Tests\Command */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ExpireTest extends TestCase { public function testExpireNonExistingUser(): void { $command = new Expire($this->getUniqueID('test'), ''); diff --git a/apps/files_versions/tests/StorageTest.php b/apps/files_versions/tests/StorageTest.php index 443cff3ee06..596271c6edf 100644 --- a/apps/files_versions/tests/StorageTest.php +++ b/apps/files_versions/tests/StorageTest.php @@ -16,9 +16,7 @@ use OCP\Server; use Test\TestCase; use Test\Traits\UserTrait; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class StorageTest extends TestCase { use UserTrait; diff --git a/apps/files_versions/tests/VersioningTest.php b/apps/files_versions/tests/VersioningTest.php index c9f7d27d7ab..a430f131e78 100644 --- a/apps/files_versions/tests/VersioningTest.php +++ b/apps/files_versions/tests/VersioningTest.php @@ -34,9 +34,8 @@ use OCP\Util; /** * Class Test_Files_versions * this class provide basic files versions test - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class VersioningTest extends \Test\TestCase { public const TEST_VERSIONS_USER = 'test-versions-user'; public const TEST_VERSIONS_USER2 = 'test-versions-user2'; diff --git a/apps/oauth2/tests/Controller/LoginRedirectorControllerTest.php b/apps/oauth2/tests/Controller/LoginRedirectorControllerTest.php index 04ac0bfbd28..bbcbe44909f 100644 --- a/apps/oauth2/tests/Controller/LoginRedirectorControllerTest.php +++ b/apps/oauth2/tests/Controller/LoginRedirectorControllerTest.php @@ -23,9 +23,7 @@ use OCP\Security\ISecureRandom; use PHPUnit\Framework\MockObject\MockObject; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class LoginRedirectorControllerTest extends TestCase { private IRequest&MockObject $request; private IURLGenerator&MockObject $urlGenerator; diff --git a/apps/oauth2/tests/Controller/SettingsControllerTest.php b/apps/oauth2/tests/Controller/SettingsControllerTest.php index 030a220e3d7..19498b16454 100644 --- a/apps/oauth2/tests/Controller/SettingsControllerTest.php +++ b/apps/oauth2/tests/Controller/SettingsControllerTest.php @@ -22,9 +22,7 @@ use OCP\Security\ISecureRandom; use OCP\Server; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class SettingsControllerTest extends TestCase { /** @var IRequest|\PHPUnit\Framework\MockObject\MockObject */ private $request; diff --git a/apps/oauth2/tests/Db/AccessTokenMapperTest.php b/apps/oauth2/tests/Db/AccessTokenMapperTest.php index 41a79fe725b..f99d12e971a 100644 --- a/apps/oauth2/tests/Db/AccessTokenMapperTest.php +++ b/apps/oauth2/tests/Db/AccessTokenMapperTest.php @@ -14,9 +14,7 @@ use OCP\IDBConnection; use OCP\Server; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class AccessTokenMapperTest extends TestCase { /** @var AccessTokenMapper */ private $accessTokenMapper; diff --git a/apps/oauth2/tests/Db/ClientMapperTest.php b/apps/oauth2/tests/Db/ClientMapperTest.php index cf072118ae2..2f0b220cb75 100644 --- a/apps/oauth2/tests/Db/ClientMapperTest.php +++ b/apps/oauth2/tests/Db/ClientMapperTest.php @@ -13,9 +13,7 @@ use OCP\IDBConnection; use OCP\Server; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ClientMapperTest extends TestCase { /** @var ClientMapper */ private $clientMapper; diff --git a/apps/provisioning_api/tests/CapabilitiesTest.php b/apps/provisioning_api/tests/CapabilitiesTest.php index 86d2bb8c4fa..43d22a71232 100644 --- a/apps/provisioning_api/tests/CapabilitiesTest.php +++ b/apps/provisioning_api/tests/CapabilitiesTest.php @@ -19,8 +19,8 @@ use Test\TestCase; * Note: group DB needed because of usage of overwriteService() * * @package OCA\Provisioning_API\Tests - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CapabilitiesTest extends TestCase { protected IAppManager&MockObject $appManager; diff --git a/apps/provisioning_api/tests/Controller/AppsControllerTest.php b/apps/provisioning_api/tests/Controller/AppsControllerTest.php index f95daeae7d3..6fc38311037 100644 --- a/apps/provisioning_api/tests/Controller/AppsControllerTest.php +++ b/apps/provisioning_api/tests/Controller/AppsControllerTest.php @@ -22,10 +22,10 @@ use PHPUnit\Framework\MockObject\MockObject; /** * Class AppsTest * - * @group DB * * @package OCA\Provisioning_API\Tests */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class AppsControllerTest extends TestCase { private IAppManager $appManager; private IAppConfig&MockObject $appConfig; diff --git a/apps/settings/tests/AppInfo/ApplicationTest.php b/apps/settings/tests/AppInfo/ApplicationTest.php index 3e895d87b06..f016c7a86a8 100644 --- a/apps/settings/tests/AppInfo/ApplicationTest.php +++ b/apps/settings/tests/AppInfo/ApplicationTest.php @@ -25,8 +25,8 @@ use Test\TestCase; * Class ApplicationTest * * @package Tests\Settings - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ApplicationTest extends TestCase { protected Application $app; protected IAppContainer $container; diff --git a/apps/settings/tests/Controller/AdminSettingsControllerTest.php b/apps/settings/tests/Controller/AdminSettingsControllerTest.php index fbdc506457b..775f19707c7 100644 --- a/apps/settings/tests/Controller/AdminSettingsControllerTest.php +++ b/apps/settings/tests/Controller/AdminSettingsControllerTest.php @@ -26,10 +26,10 @@ use Test\TestCase; /** * Class AdminSettingsControllerTest * - * @group DB * * @package Tests\Settings\Controller */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class AdminSettingsControllerTest extends TestCase { private IRequest&MockObject $request; diff --git a/apps/settings/tests/Controller/AppSettingsControllerTest.php b/apps/settings/tests/Controller/AppSettingsControllerTest.php index 392bb7b561d..de96a10d31e 100644 --- a/apps/settings/tests/Controller/AppSettingsControllerTest.php +++ b/apps/settings/tests/Controller/AppSettingsControllerTest.php @@ -34,9 +34,8 @@ use Test\TestCase; * Class AppSettingsControllerTest * * @package Tests\Settings\Controller - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class AppSettingsControllerTest extends TestCase { private IRequest&MockObject $request; private IL10N&MockObject $l10n; diff --git a/apps/settings/tests/Controller/CheckSetupControllerTest.php b/apps/settings/tests/Controller/CheckSetupControllerTest.php index 2eaedf5d4ff..1582fe3f371 100644 --- a/apps/settings/tests/Controller/CheckSetupControllerTest.php +++ b/apps/settings/tests/Controller/CheckSetupControllerTest.php @@ -25,9 +25,9 @@ use Test\TestCase; /** * Class CheckSetupControllerTest * - * @backupStaticAttributes * @package Tests\Settings\Controller */ +#[\PHPUnit\Framework\Attributes\BackupStaticProperties(true)] class CheckSetupControllerTest extends TestCase { private IRequest&MockObject $request; private IConfig&MockObject $config; diff --git a/apps/settings/tests/Controller/UsersControllerTest.php b/apps/settings/tests/Controller/UsersControllerTest.php index 1012557bfc4..662589d3d45 100644 --- a/apps/settings/tests/Controller/UsersControllerTest.php +++ b/apps/settings/tests/Controller/UsersControllerTest.php @@ -37,10 +37,9 @@ use OCP\Mail\IMailer; use PHPUnit\Framework\MockObject\MockObject; /** - * @group DB - * * @package Tests\Settings\Controller */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class UsersControllerTest extends \Test\TestCase { private IGroupManager&MockObject $groupManager; private UserManager&MockObject $userManager; diff --git a/apps/settings/tests/Settings/Admin/ServerTest.php b/apps/settings/tests/Settings/Admin/ServerTest.php index e2ca4cff3c6..81ca77bd6a4 100644 --- a/apps/settings/tests/Settings/Admin/ServerTest.php +++ b/apps/settings/tests/Settings/Admin/ServerTest.php @@ -21,9 +21,7 @@ use OCP\IUrlGenerator; use PHPUnit\Framework\MockObject\MockObject; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ServerTest extends TestCase { private IDBConnection $connection; private Server&MockObject $admin; diff --git a/apps/settings/tests/SetupChecks/SupportedDatabaseTest.php b/apps/settings/tests/SetupChecks/SupportedDatabaseTest.php index 6c75df47aa0..dc16d044dc1 100644 --- a/apps/settings/tests/SetupChecks/SupportedDatabaseTest.php +++ b/apps/settings/tests/SetupChecks/SupportedDatabaseTest.php @@ -16,9 +16,7 @@ use OCP\Server; use OCP\SetupCheck\SetupResult; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class SupportedDatabaseTest extends TestCase { private IL10N $l10n; private IUrlGenerator $urlGenerator; diff --git a/apps/settings/tests/UserMigration/AccountMigratorTest.php b/apps/settings/tests/UserMigration/AccountMigratorTest.php index b8f8301f777..162eeef526f 100644 --- a/apps/settings/tests/UserMigration/AccountMigratorTest.php +++ b/apps/settings/tests/UserMigration/AccountMigratorTest.php @@ -23,9 +23,7 @@ use Sabre\VObject\UUIDUtil; use Symfony\Component\Console\Output\OutputInterface; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class AccountMigratorTest extends TestCase { private IUserManager $userManager; private IAvatarManager $avatarManager; diff --git a/apps/sharebymail/tests/ShareByMailProviderTest.php b/apps/sharebymail/tests/ShareByMailProviderTest.php index be72e9e2df9..68e7e8691fa 100644 --- a/apps/sharebymail/tests/ShareByMailProviderTest.php +++ b/apps/sharebymail/tests/ShareByMailProviderTest.php @@ -47,8 +47,8 @@ use Test\Traits\EmailValidatorTrait; * Class ShareByMailProviderTest * * @package OCA\ShareByMail\Tests - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ShareByMailProviderTest extends TestCase { use EmailValidatorTrait; diff --git a/apps/theming/tests/Migration/Version2006Date20240905111627Test.php b/apps/theming/tests/Migration/Version2006Date20240905111627Test.php index 5f7458db11a..430b0b45e24 100644 --- a/apps/theming/tests/Migration/Version2006Date20240905111627Test.php +++ b/apps/theming/tests/Migration/Version2006Date20240905111627Test.php @@ -20,9 +20,7 @@ use OCP\Server; use PHPUnit\Framework\MockObject\MockObject; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class Version2006Date20240905111627Test extends TestCase { private IAppConfig&MockObject $appConfig; @@ -75,9 +73,7 @@ class Version2006Date20240905111627Test extends TestCase { ], $setValueCalls); } - /** - * @group DB - */ + #[\PHPUnit\Framework\Attributes\Group('DB')] public function testRestoreUserColors(): void { $this->appConfig->expects(self::once()) ->method('getValueString') @@ -126,8 +122,8 @@ class Version2006Date20240905111627Test extends TestCase { /** * Ensure only users with background color but no primary color are migrated - * @group DB */ + #[\PHPUnit\Framework\Attributes\Group('DB')] public function testRestoreUserColorsWithConflicts(): void { $this->appConfig->expects(self::once()) ->method('getValueString') diff --git a/apps/theming/tests/ServicesTest.php b/apps/theming/tests/ServicesTest.php index 3971c9b6698..7bacd43cf08 100644 --- a/apps/theming/tests/ServicesTest.php +++ b/apps/theming/tests/ServicesTest.php @@ -24,9 +24,9 @@ use Test\TestCase; /** * Class ServicesTest * - * @group DB * @package OCA\Theming\Tests */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ServicesTest extends TestCase { protected App $app; diff --git a/apps/twofactor_backupcodes/tests/Db/BackupCodeMapperTest.php b/apps/twofactor_backupcodes/tests/Db/BackupCodeMapperTest.php index 79006887a62..b0c5be0f9c6 100644 --- a/apps/twofactor_backupcodes/tests/Db/BackupCodeMapperTest.php +++ b/apps/twofactor_backupcodes/tests/Db/BackupCodeMapperTest.php @@ -15,9 +15,7 @@ use OCP\IUser; use OCP\Server; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class BackupCodeMapperTest extends TestCase { private IDBConnection $db; private BackupCodeMapper $mapper; diff --git a/apps/twofactor_backupcodes/tests/Service/BackupCodeStorageTest.php b/apps/twofactor_backupcodes/tests/Service/BackupCodeStorageTest.php index cfc35e7cb1c..afba7d6d985 100644 --- a/apps/twofactor_backupcodes/tests/Service/BackupCodeStorageTest.php +++ b/apps/twofactor_backupcodes/tests/Service/BackupCodeStorageTest.php @@ -16,9 +16,7 @@ use OCP\Server; use PHPUnit\Framework\MockObject\MockObject; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class BackupCodeStorageTest extends TestCase { private IManager&MockObject $notificationManager; private string $testUID = 'test123456789'; diff --git a/apps/user_ldap/tests/AccessTest.php b/apps/user_ldap/tests/AccessTest.php index 54be29d0f86..5bda7b8086b 100644 --- a/apps/user_ldap/tests/AccessTest.php +++ b/apps/user_ldap/tests/AccessTest.php @@ -37,10 +37,10 @@ use Test\TestCase; /** * Class AccessTest * - * @group DB * * @package OCA\User_LDAP\Tests */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class AccessTest extends TestCase { protected UserMapping&MockObject $userMapper; protected IManager&MockObject $shareManager; diff --git a/apps/user_ldap/tests/ConnectionTest.php b/apps/user_ldap/tests/ConnectionTest.php index 7116e15898f..ad99636f62c 100644 --- a/apps/user_ldap/tests/ConnectionTest.php +++ b/apps/user_ldap/tests/ConnectionTest.php @@ -16,10 +16,10 @@ use PHPUnit\Framework\MockObject\MockObject; /** * Class Test_Connection * - * @group DB * * @package OCA\User_LDAP\Tests */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ConnectionTest extends \Test\TestCase { protected ILDAPWrapper&MockObject $ldap; protected Connection $connection; diff --git a/apps/user_ldap/tests/Group_LDAPTest.php b/apps/user_ldap/tests/Group_LDAPTest.php index 10182111768..28c37ec6053 100644 --- a/apps/user_ldap/tests/Group_LDAPTest.php +++ b/apps/user_ldap/tests/Group_LDAPTest.php @@ -30,10 +30,10 @@ use Test\TestCase; /** * Class GroupLDAPTest * - * @group DB * * @package OCA\User_LDAP\Tests */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class Group_LDAPTest extends TestCase { private Access&MockObject $access; private GroupPluginManager&MockObject $pluginManager; diff --git a/apps/user_ldap/tests/HelperTest.php b/apps/user_ldap/tests/HelperTest.php index adea600d900..86df86da2e4 100644 --- a/apps/user_ldap/tests/HelperTest.php +++ b/apps/user_ldap/tests/HelperTest.php @@ -13,9 +13,7 @@ use OCP\IDBConnection; use OCP\Server; use PHPUnit\Framework\MockObject\MockObject; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class HelperTest extends \Test\TestCase { private IAppConfig&MockObject $appConfig; diff --git a/apps/user_ldap/tests/Jobs/SyncTest.php b/apps/user_ldap/tests/Jobs/SyncTest.php index f6ecf984ab0..0e46685375c 100644 --- a/apps/user_ldap/tests/Jobs/SyncTest.php +++ b/apps/user_ldap/tests/Jobs/SyncTest.php @@ -27,9 +27,7 @@ use PHPUnit\Framework\MockObject\MockObject; use Psr\Log\LoggerInterface; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class SyncTest extends TestCase { protected Helper&MockObject $helper; protected LDAP&MockObject $ldapWrapper; diff --git a/apps/user_ldap/tests/LDAPProviderTest.php b/apps/user_ldap/tests/LDAPProviderTest.php index 57323e374aa..5548f47d445 100644 --- a/apps/user_ldap/tests/LDAPProviderTest.php +++ b/apps/user_ldap/tests/LDAPProviderTest.php @@ -28,10 +28,10 @@ use Psr\Log\LoggerInterface; /** * Class LDAPProviderTest * - * @group DB * * @package OCA\User_LDAP\Tests */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class LDAPProviderTest extends \Test\TestCase { private function getServerMock(IUserLDAP $userBackend, IGroupLDAP $groupBackend) { $server = $this->getMockBuilder('OC\Server') diff --git a/apps/user_ldap/tests/Mapping/GroupMappingTest.php b/apps/user_ldap/tests/Mapping/GroupMappingTest.php index df64149577f..efa1c11f482 100644 --- a/apps/user_ldap/tests/Mapping/GroupMappingTest.php +++ b/apps/user_ldap/tests/Mapping/GroupMappingTest.php @@ -16,10 +16,10 @@ use OCP\IDBConnection; /** * Class GroupMappingTest * - * @group DB * * @package OCA\User_LDAP\Tests\Mapping */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class GroupMappingTest extends AbstractMappingTestCase { public function getMapper(IDBConnection $dbMock, ICacheFactory $cacheFactory, IAppConfig $appConfig): GroupMapping { return new GroupMapping($dbMock, $cacheFactory, $appConfig, true); diff --git a/apps/user_ldap/tests/Mapping/UserMappingTest.php b/apps/user_ldap/tests/Mapping/UserMappingTest.php index c79a5fa77e9..5fc57e44c7b 100644 --- a/apps/user_ldap/tests/Mapping/UserMappingTest.php +++ b/apps/user_ldap/tests/Mapping/UserMappingTest.php @@ -17,10 +17,10 @@ use OCP\Support\Subscription\IAssertion; /** * Class UserMappingTest * - * @group DB * * @package OCA\User_LDAP\Tests\Mapping */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class UserMappingTest extends AbstractMappingTestCase { public function getMapper(IDBConnection $dbMock, ICacheFactory $cacheFactory, IAppConfig $appConfig): UserMapping { return new UserMapping($dbMock, $cacheFactory, $appConfig, true, $this->createMock(IAssertion::class)); diff --git a/apps/user_ldap/tests/Migration/UUIDFixGroupTest.php b/apps/user_ldap/tests/Migration/UUIDFixGroupTest.php index 89d880f4acb..9ba54d046af 100644 --- a/apps/user_ldap/tests/Migration/UUIDFixGroupTest.php +++ b/apps/user_ldap/tests/Migration/UUIDFixGroupTest.php @@ -15,8 +15,8 @@ use OCA\User_LDAP\Migration\UUIDFixGroup; * Class UUIDFixGroupTest * * @package OCA\Group_LDAP\Tests\Migration - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class UUIDFixGroupTest extends AbstractUUIDFixTestCase { protected function setUp(): void { $this->isUser = false; diff --git a/apps/user_ldap/tests/Migration/UUIDFixUserTest.php b/apps/user_ldap/tests/Migration/UUIDFixUserTest.php index a582fd677fa..0c0db000953 100644 --- a/apps/user_ldap/tests/Migration/UUIDFixUserTest.php +++ b/apps/user_ldap/tests/Migration/UUIDFixUserTest.php @@ -15,8 +15,8 @@ use OCA\User_LDAP\User_Proxy; * Class UUIDFixUserTest * * @package OCA\User_LDAP\Tests\Migration - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class UUIDFixUserTest extends AbstractUUIDFixTestCase { protected function setUp(): void { $this->isUser = true; diff --git a/apps/user_ldap/tests/Settings/AdminTest.php b/apps/user_ldap/tests/Settings/AdminTest.php index eb6d4baa26e..81078491461 100644 --- a/apps/user_ldap/tests/Settings/AdminTest.php +++ b/apps/user_ldap/tests/Settings/AdminTest.php @@ -18,9 +18,9 @@ use PHPUnit\Framework\MockObject\MockObject; use Test\TestCase; /** - * @group DB * @package OCA\User_LDAP\Tests\Settings */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class AdminTest extends TestCase { private IL10N&MockObject $l10n; private ITemplateManager $templateManager; diff --git a/apps/user_ldap/tests/User/DeletedUsersIndexTest.php b/apps/user_ldap/tests/User/DeletedUsersIndexTest.php index b245e52fe6e..96a8913195c 100644 --- a/apps/user_ldap/tests/User/DeletedUsersIndexTest.php +++ b/apps/user_ldap/tests/User/DeletedUsersIndexTest.php @@ -18,10 +18,10 @@ use PHPUnit\Framework\MockObject\MockObject; /** * Class DeletedUsersIndexTest * - * @group DB * * @package OCA\User_LDAP\Tests\User */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class DeletedUsersIndexTest extends \Test\TestCase { protected DeletedUsersIndex $dui; protected IConfig $config; diff --git a/apps/user_ldap/tests/User/ManagerTest.php b/apps/user_ldap/tests/User/ManagerTest.php index bf9d1f5746f..a8ce9cb3120 100644 --- a/apps/user_ldap/tests/User/ManagerTest.php +++ b/apps/user_ldap/tests/User/ManagerTest.php @@ -26,10 +26,10 @@ use Psr\Log\LoggerInterface; /** * Class Test_User_Manager * - * @group DB * * @package OCA\User_LDAP\Tests\User */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ManagerTest extends \Test\TestCase { protected Access&MockObject $access; protected IConfig&MockObject $config; diff --git a/apps/user_ldap/tests/User/UserTest.php b/apps/user_ldap/tests/User/UserTest.php index 00edd8b3f9b..737ab5f6a86 100644 --- a/apps/user_ldap/tests/User/UserTest.php +++ b/apps/user_ldap/tests/User/UserTest.php @@ -27,10 +27,10 @@ use Psr\Log\LoggerInterface; /** * Class UserTest * - * @group DB * * @package OCA\User_LDAP\Tests\User */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class UserTest extends \Test\TestCase { protected Access&MockObject $access; protected Connection&MockObject $connection; diff --git a/apps/user_ldap/tests/User_LDAPTest.php b/apps/user_ldap/tests/User_LDAPTest.php index f8900c3cca4..0949fef0042 100644 --- a/apps/user_ldap/tests/User_LDAPTest.php +++ b/apps/user_ldap/tests/User_LDAPTest.php @@ -35,10 +35,10 @@ use Test\TestCase; /** * Class Test_User_Ldap_Direct * - * @group DB * * @package OCA\User_LDAP\Tests */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class User_LDAPTest extends TestCase { protected Access&MockObject $access; protected OfflineUser&MockObject $offlineUser; @@ -48,7 +48,7 @@ class User_LDAPTest extends TestCase { protected Manager&MockObject $userManager; protected LoggerInterface&MockObject $logger; protected DeletedUsersIndex&MockObject $deletedUsersIndex; - protected User_LDAP $backend; + protected UserLDAP $backend; protected function setUp(): void { parent::setUp(); @@ -72,7 +72,7 @@ class User_LDAPTest extends TestCase { $this->deletedUsersIndex = $this->createMock(DeletedUsersIndex::class); - $this->backend = new User_LDAP( + $this->backend = new UserLDAP( $this->access, $this->notificationManager, $this->pluginManager, diff --git a/apps/user_ldap/tests/WizardTest.php b/apps/user_ldap/tests/WizardTest.php index 3ae9a409e88..b492b932568 100644 --- a/apps/user_ldap/tests/WizardTest.php +++ b/apps/user_ldap/tests/WizardTest.php @@ -18,10 +18,10 @@ use Test\TestCase; /** * Class Test_Wizard * - * @group DB * * @package OCA\User_LDAP\Tests */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class WizardTest extends TestCase { protected function setUp(): void { parent::setUp(); diff --git a/apps/user_status/tests/Integration/Service/StatusServiceIntegrationTest.php b/apps/user_status/tests/Integration/Service/StatusServiceIntegrationTest.php index 8a21052b09f..61c07db51e9 100644 --- a/apps/user_status/tests/Integration/Service/StatusServiceIntegrationTest.php +++ b/apps/user_status/tests/Integration/Service/StatusServiceIntegrationTest.php @@ -18,9 +18,7 @@ use Test\TestCase; use function sleep; use function time; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class StatusServiceIntegrationTest extends TestCase { private StatusService $service; diff --git a/apps/webhook_listeners/tests/Db/WebhookListenerMapperTest.php b/apps/webhook_listeners/tests/Db/WebhookListenerMapperTest.php index 45d66090eae..ee6f2fc909e 100644 --- a/apps/webhook_listeners/tests/Db/WebhookListenerMapperTest.php +++ b/apps/webhook_listeners/tests/Db/WebhookListenerMapperTest.php @@ -18,9 +18,7 @@ use OCP\Server; use OCP\User\Events\UserCreatedEvent; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class WebhookListenerMapperTest extends TestCase { private IDBConnection $connection; private WebhookListenerMapper $mapper; diff --git a/apps/workflowengine/tests/Check/FileMimeTypeTest.php b/apps/workflowengine/tests/Check/FileMimeTypeTest.php index 55aea3db172..89e0676392e 100644 --- a/apps/workflowengine/tests/Check/FileMimeTypeTest.php +++ b/apps/workflowengine/tests/Check/FileMimeTypeTest.php @@ -25,9 +25,7 @@ class TemporaryNoLocal extends Temporary { } } -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class FileMimeTypeTest extends TestCase { /** @var IL10N */ private $l10n; diff --git a/apps/workflowengine/tests/ManagerTest.php b/apps/workflowengine/tests/ManagerTest.php index 4e547db387b..3f2e5064fbe 100644 --- a/apps/workflowengine/tests/ManagerTest.php +++ b/apps/workflowengine/tests/ManagerTest.php @@ -40,8 +40,8 @@ use Test\TestCase; * Class ManagerTest * * @package OCA\WorkflowEngine\Tests - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ManagerTest extends TestCase { protected Manager $manager; protected IDBConnection $db; diff --git a/build/rector.php b/build/rector.php index fe6952d2b2b..e32a064234a 100644 --- a/build/rector.php +++ b/build/rector.php @@ -12,9 +12,7 @@ use PhpParser\Node; use Rector\CodingStyle\Contract\ClassNameImport\ClassNameImportSkipVoterInterface; use Rector\Config\RectorConfig; use Rector\Php80\Rector\Class_\ClassPropertyAssignToConstructorPromotionRector; -use Rector\PHPUnit\AnnotationsToAttributes\Rector\ClassMethod\DataProviderAnnotationToAttributeRector; -use Rector\PHPUnit\CodeQuality\Rector\MethodCall\UseSpecificWillMethodRector; -use Rector\PHPUnit\PHPUnit100\Rector\Class_\StaticDataProviderClassMethodRector; +use Rector\PHPUnit\Set\PHPUnitSetList; use Rector\StaticTypeMapper\ValueObject\Type\FullyQualifiedObjectType; use Rector\ValueObject\Application\File; @@ -86,17 +84,13 @@ $config = RectorConfig::configure() // ->withPhpSets() ->withImportNames(importShortClasses:false) ->withTypeCoverageLevel(0) - ->withRules([ - UseSpecificWillMethodRector::class, - StaticDataProviderClassMethodRector::class, - DataProviderAnnotationToAttributeRector::class, - ]) ->withConfiguredRule(ClassPropertyAssignToConstructorPromotionRector::class, [ 'inline_public' => true, 'rename_property' => true, ]) ->withSets([ NextcloudSets::NEXTCLOUD_27, + PHPUnitSetList::PHPUNIT_100, ]); $config->registerService(NextcloudNamespaceSkipVoter::class, tag:ClassNameImportSkipVoterInterface::class); diff --git a/core/Controller/WhatsNewController.php b/core/Controller/WhatsNewController.php index 7fa22b0faec..e54fb13b892 100644 --- a/core/Controller/WhatsNewController.php +++ b/core/Controller/WhatsNewController.php @@ -12,6 +12,7 @@ use OCP\AppFramework\Http; use OCP\AppFramework\Http\Attribute\ApiRoute; use OCP\AppFramework\Http\Attribute\NoAdminRequired; use OCP\AppFramework\Http\DataResponse; +use OCP\AppFramework\OCSController; use OCP\Defaults; use OCP\IConfig; use OCP\IRequest; @@ -19,7 +20,7 @@ use OCP\IUserSession; use OCP\L10N\IFactory; use OCP\PreConditionNotMetException; -class WhatsNewController extends \OCP\AppFramework\OCSController { +class WhatsNewController extends OCSController { public function __construct( string $appName, IRequest $request, diff --git a/cron.php b/cron.php index cb858ab09b1..f86d93b9a73 100644 --- a/cron.php +++ b/cron.php @@ -41,7 +41,7 @@ Options: $jobClasses = empty($jobClasses) ? null : $jobClasses; if ($verbose) { - $cronService->registerVerboseCallback(function (string $message) { + $cronService->registerVerboseCallback(function (string $message): void { echo $message . PHP_EOL; }); } diff --git a/tests/Core/Command/Apps/AppsDisableTest.php b/tests/Core/Command/Apps/AppsDisableTest.php index 117af958054..26bd1878d0d 100644 --- a/tests/Core/Command/Apps/AppsDisableTest.php +++ b/tests/Core/Command/Apps/AppsDisableTest.php @@ -16,9 +16,8 @@ use Test\TestCase; /** * Class AppsDisableTest - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class AppsDisableTest extends TestCase { /** @var CommandTester */ private $commandTester; diff --git a/tests/Core/Command/Apps/AppsEnableTest.php b/tests/Core/Command/Apps/AppsEnableTest.php index 604c670ae15..86f7ca6fcce 100644 --- a/tests/Core/Command/Apps/AppsEnableTest.php +++ b/tests/Core/Command/Apps/AppsEnableTest.php @@ -18,9 +18,8 @@ use Test\TestCase; /** * Class AppsEnableTest - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class AppsEnableTest extends TestCase { /** @var CommandTester */ private $commandTester; diff --git a/tests/lib/Accounts/AccountManagerTest.php b/tests/lib/Accounts/AccountManagerTest.php index 824e49e9e9a..8d66ba15ce1 100644 --- a/tests/lib/Accounts/AccountManagerTest.php +++ b/tests/lib/Accounts/AccountManagerTest.php @@ -39,9 +39,9 @@ use Test\TestCase; /** * Class AccountManagerTest * - * @group DB * @package Test\Accounts */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class AccountManagerTest extends TestCase { /** accounts table name */ diff --git a/tests/lib/Accounts/HooksTest.php b/tests/lib/Accounts/HooksTest.php index 0fb076e203e..61aba064f22 100644 --- a/tests/lib/Accounts/HooksTest.php +++ b/tests/lib/Accounts/HooksTest.php @@ -23,8 +23,8 @@ use Test\TestCase; * Class HooksTest * * @package Test\Accounts - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class HooksTest extends TestCase { private LoggerInterface&MockObject $logger; diff --git a/tests/lib/AllConfigTest.php b/tests/lib/AllConfigTest.php index 80a38d590f4..9c1020028b0 100644 --- a/tests/lib/AllConfigTest.php +++ b/tests/lib/AllConfigTest.php @@ -8,19 +8,18 @@ namespace Test; -/** - * Class AllConfigTest - * - * @group DB - * - * @package Test - */ use OC\AllConfig; use OC\SystemConfig; use OCP\IDBConnection; use OCP\PreConditionNotMetException; use OCP\Server; +/** + * Class AllConfigTest + * + * @package Test + */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class AllConfigTest extends \Test\TestCase { /** @var IDBConnection */ protected $connection; diff --git a/tests/lib/AppConfigIntegrationTest.php b/tests/lib/AppConfigIntegrationTest.php index 4f821e00a63..7259acb10c5 100644 --- a/tests/lib/AppConfigIntegrationTest.php +++ b/tests/lib/AppConfigIntegrationTest.php @@ -23,10 +23,9 @@ use PHPUnit\Framework\MockObject\MockObject; use Psr\Log\LoggerInterface; /** - * @group DB - * * @package Test */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class AppConfigIntegrationTest extends TestCase { protected IAppConfig $appConfig; protected IDBConnection $connection; diff --git a/tests/lib/AppFramework/Db/QBMapperDBTest.php b/tests/lib/AppFramework/Db/QBMapperDBTest.php index 14477d98a96..bf149f1c2e0 100644 --- a/tests/lib/AppFramework/Db/QBMapperDBTest.php +++ b/tests/lib/AppFramework/Db/QBMapperDBTest.php @@ -61,8 +61,8 @@ class QBDBTestMapper extends QBMapper { /** * Test real database handling (serialization) - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class QBMapperDBTest extends TestCase { protected IDBConnection $connection; protected bool $schemaSetup = false; diff --git a/tests/lib/AppFramework/DependencyInjection/DIContainerTest.php b/tests/lib/AppFramework/DependencyInjection/DIContainerTest.php index 6056f87dd42..4fbe5c8307b 100644 --- a/tests/lib/AppFramework/DependencyInjection/DIContainerTest.php +++ b/tests/lib/AppFramework/DependencyInjection/DIContainerTest.php @@ -22,9 +22,7 @@ use OCP\IConfig; use OCP\IRequestId; use PHPUnit\Framework\MockObject\MockObject; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class DIContainerTest extends \Test\TestCase { private DIContainer&MockObject $container; diff --git a/tests/lib/AppFramework/DependencyInjection/DIIntergrationTests.php b/tests/lib/AppFramework/DependencyInjection/DIIntergrationTests.php index ac84ae07a36..f1e1df8bec4 100644 --- a/tests/lib/AppFramework/DependencyInjection/DIIntergrationTests.php +++ b/tests/lib/AppFramework/DependencyInjection/DIIntergrationTests.php @@ -34,6 +34,9 @@ class ClassB { } class DIIntergrationTests extends TestCase { + public function __construct() { + parent::__construct(static::class); + } private DIContainer $container; private ServerContainer $server; diff --git a/tests/lib/AppFramework/Http/DispatcherTest.php b/tests/lib/AppFramework/Http/DispatcherTest.php index 86c78e840e0..19b587116f9 100644 --- a/tests/lib/AppFramework/Http/DispatcherTest.php +++ b/tests/lib/AppFramework/Http/DispatcherTest.php @@ -76,8 +76,8 @@ class TestController extends Controller { * Class DispatcherTest * * @package Test\AppFramework\Http - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class DispatcherTest extends \Test\TestCase { /** @var MiddlewareDispatcher */ private $middlewareDispatcher; diff --git a/tests/lib/AppFramework/Middleware/Security/RateLimitingMiddlewareTest.php b/tests/lib/AppFramework/Middleware/Security/RateLimitingMiddlewareTest.php index b3e02c853e2..6624be62644 100644 --- a/tests/lib/AppFramework/Middleware/Security/RateLimitingMiddlewareTest.php +++ b/tests/lib/AppFramework/Middleware/Security/RateLimitingMiddlewareTest.php @@ -25,9 +25,7 @@ use PHPUnit\Framework\MockObject\MockObject; use Test\AppFramework\Middleware\Security\Mock\RateLimitingMiddlewareController; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class RateLimitingMiddlewareTest extends TestCase { private IRequest|MockObject $request; private IUserSession|MockObject $userSession; diff --git a/tests/lib/AppFramework/Utility/ControllerMethodReflectorTest.php b/tests/lib/AppFramework/Utility/ControllerMethodReflectorTest.php index 00ae4792824..61d2e42ae60 100644 --- a/tests/lib/AppFramework/Utility/ControllerMethodReflectorTest.php +++ b/tests/lib/AppFramework/Utility/ControllerMethodReflectorTest.php @@ -134,9 +134,6 @@ class ControllerMethodReflectorTest extends \Test\TestCase { public function arguments3($a, float $b, int $c, $d) { } - /** - * @requires PHP 7 - */ public function testReadTypeIntAnnotationsScalarTypes(): void { $reader = new ControllerMethodReflector(); $reader->reflect( diff --git a/tests/lib/AppScriptSortTest.php b/tests/lib/AppScriptSortTest.php index 7dc2bde1897..c2407705060 100644 --- a/tests/lib/AppScriptSortTest.php +++ b/tests/lib/AppScriptSortTest.php @@ -16,8 +16,8 @@ use Psr\Log\LoggerInterface; * Class AppScriptSortTest * * @package Test - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class AppScriptSortTest extends \Test\TestCase { private $logger; diff --git a/tests/lib/AppTest.php b/tests/lib/AppTest.php index 71186f5ffb0..83944ce2f7e 100644 --- a/tests/lib/AppTest.php +++ b/tests/lib/AppTest.php @@ -27,9 +27,8 @@ use Psr\Log\LoggerInterface; /** * Class AppTest - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class AppTest extends \Test\TestCase { public const TEST_USER1 = 'user1'; public const TEST_USER2 = 'user2'; diff --git a/tests/lib/Authentication/Token/PublicKeyTokenMapperTest.php b/tests/lib/Authentication/Token/PublicKeyTokenMapperTest.php index e6c5296c216..2f99024e0f4 100644 --- a/tests/lib/Authentication/Token/PublicKeyTokenMapperTest.php +++ b/tests/lib/Authentication/Token/PublicKeyTokenMapperTest.php @@ -18,9 +18,7 @@ use OCP\IUser; use OCP\Server; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class PublicKeyTokenMapperTest extends TestCase { /** @var PublicKeyTokenMapper */ private $mapper; diff --git a/tests/lib/Authentication/Token/RemoteWipeTest.php b/tests/lib/Authentication/Token/RemoteWipeTest.php index ca09767c759..b1676e90b9d 100644 --- a/tests/lib/Authentication/Token/RemoteWipeTest.php +++ b/tests/lib/Authentication/Token/RemoteWipeTest.php @@ -12,7 +12,6 @@ namespace Test\Authentication\Token; use OC\Authentication\Events\RemoteWipeFinished; use OC\Authentication\Events\RemoteWipeStarted; use OC\Authentication\Exceptions\WipeTokenException; -use OC\Authentication\Token\IProvider; use OC\Authentication\Token\IProvider as ITokenProvider; use OC\Authentication\Token\IToken; use OC\Authentication\Token\IWipeableToken; @@ -39,7 +38,7 @@ class RemoteWipeTest extends TestCase { protected function setUp(): void { parent::setUp(); - $this->tokenProvider = $this->createMock(IProvider::class); + $this->tokenProvider = $this->createMock(ITokenProvider::class); $this->eventDispatcher = $this->createMock(IEventDispatcher::class); $this->logger = $this->createMock(LoggerInterface::class); diff --git a/tests/lib/Authentication/TwoFactorAuth/Db/ProviderUserAssignmentDaoTest.php b/tests/lib/Authentication/TwoFactorAuth/Db/ProviderUserAssignmentDaoTest.php index f3fd8ee0b63..bd52dc21821 100644 --- a/tests/lib/Authentication/TwoFactorAuth/Db/ProviderUserAssignmentDaoTest.php +++ b/tests/lib/Authentication/TwoFactorAuth/Db/ProviderUserAssignmentDaoTest.php @@ -14,9 +14,7 @@ use OCP\IDBConnection; use OCP\Server; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ProviderUserAssignmentDaoTest extends TestCase { /** @var IDBConnection */ private $dbConn; diff --git a/tests/lib/BackgroundJob/JobListTest.php b/tests/lib/BackgroundJob/JobListTest.php index 6f946187c52..7185acf2beb 100644 --- a/tests/lib/BackgroundJob/JobListTest.php +++ b/tests/lib/BackgroundJob/JobListTest.php @@ -23,9 +23,9 @@ use Test\TestCase; /** * Class JobList * - * @group DB * @package Test\BackgroundJob */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class JobListTest extends TestCase { /** @var \OC\BackgroundJob\JobList */ protected $instance; diff --git a/tests/lib/Cache/FileCacheTest.php b/tests/lib/Cache/FileCacheTest.php index 4daa8d3b7ef..be27ff04f09 100644 --- a/tests/lib/Cache/FileCacheTest.php +++ b/tests/lib/Cache/FileCacheTest.php @@ -24,10 +24,10 @@ use Test\Traits\UserTrait; /** * Class FileCacheTest * - * @group DB * * @package Test\Cache */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class FileCacheTest extends TestCache { use UserTrait; diff --git a/tests/lib/Command/BackgroundModeTest.php b/tests/lib/Command/BackgroundModeTest.php index ab036ef87ee..8389126fe8e 100644 --- a/tests/lib/Command/BackgroundModeTest.php +++ b/tests/lib/Command/BackgroundModeTest.php @@ -16,9 +16,7 @@ use Symfony\Component\Console\Input\InputDefinition; use Symfony\Component\Console\Tester\CommandTester; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class BackgroundModeTest extends TestCase { private IAppConfig $appConfig; diff --git a/tests/lib/Command/CronBusTest.php b/tests/lib/Command/CronBusTest.php index c86cdcb1da0..a4675f10fdb 100644 --- a/tests/lib/Command/CronBusTest.php +++ b/tests/lib/Command/CronBusTest.php @@ -11,9 +11,7 @@ use OC\Command\CronBus; use OCP\BackgroundJob\IJobList; use Test\BackgroundJob\DummyJobList; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CronBusTest extends AsyncBusTestCase { /** * @var IJobList diff --git a/tests/lib/Comments/ManagerTest.php b/tests/lib/Comments/ManagerTest.php index bd991a5692a..d88e062a8ec 100644 --- a/tests/lib/Comments/ManagerTest.php +++ b/tests/lib/Comments/ManagerTest.php @@ -31,9 +31,8 @@ use Test\TestCase; /** * Class ManagerTest - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ManagerTest extends TestCase { /** @var IDBConnection */ private $connection; diff --git a/tests/lib/Config/LexiconTest.php b/tests/lib/Config/LexiconTest.php index b422588f662..d3809892c0e 100644 --- a/tests/lib/Config/LexiconTest.php +++ b/tests/lib/Config/LexiconTest.php @@ -25,10 +25,10 @@ use Test\TestCase; /** * Class UserPreferencesTest * - * @group DB * * @package Test */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class LexiconTest extends TestCase { /** @var AppConfig */ private IAppConfig $appConfig; diff --git a/tests/lib/Config/UserConfigTest.php b/tests/lib/Config/UserConfigTest.php index d570bf020f1..ca51319997f 100644 --- a/tests/lib/Config/UserConfigTest.php +++ b/tests/lib/Config/UserConfigTest.php @@ -25,10 +25,10 @@ use Test\TestCase; /** * Class UserPreferencesTest * - * @group DB * * @package Test */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class UserConfigTest extends TestCase { protected IDBConnection $connection; private IConfig $config; diff --git a/tests/lib/DB/ConnectionTest.php b/tests/lib/DB/ConnectionTest.php index 3f06082ff0c..20348862b7d 100644 --- a/tests/lib/DB/ConnectionTest.php +++ b/tests/lib/DB/ConnectionTest.php @@ -17,9 +17,7 @@ use OC\DB\Adapter; use OC\DB\Connection; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ConnectionTest extends TestCase { public function testSingleNodeConnectsToPrimaryOnly(): void { diff --git a/tests/lib/DB/MigratorTest.php b/tests/lib/DB/MigratorTest.php index 9d8ee6791a9..7e6438dd4a0 100644 --- a/tests/lib/DB/MigratorTest.php +++ b/tests/lib/DB/MigratorTest.php @@ -24,10 +24,10 @@ use OCP\Server; /** * Class MigratorTest * - * @group DB * * @package Test\DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class MigratorTest extends \Test\TestCase { /** * @var \Doctrine\DBAL\Connection $connection diff --git a/tests/lib/DB/OCPostgreSqlPlatformTest.php b/tests/lib/DB/OCPostgreSqlPlatformTest.php index af17b01fc13..d76089e6fe5 100644 --- a/tests/lib/DB/OCPostgreSqlPlatformTest.php +++ b/tests/lib/DB/OCPostgreSqlPlatformTest.php @@ -19,10 +19,10 @@ use Doctrine\DBAL\Types\Types; * custom OCPostgreSqlPlatform behavior has been upstreamed, test is left to * ensure behavior stays correct. * - * @group DB * * @package Test\DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class OCPostgreSqlPlatformTest extends \Test\TestCase { public function testAlterBigint(): void { $platform = new PostgreSQLPlatform(); diff --git a/tests/lib/DB/QueryBuilder/ExpressionBuilderDBTest.php b/tests/lib/DB/QueryBuilder/ExpressionBuilderDBTest.php index a13490c52f3..112bfe2ca16 100644 --- a/tests/lib/DB/QueryBuilder/ExpressionBuilderDBTest.php +++ b/tests/lib/DB/QueryBuilder/ExpressionBuilderDBTest.php @@ -16,9 +16,7 @@ use OCP\IDBConnection; use OCP\Server; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ExpressionBuilderDBTest extends TestCase { /** @var \Doctrine\DBAL\Connection|IDBConnection */ protected $connection; diff --git a/tests/lib/DB/QueryBuilder/ExpressionBuilderTest.php b/tests/lib/DB/QueryBuilder/ExpressionBuilderTest.php index 1373fde5407..4adf6618e81 100644 --- a/tests/lib/DB/QueryBuilder/ExpressionBuilderTest.php +++ b/tests/lib/DB/QueryBuilder/ExpressionBuilderTest.php @@ -19,10 +19,10 @@ use Test\TestCase; /** * Class ExpressionBuilderTest * - * @group DB * * @package Test\DB\QueryBuilder */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ExpressionBuilderTest extends TestCase { /** @var ExpressionBuilder */ protected $expressionBuilder; diff --git a/tests/lib/DB/QueryBuilder/FunctionBuilderTest.php b/tests/lib/DB/QueryBuilder/FunctionBuilderTest.php index eea5c952c39..2202e6879f3 100644 --- a/tests/lib/DB/QueryBuilder/FunctionBuilderTest.php +++ b/tests/lib/DB/QueryBuilder/FunctionBuilderTest.php @@ -16,10 +16,10 @@ use Test\TestCase; /** * Class FunctionBuilderTest * - * @group DB * * @package Test\DB\QueryBuilder */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class FunctionBuilderTest extends TestCase { /** @var \Doctrine\DBAL\Connection|IDBConnection */ protected $connection; diff --git a/tests/lib/DB/QueryBuilder/Partitioned/PartitionedQueryBuilderTest.php b/tests/lib/DB/QueryBuilder/Partitioned/PartitionedQueryBuilderTest.php index f99adc73aa8..80ed5cd17e5 100644 --- a/tests/lib/DB/QueryBuilder/Partitioned/PartitionedQueryBuilderTest.php +++ b/tests/lib/DB/QueryBuilder/Partitioned/PartitionedQueryBuilderTest.php @@ -18,9 +18,7 @@ use OCP\IDBConnection; use OCP\Server; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class PartitionedQueryBuilderTest extends TestCase { private IDBConnection $connection; private ShardConnectionManager $shardConnectionManager; diff --git a/tests/lib/DB/QueryBuilder/QueryBuilderTest.php b/tests/lib/DB/QueryBuilder/QueryBuilderTest.php index a1c3ae3d905..9c4379e6f28 100644 --- a/tests/lib/DB/QueryBuilder/QueryBuilderTest.php +++ b/tests/lib/DB/QueryBuilder/QueryBuilderTest.php @@ -27,10 +27,10 @@ use Psr\Log\LoggerInterface; /** * Class QueryBuilderTest * - * @group DB * * @package Test\DB\QueryBuilder */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class QueryBuilderTest extends \Test\TestCase { private SystemConfig&MockObject $config; private LoggerInterface&MockObject $logger; diff --git a/tests/lib/DB/QueryBuilder/Sharded/SharedQueryBuilderTest.php b/tests/lib/DB/QueryBuilder/Sharded/SharedQueryBuilderTest.php index d0f232cb03f..145a0214570 100644 --- a/tests/lib/DB/QueryBuilder/Sharded/SharedQueryBuilderTest.php +++ b/tests/lib/DB/QueryBuilder/Sharded/SharedQueryBuilderTest.php @@ -19,9 +19,7 @@ use OCP\IDBConnection; use OCP\Server; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class SharedQueryBuilderTest extends TestCase { private IDBConnection $connection; private AutoIncrementHandler $autoIncrementHandler; diff --git a/tests/lib/DirectEditing/ManagerTest.php b/tests/lib/DirectEditing/ManagerTest.php index 2ad7f98df49..f519aeb4ce1 100644 --- a/tests/lib/DirectEditing/ManagerTest.php +++ b/tests/lib/DirectEditing/ManagerTest.php @@ -83,8 +83,8 @@ class Editor implements IEditor { * Class ManagerTest * * @package Test\DirectEditing - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ManagerTest extends TestCase { private $manager; /** diff --git a/tests/lib/Encryption/DecryptAllTest.php b/tests/lib/Encryption/DecryptAllTest.php index 5b56ac271c5..f7a6497e011 100644 --- a/tests/lib/Encryption/DecryptAllTest.php +++ b/tests/lib/Encryption/DecryptAllTest.php @@ -28,10 +28,10 @@ use Test\TestCase; /** * Class DecryptAllTest * - * @group DB * * @package Test\Encryption */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class DecryptAllTest extends TestCase { private IUserManager&MockObject $userManager; private Manager&MockObject $encryptionManager; diff --git a/tests/lib/Encryption/ManagerTest.php b/tests/lib/Encryption/ManagerTest.php index e9b6ddae8a0..6a6e55b1925 100644 --- a/tests/lib/Encryption/ManagerTest.php +++ b/tests/lib/Encryption/ManagerTest.php @@ -87,9 +87,7 @@ class ManagerTest extends TestCase { return $this->manager; } - /** - * @depends testModuleRegistration - */ + #[\PHPUnit\Framework\Attributes\Depends('testModuleRegistration')] public function testModuleReRegistration($manager): void { $this->expectException(ModuleAlreadyExistsException::class); $this->expectExceptionMessage('Id "ID0" already used by encryption module "TestDummyModule0"'); diff --git a/tests/lib/Federation/CloudIdManagerTest.php b/tests/lib/Federation/CloudIdManagerTest.php index cf97c895dc1..391baa79960 100644 --- a/tests/lib/Federation/CloudIdManagerTest.php +++ b/tests/lib/Federation/CloudIdManagerTest.php @@ -19,9 +19,7 @@ use OCP\IURLGenerator; use OCP\IUserManager; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CloudIdManagerTest extends TestCase { /** @var IManager|\PHPUnit\Framework\MockObject\MockObject */ protected $contactsManager; diff --git a/tests/lib/Federation/CloudIdTest.php b/tests/lib/Federation/CloudIdTest.php index 67683861e40..cfda35b545e 100644 --- a/tests/lib/Federation/CloudIdTest.php +++ b/tests/lib/Federation/CloudIdTest.php @@ -15,9 +15,7 @@ use OCP\Federation\ICloudIdManager; use PHPUnit\Framework\MockObject\MockObject; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CloudIdTest extends TestCase { protected CloudIdManager&MockObject $cloudIdManager; diff --git a/tests/lib/Files/Cache/CacheTest.php b/tests/lib/Files/Cache/CacheTest.php index 0ce5a5ac2b3..c2dca753f5a 100644 --- a/tests/lib/Files/Cache/CacheTest.php +++ b/tests/lib/Files/Cache/CacheTest.php @@ -33,10 +33,10 @@ class LongId extends Temporary { /** * Class CacheTest * - * @group DB * * @package Test\Files\Cache */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CacheTest extends \Test\TestCase { /** * @var Temporary $storage ; diff --git a/tests/lib/Files/Cache/FileAccessTest.php b/tests/lib/Files/Cache/FileAccessTest.php index 9a786f9430a..36658c5a721 100644 --- a/tests/lib/Files/Cache/FileAccessTest.php +++ b/tests/lib/Files/Cache/FileAccessTest.php @@ -19,9 +19,7 @@ use OCP\Server; use Psr\Log\LoggerInterface; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class FileAccessTest extends TestCase { private IDBConnection $dbConnection; private FileAccess $fileAccess; diff --git a/tests/lib/Files/Cache/HomeCacheTest.php b/tests/lib/Files/Cache/HomeCacheTest.php index 86fda615fd7..f5afecca888 100644 --- a/tests/lib/Files/Cache/HomeCacheTest.php +++ b/tests/lib/Files/Cache/HomeCacheTest.php @@ -42,10 +42,10 @@ class DummyUser extends User { /** * Class HomeCacheTest * - * @group DB * * @package Test\Files\Cache */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class HomeCacheTest extends \Test\TestCase { /** * @var \OC\Files\Storage\Home $storage diff --git a/tests/lib/Files/Cache/LocalRootScannerTest.php b/tests/lib/Files/Cache/LocalRootScannerTest.php index 727da2ed698..12f785103d1 100644 --- a/tests/lib/Files/Cache/LocalRootScannerTest.php +++ b/tests/lib/Files/Cache/LocalRootScannerTest.php @@ -13,9 +13,7 @@ use OCP\ITempManager; use OCP\Server; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class LocalRootScannerTest extends TestCase { /** @var LocalRootStorage */ private $storage; diff --git a/tests/lib/Files/Cache/MoveFromCacheTraitTest.php b/tests/lib/Files/Cache/MoveFromCacheTraitTest.php index d580cdca4b9..c7e20f55fbe 100644 --- a/tests/lib/Files/Cache/MoveFromCacheTraitTest.php +++ b/tests/lib/Files/Cache/MoveFromCacheTraitTest.php @@ -19,9 +19,8 @@ class FallBackCrossCacheMoveCache extends Cache { /** * Class MoveFromCacheTraitTest - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class MoveFromCacheTraitTest extends CacheTest { protected function setUp(): void { parent::setUp(); diff --git a/tests/lib/Files/Cache/PropagatorTest.php b/tests/lib/Files/Cache/PropagatorTest.php index 2ab213e9567..71e30eea4b0 100644 --- a/tests/lib/Files/Cache/PropagatorTest.php +++ b/tests/lib/Files/Cache/PropagatorTest.php @@ -13,9 +13,7 @@ use OCP\Files\Cache\ICacheEntry; use OCP\Files\Storage\IStorage; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class PropagatorTest extends TestCase { /** @var IStorage */ private $storage; diff --git a/tests/lib/Files/Cache/ScannerTest.php b/tests/lib/Files/Cache/ScannerTest.php index 370ad5f60dc..e8dbc5006f1 100644 --- a/tests/lib/Files/Cache/ScannerTest.php +++ b/tests/lib/Files/Cache/ScannerTest.php @@ -22,10 +22,10 @@ use Test\TestCase; /** * Class ScannerTest * - * @group DB * * @package Test\Files\Cache */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ScannerTest extends TestCase { private Storage $storage; private Scanner $scanner; diff --git a/tests/lib/Files/Cache/SearchBuilderTest.php b/tests/lib/Files/Cache/SearchBuilderTest.php index 89589fcd342..28fe6aefc77 100644 --- a/tests/lib/Files/Cache/SearchBuilderTest.php +++ b/tests/lib/Files/Cache/SearchBuilderTest.php @@ -21,9 +21,7 @@ use OCP\IDBConnection; use OCP\Server; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class SearchBuilderTest extends TestCase { /** @var IQueryBuilder */ private $builder; diff --git a/tests/lib/Files/Cache/UpdaterLegacyTest.php b/tests/lib/Files/Cache/UpdaterLegacyTest.php index c71fac2d2dc..fb8083d9670 100644 --- a/tests/lib/Files/Cache/UpdaterLegacyTest.php +++ b/tests/lib/Files/Cache/UpdaterLegacyTest.php @@ -21,10 +21,10 @@ use OCP\Server; /** * Class UpdaterLegacyTest * - * @group DB * * @package Test\Files\Cache */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class UpdaterLegacyTest extends \Test\TestCase { /** * @var Storage $storage diff --git a/tests/lib/Files/Cache/UpdaterTest.php b/tests/lib/Files/Cache/UpdaterTest.php index 65c47cb9ae6..f651617e031 100644 --- a/tests/lib/Files/Cache/UpdaterTest.php +++ b/tests/lib/Files/Cache/UpdaterTest.php @@ -20,10 +20,10 @@ use OCP\Files\Storage\IStorage; /** * Class UpdaterTest * - * @group DB * * @package Test\Files\Cache */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class UpdaterTest extends \Test\TestCase { /** * @var Storage diff --git a/tests/lib/Files/Cache/WatcherTest.php b/tests/lib/Files/Cache/WatcherTest.php index 6d0a8e0886b..fecafb004b7 100644 --- a/tests/lib/Files/Cache/WatcherTest.php +++ b/tests/lib/Files/Cache/WatcherTest.php @@ -15,10 +15,10 @@ use OC\Files\Storage\Temporary; /** * Class WatcherTest * - * @group DB * * @package Test\Files\Cache */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class WatcherTest extends \Test\TestCase { /** * @var Storage[] $storages diff --git a/tests/lib/Files/Cache/Wrapper/CacheJailTest.php b/tests/lib/Files/Cache/Wrapper/CacheJailTest.php index 8ac3492fbd2..f5fecdab92b 100644 --- a/tests/lib/Files/Cache/Wrapper/CacheJailTest.php +++ b/tests/lib/Files/Cache/Wrapper/CacheJailTest.php @@ -23,10 +23,10 @@ use Test\Files\Cache\CacheTest; /** * Class CacheJail * - * @group DB * * @package Test\Files\Cache\Wrapper */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CacheJailTest extends CacheTest { /** * @var Cache $sourceCache diff --git a/tests/lib/Files/Cache/Wrapper/CachePermissionsMaskTest.php b/tests/lib/Files/Cache/Wrapper/CachePermissionsMaskTest.php index 4fbeafc9270..f763fd1608c 100644 --- a/tests/lib/Files/Cache/Wrapper/CachePermissionsMaskTest.php +++ b/tests/lib/Files/Cache/Wrapper/CachePermissionsMaskTest.php @@ -16,10 +16,10 @@ use Test\Files\Cache\CacheTest; /** * Class CachePermissionsMask * - * @group DB * * @package Test\Files\Cache\Wrapper */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CachePermissionsMaskTest extends CacheTest { /** * @var Cache $sourceCache diff --git a/tests/lib/Files/Config/UserMountCacheTest.php b/tests/lib/Files/Config/UserMountCacheTest.php index bf957e04ed4..31a2ea36998 100644 --- a/tests/lib/Files/Config/UserMountCacheTest.php +++ b/tests/lib/Files/Config/UserMountCacheTest.php @@ -32,9 +32,7 @@ use Psr\Log\LoggerInterface; use Test\TestCase; use Test\Util\User\Dummy; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class UserMountCacheTest extends TestCase { private IDBConnection $connection; private IUserManager $userManager; diff --git a/tests/lib/Files/EtagTest.php b/tests/lib/Files/EtagTest.php index d1b344ee997..ccf1319a6a8 100644 --- a/tests/lib/Files/EtagTest.php +++ b/tests/lib/Files/EtagTest.php @@ -23,10 +23,10 @@ use Psr\Log\LoggerInterface; /** * Class EtagTest * - * @group DB * * @package Test\Files */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class EtagTest extends \Test\TestCase { private $datadir; diff --git a/tests/lib/Files/FilesystemTest.php b/tests/lib/Files/FilesystemTest.php index a819acb1620..b838310d28b 100644 --- a/tests/lib/Files/FilesystemTest.php +++ b/tests/lib/Files/FilesystemTest.php @@ -49,10 +49,10 @@ class DummyMountProvider implements IMountProvider { /** * Class FilesystemTest * - * @group DB * * @package Test\Files */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class FilesystemTest extends \Test\TestCase { public const TEST_FILESYSTEM_USER1 = 'test-filesystem-user1'; public const TEST_FILESYSTEM_USER2 = 'test-filesystem-user1'; diff --git a/tests/lib/Files/Mount/RootMountProviderTest.php b/tests/lib/Files/Mount/RootMountProviderTest.php index bf29bfa070a..fedd62d5663 100644 --- a/tests/lib/Files/Mount/RootMountProviderTest.php +++ b/tests/lib/Files/Mount/RootMountProviderTest.php @@ -18,9 +18,7 @@ use OCP\App\IAppManager; use OCP\IConfig; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class RootMountProviderTest extends TestCase { private StorageFactory $loader; diff --git a/tests/lib/Files/Node/FileTest.php b/tests/lib/Files/Node/FileTest.php index 0ab8a32b6cb..027f48498ea 100644 --- a/tests/lib/Files/Node/FileTest.php +++ b/tests/lib/Files/Node/FileTest.php @@ -16,10 +16,10 @@ use OCP\Files\NotPermittedException; /** * Class FileTest * - * @group DB * * @package Test\Files\Node */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class FileTest extends NodeTestCase { protected function createTestNode($root, $view, $path, array $data = [], $internalPath = '', $storage = null) { if ($data || $internalPath || $storage) { diff --git a/tests/lib/Files/Node/FolderTest.php b/tests/lib/Files/Node/FolderTest.php index fc8b778cb34..8ba0745d4e1 100644 --- a/tests/lib/Files/Node/FolderTest.php +++ b/tests/lib/Files/Node/FolderTest.php @@ -42,10 +42,10 @@ use PHPUnit\Framework\MockObject\MockObject; /** * Class FolderTest * - * @group DB * * @package Test\Files\Node */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class FolderTest extends NodeTestCase { protected function createTestNode($root, $view, $path, array $data = [], $internalPath = '', $storage = null) { $view->expects($this->any()) diff --git a/tests/lib/Files/Node/HookConnectorTest.php b/tests/lib/Files/Node/HookConnectorTest.php index f108b279179..949fcf7be8b 100644 --- a/tests/lib/Files/Node/HookConnectorTest.php +++ b/tests/lib/Files/Node/HookConnectorTest.php @@ -45,10 +45,10 @@ use Test\Traits\UserTrait; /** * Class HookConnectorTest * - * @group DB * * @package Test\Files\Node */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class HookConnectorTest extends TestCase { use UserTrait; use MountProviderTrait; diff --git a/tests/lib/Files/Node/IntegrationTest.php b/tests/lib/Files/Node/IntegrationTest.php index d7a56a0f99d..af54a2e162d 100644 --- a/tests/lib/Files/Node/IntegrationTest.php +++ b/tests/lib/Files/Node/IntegrationTest.php @@ -26,10 +26,10 @@ use Test\Traits\UserTrait; /** * Class IntegrationTest * - * @group DB * * @package Test\Files\Node */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class IntegrationTest extends \Test\TestCase { use UserTrait; diff --git a/tests/lib/Files/ObjectStore/AzureTest.php b/tests/lib/Files/ObjectStore/AzureTest.php index 52d2b9e8657..76d1d7f1d87 100644 --- a/tests/lib/Files/ObjectStore/AzureTest.php +++ b/tests/lib/Files/ObjectStore/AzureTest.php @@ -11,9 +11,7 @@ use OC\Files\ObjectStore\Azure; use OCP\IConfig; use OCP\Server; -/** - * @group PRIMARY-azure - */ +#[\PHPUnit\Framework\Attributes\Group('PRIMARY-azure')] class AzureTest extends ObjectStoreTestCase { protected function getInstance() { $config = Server::get(IConfig::class)->getSystemValue('objectstore'); diff --git a/tests/lib/Files/ObjectStore/ObjectStoreScannerTest.php b/tests/lib/Files/ObjectStore/ObjectStoreScannerTest.php index ea6ac682c70..f5af264a25e 100644 --- a/tests/lib/Files/ObjectStore/ObjectStoreScannerTest.php +++ b/tests/lib/Files/ObjectStore/ObjectStoreScannerTest.php @@ -14,9 +14,7 @@ use OCP\Files\Cache\ICache; use OCP\Files\Storage\IStorage; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ObjectStoreScannerTest extends TestCase { private IStorage $storage; private ICache $cache; diff --git a/tests/lib/Files/ObjectStore/ObjectStoreStorageTest.php b/tests/lib/Files/ObjectStore/ObjectStoreStorageTest.php index 3387808445a..f980b49172f 100644 --- a/tests/lib/Files/ObjectStore/ObjectStoreStorageTest.php +++ b/tests/lib/Files/ObjectStore/ObjectStoreStorageTest.php @@ -15,9 +15,7 @@ use OCP\Constants; use OCP\Files\ObjectStore\IObjectStore; use Test\Files\Storage\Storage; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ObjectStoreStorageTest extends Storage { /** @var ObjectStoreStorageOverwrite */ protected $instance; diff --git a/tests/lib/Files/ObjectStore/ObjectStoreStoragesDifferentBucketTest.php b/tests/lib/Files/ObjectStore/ObjectStoreStoragesDifferentBucketTest.php index d39426ee821..d84df898a12 100644 --- a/tests/lib/Files/ObjectStore/ObjectStoreStoragesDifferentBucketTest.php +++ b/tests/lib/Files/ObjectStore/ObjectStoreStoragesDifferentBucketTest.php @@ -13,9 +13,7 @@ use OC\Files\Storage\Temporary; use OCP\Files\ObjectStore\IObjectStore; use Test\Files\Storage\StoragesTestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ObjectStoreStoragesDifferentBucketTest extends StoragesTestCase { /** * @var IObjectStore diff --git a/tests/lib/Files/ObjectStore/ObjectStoreStoragesSameBucketTest.php b/tests/lib/Files/ObjectStore/ObjectStoreStoragesSameBucketTest.php index 4e42668cd3f..5779c0c4336 100644 --- a/tests/lib/Files/ObjectStore/ObjectStoreStoragesSameBucketTest.php +++ b/tests/lib/Files/ObjectStore/ObjectStoreStoragesSameBucketTest.php @@ -13,9 +13,7 @@ use OC\Files\Storage\Temporary; use OCP\Files\ObjectStore\IObjectStore; use Test\Files\Storage\StoragesTestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ObjectStoreStoragesSameBucketTest extends StoragesTestCase { /** * @var IObjectStore diff --git a/tests/lib/Files/ObjectStore/S3Test.php b/tests/lib/Files/ObjectStore/S3Test.php index b75536a214e..0a20653f88e 100644 --- a/tests/lib/Files/ObjectStore/S3Test.php +++ b/tests/lib/Files/ObjectStore/S3Test.php @@ -44,9 +44,7 @@ class NonSeekableStream extends Wrapper { } } -/** - * @group PRIMARY-s3 - */ +#[\PHPUnit\Framework\Attributes\Group('PRIMARY-s3')] class S3Test extends ObjectStoreTestCase { public function setUp(): void { parent::setUp(); diff --git a/tests/lib/Files/ObjectStore/SwiftTest.php b/tests/lib/Files/ObjectStore/SwiftTest.php index 3f919c0dd48..a5d97256dd6 100644 --- a/tests/lib/Files/ObjectStore/SwiftTest.php +++ b/tests/lib/Files/ObjectStore/SwiftTest.php @@ -13,9 +13,7 @@ use OCP\Files\ObjectStore\IObjectStore; use OCP\IConfig; use OCP\Server; -/** - * @group PRIMARY-swift - */ +#[\PHPUnit\Framework\Attributes\Group('PRIMARY-swift')] class SwiftTest extends ObjectStoreTestCase { /** * @return IObjectStore diff --git a/tests/lib/Files/PathVerificationTest.php b/tests/lib/Files/PathVerificationTest.php index e13dbe4f207..6ba9af5d9eb 100644 --- a/tests/lib/Files/PathVerificationTest.php +++ b/tests/lib/Files/PathVerificationTest.php @@ -17,10 +17,10 @@ use OCP\Server; /** * Class PathVerificationTest * - * @group DB * * @package Test\Files */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class PathVerificationTest extends \Test\TestCase { /** * @var View diff --git a/tests/lib/Files/Search/QueryOptimizer/CombinedTests.php b/tests/lib/Files/Search/QueryOptimizer/CombinedTests.php index 665224cb63e..3590af850c8 100644 --- a/tests/lib/Files/Search/QueryOptimizer/CombinedTests.php +++ b/tests/lib/Files/Search/QueryOptimizer/CombinedTests.php @@ -14,6 +14,9 @@ use OCP\Files\Search\ISearchComparison; use Test\TestCase; class CombinedTests extends TestCase { + public function __construct() { + parent::__construct(static::class); + } private QueryOptimizer $optimizer; protected function setUp(): void { diff --git a/tests/lib/Files/Search/SearchIntegrationTest.php b/tests/lib/Files/Search/SearchIntegrationTest.php index e3e99bbfadd..c1efb839eab 100644 --- a/tests/lib/Files/Search/SearchIntegrationTest.php +++ b/tests/lib/Files/Search/SearchIntegrationTest.php @@ -14,9 +14,7 @@ use OCP\Files\Search\ISearchBinaryOperator; use OCP\Files\Search\ISearchComparison; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class SearchIntegrationTest extends TestCase { private $cache; private $storage; diff --git a/tests/lib/Files/SimpleFS/SimpleFolderTest.php b/tests/lib/Files/SimpleFS/SimpleFolderTest.php index 50038b286a9..9232bdce42f 100644 --- a/tests/lib/Files/SimpleFS/SimpleFolderTest.php +++ b/tests/lib/Files/SimpleFS/SimpleFolderTest.php @@ -16,9 +16,7 @@ use OCP\Files\SimpleFS\ISimpleFolder; use Test\Traits\MountProviderTrait; use Test\Traits\UserTrait; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class SimpleFolderTest extends \Test\TestCase { use MountProviderTrait; use UserTrait; diff --git a/tests/lib/Files/Storage/CommonTest.php b/tests/lib/Files/Storage/CommonTest.php index c5ff6bb0b5f..f460d579ce2 100644 --- a/tests/lib/Files/Storage/CommonTest.php +++ b/tests/lib/Files/Storage/CommonTest.php @@ -21,10 +21,10 @@ use PHPUnit\Framework\MockObject\MockObject; /** * Class CommonTest * - * @group DB * * @package Test\Files\Storage */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CommonTest extends Storage { private string $tmpDir; diff --git a/tests/lib/Files/Storage/CopyDirectoryTest.php b/tests/lib/Files/Storage/CopyDirectoryTest.php index b593b7c984f..141f6532a3d 100644 --- a/tests/lib/Files/Storage/CopyDirectoryTest.php +++ b/tests/lib/Files/Storage/CopyDirectoryTest.php @@ -27,10 +27,10 @@ class CopyDirectoryStorage extends StorageNoRecursiveCopy { /** * Class CopyDirectoryTest * - * @group DB * * @package Test\Files\Storage */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CopyDirectoryTest extends Storage { protected function setUp(): void { parent::setUp(); diff --git a/tests/lib/Files/Storage/HomeTest.php b/tests/lib/Files/Storage/HomeTest.php index 84a9816cb0c..2ca9737bf79 100644 --- a/tests/lib/Files/Storage/HomeTest.php +++ b/tests/lib/Files/Storage/HomeTest.php @@ -37,10 +37,10 @@ class DummyUser extends User { /** * Class Home * - * @group DB * * @package Test\Files\Storage */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class HomeTest extends Storage { /** * @var string tmpDir diff --git a/tests/lib/Files/Storage/LocalTest.php b/tests/lib/Files/Storage/LocalTest.php index 89449a51351..1e05e295ded 100644 --- a/tests/lib/Files/Storage/LocalTest.php +++ b/tests/lib/Files/Storage/LocalTest.php @@ -19,10 +19,10 @@ use OCP\Server; /** * Class LocalTest * - * @group DB * * @package Test\Files\Storage */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class LocalTest extends Storage { /** * @var string tmpDir diff --git a/tests/lib/Files/Storage/Wrapper/KnownMtimeTest.php b/tests/lib/Files/Storage/Wrapper/KnownMtimeTest.php index b1b5582b4ed..5a8261289aa 100644 --- a/tests/lib/Files/Storage/Wrapper/KnownMtimeTest.php +++ b/tests/lib/Files/Storage/Wrapper/KnownMtimeTest.php @@ -13,9 +13,7 @@ use PHPUnit\Framework\MockObject\MockObject; use Psr\Clock\ClockInterface; use Test\Files\Storage\Storage; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class KnownMtimeTest extends Storage { /** @var Temporary */ private $sourceStorage; diff --git a/tests/lib/Files/Storage/Wrapper/PermissionsMaskTest.php b/tests/lib/Files/Storage/Wrapper/PermissionsMaskTest.php index a2f3460c58c..1b4a55b010b 100644 --- a/tests/lib/Files/Storage/Wrapper/PermissionsMaskTest.php +++ b/tests/lib/Files/Storage/Wrapper/PermissionsMaskTest.php @@ -14,9 +14,7 @@ use OC\Files\Storage\Wrapper\Wrapper; use OCP\Constants; use OCP\Files\Cache\IScanner; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class PermissionsMaskTest extends \Test\Files\Storage\Storage { /** * @var Temporary diff --git a/tests/lib/Files/Storage/Wrapper/QuotaTest.php b/tests/lib/Files/Storage/Wrapper/QuotaTest.php index f83f9ed7f6b..3d313666a93 100644 --- a/tests/lib/Files/Storage/Wrapper/QuotaTest.php +++ b/tests/lib/Files/Storage/Wrapper/QuotaTest.php @@ -19,10 +19,10 @@ use OCP\Server; /** * Class QuotaTest * - * @group DB * * @package Test\Files\Storage\Wrapper */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class QuotaTest extends \Test\Files\Storage\Storage { /** * @var string tmpDir diff --git a/tests/lib/Files/Utils/ScannerTest.php b/tests/lib/Files/Utils/ScannerTest.php index 49399ef70a6..3e60c0234cb 100644 --- a/tests/lib/Files/Utils/ScannerTest.php +++ b/tests/lib/Files/Utils/ScannerTest.php @@ -43,10 +43,10 @@ class TestScanner extends Scanner { /** * Class ScannerTest * - * @group DB * * @package Test\Files\Utils */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ScannerTest extends \Test\TestCase { /** * @var \Test\Util\User\Dummy diff --git a/tests/lib/Files/ViewTest.php b/tests/lib/Files/ViewTest.php index e7ad9fcc304..3a6a223b135 100644 --- a/tests/lib/Files/ViewTest.php +++ b/tests/lib/Files/ViewTest.php @@ -90,10 +90,10 @@ class TestEventHandler { /** * Class ViewTest * - * @group DB * * @package Test\Files */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ViewTest extends \Test\TestCase { use UserTrait; diff --git a/tests/lib/FilesMetadata/FilesMetadataManagerTest.php b/tests/lib/FilesMetadata/FilesMetadataManagerTest.php index 2f9edba0015..a19ff451012 100644 --- a/tests/lib/FilesMetadata/FilesMetadataManagerTest.php +++ b/tests/lib/FilesMetadata/FilesMetadataManagerTest.php @@ -26,9 +26,7 @@ use Test\TestCase; use Test\Traits\MountProviderTrait; use Test\Traits\UserTrait; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class FilesMetadataManagerTest extends TestCase { use UserTrait; use MountProviderTrait; @@ -49,7 +47,7 @@ class FilesMetadataManagerTest extends TestCase { $this->jobList = $this->createMock(JobList::class); $this->eventDispatcher = $this->createMock(IEventDispatcher::class); - $this->eventDispatcher->method('dispatchTyped')->willReturnCallback(function (Event $event) { + $this->eventDispatcher->method('dispatchTyped')->willReturnCallback(function (Event $event): void { if ($event instanceof AMetadataEvent) { $name = $event->getNode()->getName(); if (isset($this->metadata[$name])) { diff --git a/tests/lib/Group/Backend.php b/tests/lib/Group/Backend.php index 3c068d8d64b..23d877012cb 100644 --- a/tests/lib/Group/Backend.php +++ b/tests/lib/Group/Backend.php @@ -10,9 +10,8 @@ namespace Test\Group; /** * Class Backend - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] abstract class Backend extends \Test\TestCase { /** * @var \OC\Group\Backend $backend diff --git a/tests/lib/Group/DatabaseTest.php b/tests/lib/Group/DatabaseTest.php index ee6b4156040..b8ec53f0d0d 100644 --- a/tests/lib/Group/DatabaseTest.php +++ b/tests/lib/Group/DatabaseTest.php @@ -12,9 +12,8 @@ use OC\Group\Database; /** * Class Database - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class DatabaseTest extends Backend { private $groups = []; diff --git a/tests/lib/Group/Dummy.php b/tests/lib/Group/Dummy.php index 5f2a81cf0aa..45e82f1a391 100644 --- a/tests/lib/Group/Dummy.php +++ b/tests/lib/Group/Dummy.php @@ -10,10 +10,12 @@ namespace Test\Group; /** * Class Dummy - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class Dummy extends Backend { + public function __construct() { + parent::__construct(static::class); + } protected function setUp(): void { parent::setUp(); $this->backend = new \Test\Util\Group\Dummy(); diff --git a/tests/lib/HelperStorageTest.php b/tests/lib/HelperStorageTest.php index 140be04ec64..81cff4a283e 100644 --- a/tests/lib/HelperStorageTest.php +++ b/tests/lib/HelperStorageTest.php @@ -19,9 +19,8 @@ use Test\Traits\UserTrait; /** * Test the storage functions of OC_Helper - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class HelperStorageTest extends \Test\TestCase { use UserTrait; diff --git a/tests/lib/ImageTest.php b/tests/lib/ImageTest.php index 02181fffff9..fe4b32a07fc 100644 --- a/tests/lib/ImageTest.php +++ b/tests/lib/ImageTest.php @@ -172,9 +172,7 @@ class ImageTest extends \Test\TestCase { $this->assertNull($img->data()); } - /** - * @depends testData - */ + #[\PHPUnit\Framework\Attributes\Depends('testData')] public function testToString(): void { $img = new Image(); $img->loadFromFile(OC::$SERVERROOT . '/tests/data/testimage.png'); diff --git a/tests/lib/InfoXmlTest.php b/tests/lib/InfoXmlTest.php index e92ee5b9c09..9506f87c1b0 100644 --- a/tests/lib/InfoXmlTest.php +++ b/tests/lib/InfoXmlTest.php @@ -14,9 +14,9 @@ use OCP\Server; /** * Class InfoXmlTest * - * @group DB * @package Test */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class InfoXmlTest extends TestCase { private IAppManager $appManager; diff --git a/tests/lib/InstallerTest.php b/tests/lib/InstallerTest.php index e763807be29..2c2cd5cbd1e 100644 --- a/tests/lib/InstallerTest.php +++ b/tests/lib/InstallerTest.php @@ -25,8 +25,8 @@ use Psr\Log\LoggerInterface; * Class InstallerTest * * @package Test - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class InstallerTest extends TestCase { private static $appid = 'testapp'; private $appstore; diff --git a/tests/lib/Lock/DBLockingProviderTest.php b/tests/lib/Lock/DBLockingProviderTest.php index 872ee55d8a8..3e6b7c5198e 100644 --- a/tests/lib/Lock/DBLockingProviderTest.php +++ b/tests/lib/Lock/DBLockingProviderTest.php @@ -17,10 +17,10 @@ use OCP\Server; /** * Class DBLockingProvider * - * @group DB * * @package Test\Lock */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class DBLockingProviderTest extends LockingProvider { /** * @var \OC\Lock\DBLockingProvider diff --git a/tests/lib/Lock/NonCachingDBLockingProviderTest.php b/tests/lib/Lock/NonCachingDBLockingProviderTest.php index 4f05f0ba892..a3b878177fa 100644 --- a/tests/lib/Lock/NonCachingDBLockingProviderTest.php +++ b/tests/lib/Lock/NonCachingDBLockingProviderTest.php @@ -13,10 +13,9 @@ use OCP\Lock\ILockingProvider; use OCP\Server; /** - * @group DB - * * @package Test\Lock */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class NonCachingDBLockingProviderTest extends DBLockingProviderTest { /** * @return ILockingProvider diff --git a/tests/lib/Lockdown/Filesystem/NoFSTest.php b/tests/lib/Lockdown/Filesystem/NoFSTest.php index b471f573fb7..b257a8b29dc 100644 --- a/tests/lib/Lockdown/Filesystem/NoFSTest.php +++ b/tests/lib/Lockdown/Filesystem/NoFSTest.php @@ -14,9 +14,7 @@ use OCP\Authentication\Token\IToken; use OCP\Server; use Test\Traits\UserTrait; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class NoFSTest extends \Test\TestCase { use UserTrait; diff --git a/tests/lib/Memcache/APCuTest.php b/tests/lib/Memcache/APCuTest.php index 199bdf298f6..3b113257196 100644 --- a/tests/lib/Memcache/APCuTest.php +++ b/tests/lib/Memcache/APCuTest.php @@ -10,10 +10,8 @@ namespace Test\Memcache; use OC\Memcache\APCu; -/** - * @group Memcache - * @group APCu - */ +#[\PHPUnit\Framework\Attributes\Group('Memcache')] +#[\PHPUnit\Framework\Attributes\Group('APCu')] class APCuTest extends Cache { protected function setUp(): void { parent::setUp(); diff --git a/tests/lib/Memcache/ArrayCacheTest.php b/tests/lib/Memcache/ArrayCacheTest.php index e71c821729c..2e27994d128 100644 --- a/tests/lib/Memcache/ArrayCacheTest.php +++ b/tests/lib/Memcache/ArrayCacheTest.php @@ -10,9 +10,7 @@ namespace Test\Memcache; use OC\Memcache\ArrayCache; -/** - * @group Memcache - */ +#[\PHPUnit\Framework\Attributes\Group('Memcache')] class ArrayCacheTest extends Cache { protected function setUp(): void { parent::setUp(); diff --git a/tests/lib/Memcache/CasTraitTest.php b/tests/lib/Memcache/CasTraitTest.php index 9de04fa2726..f3a193ccc92 100644 --- a/tests/lib/Memcache/CasTraitTest.php +++ b/tests/lib/Memcache/CasTraitTest.php @@ -11,9 +11,7 @@ namespace Test\Memcache; use OC\Memcache\ArrayCache; use Test\TestCase; -/** - * @group Memcache - */ +#[\PHPUnit\Framework\Attributes\Group('Memcache')] class CasTraitTest extends TestCase { /** * @return \OC\Memcache\CasTrait diff --git a/tests/lib/Memcache/FactoryTest.php b/tests/lib/Memcache/FactoryTest.php index 31500f31b65..338d2888c71 100644 --- a/tests/lib/Memcache/FactoryTest.php +++ b/tests/lib/Memcache/FactoryTest.php @@ -51,9 +51,7 @@ class Test_Factory_Unavailable_Cache2 extends NullCache { } } -/** - * @group Memcache - */ +#[\PHPUnit\Framework\Attributes\Group('Memcache')] class FactoryTest extends \Test\TestCase { public const AVAILABLE1 = '\\Test\\Memcache\\Test_Factory_Available_Cache1'; public const AVAILABLE2 = '\\Test\\Memcache\\Test_Factory_Available_Cache2'; diff --git a/tests/lib/Memcache/MemcachedTest.php b/tests/lib/Memcache/MemcachedTest.php index 61e2f42e3d6..54ab47c21b5 100644 --- a/tests/lib/Memcache/MemcachedTest.php +++ b/tests/lib/Memcache/MemcachedTest.php @@ -10,10 +10,8 @@ namespace Test\Memcache; use OC\Memcache\Memcached; -/** - * @group Memcache - * @group Memcached - */ +#[\PHPUnit\Framework\Attributes\Group('Memcache')] +#[\PHPUnit\Framework\Attributes\Group('Memcached')] class MemcachedTest extends Cache { public static function setUpBeforeClass(): void { parent::setUpBeforeClass(); diff --git a/tests/lib/Memcache/RedisTest.php b/tests/lib/Memcache/RedisTest.php index c1dcc954925..18c08b87c98 100644 --- a/tests/lib/Memcache/RedisTest.php +++ b/tests/lib/Memcache/RedisTest.php @@ -12,10 +12,8 @@ use OC\Memcache\Redis; use OCP\IConfig; use OCP\Server; -/** - * @group Memcache - * @group Redis - */ +#[\PHPUnit\Framework\Attributes\Group('Memcache')] +#[\PHPUnit\Framework\Attributes\Group('Redis')] class RedisTest extends Cache { /** * @var Redis cache; diff --git a/tests/lib/Preview/BackgroundCleanupJobTest.php b/tests/lib/Preview/BackgroundCleanupJobTest.php index 80df690ad76..23f0dabba98 100644 --- a/tests/lib/Preview/BackgroundCleanupJobTest.php +++ b/tests/lib/Preview/BackgroundCleanupJobTest.php @@ -25,10 +25,10 @@ use Test\Traits\UserTrait; /** * Class BackgroundCleanupJobTest * - * @group DB * * @package Test\Preview */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class BackgroundCleanupJobTest extends \Test\TestCase { use MountProviderTrait; use UserTrait; diff --git a/tests/lib/Preview/BitmapTest.php b/tests/lib/Preview/BitmapTest.php index 36e768010a9..1758523a3b6 100644 --- a/tests/lib/Preview/BitmapTest.php +++ b/tests/lib/Preview/BitmapTest.php @@ -13,10 +13,10 @@ use OC\Preview\Postscript; /** * Class BitmapTest * - * @group DB * * @package Test\Preview */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class BitmapTest extends Provider { protected function setUp(): void { parent::setUp(); diff --git a/tests/lib/Preview/HEICTest.php b/tests/lib/Preview/HEICTest.php index def113fbf08..bac8e1b816f 100644 --- a/tests/lib/Preview/HEICTest.php +++ b/tests/lib/Preview/HEICTest.php @@ -12,10 +12,10 @@ use OC\Preview\HEIC; /** * Class BitmapTest * - * @group DB * * @package Test\Preview */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class HEICTest extends Provider { protected function setUp(): void { if (!in_array('HEIC', \Imagick::queryFormats('HEI*'))) { diff --git a/tests/lib/Preview/ImageTest.php b/tests/lib/Preview/ImageTest.php index 88c3090557f..65cd194277f 100644 --- a/tests/lib/Preview/ImageTest.php +++ b/tests/lib/Preview/ImageTest.php @@ -13,10 +13,10 @@ use OC\Preview\JPEG; /** * Class ImageTest * - * @group DB * * @package Test\Preview */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ImageTest extends Provider { protected function setUp(): void { parent::setUp(); diff --git a/tests/lib/Preview/MP3Test.php b/tests/lib/Preview/MP3Test.php index faa06fe42a1..4cd41e4a0b0 100644 --- a/tests/lib/Preview/MP3Test.php +++ b/tests/lib/Preview/MP3Test.php @@ -13,10 +13,10 @@ use OC\Preview\MP3; /** * Class MP3Test * - * @group DB * * @package Test\Preview */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class MP3Test extends Provider { protected function setUp(): void { parent::setUp(); diff --git a/tests/lib/Preview/MovePreviewJobTest.php b/tests/lib/Preview/MovePreviewJobTest.php index 8c9df4274f0..69f730474e7 100644 --- a/tests/lib/Preview/MovePreviewJobTest.php +++ b/tests/lib/Preview/MovePreviewJobTest.php @@ -29,9 +29,7 @@ use PHPUnit\Framework\MockObject\MockObject; use Psr\Log\LoggerInterface; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class MovePreviewJobTest extends TestCase { private IAppData $previewAppData; private PreviewMapper $previewMapper; diff --git a/tests/lib/Preview/MovieBrokenStuckFfmpegTest.php b/tests/lib/Preview/MovieBrokenStuckFfmpegTest.php index e66d5e64649..7b995c24b25 100644 --- a/tests/lib/Preview/MovieBrokenStuckFfmpegTest.php +++ b/tests/lib/Preview/MovieBrokenStuckFfmpegTest.php @@ -11,10 +11,10 @@ namespace Test\Preview; /** * Class MovieTest * - * @group DB * * @package Test\Preview */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class MovieBrokenStuckFfmpegTest extends MovieTest { protected string $fileName = 'broken-video.webm'; } diff --git a/tests/lib/Preview/MovieTest.php b/tests/lib/Preview/MovieTest.php index b88f057cdd0..770e74a4ba9 100644 --- a/tests/lib/Preview/MovieTest.php +++ b/tests/lib/Preview/MovieTest.php @@ -15,10 +15,10 @@ use OCP\Server; /** * Class MovieTest * - * @group DB * * @package Test\Preview */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class MovieTest extends Provider { protected string $fileName = 'testimage.mp4'; protected int $width = 560; diff --git a/tests/lib/Preview/MovieTestRemoteFile.php b/tests/lib/Preview/MovieTestRemoteFile.php index 022c4e29cfb..5a635d34c59 100644 --- a/tests/lib/Preview/MovieTestRemoteFile.php +++ b/tests/lib/Preview/MovieTestRemoteFile.php @@ -18,11 +18,14 @@ use OCP\Server; /** * Class MovieTestRemoteFile * - * @group DB * * @package Test\Preview */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class MovieTestRemoteFile extends Provider { + public function __construct() { + parent::__construct(static::class); + } // 1080p (1920x1080) 30 FPS HEVC/H264, 10 secs, avg. bitrate: ~10 Mbps protected string $fileName = 'testvideo-remote-file.mp4'; protected int $width = 1920; diff --git a/tests/lib/Preview/OfficeTest.php b/tests/lib/Preview/OfficeTest.php index 62a4767fd2e..1095bdc2e17 100644 --- a/tests/lib/Preview/OfficeTest.php +++ b/tests/lib/Preview/OfficeTest.php @@ -15,10 +15,10 @@ use OCP\Server; /** * Class OfficeTest * - * @group DB * * @package Test\Preview */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class OfficeTest extends Provider { protected function setUp(): void { $binaryFinder = Server::get(IBinaryFinder::class); diff --git a/tests/lib/Preview/PreviewMapperTest.php b/tests/lib/Preview/PreviewMapperTest.php index 8e27a642473..0fc5420e235 100644 --- a/tests/lib/Preview/PreviewMapperTest.php +++ b/tests/lib/Preview/PreviewMapperTest.php @@ -16,9 +16,7 @@ use OCP\IDBConnection; use OCP\Server; use Test\TestCase; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class PreviewMapperTest extends TestCase { private PreviewMapper $previewMapper; private IDBConnection $connection; diff --git a/tests/lib/Preview/PreviewServiceTest.php b/tests/lib/Preview/PreviewServiceTest.php index f3f9c8ae895..09f6a891402 100644 --- a/tests/lib/Preview/PreviewServiceTest.php +++ b/tests/lib/Preview/PreviewServiceTest.php @@ -17,10 +17,8 @@ use OCP\Server; use PHPUnit\Framework\Attributes\CoversClass; use PHPUnit\Framework\TestCase; -/** - * @group DB - */ #[CoversClass(PreviewService::class)] +#[\PHPUnit\Framework\Attributes\Group('DB')] class PreviewServiceTest extends TestCase { private PreviewService $previewService; private PreviewMapper $previewMapper; diff --git a/tests/lib/Preview/Provider.php b/tests/lib/Preview/Provider.php index d8f10c430e4..22b64b58292 100644 --- a/tests/lib/Preview/Provider.php +++ b/tests/lib/Preview/Provider.php @@ -72,12 +72,12 @@ abstract class Provider extends \Test\TestCase { /** * Launches all the tests we have * - * @requires extension imagick * * @param int $widthAdjustment * @param int $heightAdjustment */ #[\PHPUnit\Framework\Attributes\DataProvider('dimensionsDataProvider')] + #[\PHPUnit\Framework\Attributes\RequiresPhpExtension('imagick')] public function testGetThumbnail($widthAdjustment, $heightAdjustment): void { $ratio = round($this->width / $this->height, 2); $this->maxWidth = $this->width - $widthAdjustment; diff --git a/tests/lib/Preview/SVGTest.php b/tests/lib/Preview/SVGTest.php index 8c6d9bb6691..2fad4720345 100644 --- a/tests/lib/Preview/SVGTest.php +++ b/tests/lib/Preview/SVGTest.php @@ -14,10 +14,10 @@ use OCP\Files\File; /** * Class SVGTest * - * @group DB * * @package Test\Preview */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class SVGTest extends Provider { protected function setUp(): void { $checkImagick = new \Imagick(); @@ -45,10 +45,8 @@ class SVGTest extends Provider { ]; } - /** - * @requires extension imagick - */ #[\PHPUnit\Framework\Attributes\DataProvider('dataGetThumbnailSVGHref')] + #[\PHPUnit\Framework\Attributes\RequiresPhpExtension('imagick')] public function testGetThumbnailSVGHref(string $content): void { $handle = fopen('php://temp', 'w+'); fwrite($handle, ' diff --git a/tests/lib/Preview/TXTTest.php b/tests/lib/Preview/TXTTest.php index d722495136c..c1c9952c68b 100644 --- a/tests/lib/Preview/TXTTest.php +++ b/tests/lib/Preview/TXTTest.php @@ -13,10 +13,10 @@ use OC\Preview\TXT; /** * Class TXTTest * - * @group DB * * @package Test\Preview */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class TXTTest extends Provider { protected function setUp(): void { parent::setUp(); diff --git a/tests/lib/Repair/CleanTagsTest.php b/tests/lib/Repair/CleanTagsTest.php index 04afabb87d9..80d16b9444f 100644 --- a/tests/lib/Repair/CleanTagsTest.php +++ b/tests/lib/Repair/CleanTagsTest.php @@ -19,10 +19,10 @@ use PHPUnit\Framework\MockObject\MockObject; /** * Tests for the cleaning the tags tables * - * @group DB * * @see \OC\Repair\CleanTags */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CleanTagsTest extends \Test\TestCase { private ?int $createdFile = null; diff --git a/tests/lib/Repair/OldGroupMembershipSharesTest.php b/tests/lib/Repair/OldGroupMembershipSharesTest.php index 099290b18ed..a385c1c9fc0 100644 --- a/tests/lib/Repair/OldGroupMembershipSharesTest.php +++ b/tests/lib/Repair/OldGroupMembershipSharesTest.php @@ -19,10 +19,10 @@ use PHPUnit\Framework\MockObject\MockObject; /** * Class OldGroupMembershipSharesTest * - * @group DB * * @package Test\Repair */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class OldGroupMembershipSharesTest extends \Test\TestCase { private IDBConnection $connection; diff --git a/tests/lib/Repair/Owncloud/UpdateLanguageCodesTest.php b/tests/lib/Repair/Owncloud/UpdateLanguageCodesTest.php index a3eb163b0d6..2c735b7bc6d 100644 --- a/tests/lib/Repair/Owncloud/UpdateLanguageCodesTest.php +++ b/tests/lib/Repair/Owncloud/UpdateLanguageCodesTest.php @@ -19,10 +19,10 @@ use Test\TestCase; /** * Class UpdateLanguageCodesTest * - * @group DB * * @package Test\Repair */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class UpdateLanguageCodesTest extends TestCase { protected IDBConnection $connection; diff --git a/tests/lib/Repair/RepairCollationTest.php b/tests/lib/Repair/RepairCollationTest.php index 3c51325562d..ea6e70b2164 100644 --- a/tests/lib/Repair/RepairCollationTest.php +++ b/tests/lib/Repair/RepairCollationTest.php @@ -31,10 +31,10 @@ class TestCollationRepair extends Collation { /** * Tests for the converting of MySQL tables to InnoDB engine * - * @group DB * * @see \OC\Repair\RepairMimeTypes */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class RepairCollationTest extends TestCase { private TestCollationRepair $repair; diff --git a/tests/lib/Repair/RepairInvalidSharesTest.php b/tests/lib/Repair/RepairInvalidSharesTest.php index 72103976da5..2171a34a6f0 100644 --- a/tests/lib/Repair/RepairInvalidSharesTest.php +++ b/tests/lib/Repair/RepairInvalidSharesTest.php @@ -20,10 +20,10 @@ use Test\TestCase; /** * Tests for repairing invalid shares * - * @group DB * * @see \OC\Repair\RepairInvalidShares */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class RepairInvalidSharesTest extends TestCase { private RepairInvalidShares $repair; diff --git a/tests/lib/Repair/RepairMimeTypesTest.php b/tests/lib/Repair/RepairMimeTypesTest.php index 0261b56ebe9..40272c18c72 100644 --- a/tests/lib/Repair/RepairMimeTypesTest.php +++ b/tests/lib/Repair/RepairMimeTypesTest.php @@ -20,10 +20,10 @@ use OCP\Server; /** * Tests for the converting of legacy storages to home storages. * - * @group DB * * @see \OC\Repair\RepairMimeTypes */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class RepairMimeTypesTest extends \Test\TestCase { private RepairMimeTypes $repair; diff --git a/tests/lib/Route/RouterTest.php b/tests/lib/Route/RouterTest.php index 560cd5d3aed..05ebc90806b 100644 --- a/tests/lib/Route/RouterTest.php +++ b/tests/lib/Route/RouterTest.php @@ -21,10 +21,10 @@ use Test\TestCase; /** * Class RouterTest * - * @group RoutingWeirdness * * @package Test\Route */ +#[\PHPUnit\Framework\Attributes\Group('RoutingWeirdness')] class RouterTest extends TestCase { private Router $router; private IAppManager&MockObject $appManager; diff --git a/tests/lib/Security/CertificateManagerTest.php b/tests/lib/Security/CertificateManagerTest.php index 4dadc824ef6..2ee77ab74fb 100644 --- a/tests/lib/Security/CertificateManagerTest.php +++ b/tests/lib/Security/CertificateManagerTest.php @@ -25,9 +25,8 @@ use Psr\Log\LoggerInterface; /** * Class CertificateManagerTest - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CertificateManagerTest extends \Test\TestCase { use \Test\Traits\UserTrait; use \Test\Traits\MountProviderTrait; diff --git a/tests/lib/Security/CredentialsManagerTest.php b/tests/lib/Security/CredentialsManagerTest.php index 4dfe8c5681d..e115a0c4321 100644 --- a/tests/lib/Security/CredentialsManagerTest.php +++ b/tests/lib/Security/CredentialsManagerTest.php @@ -13,9 +13,7 @@ namespace Test\Security; use OCP\Security\ICredentialsManager; use OCP\Server; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class CredentialsManagerTest extends \Test\TestCase { #[\PHPUnit\Framework\Attributes\DataProvider('credentialsProvider')] public function testWithDB($userId, $identifier): void { diff --git a/tests/lib/ServerTest.php b/tests/lib/ServerTest.php index b070641e856..d478bea089e 100644 --- a/tests/lib/ServerTest.php +++ b/tests/lib/ServerTest.php @@ -16,10 +16,10 @@ use OCP\Comments\ICommentsManager; /** * Class Server * - * @group DB * * @package Test */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ServerTest extends \Test\TestCase { /** @var Server */ protected $server; diff --git a/tests/lib/Share/HelperTest.php b/tests/lib/Share/HelperTest.php index 3438f108749..de3b407612b 100644 --- a/tests/lib/Share/HelperTest.php +++ b/tests/lib/Share/HelperTest.php @@ -10,10 +10,7 @@ namespace Test\Share; use OC\Share\Helper; -/** - * @group DB - * Class Helper - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class HelperTest extends \Test\TestCase { public static function expireDateProvider(): array { return [ diff --git a/tests/lib/Share/ShareTest.php b/tests/lib/Share/ShareTest.php index 9690d242067..ba86f3c3d3b 100644 --- a/tests/lib/Share/ShareTest.php +++ b/tests/lib/Share/ShareTest.php @@ -21,9 +21,8 @@ use OCP\Server; /** * Class Test_Share - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ShareTest extends \Test\TestCase { protected $itemType; diff --git a/tests/lib/Share20/DefaultShareProviderTest.php b/tests/lib/Share20/DefaultShareProviderTest.php index 6b4b0356061..d79dbbada3d 100644 --- a/tests/lib/Share20/DefaultShareProviderTest.php +++ b/tests/lib/Share20/DefaultShareProviderTest.php @@ -41,8 +41,8 @@ use Psr\Log\LoggerInterface; * Class DefaultShareProviderTest * * @package Test\Share20 - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class DefaultShareProviderTest extends \Test\TestCase { /** @var IDBConnection */ protected $dbConn; diff --git a/tests/lib/Share20/ManagerTest.php b/tests/lib/Share20/ManagerTest.php index aa42b685ffa..7c5e5a6e743 100644 --- a/tests/lib/Share20/ManagerTest.php +++ b/tests/lib/Share20/ManagerTest.php @@ -74,8 +74,8 @@ class DummyShareManagerListener { * Class ManagerTest * * @package Test\Share20 - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ManagerTest extends \Test\TestCase { /** @var Manager */ protected $manager; diff --git a/tests/lib/Share20/ShareByMailProviderTest.php b/tests/lib/Share20/ShareByMailProviderTest.php index 5180f49f227..c2fd0a77ee9 100644 --- a/tests/lib/Share20/ShareByMailProviderTest.php +++ b/tests/lib/Share20/ShareByMailProviderTest.php @@ -35,8 +35,8 @@ use Test\Traits\EmailValidatorTrait; * Class ShareByMailProviderTest * * @package Test\Share20 - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ShareByMailProviderTest extends TestCase { use EmailValidatorTrait; diff --git a/tests/lib/SubAdminTest.php b/tests/lib/SubAdminTest.php index 8e90ff2e1b2..4c0b1c33434 100644 --- a/tests/lib/SubAdminTest.php +++ b/tests/lib/SubAdminTest.php @@ -19,9 +19,7 @@ use OCP\IUser; use OCP\IUserManager; use OCP\Server; -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class SubAdminTest extends \Test\TestCase { /** @var IUserManager */ private $userManager; diff --git a/tests/lib/SystemTag/SystemTagManagerTest.php b/tests/lib/SystemTag/SystemTagManagerTest.php index 4d2741ba6c5..897d8e15d37 100644 --- a/tests/lib/SystemTag/SystemTagManagerTest.php +++ b/tests/lib/SystemTag/SystemTagManagerTest.php @@ -26,9 +26,9 @@ use Test\TestCase; /** * Class TestSystemTagManager * - * @group DB * @package Test\SystemTag */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class SystemTagManagerTest extends TestCase { private ISystemTagManager $tagManager; private IDBConnection $connection; diff --git a/tests/lib/SystemTag/SystemTagObjectMapperTest.php b/tests/lib/SystemTag/SystemTagObjectMapperTest.php index e459b81512d..5542d046d88 100644 --- a/tests/lib/SystemTag/SystemTagObjectMapperTest.php +++ b/tests/lib/SystemTag/SystemTagObjectMapperTest.php @@ -26,9 +26,9 @@ use Test\TestCase; /** * Class TestSystemTagObjectMapper * - * @group DB * @package Test\SystemTag */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class SystemTagObjectMapperTest extends TestCase { /** * @var ISystemTagManager diff --git a/tests/lib/TagsTest.php b/tests/lib/TagsTest.php index 4aeaeeed86e..9528fc29184 100644 --- a/tests/lib/TagsTest.php +++ b/tests/lib/TagsTest.php @@ -24,9 +24,8 @@ use Psr\Log\LoggerInterface; /** * Class TagsTest - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class TagsTest extends \Test\TestCase { protected $objectType; /** @var IUser */ @@ -269,9 +268,7 @@ class TagsTest extends \Test\TestCase { $this->assertEquals(9, count($tagger->getIdsForTag('Family'))); } - /** - * @depends testTagAs - */ + #[\PHPUnit\Framework\Attributes\Depends('testTagAs')] public function testUnTag(): void { $objIds = [1, 2, 3, 4, 5, 6, 7, 8, 9]; diff --git a/tests/lib/TaskProcessing/TaskProcessingTest.php b/tests/lib/TaskProcessing/TaskProcessingTest.php index 9e663102ace..395bd467b37 100644 --- a/tests/lib/TaskProcessing/TaskProcessingTest.php +++ b/tests/lib/TaskProcessing/TaskProcessingTest.php @@ -570,9 +570,7 @@ class ConflictingExternalTaskType implements ITaskType { } } -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class TaskProcessingTest extends \Test\TestCase { private IManager $manager; private Coordinator $coordinator; diff --git a/tests/lib/TestCase.php b/tests/lib/TestCase.php index e5c999ffc9d..54368c93b8b 100644 --- a/tests/lib/TestCase.php +++ b/tests/lib/TestCase.php @@ -10,7 +10,9 @@ namespace Test; use DOMDocument; use DOMNode; +use OC\App\AppStore\Fetcher\AppFetcher; use OC\Command\QueueBus; +use OC\Files\AppData\Factory; use OC\Files\Cache\Storage; use OC\Files\Config\MountProviderCollection; use OC\Files\Filesystem; @@ -20,7 +22,9 @@ use OC\Files\Mount\RootMountProvider; use OC\Files\ObjectStore\PrimaryObjectStoreConfig; use OC\Files\SetupManager; use OC\Files\View; +use OC\Installer; use OC\Template\Base; +use OC\Updater; use OCP\AppFramework\QueryException; use OCP\Command\IBus; use OCP\DB\QueryBuilder\IQueryBuilder; @@ -338,10 +342,10 @@ abstract class TestCase extends \PHPUnit\Framework\TestCase { self::tearDownAfterClassCleanStrayLocks(); // Ensure we start with fresh instances of some classes to reduce side-effects between tests - unset(\OC::$server[\OC\Files\AppData\Factory::class]); - unset(\OC::$server[\OC\App\AppStore\Fetcher\AppFetcher::class]); - unset(\OC::$server[\OC\Installer::class]); - unset(\OC::$server[\OC\Updater::class]); + unset(\OC::$server[Factory::class]); + unset(\OC::$server[AppFetcher::class]); + unset(\OC::$server[Installer::class]); + unset(\OC::$server[Updater::class]); /** @var SetupManager $setupManager */ $setupManager = Server::get(SetupManager::class); diff --git a/tests/lib/TextProcessing/TextProcessingTest.php b/tests/lib/TextProcessing/TextProcessingTest.php index bdcb4a57d7c..c25e5005467 100644 --- a/tests/lib/TextProcessing/TextProcessingTest.php +++ b/tests/lib/TextProcessing/TextProcessingTest.php @@ -88,9 +88,7 @@ class FreePromptProvider implements IProvider { } } -/** - * @group DB - */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class TextProcessingTest extends \Test\TestCase { private IManager $manager; private Coordinator $coordinator; diff --git a/tests/lib/User/DatabaseTest.php b/tests/lib/User/DatabaseTest.php index 33101173c0a..0afd54c345c 100644 --- a/tests/lib/User/DatabaseTest.php +++ b/tests/lib/User/DatabaseTest.php @@ -18,9 +18,8 @@ use PHPUnit\Framework\MockObject\MockObject; /** * Class DatabaseTest - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class DatabaseTest extends Backend { /** @var array */ private $users; diff --git a/tests/lib/User/Dummy.php b/tests/lib/User/Dummy.php index ec5be8ec60a..4b59bc5a4f5 100644 --- a/tests/lib/User/Dummy.php +++ b/tests/lib/User/Dummy.php @@ -9,6 +9,9 @@ namespace Test\User; class Dummy extends Backend { + public function __construct() { + parent::__construct(static::class); + } protected function setUp(): void { parent::setUp(); $this->backend = new \Test\Util\User\Dummy(); diff --git a/tests/lib/User/ManagerTest.php b/tests/lib/User/ManagerTest.php index d5872787d0a..7832fc38890 100644 --- a/tests/lib/User/ManagerTest.php +++ b/tests/lib/User/ManagerTest.php @@ -26,10 +26,10 @@ use Test\TestCase; /** * Class ManagerTest * - * @group DB * * @package Test\User */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class ManagerTest extends TestCase { /** @var IConfig */ private $config; @@ -694,10 +694,8 @@ class ManagerTest extends TestCase { $user4->delete(); } - /** - * @runInSeparateProcess - * @preserveGlobalState disabled - */ + #[\PHPUnit\Framework\Attributes\RunInSeparateProcess] + #[\PHPUnit\Framework\Attributes\PreserveGlobalState(enabled: false)] public function testRecentlyActive(): void { $config = Server::get(IConfig::class); $manager = Server::get(IUserManager::class); diff --git a/tests/lib/User/SessionTest.php b/tests/lib/User/SessionTest.php index 50c449559a0..e74dcaae6f2 100644 --- a/tests/lib/User/SessionTest.php +++ b/tests/lib/User/SessionTest.php @@ -42,9 +42,9 @@ use function array_diff; use function get_class_methods; /** - * @group DB * @package Test\User */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class SessionTest extends \Test\TestCase { /** @var ITimeFactory|MockObject */ private $timeFactory; diff --git a/tests/lib/User/UserTest.php b/tests/lib/User/UserTest.php index 05056c92193..c1e07a0f153 100644 --- a/tests/lib/User/UserTest.php +++ b/tests/lib/User/UserTest.php @@ -30,10 +30,10 @@ use Test\TestCase; /** * Class UserTest * - * @group DB * * @package Test\User */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class UserTest extends TestCase { /** @var IEventDispatcher|MockObject */ protected $dispatcher; diff --git a/tests/lib/UtilCheckServerTest.php b/tests/lib/UtilCheckServerTest.php index 087c67120a3..210085ff1dd 100644 --- a/tests/lib/UtilCheckServerTest.php +++ b/tests/lib/UtilCheckServerTest.php @@ -16,9 +16,8 @@ use OCP\Util; /** * Tests for server check functions - * - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class UtilCheckServerTest extends \Test\TestCase { private $datadir; diff --git a/tests/lib/UtilTest.php b/tests/lib/UtilTest.php index 6d995be2434..6738cafdb1f 100644 --- a/tests/lib/UtilTest.php +++ b/tests/lib/UtilTest.php @@ -20,8 +20,8 @@ use OCP\Util; * Class UtilTest * * @package Test - * @group DB */ +#[\PHPUnit\Framework\Attributes\Group('DB')] class UtilTest extends \Test\TestCase { public function testGetVersion(): void { $version = Util::getVersion();