Avoid name collision with parent method (#193)

Since symfony/symfony@4f91020 , that ships with 4.3.0, there is a
collision between a private static method in a trait use in an ancestor
class of BaseTestCase and the method in BaseTestCase.
See https://github.com/symfony/symfony/blob/950306adaad8b23725c4efeaf6c570d89e17f164/src/Symfony/Bundle/FrameworkBundle/Test/WebTestAssertionsTrait.php#L189
Closes #192

Author: greg0ire

phpunit.xml.dist

@@ -12,4 +12,8 @@
         </testsuite>
     </testsuites>
 
+    <listeners>
+       <listener class="Symfony\Bridge\PhpUnit\SymfonyTestsListener" />
+    </listeners>
+
 </phpunit>

src/Functional/BaseTestCase.php

@@ -54,12 +54,26 @@ protected function getKernelConfiguration()
         return [];
     }
 
+    public function __call($name, $arguments)
+    {
+        if ('getClient' === $name) {
+            @trigger_error(sprintf(
+                '"%s::getClient()" is deprecated in favor of getFrameworkBundleClient() since symfony-cmf/testing 2.1 and will no longer be callable in 3.0',
+                __CLASS__
+            ), E_USER_DEPRECATED);
+
+            return $this->getFrameworkBundleClient();
+        }
+
+        return parent::$name(...$arguments);
+    }
+
     /**
      * Gets the Client.
      *
      * @return Client
      */
-    public function getClient()
+    public function getFrameworkBundleClient()
     {
         if (null === $this->client) {
             $this->client = $this->createClient($this->getKernelConfiguration());

tests/Functional/BaseTestCaseTest.php

@@ -76,6 +76,20 @@ public function provideTestDb()
         ];
     }
 
+    /**
+     * @group legacy
+     * @expectedDeprecation "Symfony\Cmf\Component\Testing\Functional\BaseTestCase::getClient()" is deprecated in favor of getFrameworkBundleClient() since symfony-cmf/testing 2.1 and will no longer be callable in 3.0
+     */
+    public function testItTriggersADeprecationErrorWhenCallingGetClient()
+    {
+        $this->assertInstanceOf(Client::class, $this->testCase->getClient());
+    }
+
+    public function testItCanProvideAFrameworkBundleClient()
+    {
+        $this->assertInstanceOf(Client::class, $this->testCase->getFrameworkBundleClient());
+    }
+
     /**
      * @dataProvider provideTestDb
      * @depends testGetContainer