Remove CoversClass and CoversTrait attributes

DamienHarper · Sep 6, 2024 · 14de0cb · 14de0cb
1 parent 941f60d
commit 14de0cb
Show file tree

Hide file tree

Showing 36 changed files with 1 addition and 1,377 deletions.
diff --git a/.php-cs-fixer.dist.php b/.php-cs-fixer.dist.php
@@ -39,6 +39,7 @@
         'numeric_literal_separator' => true,
         'string_implicit_backslashes' => true, // https://github.com/PHP-CS-Fixer/PHP-CS-Fixer/pull/7786
         'php_unit_test_case_static_method_calls' => false,
+        'php_unit_test_class_requires_covers' => false,
     ])
     ->setFinder(
         (new Finder())

diff --git a/tests/AuditorTest.php b/tests/AuditorTest.php
@@ -4,18 +4,14 @@
 
 namespace DH\Auditor\Tests;
 
-use DH\Auditor\Auditor;
 use DH\Auditor\Configuration;
-use DH\Auditor\EventSubscriber\AuditEventSubscriber;
 use DH\Auditor\Exception\InvalidArgumentException;
 use DH\Auditor\Exception\ProviderException;
-use DH\Auditor\Provider\AbstractProvider;
 use DH\Auditor\Tests\Fixtures\Provider\AuditNoStorageProvider;
 use DH\Auditor\Tests\Fixtures\Provider\NoStorageNoAuditProvider;
 use DH\Auditor\Tests\Fixtures\Provider\StorageAndAuditProvider;
 use DH\Auditor\Tests\Fixtures\Provider\StorageNoAuditProvider;
 use DH\Auditor\Tests\Traits\AuditorTrait;
-use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\Attributes\Depends;
 use PHPUnit\Framework\Attributes\Small;
 use PHPUnit\Framework\TestCase;
@@ -25,10 +21,6 @@
  * @internal
  */
 #[Small]
-#[CoversClass(Auditor::class)]
-#[CoversClass(Configuration::class)]
-#[CoversClass(AuditEventSubscriber::class)]
-#[CoversClass(AbstractProvider::class)]
 final class AuditorTest extends TestCase
 {
     use AuditorTrait;

diff --git a/tests/ConfigurationTest.php b/tests/ConfigurationTest.php
@@ -4,19 +4,14 @@
 
 namespace DH\Auditor\Tests;
 
-use DH\Auditor\Configuration;
 use DH\Auditor\Tests\Traits\AuditorConfigurationTrait;
-use PHPUnit\Framework\Attributes\CoversClass;
-use PHPUnit\Framework\Attributes\CoversTrait;
 use PHPUnit\Framework\Attributes\Small;
 use PHPUnit\Framework\TestCase;
 
 /**
  * @internal
  */
 #[Small]
-#[CoversClass(Configuration::class)]
-#[CoversTrait(AuditorConfigurationTrait::class)]
 final class ConfigurationTest extends TestCase
 {
     use AuditorConfigurationTrait;

diff --git a/tests/Event/LifecycleEventTest.php b/tests/Event/LifecycleEventTest.php
@@ -7,20 +7,14 @@
 use DH\Auditor\Event\LifecycleEvent;
 use DH\Auditor\EventSubscriber\AuditEventSubscriber;
 use DH\Auditor\Exception\InvalidArgumentException;
-use DH\Auditor\Provider\Doctrine\Persistence\Helper\DoctrineHelper;
-use DH\Auditor\Provider\Doctrine\Persistence\Helper\SchemaHelper;
 use DH\Auditor\Tests\Traits\AuditorTrait;
-use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\Attributes\Small;
 use PHPUnit\Framework\TestCase;
 
 /**
  * @internal
  */
 #[Small]
-#[CoversClass(LifecycleEvent::class)]
-#[CoversClass(DoctrineHelper::class)]
-#[CoversClass(SchemaHelper::class)]
 final class LifecycleEventTest extends TestCase
 {
     use AuditorTrait;

diff --git a/tests/EventSubscriber/AuditEventSubscriberTest.php b/tests/EventSubscriber/AuditEventSubscriberTest.php
@@ -4,29 +4,17 @@
 
 namespace DH\Auditor\Tests\EventSubscriber;
 
-use DH\Auditor\Auditor;
-use DH\Auditor\Configuration;
-use DH\Auditor\Event\AuditEvent;
 use DH\Auditor\Event\LifecycleEvent;
 use DH\Auditor\EventSubscriber\AuditEventSubscriber;
-use DH\Auditor\Provider\Doctrine\Persistence\Helper\DoctrineHelper;
-use DH\Auditor\Provider\Doctrine\Persistence\Helper\SchemaHelper;
 use DH\Auditor\Tests\Fixtures\EventSubscriber\CustomAuditEventSubscriber;
 use DH\Auditor\Tests\Traits\AuditorTrait;
-use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\Attributes\Small;
 use PHPUnit\Framework\TestCase;
 
 /**
  * @internal
  */
 #[Small]
-#[CoversClass(AuditEventSubscriber::class)]
-#[CoversClass(Auditor::class)]
-#[CoversClass(Configuration::class)]
-#[CoversClass(AuditEvent::class)]
-#[CoversClass(DoctrineHelper::class)]
-#[CoversClass(SchemaHelper::class)]
 final class AuditEventSubscriberTest extends TestCase
 {
     use AuditorTrait;

diff --git a/tests/Model/EntryTest.php b/tests/Model/EntryTest.php
@@ -5,15 +5,13 @@
 namespace DH\Auditor\Tests\Model;
 
 use DH\Auditor\Model\Entry;
-use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\Attributes\Small;
 use PHPUnit\Framework\TestCase;
 
 /**
  * @internal
  */
 #[Small]
-#[CoversClass(Entry::class)]
 final class EntryTest extends TestCase
 {
     public function testAccessors(): void

diff --git a/tests/Model/TransactionTest.php b/tests/Model/TransactionTest.php
@@ -5,15 +5,13 @@
 namespace DH\Auditor\Tests\Model;
 
 use DH\Auditor\Model\Transaction;
-use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\Attributes\Small;
 use PHPUnit\Framework\TestCase;
 
 /**
  * @internal
  */
 #[Small]
-#[CoversClass(Transaction::class)]
 final class TransactionTest extends TestCase
 {
     protected function setUp(): void {}

diff --git a/tests/Provider/Doctrine/Auditing/Annotation/AnnotationLoaderTest.php b/tests/Provider/Doctrine/Auditing/Annotation/AnnotationLoaderTest.php
@@ -5,28 +5,14 @@
 namespace DH\Auditor\Tests\Provider\Doctrine\Auditing\Annotation;
 
 use DH\Auditor\Provider\Doctrine\Auditing\Annotation\AnnotationLoader;
-use DH\Auditor\Provider\Doctrine\Auditing\Annotation\Auditable;
-use DH\Auditor\Provider\Doctrine\Auditing\Annotation\Security;
-use DH\Auditor\Provider\Doctrine\Auditing\DBAL\Middleware\AuditorConnection;
-use DH\Auditor\Provider\Doctrine\Auditing\DBAL\Middleware\AuditorDriver;
-use DH\Auditor\Provider\Doctrine\Auditing\DBAL\Middleware\AuditorMiddleware;
-use DH\Auditor\Provider\Doctrine\Persistence\Helper\DoctrineHelper;
 use DH\Auditor\Tests\Provider\Doctrine\Traits\EntityManagerInterfaceTrait;
-use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\Attributes\Small;
 use PHPUnit\Framework\TestCase;
 
 /**
  * @internal
  */
 #[Small]
-#[CoversClass(AnnotationLoader::class)]
-#[CoversClass(Auditable::class)]
-#[CoversClass(Security::class)]
-#[CoversClass(AuditorConnection::class)]
-#[CoversClass(AuditorDriver::class)]
-#[CoversClass(AuditorMiddleware::class)]
-#[CoversClass(DoctrineHelper::class)]
 final class AnnotationLoaderTest extends TestCase
 {
     use EntityManagerInterfaceTrait;

diff --git a/tests/Provider/Doctrine/Auditing/Transaction/TransactionProcessorTest.php b/tests/Provider/Doctrine/Auditing/Transaction/TransactionProcessorTest.php
@@ -4,87 +4,23 @@
 
 namespace DH\Auditor\Tests\Provider\Doctrine\Auditing\Transaction;
 
-use DH\Auditor\Auditor;
-use DH\Auditor\Configuration;
-use DH\Auditor\Event\AuditEvent;
-use DH\Auditor\Event\Dto\AbstractAssociationEventDto;
-use DH\Auditor\Event\Dto\AbstractEventDto;
-use DH\Auditor\Event\Dto\InsertEventDto;
-use DH\Auditor\Event\Dto\RemoveEventDto;
-use DH\Auditor\Event\Dto\UpdateEventDto;
-use DH\Auditor\EventSubscriber\AuditEventSubscriber;
-use DH\Auditor\Model\Entry;
-use DH\Auditor\Provider\AbstractProvider;
-use DH\Auditor\Provider\Doctrine\Auditing\Annotation\AnnotationLoader;
-use DH\Auditor\Provider\Doctrine\Auditing\DBAL\Middleware\AuditorConnection;
-use DH\Auditor\Provider\Doctrine\Auditing\DBAL\Middleware\AuditorDriver;
-use DH\Auditor\Provider\Doctrine\Auditing\DBAL\Middleware\AuditorMiddleware;
-use DH\Auditor\Provider\Doctrine\Auditing\Event\DoctrineSubscriber;
-use DH\Auditor\Provider\Doctrine\Auditing\Transaction\TransactionHydrator;
-use DH\Auditor\Provider\Doctrine\Auditing\Transaction\TransactionManager;
 use DH\Auditor\Provider\Doctrine\Auditing\Transaction\TransactionProcessor;
-use DH\Auditor\Provider\Doctrine\DoctrineProvider;
 use DH\Auditor\Provider\Doctrine\Model\Transaction;
-use DH\Auditor\Provider\Doctrine\Persistence\Event\CreateSchemaListener;
-use DH\Auditor\Provider\Doctrine\Persistence\Event\TableSchemaListener;
-use DH\Auditor\Provider\Doctrine\Persistence\Helper\DoctrineHelper;
-use DH\Auditor\Provider\Doctrine\Persistence\Helper\PlatformHelper;
-use DH\Auditor\Provider\Doctrine\Persistence\Helper\SchemaHelper;
-use DH\Auditor\Provider\Doctrine\Persistence\Reader\Query;
 use DH\Auditor\Provider\Doctrine\Persistence\Reader\Reader;
-use DH\Auditor\Provider\Doctrine\Persistence\Schema\SchemaManager;
-use DH\Auditor\Provider\Doctrine\Service\DoctrineService;
 use DH\Auditor\Provider\Doctrine\Service\StorageService;
-use DH\Auditor\Provider\Service\AbstractService;
 use DH\Auditor\Tests\Provider\Doctrine\Fixtures\Entity\Standard\Blog\Author;
 use DH\Auditor\Tests\Provider\Doctrine\Fixtures\Entity\Standard\Blog\Comment;
 use DH\Auditor\Tests\Provider\Doctrine\Fixtures\Entity\Standard\Blog\Post;
 use DH\Auditor\Tests\Provider\Doctrine\Fixtures\Entity\Standard\Blog\Tag;
 use DH\Auditor\Tests\Provider\Doctrine\Traits\Schema\DefaultSchemaSetupTrait;
 use DH\Auditor\Tests\Traits\ReflectionTrait;
-use DH\Auditor\User\User;
-use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\Attributes\Small;
 use PHPUnit\Framework\TestCase;
 
 /**
  * @internal
  */
 #[Small]
-#[CoversClass(TransactionProcessor::class)]
-#[CoversClass(Auditor::class)]
-#[CoversClass(Configuration::class)]
-#[CoversClass(AuditEventSubscriber::class)]
-#[CoversClass(AuditEvent::class)]
-#[CoversClass(Entry::class)]
-#[CoversClass(AbstractProvider::class)]
-#[CoversClass(AnnotationLoader::class)]
-#[CoversClass(DoctrineSubscriber::class)]
-#[CoversClass(AuditorConnection::class)]
-#[CoversClass(AuditorDriver::class)]
-#[CoversClass(AuditorMiddleware::class)]
-#[CoversClass(TransactionHydrator::class)]
-#[CoversClass(TransactionManager::class)]
-#[CoversClass(\DH\Auditor\Provider\Doctrine\Configuration::class)]
-#[CoversClass(DoctrineProvider::class)]
-#[CoversClass(CreateSchemaListener::class)]
-#[CoversClass(TableSchemaListener::class)]
-#[CoversClass(DoctrineHelper::class)]
-#[CoversClass(PlatformHelper::class)]
-#[CoversClass(SchemaHelper::class)]
-#[CoversClass(Query::class)]
-#[CoversClass(Reader::class)]
-#[CoversClass(SchemaManager::class)]
-#[CoversClass(DoctrineService::class)]
-#[CoversClass(AbstractService::class)]
-#[CoversClass(User::class)]
-#[CoversClass(AbstractAssociationEventDto::class)]
-#[CoversClass(AbstractEventDto::class)]
-#[CoversClass(\DH\Auditor\Model\Transaction::class)]
-#[CoversClass(Transaction::class)]
-#[CoversClass(InsertEventDto::class)]
-#[CoversClass(RemoveEventDto::class)]
-#[CoversClass(UpdateEventDto::class)]
 final class TransactionProcessorTest extends TestCase
 {
     use DefaultSchemaSetupTrait;

diff --git a/tests/Provider/Doctrine/ConfigurationTest.php b/tests/Provider/Doctrine/ConfigurationTest.php
@@ -5,21 +5,18 @@
 namespace DH\Auditor\Tests\Provider\Doctrine;
 
 use DH\Auditor\Provider\Doctrine\Auditing\Annotation\Security;
-use DH\Auditor\Provider\Doctrine\Configuration;
 use DH\Auditor\Tests\Provider\Doctrine\Fixtures\Entity\Annotation\AuditableButUnauditedEntity;
 use DH\Auditor\Tests\Provider\Doctrine\Fixtures\Entity\Annotation\AuditedEntity;
 use DH\Auditor\Tests\Provider\Doctrine\Fixtures\Entity\Standard\Blog\Comment;
 use DH\Auditor\Tests\Provider\Doctrine\Fixtures\Entity\Standard\Blog\Post;
 use DH\Auditor\Tests\Provider\Doctrine\Traits\ProviderConfigurationTrait;
-use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\Attributes\Small;
 use PHPUnit\Framework\TestCase;
 
 /**
  * @internal
  */
 #[Small]
-#[CoversClass(Configuration::class)]
 final class ConfigurationTest extends TestCase
 {
     use ProviderConfigurationTrait;

diff --git a/tests/Provider/Doctrine/DoctrineProviderTest.php b/tests/Provider/Doctrine/DoctrineProviderTest.php
@@ -4,30 +4,13 @@
 
 namespace DH\Auditor\Tests\Provider\Doctrine;
 
-use DH\Auditor\Auditor;
-use DH\Auditor\EventSubscriber\AuditEventSubscriber;
 use DH\Auditor\Exception\InvalidArgumentException;
 use DH\Auditor\Exception\ProviderException;
 use DH\Auditor\Provider\Doctrine\Auditing\Annotation\AnnotationLoader;
-use DH\Auditor\Provider\Doctrine\Auditing\Annotation\Auditable;
-use DH\Auditor\Provider\Doctrine\Auditing\Annotation\Security;
-use DH\Auditor\Provider\Doctrine\Auditing\DBAL\Middleware\AuditorConnection;
-use DH\Auditor\Provider\Doctrine\Auditing\DBAL\Middleware\AuditorDriver;
-use DH\Auditor\Provider\Doctrine\Auditing\DBAL\Middleware\AuditorMiddleware;
-use DH\Auditor\Provider\Doctrine\Auditing\Event\DoctrineSubscriber;
-use DH\Auditor\Provider\Doctrine\Auditing\Transaction\TransactionHydrator;
-use DH\Auditor\Provider\Doctrine\Auditing\Transaction\TransactionManager;
-use DH\Auditor\Provider\Doctrine\Auditing\Transaction\TransactionProcessor;
 use DH\Auditor\Provider\Doctrine\Configuration;
 use DH\Auditor\Provider\Doctrine\DoctrineProvider;
-use DH\Auditor\Provider\Doctrine\Persistence\Event\CreateSchemaListener;
-use DH\Auditor\Provider\Doctrine\Persistence\Event\TableSchemaListener;
-use DH\Auditor\Provider\Doctrine\Persistence\Helper\DoctrineHelper;
-use DH\Auditor\Provider\Doctrine\Persistence\Schema\SchemaManager;
 use DH\Auditor\Provider\Doctrine\Service\AuditingService;
-use DH\Auditor\Provider\Doctrine\Service\DoctrineService;
 use DH\Auditor\Provider\Doctrine\Service\StorageService;
-use DH\Auditor\Provider\Service\AbstractService;
 use DH\Auditor\Provider\Service\StorageServiceInterface;
 use DH\Auditor\Security\RoleCheckerInterface;
 use DH\Auditor\Security\SecurityProviderInterface;
@@ -42,7 +25,6 @@
 use DH\Auditor\User\User;
 use DH\Auditor\User\UserInterface;
 use DH\Auditor\User\UserProviderInterface;
-use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\Attributes\Depends;
 use PHPUnit\Framework\Attributes\Small;
 use PHPUnit\Framework\TestCase;
@@ -51,28 +33,6 @@
  * @internal
  */
 #[Small]
-#[CoversClass(DoctrineProvider::class)]
-#[CoversClass(Auditor::class)]
-#[CoversClass(\DH\Auditor\Configuration::class)]
-#[CoversClass(AuditEventSubscriber::class)]
-#[CoversClass(AnnotationLoader::class)]
-#[CoversClass(DoctrineSubscriber::class)]
-#[CoversClass(TransactionHydrator::class)]
-#[CoversClass(TransactionManager::class)]
-#[CoversClass(TransactionProcessor::class)]
-#[CoversClass(Configuration::class)]
-#[CoversClass(CreateSchemaListener::class)]
-#[CoversClass(TableSchemaListener::class)]
-#[CoversClass(DoctrineHelper::class)]
-#[CoversClass(SchemaManager::class)]
-#[CoversClass(DoctrineService::class)]
-#[CoversClass(AbstractService::class)]
-#[CoversClass(Auditable::class)]
-#[CoversClass(Security::class)]
-#[CoversClass(AuditorConnection::class)]
-#[CoversClass(AuditorDriver::class)]
-#[CoversClass(AuditorMiddleware::class)]
-#[CoversClass(User::class)]
 final class DoctrineProviderTest extends TestCase
 {
     use DoctrineProviderTrait;

diff --git a/tests/Provider/Doctrine/Event/DoctrineSubscriberTest.php b/tests/Provider/Doctrine/Event/DoctrineSubscriberTest.php
@@ -6,7 +6,6 @@
 
 use DH\Auditor\Provider\Doctrine\Auditing\DBAL\Middleware\AuditorDriver;
 use DH\Auditor\Provider\Doctrine\Auditing\Event\DoctrineSubscriber;
-use DH\Auditor\Provider\Doctrine\Model\Transaction;
 use DH\Auditor\Transaction\TransactionManagerInterface;
 use Doctrine\DBAL\Configuration;
 use Doctrine\DBAL\Connection as ConnectionDbal;
@@ -19,17 +18,13 @@
 use Doctrine\DBAL\VersionAwarePlatformDriver;
 use Doctrine\ORM\EntityManagerInterface;
 use Doctrine\ORM\Event\OnFlushEventArgs;
-use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\Attributes\Small;
 use PHPUnit\Framework\TestCase;
 
 /**
  * @internal
  */
 #[Small]
-#[CoversClass(DoctrineSubscriber::class)]
-#[CoversClass(Transaction::class)]
-#[CoversClass(AuditorDriver::class)]
 final class DoctrineSubscriberTest extends TestCase
 {
     public function testIssue184IfAbstractDriverMiddleware(): void