saper has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/99640


Change subject: Don't assume test user has ID 1 in SpecialPageTest
......................................................................

Don't assume test user has ID 1 in SpecialPageTest

SpecialPageTest seems to assume
the first database user has ID number 1.

In case of the current PostgreSQL test
setup it does not have to be the case.

This query:

Query pgtest01 (101) (slave):
SELECT /* User::loadFromDatabase 127.0.0.1
*/  user_id /* more fields */ FROM "unittest_mwuser"
WHERE user_id = '1'  LIMIT 1

fails, because UTSysop has been added by the
test setup before with some other ID:

Query pgtest01 (65) (slave):

INSERT /* User::addToDatabase 127.0.0.1 */
INTO "unittest _mwuser"
        (user_id,user_name /* more fields */
VALUES ('322','UTSysop' /* more values */

Change-Id: I7b0a6a9001765111669051d57e675e588b71ba17
---
M tests/phpunit/includes/SpecialPageTest.php
1 file changed, 1 insertion(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/40/99640/1

diff --git a/tests/phpunit/includes/SpecialPageTest.php 
b/tests/phpunit/includes/SpecialPageTest.php
index 65057a5..0ee335a 100644
--- a/tests/phpunit/includes/SpecialPageTest.php
+++ b/tests/phpunit/includes/SpecialPageTest.php
@@ -102,8 +102,7 @@
        public function testRequireLoginNotAnon() {
                $specialPage = new SpecialPage( 'Watchlist', 'viewmywatchlist' 
);
 
-               $user = User::newFromId( 0 );
-               $user->setId( 1 );
+               $user = User::newFromName( "UTSysop" );
                $specialPage->getContext()->setUser( $user );
 
                $specialPage->requireLogin();

-- 
To view, visit https://gerrit.wikimedia.org/r/99640
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7b0a6a9001765111669051d57e675e588b71ba17
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: saper <sa...@saper.info>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to