Removed testBug29116LoadWithEmptyIp. Fails due to wfDeprecated() call in load() and...
authorAaron Schulz <aaron@users.mediawiki.org>
Mon, 18 Jul 2011 22:30:38 +0000 (22:30 +0000)
committerAaron Schulz <aaron@users.mediawiki.org>
Mon, 18 Jul 2011 22:30:38 +0000 (22:30 +0000)
tests/phpunit/includes/BlockTest.php

index 2f224ba..dff5dfe 100644 (file)
@@ -82,25 +82,6 @@ class BlockTest extends MediaWikiLangTestCase {
 
        }
 
-       /**
-        * This is the method previously used to load block info in CheckUser etc
-        * passing an empty value (empty string, null, etc) as the ip parameter bypasses IP lookup checks.
-        *
-        * This stopped working with r84475 and friends: regression being fixed for bug 29116.
-        *
-        * @dataProvider dataBug29116
-        */
-       function testBug29116LoadWithEmptyIp( $vagueTarget ) {
-               $uid = User::idFromName( 'UTBlockee' );
-               $this->assertTrue( ($uid > 0), 'Must be able to look up the target user during tests' );
-
-               $block = new Block();
-               $ok = $block->load( $vagueTarget, $uid );
-               $this->assertTrue( $ok, "Block->load() with empty IP and user ID '$uid' should return a block" );
-
-               $this->assertTrue( $this->block->equals( $block ), "Block->load() returns the same block as the one that was made when given empty ip param " . var_export( $vagueTarget, true ) );
-       }
-
        /**
         * CheckUser since being changed to use Block::newFromTarget started failing
         * because the new function didn't accept empty strings like Block::load()