Merge "Remove no-op parameter from UIDGeneratorTest"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Thu, 25 Feb 2016 17:11:58 +0000 (17:11 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Thu, 25 Feb 2016 17:11:58 +0000 (17:11 +0000)
1  2 
tests/phpunit/includes/utils/UIDGeneratorTest.php

@@@ -14,16 -14,16 +14,16 @@@ class UIDGeneratorTest extends PHPUnit_
         * @covers UIDGenerator::newTimestampedUID88
         */
        public function testTimestampedUID( $method, $digitlen, $bits, $tbits, $hostbits ) {
 -              $id = call_user_func( array( 'UIDGenerator', $method ) );
 +              $id = call_user_func( [ 'UIDGenerator', $method ] );
                $this->assertEquals( true, ctype_digit( $id ), "UID made of digit characters" );
                $this->assertLessThanOrEqual( $digitlen, strlen( $id ),
                        "UID has the right number of digits" );
                $this->assertLessThanOrEqual( $bits, strlen( Wikimedia\base_convert( $id, 10, 2 ) ),
                        "UID has the right number of bits" );
  
 -              $ids = array();
 +              $ids = [];
                for ( $i = 0; $i < 300; $i++ ) {
 -                      $ids[] = call_user_func( array( 'UIDGenerator', $method ) );
 +                      $ids[] = call_user_func( [ 'UIDGenerator', $method ] );
                }
  
                $lastId = array_shift( $ids );
         * NOTE: When adding a new method name here please update the covers tags for the tests!
         */
        public static function provider_testTimestampedUID() {
 -              return array(
 -                      array( 'newTimestampedUID128', 39, 128, 46, 48 ),
 -                      array( 'newTimestampedUID128', 39, 128, 46, 48 ),
 -                      array( 'newTimestampedUID88', 27, 88, 46, 32 ),
 -              );
 +              return [
 +                      [ 'newTimestampedUID128', 39, 128, 46, 48 ],
 +                      [ 'newTimestampedUID128', 39, 128, 46, 48 ],
 +                      [ 'newTimestampedUID88', 27, 88, 46, 32 ],
 +              ];
        }
  
        /**
         * @covers UIDGenerator::newUUIDv1
         */
        public function testUUIDv1() {
 -              $ids = array();
 +              $ids = [];
                for ( $i = 0; $i < 100; $i++ ) {
                        $id = UIDGenerator::newUUIDv1();
                        $this->assertEquals( true,
@@@ -79,7 -79,7 +79,7 @@@
                                preg_match( '!^[0-9a-f]{12}1[0-9a-f]{3}[89ab][0-9a-f]{15}$!', $id ),
                                "UID $id has the right format" );
  
-                       $id = UIDGenerator::newRawUUIDv1( UIDGenerator::QUICK_RAND );
+                       $id = UIDGenerator::newRawUUIDv1();
                        $this->assertEquals( true,
                                preg_match( '!^[0-9a-f]{12}1[0-9a-f]{3}[89ab][0-9a-f]{15}$!', $id ),
                                "UID $id has the right format" );
@@@ -92,7 -92,7 +92,7 @@@
         * @covers UIDGenerator::newUUIDv4
         */
        public function testUUIDv4() {
 -              $ids = array();
 +              $ids = [];
                for ( $i = 0; $i < 100; $i++ ) {
                        $id = UIDGenerator::newUUIDv4();
                        $ids[] = $id;