Reworked unconditionally skipped tests

tests/Functional/LoggingTest.php

@@ -23,22 +23,6 @@ class LoggingTest extends FunctionalTestCase
         $this->connection->executeQuery($sql, []);
     }
 
-    public function testLogExecuteUpdate() : void
-    {
-        $this->markTestSkipped('Test breaks MySQL but works on all other platforms (Unbuffered Queries stuff).');
-
-        $sql = $this->connection->getDatabasePlatform()->getDummySelectSQL();
-
-        $logMock = $this->createMock(SQLLogger::class);
-        $logMock->expects($this->at(0))
-                ->method('startQuery')
-                ->with($this->equalTo($sql), $this->equalTo([]), $this->equalTo([]));
-        $logMock->expects($this->at(1))
-                ->method('stopQuery');
-        $this->connection->getConfiguration()->setSQLLogger($logMock);
-        $this->connection->executeUpdate($sql, []);
-    }
-
     public function testLogPrepareExecute() : void
     {
         $sql = $this->connection->getDatabasePlatform()->getDummySelectSQL();

tests/Functional/Schema/PostgreSqlSchemaManagerTest.php

@@ -5,7 +5,6 @@ declare(strict_types=1);
 namespace Doctrine\DBAL\Tests\Functional\Schema;
 
 use Doctrine\DBAL\Platforms\AbstractPlatform;
-use Doctrine\DBAL\Platforms\PostgreSQL94Platform;
 use Doctrine\DBAL\Schema;
 use Doctrine\DBAL\Schema\Comparator;
 use Doctrine\DBAL\Schema\ForeignKeyConstraint;
@@ -417,12 +416,6 @@ class PostgreSqlSchemaManagerTest extends SchemaManagerFunctionalTestCase
 
     public function testJsonbColumn() : void
     {
-        if (! $this->schemaManager->getDatabasePlatform() instanceof PostgreSQL94Platform) {
-            $this->markTestSkipped('Requires PostgresSQL 9.4+');
-
-            return;
-        }
-
         $table = new Schema\Table('test_jsonb');
         $table->addColumn('foo', Types::JSON)->setPlatformOption('jsonb', true);
         $this->schemaManager->dropAndCreateTable($table);

tests/Schema/ComparatorTest.php

@@ -816,8 +816,6 @@ class ComparatorTest extends TestCase
 
     public function testDetectChangeIdentifierType() : void
     {
-        $this->markTestSkipped('DBAL-2 was reopened, this test cannot work anymore.');
-
         $tableA = new Table('foo');
         $tableA->addColumn('id', 'integer', ['autoincrement' => false]);