#714 - Cleaning up test case (dedicated private method for skipped cases)

tests/Doctrine/Tests/DBAL/Driver/PDOPgSql/DriverTest.php

@@ -15,17 +15,9 @@ class DriverTest extends AbstractPostgreSQLDriverTest
 
     public function testConnectionDisablesPreparesOnPhp56()
     {
-        if (PHP_VERSION_ID < 50600) {
-            $this->markTestSkipped('Test requires PHP 5.6+');
-        }
-
-        if ($GLOBALS['db_type'] !== 'pdo_pgsql') {
-            $this->markTestSkipped('Test enabled only when using pdo_pgsql specific phpunit.xml');
-        }
-
-        $driver = $this->createDriver();
+        $this->skipWhenNotUsingPhp56AndPdoPgsql();
 
-        $connection = $driver->connect(
+        $connection = $this->createDriver()->connect(
             array(
                 'host'   => $GLOBALS['db_host'],
                 'dbname' => $GLOBALS['db_name'],
@@ -46,4 +38,18 @@ class DriverTest extends AbstractPostgreSQLDriverTest
     {
         return new Driver();
     }
+
+    /**
+     * @throws \PHPUnit_Framework_SkippedTestError
+     */
+    private function skipWhenNotUsingPhp56AndPdoPgsql()
+    {
+        if (PHP_VERSION_ID < 50600) {
+            $this->markTestSkipped('Test requires PHP 5.6+');
+        }
+
+        if ($GLOBALS['db_type'] !== 'pdo_pgsql') {
+            $this->markTestSkipped('Test enabled only when using pdo_pgsql specific phpunit.xml');
+        }
+    }
 }