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

Timestamp:

10/11/2015 10:43:59 PM (11 years ago)

Author:

jeremyfelt

Message:

MS: Adjust _network_option() parameter order, $network_id is first.

This better aligns with expectations and matches the structure used by _blog_option(). The _site_option() functions remain as an appropriate method for working with the current network.

See #28290.

File:

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

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

@@ -16 +16 @@
                 $value = rand_str();
 
-                add_network_option( $option, $value );
-                $this->assertFalse( get_network_option( $option, false, $id ) );
+                add_site_option( $option, $value );
+                $this->assertFalse( get_network_option( $id, $option, false ) );
         }
 
@@ -25 +25 @@
                 $value = rand_str();
 
-                add_network_option( $option, $value, $id );
-                $this->assertEquals( $value, get_network_option( $option, false, $id ) );
+                add_network_option( $id, $option, $value );
+                $this->assertEquals( $value, get_network_option( $id, $option, false ) );
         }
 
@@ -34 +34 @@
                 $value = rand_str();
 
-                add_network_option( $option, $value );
-                add_network_option( $option, $value, $id );
-                delete_network_option( $option );
-                $this->assertEquals( $value, get_network_option( $option, false, $id ) );
+                add_site_option( $option, $value );
+                add_network_option( $id, $option, $value );
+                delete_site_option( $option );
+                $this->assertEquals( $value, get_network_option( $id, $option, false ) );
         }
 }