Merge pull request #526 from alexislefebvre/revert-add-compatibility-…

…for-symfony-4.3.0-512 [3.x] Revert add compatibility for symfony 4.3.0 512
liip · Jun 12, 2019 · 63ff8eb · 63ff8eb
2 parents fd3383b + cad5563
commit 63ff8eb
Show file tree

Hide file tree

Showing 9 changed files with 74 additions and 75 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -23,8 +23,6 @@ matrix:
       env: SYMFONY_VERSION="^4.1"
     - php: 7.3
       env: SYMFONY_VERSION="^4.1"
-    - php: 7.3
-      env: SYMFONY_VERSION="^4.3@beta"
   allow_failures:
     - php: 7.2
       env: SYMFONY_VERSION="dev-master"

diff --git a/composer.json b/composer.json
@@ -30,6 +30,9 @@
         "symfony/symfony": "^3.4 || ^4.1",
         "twig/twig": "^2.0"
     },
+    "conflict": {
+        "symfony/framework-bundle": "4.3.0"
+    },
     "suggest": {
         "brianium/paratest": "Required when using paratest to parallelize tests",
         "doctrine/dbal": "Required when using the fixture loading functionality with an ORM and SQLite",

diff --git a/src/DependencyInjection/Compiler/SetTestClientPass.php b/src/DependencyInjection/Compiler/SetTestClientPass.php
@@ -33,12 +33,6 @@ public function process(ContainerBuilder $container): void
             $definition = $container->getDefinition('test.client');
             $definition->setPublic(false);
             $container->setDefinition('liip_functional_test.query.count_client.parent', $definition);
-        } elseif ($container->hasAlias('test.client')) {
-            // Symfony <2.8
-            $container->setAlias(
-                'liip_functional_test.query.count_client.parent',
-                new Alias((string) $container->getAlias('test.client'), false)
-            );
         } else {
             throw new \Exception('The LiipFunctionalTestBundle\'s Query Counter can only be used in the test environment.'.PHP_EOL.'See https://github.com/liip/LiipFunctionalTestBundle#only-in-test-environment');
         }

diff --git a/src/QueryCountClient.php b/src/QueryCountClient.php
@@ -15,13 +15,19 @@
 
 use Symfony\Bundle\FrameworkBundle\Client;
 use Symfony\Bundle\FrameworkBundle\DependencyInjection\Compiler\CompilerDebugDumpPass;
+use Symfony\Bundle\FrameworkBundle\KernelBrowser;
 
 // Symfony <4 BC
 if (class_exists(CompilerDebugDumpPass::class)) {
     class_alias(QueryCountClientSymfony3Trait::class, QueryCountClientTrait::class);
 }
 
-class QueryCountClient extends Client
+// Symfony <4.3.1 BC
+if (!class_exists(KernelBrowser::class)) {
+    class_alias(Client::class, KernelBrowser::class);
+}
+
+class QueryCountClient extends KernelBrowser
 {
     /*
      * We use trait only because of Client::request signature strict type mismatch between Symfony 3 and 4.

diff --git a/src/Test/WebTestCase.php b/src/Test/WebTestCase.php
@@ -36,9 +36,6 @@
  */
 abstract class WebTestCase extends BaseWebTestCase
 {
-    /** @var Client|null */
-    protected static $client;
-
     protected $environment = 'test';
 
     protected $containers;
@@ -472,11 +469,6 @@ protected function createClientWithParams(array $params, ?string $username = nul
             $session->save();
         }
 
-        return static::$client = $client;
+        return $client;
     }
 }
-
-// Compatibility layer for Symfony 4.3+
-if (class_exists('Symfony\Bundle\FrameworkBundle\KernelBrowser')) {
-    class_alias('Symfony\Bundle\FrameworkBundle\KernelBrowser', 'Symfony\Bundle\FrameworkBundle\Client');
-}
diff --git a/tests/DependencyInjection/ConfigurationTest.php b/tests/DependencyInjection/ConfigurationTest.php
@@ -25,8 +25,8 @@ class ConfigurationTest extends WebTestCase
 
     public function setUp(): void
     {
-        static::$client = $this->makeClient();
-        $this->clientContainer = static::$client->getContainer();
+        $client = static::makeClient();
+        $this->clientContainer = $client->getContainer();
     }
 
     /**

diff --git a/tests/Test/WebTestCaseConfigLeanFrameworkTest.php b/tests/Test/WebTestCaseConfigLeanFrameworkTest.php
@@ -35,7 +35,7 @@ protected static function getKernelClass(): string
 
     public function testAssertStatusCode(): void
     {
-        $client = $this->makeClient();
+        $client = static::makeClient();
 
         $path = '/';
         $client->request('GET', $path);
@@ -45,7 +45,7 @@ public function testAssertStatusCode(): void
 
     public function testAssertValidationErrorsTriggersError(): void
     {
-        $client = $this->makeClient();
+        $client = static::makeClient();
 
         $path = '/form';
         $client->request('GET', $path);

diff --git a/tests/Test/WebTestCaseConfigTest.php b/tests/Test/WebTestCaseConfigTest.php
@@ -38,6 +38,9 @@ class WebTestCaseConfigTest extends WebTestCase
 {
     use LiipAcmeFixturesTrait;
 
+    /** @var \Symfony\Bundle\FrameworkBundle\Client client */
+    private $client = null;
+
     protected static function getKernelClass(): string
     {
         return AppConfigKernel::class;
@@ -48,13 +51,13 @@ protected static function getKernelClass(): string
      */
     public function testIndexClientWithCredentials(): void
     {
-        static::$client = static::makeClientWithCredentials('foobar', '12341234');
+        $this->client = static::makeClientWithCredentials('foobar', '12341234');
 
         $path = '/';
 
-        $crawler = static::$client->request('GET', $path);
+        $crawler = $this->client->request('GET', $path);
 
-        $this->assertStatusCode(200, static::$client);
+        $this->assertStatusCode(200, $this->client);
 
         $this->assertSame(1,
             $crawler->filter('html > body')->count());
@@ -77,13 +80,13 @@ public function testIndexClientWithCredentials(): void
      */
     public function testIndexAuthenticatedClient(): void
     {
-        static::$client = static::makeAuthenticatedClient();
+        $this->client = static::makeAuthenticatedClient();
 
         $path = '/';
 
-        $crawler = static::$client->request('GET', $path);
+        $crawler = $this->client->request('GET', $path);
 
-        $this->assertStatusCode(200, static::$client);
+        $this->assertStatusCode(200, $this->client);
 
         $this->assertSame(1,
             $crawler->filter('html > body')->count());
@@ -114,13 +117,13 @@ public function testIndexAuthenticationLoginAs(): void
             $loginAs
         );
 
-        static::$client = $this->makeClient();
+        $this->client = static::makeClient();
 
         $path = '/';
 
-        $crawler = static::$client->request('GET', $path);
+        $crawler = $this->client->request('GET', $path);
 
-        $this->assertStatusCode(200, static::$client);
+        $this->assertStatusCode(200, $this->client);
 
         $this->assertSame(1,
             $crawler->filter('html > body')->count());
@@ -152,12 +155,12 @@ public function testAllowedQueriesExceededException(): void
 
         $this->loginAs($user, 'secured_area');
 
-        static::$client = $this->makeClient();
+        $this->client = static::makeClient();
 
         // One another query to load the second user.
         $path = '/user/2';
 
-        static::$client->request('GET', $path);
+        $this->client->request('GET', $path);
     }
 
     /**
@@ -172,11 +175,11 @@ public function testAllowedQueriesExceededException(): void
      */
     public function testAnnotationAndException(): void
     {
-        static::$client = $this->makeClient();
+        $this->client = static::makeClient();
 
         // One query to load the second user
         $path = '/user/1';
 
-        static::$client->request('GET', $path);
+        $this->client->request('GET', $path);
     }
 }