Changeset 54402 for trunk/tests/phpunit/tests/option/networkOption.php

Timestamp:

10/07/2022 01:02:07 AM (19 months ago)

Author:

desrosj

Message:

Tests: Replace some occurrences of assertEquals() with assertSame().

This ensures that not only the return values match the expected results, but also that their type is the same.

Props costdev, desrosj.
See #55654.

File:

• trunk/tests/phpunit/tests/option/networkOption.php (modified) (3 diffs)

trunk/tests/phpunit/tests/option/networkOption.php

@@ -177 +177 @@
 
         // Check they exists.
-        $this->assertEquals( $funky_meta, get_network_option( $network_id, $option ) );
+        $this->assertEqualSets( $funky_meta, get_network_option( $network_id, $option ) );
     }
 
@@ -191 +191 @@
         add_metadata( 'site', $network_id, $option, 'tuesday', true );
         add_metadata( 'site', $network_id, $option, 'wednesday', true );
-        $this->assertEquals( 'monday', get_network_option( $network_id, $option, true ) );
+        $this->assertSame( 'monday', get_network_option( $network_id, $option, true ) );
     }
 
@@ -280 +280 @@
 
         add_metadata( 'site', $network_id, $option, $array_w_object, true );
-        $this->assertEquals( $array_w_object, get_network_option( $network_id, $option ) );
+        $this->assertEqualSets( $array_w_object, get_network_option( $network_id, $option ) );
     }