Convert all array() syntax to []
[lhc/web/wiklou.git] / tests / phpunit / includes / db / DatabaseSqliteTest.php
index 451d82a..18de9be 100644 (file)
@@ -3,7 +3,7 @@
 class DatabaseSqliteMock extends DatabaseSqlite {
        private $lastQuery;
 
-       public static function newInstance( array $p = array() ) {
+       public static function newInstance( array $p = [] ) {
                $p['dbFilePath'] = ':memory:';
                $p['schema'] = false;
 
@@ -64,26 +64,26 @@ class DatabaseSqliteTest extends MediaWikiTestCase {
        }
 
        public static function provideAddQuotes() {
-               return array(
-                       array( // #0: empty
+               return [
+                       [ // #0: empty
                                '', "''"
-                       ),
-                       array( // #1: simple
+                       ],
+                       [ // #1: simple
                                'foo bar', "'foo bar'"
-                       ),
-                       array( // #2: including quote
+                       ],
+                       [ // #2: including quote
                                'foo\'bar', "'foo''bar'"
-                       ),
+                       ],
                        // #3: including \0 (must be represented as hex, per https://bugs.php.net/bug.php?id=63419)
-                       array(
+                       [
                                "x\0y",
                                "x'780079'",
-                       ),
-                       array( // #4: blob object (must be represented as hex)
+                       ],
+                       [ // #4: blob object (must be represented as hex)
                                new Blob( "hello" ),
                                "x'68656c6c6f'",
-                       ),
-               );
+                       ],
+               ];
        }
 
        /**
@@ -100,7 +100,8 @@ class DatabaseSqliteTest extends MediaWikiTestCase {
 
                $this->assertTrue( $re !== false, 'query failed' );
 
-               if ( $row = $re->fetchRow() ) {
+               $row = $re->fetchRow();
+               if ( $row ) {
                        if ( $value instanceof Blob ) {
                                $value = $value->fetch();
                        }
@@ -189,20 +190,36 @@ class DatabaseSqliteTest extends MediaWikiTestCase {
        public function testDuplicateTableStructure() {
                $db = DatabaseSqlite::newStandaloneInstance( ':memory:' );
                $db->query( 'CREATE TABLE foo(foo, barfoo)' );
+               $db->query( 'CREATE INDEX index1 ON foo(foo)' );
+               $db->query( 'CREATE UNIQUE INDEX index2 ON foo(barfoo)' );
 
                $db->duplicateTableStructure( 'foo', 'bar' );
                $this->assertEquals( 'CREATE TABLE "bar"(foo, barfoo)',
-                       $db->selectField( 'sqlite_master', 'sql', array( 'name' => 'bar' ) ),
+                       $db->selectField( 'sqlite_master', 'sql', [ 'name' => 'bar' ] ),
                        'Normal table duplication'
                );
+               $indexList = $db->query( 'PRAGMA INDEX_LIST("bar")' );
+               $index = $indexList->next();
+               $this->assertEquals( 'bar_index1', $index->name );
+               $this->assertEquals( '0', $index->unique );
+               $index = $indexList->next();
+               $this->assertEquals( 'bar_index2', $index->name );
+               $this->assertEquals( '1', $index->unique );
 
                $db->duplicateTableStructure( 'foo', 'baz', true );
                $this->assertEquals( 'CREATE TABLE "baz"(foo, barfoo)',
-                       $db->selectField( 'sqlite_temp_master', 'sql', array( 'name' => 'baz' ) ),
+                       $db->selectField( 'sqlite_temp_master', 'sql', [ 'name' => 'baz' ] ),
                        'Creation of temporary duplicate'
                );
+               $indexList = $db->query( 'PRAGMA INDEX_LIST("baz")' );
+               $index = $indexList->next();
+               $this->assertEquals( 'baz_index1', $index->name );
+               $this->assertEquals( '0', $index->unique );
+               $index = $indexList->next();
+               $this->assertEquals( 'baz_index2', $index->name );
+               $this->assertEquals( '1', $index->unique );
                $this->assertEquals( 0,
-                       $db->selectField( 'sqlite_master', 'COUNT(*)', array( 'name' => 'baz' ) ),
+                       $db->selectField( 'sqlite_master', 'COUNT(*)', [ 'name' => 'baz' ] ),
                        'Create a temporary duplicate only'
                );
        }
@@ -219,13 +236,13 @@ class DatabaseSqliteTest extends MediaWikiTestCase {
 
                $db->duplicateTableStructure( 'foo', 'bar' );
                $this->assertEquals( 'CREATE VIRTUAL TABLE "bar" USING FTS3(foobar)',
-                       $db->selectField( 'sqlite_master', 'sql', array( 'name' => 'bar' ) ),
+                       $db->selectField( 'sqlite_master', 'sql', [ 'name' => 'bar' ] ),
                        'Duplication of virtual tables'
                );
 
                $db->duplicateTableStructure( 'foo', 'baz', true );
                $this->assertEquals( 'CREATE VIRTUAL TABLE "baz" USING FTS3(foobar)',
-                       $db->selectField( 'sqlite_master', 'sql', array( 'name' => 'baz' ) ),
+                       $db->selectField( 'sqlite_master', 'sql', [ 'name' => 'baz' ] ),
                        "Can't create temporary virtual tables, should fall back to non-temporary duplication"
                );
        }
@@ -237,25 +254,25 @@ class DatabaseSqliteTest extends MediaWikiTestCase {
                $db = DatabaseSqlite::newStandaloneInstance( ':memory:' );
                $db->query( 'CREATE TABLE a (a_1)', __METHOD__ );
                $db->query( 'CREATE TABLE b (b_1, b_2)', __METHOD__ );
-               $db->insert( 'a', array(
-                               array( 'a_1' => 1 ),
-                               array( 'a_1' => 2 ),
-                               array( 'a_1' => 3 ),
-                       ),
+               $db->insert( 'a', [
+                               [ 'a_1' => 1 ],
+                               [ 'a_1' => 2 ],
+                               [ 'a_1' => 3 ],
+                       ],
                        __METHOD__
                );
-               $db->insert( 'b', array(
-                               array( 'b_1' => 2, 'b_2' => 'a' ),
-                               array( 'b_1' => 3, 'b_2' => 'b' ),
-                       ),
+               $db->insert( 'b', [
+                               [ 'b_1' => 2, 'b_2' => 'a' ],
+                               [ 'b_1' => 3, 'b_2' => 'b' ],
+                       ],
                        __METHOD__
                );
-               $db->deleteJoin( 'a', 'b', 'a_1', 'b_1', array( 'b_2' => 'a' ), __METHOD__ );
+               $db->deleteJoin( 'a', 'b', 'a_1', 'b_1', [ 'b_2' => 'a' ], __METHOD__ );
                $res = $db->query( "SELECT * FROM a", __METHOD__ );
-               $this->assertResultIs( array(
-                               array( 'a_1' => 1 ),
-                               array( 'a_1' => 3 ),
-                       ),
+               $this->assertResultIs( [
+                               [ 'a_1' => 1 ],
+                               [ 'a_1' => 3 ],
+                       ],
                        $res
                );
        }
@@ -278,19 +295,19 @@ class DatabaseSqliteTest extends MediaWikiTestCase {
                global $IP, $wgVersion, $wgProfiler;
 
                // Versions tested
-               $versions = array(
-                       //'1.13', disabled for now, was totally screwed up
+               $versions = [
+                       // '1.13', disabled for now, was totally screwed up
                        // SQLite wasn't included in 1.14
                        '1.15',
                        '1.16',
                        '1.17',
                        '1.18',
-               );
+               ];
 
                // Mismatches for these columns we can safely ignore
-               $ignoredColumns = array(
+               $ignoredColumns = [
                        'user_newtalk.user_last_timestamp', // r84185
-               );
+               ];
 
                $currentDB = DatabaseSqlite::newStandaloneInstance( ':memory:' );
                $currentDB->sourceFile( "$IP/maintenance/tables.sql" );
@@ -365,7 +382,7 @@ class DatabaseSqliteTest extends MediaWikiTestCase {
                $databaseCreation = $db->query( 'CREATE TABLE a ( a_1 )', __METHOD__ );
                $this->assertInstanceOf( 'ResultWrapper', $databaseCreation, "Database creation" );
 
-               $insertion = $db->insert( 'a', array( 'a_1' => 10 ), __METHOD__ );
+               $insertion = $db->insert( 'a', [ 'a_1' => 10 ], __METHOD__ );
                $this->assertTrue( $insertion, "Insertion worked" );
 
                $this->assertInternalType( 'integer', $db->insertId(), "Actual typecheck" );
@@ -376,21 +393,21 @@ class DatabaseSqliteTest extends MediaWikiTestCase {
                static $maint = null;
                if ( $maint === null ) {
                        $maint = new FakeMaintenance();
-                       $maint->loadParamsAndArgs( null, array( 'quiet' => 1 ) );
+                       $maint->loadParamsAndArgs( null, [ 'quiet' => 1 ] );
                }
 
                global $IP;
                $db = DatabaseSqlite::newStandaloneInstance( ':memory:' );
                $db->sourceFile( "$IP/tests/phpunit/data/db/sqlite/tables-$version.sql" );
                $updater = DatabaseUpdater::newForDB( $db, false, $maint );
-               $updater->doUpdates( array( 'core' ) );
+               $updater->doUpdates( [ 'core' ] );
 
                return $db;
        }
 
        private function getTables( $db ) {
                $list = array_flip( $db->listTables() );
-               $excluded = array(
+               $excluded = [
                        'external_user', // removed from core in 1.22
                        'math', // moved out of core in 1.18
                        'trackbacks', // removed from core in 1.19
@@ -401,7 +418,7 @@ class DatabaseSqliteTest extends MediaWikiTestCase {
                        // FTS4 ready!!1
                        'searchindex_docsize',
                        'searchindex_stat',
-               );
+               ];
                foreach ( $excluded as $t ) {
                        unset( $list[$t] );
                }
@@ -412,7 +429,7 @@ class DatabaseSqliteTest extends MediaWikiTestCase {
        }
 
        private function getColumns( $db, $table ) {
-               $cols = array();
+               $cols = [];
                $res = $db->query( "PRAGMA table_info($table)" );
                $this->assertNotNull( $res );
                foreach ( $res as $col ) {
@@ -424,13 +441,13 @@ class DatabaseSqliteTest extends MediaWikiTestCase {
        }
 
        private function getIndexes( $db, $table ) {
-               $indexes = array();
+               $indexes = [];
                $res = $db->query( "PRAGMA index_list($table)" );
                $this->assertNotNull( $res );
                foreach ( $res as $index ) {
                        $res2 = $db->query( "PRAGMA index_info({$index->name})" );
                        $this->assertNotNull( $res2 );
-                       $index->columns = array();
+                       $index->columns = [];
                        foreach ( $res2 as $col ) {
                                $index->columns[] = $col;
                        }
@@ -459,11 +476,19 @@ class DatabaseSqliteTest extends MediaWikiTestCase {
                $this->assertInstanceOf( 'ResultWrapper', $databaseCreation, "Failed to create table a" );
                $res = $db->select( 'a', '*' );
                $this->assertEquals( 0, $db->numFields( $res ), "expects to get 0 fields for an empty table" );
-               $insertion = $db->insert( 'a', array( 'a_1' => 10 ), __METHOD__ );
+               $insertion = $db->insert( 'a', [ 'a_1' => 10 ], __METHOD__ );
                $this->assertTrue( $insertion, "Insertion failed" );
                $res = $db->select( 'a', '*' );
                $this->assertEquals( 1, $db->numFields( $res ), "wrong number of fields" );
 
                $this->assertTrue( $db->close(), "closing database" );
        }
+
+       public function testToString() {
+               $db = DatabaseSqlite::newStandaloneInstance( ':memory:' );
+
+               $toString = (string)$db;
+
+               $this->assertContains( 'SQLite ', $toString );
+       }
 }