Move dbSetup=true; follow-up d920237
authorMatthew Flaschen <mflaschen@wikimedia.org>
Tue, 15 Mar 2016 02:49:13 +0000 (22:49 -0400)
committerMatthew Flaschen <mflaschen@wikimedia.org>
Tue, 15 Mar 2016 02:52:46 +0000 (22:52 -0400)
See https://gerrit.wikimedia.org/r/#/c/274643/8/tests/phpunit/MediaWikiTestCase.php

I unintentionally changed it so dbSetup wasn't always set to true
(it was in the early return case in the old code, but not the new)

Bug: T95870
Change-Id: I2119b0233330b88a61d15fe4dc55fe855fce4b6a

tests/phpunit/MediaWikiTestCase.php

index f3306be..045c3d4 100644 (file)
@@ -764,6 +764,8 @@ abstract class MediaWikiTestCase extends PHPUnit_Framework_TestCase {
                        return;
                }
 
+               self::$dbSetup = true;
+
                if ( !self::setupDatabaseWithTestPrefix( $db, $prefix ) ) {
                        return;
                }
@@ -773,8 +775,6 @@ abstract class MediaWikiTestCase extends PHPUnit_Framework_TestCase {
                if ( $db->getType() == 'oracle' ) {
                        $db->query( 'BEGIN FILL_WIKI_INFO; END;' );
                }
-
-               self::$dbSetup = true;
        }
 
        /**