- Refactor test infrastructure: replace ApiTestCase implementations with WebTestCase and refactor traits for shared logic.

- Update PHPUnit assertions to use stricter methods for status code verification.
- Adjust composer constraints for improved compatibility with future Symfony and Doctrine releases.
- Simplify and enhance XML and JSON test helpers for better reusability across controllers.

Author: tomkalon

.github/workflows/build.yml

@@ -23,7 +23,7 @@ jobs:
             fail-fast: false
             matrix:
                 orm: ['2.*', '3.*']
-                php: ["8.1", "8.2", "8.3"]
+                php: ["8.1", "8.2", "8.3", "8.4"]
                 composer-flags: ['--no-scripts --prefer-stable --prefer-dist']
                 symfony: ["^6.4", "^7.1"]
                 app_env: ["test"]
@@ -90,13 +90,6 @@ jobs:
                     composer analyse
                     (cd src/Component && composer validate --strict)
 
-            -
-                name: Run Phpspec tests
-                run: |
-                    vendor/bin/phpspec run --ansi --no-interaction
-                    (cd src/Component && vendor/bin/phpspec run --no-interaction)
-                    (cd src/Component && vendor/bin/phpspec run --no-interaction --config legacy/phpspec.yml.dist)
-
             -
                 name: Run PHPUnit tests
                 run: vendor/bin/phpunit --colors=always

composer.json

@@ -32,7 +32,7 @@
         "doctrine/collections": "^2.2",
         "doctrine/event-manager": "^1.1 || ^2.0",
         "doctrine/inflector": "^2.0",
-        "doctrine/persistence": "^3.3",
+        "doctrine/persistence": "^3.3 || ^4.0",
         "gedmo/doctrine-extensions": "^3.17.1",
         "sylius/registry": "^1.2",
         "symfony/config": "^6.4 || ^7.1",
@@ -56,14 +56,13 @@
         "sylius/resource": "self.version"
     },
     "require-dev": {
-        "doctrine/doctrine-bundle": "^2.13",
+        "doctrine/doctrine-bundle": "^2.13 || ^3.0",
         "doctrine/orm": "^2.18 || ^3.3",
         "doctrine/phpcr-odm": "^2.1",
         "friendsofsymfony/rest-bundle": "^3.7",
         "jackalope/jackalope": "^2.0",
         "jackalope/jackalope-doctrine-dbal": "^2.0",
         "jms/serializer-bundle": "^3.5 || ^4.0 || ^5.0",
-        "lchrusciel/api-test-case": "^5.0",
         "matthiasnoback/symfony-dependency-injection-test": "^6.1.0",
         "pagerfanta/pagerfanta": "^4.4",
         "pamil/phpspec-skip-example-extension": "^4.2",
@@ -77,9 +76,11 @@
         "rector/rector": "^0.18.2",
         "sylius-labs/coding-standard": "^4.4",
         "sylius/grid-bundle": "^1.13",
+        "symfony/browser-kit": "^6.4 || ^7.1",
         "symfony/console": "^6.4 || ^7.1",
         "symfony/css-selector": "^6.4 || ^7.1",
         "symfony/dependency-injection": "^6.4 || ^7.1",
+        "symfony/dom-crawler": "^6.4 || ^7.1",
         "symfony/dotenv": "^6.4 || ^7.1",
         "symfony/http-kernel": "^6.4 || ^7.1",
         "symfony/messenger": "^6.4 || ^7.1",
@@ -96,7 +97,6 @@
     },
     "conflict": {
         "doctrine/orm": "<2.18",
-        "doctrine/doctrine-bundle": "<2.0 || ^3.0",
         "friendsofsymfony/rest-bundle": "<3.0",
         "jms/serializer-bundle": "<3.5",
         "pagerfanta/pagerfanta" : "<4.4",

src/Component/tests/Doctrine/Common/State/PersistProcessorTest.php

@@ -64,9 +64,17 @@ public function testItOnlyFlushesWhenManagerContainsTheResource(): void
         $operation = $this->prophesize(Operation::class);
         $data = new \stdClass();
 
-        $this->managerRegistry->getManagerForClass(\stdClass::class)->willReturn($manager);
+        $metadata = $this->prophesize(ClassMetadata::class);
+
+        $metadata->isChangeTrackingDeferredExplicit()->willReturn(false);
+
+        $this->managerRegistry
+            ->getManagerForClass(\stdClass::class)
+            ->willReturn($manager->reveal())
+        ;
+
         $manager->contains($data)->willReturn(true);
-        $manager->getClassMetadata(\stdClass::class)->willReturn($data);
+        $manager->getClassMetadata(\stdClass::class)->willReturn($metadata->reveal());
 
         $manager->persist($data)->shouldNotBeCalled();
         $manager->flush()->shouldBeCalled();

tests/Application/config/bundles.php

@@ -14,10 +14,8 @@
 use BabDev\PagerfantaBundle\BabDevPagerfantaBundle;
 use Bazinga\Bundle\HateoasBundle\BazingaHateoasBundle;
 use Doctrine\Bundle\DoctrineBundle\DoctrineBundle;
-use Fidry\AliceDataFixtures\Bridge\Symfony\FidryAliceDataFixturesBundle;
 use FOS\RestBundle\FOSRestBundle;
 use JMS\SerializerBundle\JMSSerializerBundle;
-use Nelmio\Alice\Bridge\Symfony\NelmioAliceBundle;
 use Sylius\Bundle\GridBundle\SyliusGridBundle;
 use Sylius\Bundle\ResourceBundle\SyliusResourceBundle;
 use Symfony\Bundle\FrameworkBundle\FrameworkBundle;
@@ -35,8 +33,6 @@
     FOSRestBundle::class => ['all' => true, 'test_without_fosrest' => false],
     JMSSerializerBundle::class => ['all' => true, 'test_without_fosrest' => false],
     BazingaHateoasBundle::class => ['all' => true, 'test_without_hateoas' => false, 'test_without_fosrest' => false, 'test_with_attributes' => false],
-    FidryAliceDataFixturesBundle::class => ['all' => true],
-    NelmioAliceBundle::class => ['all' => true],
     winzouStateMachineBundle::class => ['all' => true, 'test_without_state_machine' => false],
     SyliusGridBundle::class => ['all' => true, 'test_without_twig' => false],
     Zenstruck\Foundry\ZenstruckFoundryBundle::class => ['dev' => true, 'test' => true],

tests/Application/config/packages/doctrine.yaml

@@ -4,7 +4,6 @@ doctrine:
         path: "%database_path%"
         charset: UTF8
     orm:
-        auto_generate_proxy_classes: true
         naming_strategy: doctrine.orm.naming_strategy.underscore_number_aware
         auto_mapping: true
         mappings:

tests/Application/src/Tests/Controller/BlogPostApiTest.php

@@ -13,16 +13,27 @@
 
 namespace App\Tests\Controller;
 
-use ApiTestCase\JsonApiTestCase;
 use App\Foundry\Factory\BlogPostFactory;
+use App\Tests\Trait\JsonApiTestTrait;
 use PHPUnit\Framework\Attributes\Test;
-use Sylius\Bundle\ResourceBundle\ResourceBundleInterface;
+use Symfony\Bundle\FrameworkBundle\KernelBrowser;
+use Symfony\Bundle\FrameworkBundle\Test\WebTestCase;
 use Symfony\Component\HttpFoundation\Response;
 use Zenstruck\Foundry\Test\Factories;
+use Zenstruck\Foundry\Test\ResetDatabase;
 
-final class BlogPostApiTest extends JsonApiTestCase
+final class BlogPostApiTest extends WebTestCase
 {
     use Factories;
+    use ResetDatabase;
+    use JsonApiTestTrait;
+
+    private KernelBrowser $client;
+
+    protected function setUp(): void
+    {
+        $this->client = static::createClient();
+    }
 
     #[Test]
     public function it_allows_creating_a_blog_post(): void
@@ -31,6 +42,7 @@ public function it_allows_creating_a_blog_post(): void
 
         $this->client->request('POST', '/blog-posts/', [], [], ['CONTENT_TYPE' => 'application/json'], '{}');
         $response = $this->client->getResponse();
+
         $this->assertResponse($response, 'blog-posts/create_response', Response::HTTP_CREATED);
     }
 
@@ -46,6 +58,7 @@ public function it_allows_reviewing_a_blog_post(): void
 
         $this->client->request('PUT', '/blog-posts/' . $blogPost->getId() . '/to_review', [], [], ['CONTENT_TYPE' => 'application/json'], '{}');
         $response = $this->client->getResponse();
+
         $this->assertResponse($response, 'blog-posts/to_review_response', Response::HTTP_OK);
     }
 
@@ -61,6 +74,7 @@ public function it_allows_publishing_a_blog_post(): void
 
         $this->client->request('PUT', '/blog-posts/' . $blogPost->getId() . '/publish', [], [], ['CONTENT_TYPE' => 'application/json'], '{}');
         $response = $this->client->getResponse();
+
         $this->assertResponse($response, 'blog-posts/publish_response', Response::HTTP_OK);
     }
 
@@ -76,6 +90,7 @@ public function it_allows_rejecting_a_blog_post(): void
 
         $this->client->request('PUT', '/blog-posts/' . $blogPost->getId() . '/reject', [], [], ['CONTENT_TYPE' => 'application/json'], '{}');
         $response = $this->client->getResponse();
+
         $this->assertResponse($response, 'blog-posts/reject_response', Response::HTTP_OK);
     }
 
@@ -91,7 +106,8 @@ public function it_does_not_allow_to_publish_a_blog_post_with_draft_status(): vo
 
         $this->client->request('PUT', '/blog-posts/' . $blogPost->getId() . '/publish', [], [], ['CONTENT_TYPE' => 'application/json'], '{}');
         $response = $this->client->getResponse();
-        $this->assertResponseCode($response, Response::HTTP_BAD_REQUEST);
+
+        $this->assertSame(Response::HTTP_BAD_REQUEST, $response->getStatusCode());
     }
 
     #[Test]
@@ -106,17 +122,7 @@ public function it_does_not_allow_to_reject_a_blog_post_with_draft_status(): voi
 
         $this->client->request('PUT', '/blog-posts/' . $blogPost->getId() . '/reject', [], [], ['CONTENT_TYPE' => 'application/json'], '{}');
         $response = $this->client->getResponse();
-        $this->assertResponseCode($response, Response::HTTP_BAD_REQUEST);
-    }
-
-    private function markAsSkippedIfNecessary(): void
-    {
-        $container = self::getContainer();
-
-        $stateMachine = $container->getParameter('sylius.resource.settings')['state_machine_component'];
 
-        if (ResourceBundleInterface::STATE_MACHINE_SYMFONY !== $stateMachine) {
-            $this->markTestSkipped();
-        }
+        $this->assertSame(Response::HTTP_BAD_REQUEST, $response->getStatusCode());
     }
 }