minor #7329 Delete an unused test which would not work anymore and replace it wit… (michaelKaefer)

This PR was merged into the 4.x branch.

Discussion
----------

Delete an unused test which would not work anymore and replace it wit…

…h a rewrite

Commits
-------

1fc8a70 Delete an unused test which would not work anymore and replace it with a rewrite

Author: javiereguiluz

phpunit.xml.dist

@@ -19,7 +19,6 @@
         <testsuite name="EasyAdmin Test Suite">
             <directory>tests/</directory>
             <exclude>tests/Controller/PrettyUrls/</exclude>
-            <exclude>tests/DataCollector/</exclude>
             <exclude>tests/EventListener/</exclude>
             <exclude>tests/Form/</exclude>
         </testsuite>

tests/DataCollector/EasyAdminDataCollectorTest.php

@@ -0,0 +1,107 @@
+<?php
+
+namespace EasyCorp\Bundle\EasyAdminBundle\Tests\Inspector;
+
+use Doctrine\ORM\EntityRepository;
+use EasyCorp\Bundle\EasyAdminBundle\Config\Action;
+use EasyCorp\Bundle\EasyAdminBundle\Config\Option\EA;
+use EasyCorp\Bundle\EasyAdminBundle\Inspector\DataCollector;
+use EasyCorp\Bundle\EasyAdminBundle\Test\AbstractCrudTestCase;
+use EasyCorp\Bundle\EasyAdminBundle\Tests\TestApplication\Controller\CategoryCrudController;
+use EasyCorp\Bundle\EasyAdminBundle\Tests\TestApplication\Controller\SecureDashboardController;
+use EasyCorp\Bundle\EasyAdminBundle\Tests\TestApplication\Entity\Category;
+use Symfony\Component\VarDumper\Cloner\Data;
+
+class DataCollectorTest extends AbstractCrudTestCase
+{
+    protected EntityRepository $categories;
+
+    protected function setUp(): void
+    {
+        parent::setUp();
+
+        $this->categories = $this->entityManager->getRepository(Category::class);
+
+        $this->client->enableProfiler();
+        $this->client->setServerParameters(['PHP_AUTH_USER' => 'admin', 'PHP_AUTH_PW' => '1234']);
+    }
+
+    protected function getControllerFqcn(): string
+    {
+        return CategoryCrudController::class;
+    }
+
+    protected function getDashboardFqcn(): string
+    {
+        return SecureDashboardController::class;
+    }
+
+    public function testIndex(): void
+    {
+        $sort = ['name' => 'DESC', 'slug' => 'ASC'];
+        $this->client->request('GET', $this->generateIndexUrl().'&'.http_build_query([EA::SORT => $sort]));
+
+        /** @var DataCollector $collector */
+        $collector = $this->client->getProfile()->getCollector('easyadmin');
+
+        $this->assertTrue($collector->isEasyAdminRequest());
+
+        $this->assertSame(
+            [
+                'CRUD Controller FQCN' => CategoryCrudController::class,
+                'CRUD Action' => Action::INDEX,
+                'Entity ID' => null,
+                'Sort' => $sort,
+            ],
+            array_map(static fn (Data $d) => $d->getValue(true), $collector->getData()),
+        );
+    }
+
+    public function testEdit(): void
+    {
+        /** @var Category $category */
+        $category = $this->categories->findOneBy([]);
+
+        $this->client->request('GET', $this->generateEditFormUrl($category->getId()));
+
+        /** @var DataCollector $collector */
+        $collector = $this->client->getProfile()->getCollector('easyadmin');
+
+        $this->assertTrue($collector->isEasyAdminRequest());
+
+        $this->assertSame(
+            [
+                'CRUD Controller FQCN' => CategoryCrudController::class,
+                'CRUD Action' => Action::EDIT,
+                'Entity ID' => (string) $category->getId(),
+                'Sort' => null,
+            ],
+            array_map(static fn (Data $d) => $d->getValue(true), $collector->getData()),
+        );
+    }
+
+    public function testReset(): void
+    {
+        $this->client->request('GET', $this->generateIndexUrl());
+
+        /** @var DataCollector $collector */
+        $collector = $this->client->getProfile()->getCollector('easyadmin');
+
+        $this->assertSame(
+            [
+                'CRUD Controller FQCN' => CategoryCrudController::class,
+                'CRUD Action' => Action::INDEX,
+                'Entity ID' => null,
+                'Sort' => null,
+            ],
+            array_map(static fn (Data $d) => $d->getValue(true), $collector->getData()),
+        );
+
+        $collector->reset();
+
+        $this->assertSame(
+            [],
+            array_map(static fn (Data $d) => $d->getValue(true), $collector->getData()),
+        );
+    }
+}

tests/Inspector/DataCollectorTest.php

@@ -22,6 +22,10 @@
         'default_uuid_version' => 7,
         'time_based_uuid_version' => 7,
     ],
+    'profiler' => [
+        'enabled' => true,
+        'collect' => false,
+    ],
 ];
 
 if (EasyCorp\Bundle\EasyAdminBundle\Tests\TestApplication\Kernel::VERSION_ID < 60000) {