fix logic bug in assertArrayEquals
authordaniel <daniel.kinzler@wikimedia.de>
Thu, 26 Jul 2012 18:08:54 +0000 (20:08 +0200)
committerdaniel <daniel.kinzler@wikimedia.de>
Thu, 26 Jul 2012 18:08:54 +0000 (20:08 +0200)
Change-Id: I442ebfe77292918f29121aba1205de664723e468

tests/phpunit/MediaWikiTestCase.php

index 8f18d6e..c96eba0 100644 (file)
@@ -400,16 +400,16 @@ abstract class MediaWikiTestCase extends PHPUnit_Framework_TestCase {
         * @param boolean $named If the keys should match
         */
        protected function assertArrayEquals( array $expected, array $actual, $ordered = false, $named = false ) {
-               if ( !$named ) {
-                       $expected = array_values( $expected );
-                       $actual = array_values( $actual );
-               }
-
                if ( !$ordered ) {
                        asort( $expected );
                        asort( $actual );
                }
 
+               if ( !$named ) {
+                       $expected = array_values( $expected );
+                       $actual = array_values( $actual );
+               }
+
                call_user_func_array(
                        array( $this, 'assertEquals' ),
                        array_merge( array( $expected, $actual ), array_slice( func_get_args(), 4 ) )