StatementTest::testFetchFromExecutedStatementWithClosedCursor() - Code Metrics - Inspection of "Statement objects should throw exception in except..." - doctrine/dbal - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions
Pull Request — develop (#3154)

by Sergei
created 2018-07-25 05:04 UTC
testFetchFromExecutedStatementWithClosedCursor() A

↳ Parent: StatementTest
Complexity

Conditions	1
Paths	1
Size

Total Lines	10
Code Lines	6
Duplication

Lines	0
Ratio	0 %
Importance

Changes
Metric	Value
eloc	6
dl	0
loc	10
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	1
<?php

namespace Doctrine\Tests\DBAL\Functional;

use Doctrine\DBAL\Driver\DriverException;
use Doctrine\DBAL\Driver\Statement;
use Doctrine\DBAL\FetchMode;
use Doctrine\DBAL\ParameterType;
use Doctrine\DBAL\Schema\Table;
use Doctrine\DBAL\Types\Type;
use function base64_decode;
use function stream_get_contents;

class StatementTest extends \Doctrine\Tests\DbalFunctionalTestCase
{
    protected function setUp()
    {
        parent::setUp();

        $table = new Table('stmt_test');
        $table->addColumn('id', 'integer');
        $table->addColumn('name', 'text', array('notnull' => false));
        $this->_conn->getSchemaManager()->dropAndCreateTable($table);
    }

    public function testStatementIsReusableAfterClosingCursor()
    {
        $this->_conn->insert('stmt_test', array('id' => 1));
        $this->_conn->insert('stmt_test', array('id' => 2));

        $stmt = $this->_conn->prepare('SELECT id FROM stmt_test ORDER BY id');

        $stmt->execute();

        $id = $stmt->fetchColumn();
        self::assertEquals(1, $id);

        $stmt->closeCursor();

        $stmt->execute();
        $id = $stmt->fetchColumn();
        self::assertEquals(1, $id);
        $id = $stmt->fetchColumn();
        self::assertEquals(2, $id);
    }

    public function testReuseStatementWithLongerResults()
    {
        $sm = $this->_conn->getSchemaManager();
        $table = new Table('stmt_longer_results');
        $table->addColumn('param', 'string');
        $table->addColumn('val', 'text');
        $sm->createTable($table);

        $row1 = array(
            'param' => 'param1',
            'val' => 'X',
        );
        $this->_conn->insert('stmt_longer_results', $row1);

        $stmt = $this->_conn->prepare('SELECT param, val FROM stmt_longer_results ORDER BY param');
        $stmt->execute();
        self::assertArraySubset(array(
            array('param1', 'X'),
        ), $stmt->fetchAll(FetchMode::NUMERIC));

        $row2 = array(
            'param' => 'param2',
            'val' => 'A bit longer value',
        );
        $this->_conn->insert('stmt_longer_results', $row2);

        $stmt->execute();
        self::assertArraySubset(array(
            array('param1', 'X'),
            array('param2', 'A bit longer value'),
        ), $stmt->fetchAll(FetchMode::NUMERIC));
    }

    public function testFetchLongBlob()
    {
        // make sure memory limit is large enough to not cause false positives,
        // but is still not enough to store a LONGBLOB of the max possible size
        $this->iniSet('memory_limit', '4G');

        $sm = $this->_conn->getSchemaManager();
        $table = new Table('stmt_long_blob');
        $table->addColumn('contents', 'blob', array(
            'length' => 0xFFFFFFFF,
        ));
        $sm->createTable($table);

        $contents = base64_decode(<<<EOF
H4sICJRACVgCA2RvY3RyaW5lLmljbwDtVNtLFHEU/ia1i9fVzVWxvJSrZmoXS6pd0zK7QhdNc03z
lrpppq1pWqJCFERZkUFEDybYBQqJhB6iUOqhh+whgl4qkF6MfGh+s87O7GVmO6OlBfUfdIZvznxn
fpzznW9gAI4unQ50XwirH2AAkEygEuIwU58ODnPBzXGv14sEq4BrwzKKL4sY++SGTz6PodcutN5x
IPvsFCa+K9CXMfS/cOL5OxesN0Wceygho0WAXVLwcUJBdDVDaqOAij4Rrz640XlXQmAxQ16PHU63
iqdvXbg4JOHLpILBUSdM7XZEVDDcfuZEbI2ASaYguUGAroSh97GMngcSeFFFerMdI+/dyGy1o+GW
Ax5FxfAbFwoviajuc+DCIwn+RTwGRmRIThXxdQJyu+z4/NUDYz2DKCsILuERWsoQfoQhqpLhyhMZ
XfcknBmU0NLvQArpTm0SsI5mqKqKuFoGc8cUcjrtqLohom1AgtujQnapmJJU+BbwCLIwhJXyiKlh
MB4TkFgvIK3JjrRmAefJm+77Eiqvi+SvCq/qJahQyWuVuEpcIa7QLh7Kbsourb9b66/pZdAd1voz
fCNfwsp46OnZQPojSX9UFcNy+mYJNDeJPHtJfqeR/nSaPTzmwlXar5dQ1adpd+B//I9/hi0xuCPQ
Nkvb5um37Wtc+auQXZsVxEVYD5hnCilxTaYYjsuxLlsxXUitzd2hs3GWHLM5UOM7Fy8t3xiat4fb
sneNxmNb/POO1pRXc7vnF2nc13Rq0cFWiyXkuHmzxuOtzUYfC7fEmK/3mx4QZd5u4E7XJWz6+dey
Za4tXHUiPyB8Vm781oaT+3fN6Y/eUFDfPkcNWetNxb+tlxEZsPqPdZMOzS4rxwJ8CDC+ABj1+Tu0
d+N0hqezcjblboJ3Bj8ARJilHX4FAAA=
EOF
    );

        $this->_conn->insert('stmt_long_blob', ['contents' => $contents], [ParameterType::LARGE_OBJECT]);

        $stmt = $this->_conn->prepare('SELECT contents FROM stmt_long_blob');
        $stmt->execute();

        $stream = Type::getType('blob')
            ->convertToPHPValue(
                $stmt->fetchColumn(),
                $this->_conn->getDatabasePlatform()
            );

        self::assertSame($contents, stream_get_contents($stream));
        self::assertSame($contents, stream_get_contents(/** @scrutinizer ignore-type */ $stream));
    }

    public function testIncompletelyFetchedStatementDoesNotBlockConnection()
    {
        $this->_conn->insert('stmt_test', array('id' => 1));
        $this->_conn->insert('stmt_test', array('id' => 2));

        $stmt1 = $this->_conn->prepare('SELECT id FROM stmt_test');
        $stmt1->execute();
        $stmt1->fetch();
        $stmt1->execute();
        // fetching only one record out of two
        $stmt1->fetch();

        $stmt2 = $this->_conn->prepare('SELECT id FROM stmt_test WHERE id = ?');
        $stmt2->execute(array(1));
        self::assertEquals(1, $stmt2->fetchColumn());
    }

    public function testReuseStatementAfterClosingCursor()
    {
        $this->_conn->insert('stmt_test', array('id' => 1));
        $this->_conn->insert('stmt_test', array('id' => 2));

        $stmt = $this->_conn->prepare('SELECT id FROM stmt_test WHERE id = ?');

        $stmt->execute(array(1));
        $id = $stmt->fetchColumn();
        self::assertEquals(1, $id);

        $stmt->closeCursor();

        $stmt->execute(array(2));
        $id = $stmt->fetchColumn();
        self::assertEquals(2, $id);
    }

    public function testReuseStatementWithParameterBoundByReference()
    {
        $this->_conn->insert('stmt_test', array('id' => 1));
        $this->_conn->insert('stmt_test', array('id' => 2));

        $stmt = $this->_conn->prepare('SELECT id FROM stmt_test WHERE id = ?');
        $stmt->bindParam(1, $id);

        $id = 1;

        $stmt->execute();
        self::assertEquals(1, $stmt->fetchColumn());

        $id = 2;
        $stmt->execute();
        self::assertEquals(2, $stmt->fetchColumn());
    }

    public function testReuseStatementWithReboundValue()
    {
        $this->_conn->insert('stmt_test', array('id' => 1));
        $this->_conn->insert('stmt_test', array('id' => 2));

        $stmt = $this->_conn->prepare('SELECT id FROM stmt_test WHERE id = ?');

        $stmt->bindValue(1, 1);
        $stmt->execute();
        self::assertEquals(1, $stmt->fetchColumn());

        $stmt->bindValue(1, 2);
        $stmt->execute();
        self::assertEquals(2, $stmt->fetchColumn());
    }

    public function testReuseStatementWithReboundParam()
    {
        $this->_conn->insert('stmt_test', array('id' => 1));
        $this->_conn->insert('stmt_test', array('id' => 2));

        $stmt = $this->_conn->prepare('SELECT id FROM stmt_test WHERE id = ?');

        $x = 1;
        $stmt->bindParam(1, $x);
        $stmt->execute();
        self::assertEquals(1, $stmt->fetchColumn());

        $y = 2;
        $stmt->bindParam(1, $y);
        $stmt->execute();
        self::assertEquals(2, $stmt->fetchColumn());
    }

    /**
     * @dataProvider fetchProvider
     */
    public function testFetchFromNonExecutedStatement(callable $fetch)
    {
        $stmt = $this->_conn->prepare('SELECT id FROM stmt_test');

        self::expectException(DriverException::class);
              expectException(DriverException::class);
        $fetch($stmt);
    }

    /**
     * @dataProvider fetchProvider
     */
    public function testCloseCursorOnNonExecutedStatement(callable $fetch)
    {
        $stmt = $this->_conn->prepare('SELECT id FROM stmt_test');
        $stmt->closeCursor();

        self::expectException(DriverException::class);
              expectException(DriverException::class);
        $fetch($stmt);
    }

    /**
     * @group DBAL-2637
     */
    public function testCloseCursorAfterCursorEnd()
    {
        $stmt = $this->_conn->prepare('SELECT name FROM stmt_test');

        $stmt->execute();
        self::assertFalse($stmt->fetch());

        $stmt->closeCursor();

        self::expectException(DriverException::class);
              expectException(DriverException::class);
        $stmt->fetch();
    }

    /**
     * @dataProvider fetchProvider
     */
    public function testFetchFromNonExecutedStatementWithClosedCursor(callable $fetch)
    {
        $stmt = $this->_conn->prepare('SELECT id FROM stmt_test');
        $stmt->closeCursor();

        self::expectException(DriverException::class);
              expectException(DriverException::class);
        $fetch($stmt);
    }

    /**
     * @dataProvider fetchProvider
     */
    public function testFetchFromExecutedStatementWithClosedCursor(callable $fetch)
    {
        $this->_conn->insert('stmt_test', array('id' => 1));

        $stmt = $this->_conn->prepare('SELECT id FROM stmt_test');
        $stmt->execute();
        $stmt->closeCursor();

        self::expectException(DriverException::class);
              expectException(DriverException::class);
        $fetch($stmt);
    }

    /**
     * @dataProvider fetchProvider
     */
    public function testFetchBeforeExecute(callable $fetch) : void
    {
        $platform = $this->_conn->getDatabasePlatform();
        $query    = $platform->getDummySelectSQL();
        $stmt     = $this->_conn->prepare($query);

        self::expectException(DriverException::class);
              expectException(DriverException::class);
        $fetch($stmt);
    }

    public static function fetchProvider()
    {
        return array(
            'fetch' => array(
                function (Statement $stmt) {
                    return $stmt->fetch();
                },
            ),
            'fetch-column' => array(
                function (Statement $stmt) {
                    return $stmt->fetchColumn();
                },
            ),
            'fetch-all' => array(
                function (Statement $stmt) {
                    return $stmt->fetchAll();
                },
            ),
        );
    }

    public function testFetchInColumnMode() : void
    {
        $platform = $this->_conn->getDatabasePlatform();
        $query    = $platform->getDummySelectSQL();
        $result   = $this->_conn->executeQuery($query)->fetch(FetchMode::COLUMN);

        self::assertEquals(1, $result);
    }
}

doctrine / dbal

Pull Request — develop (#3154)

testFetchFromExecutedStatementWithClosedCursor() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like
