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

Timestamp:

10/11/2015 11:29:55 PM (10 years ago)

Author:

jeremyfelt

Message:

MS: Reject truthy, non-numeric network ids in _network_option().

A valid $network_id or null/false is expected as the first parameter for _network_option(). If something other than that is passed, we immediately return false rather than attempting to guess what network was intended.

See #28290.

File:

• trunk/tests/phpunit/tests/option/networkOption.php (modified) (1 diff)

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

@@ -39 +39 @@
         $this->assertEquals( $value, get_network_option( $id, $option, false ) );
     }
+
+    /**
+     * @dataProvider data_network_id_parameter
+     *
+     * @param $network_id
+     * @param $expected_response
+     */
+    function test_add_network_option_network_id_parameter( $network_id, $expected_response ) {
+        $option = rand_str();
+        $value = rand_str();
+
+        $this->assertEquals( $expected_response, add_network_option( $network_id, $option, $value ) );
+    }
+
+    /**
+     * @dataProvider data_network_id_parameter
+     *
+     * @param $network_id
+     * @param $expected_response
+     */
+    function test_get_network_option_network_id_parameter( $network_id, $expected_response ) {
+        $option = rand_str();
+
+        $this->assertEquals( $expected_response, get_network_option( $network_id, $option, true ) );
+    }
+
+    function data_network_id_parameter() {
+        return array(
+            // Numeric values should always be accepted.
+            array( 1,   true ),
+            array( '1', true ),
+            array( 2,   true ),
+
+            // Null, false, and zero will be treated as the current network.
+            array( null,  true ),
+            array( false, true ),
+            array( 0,     true ),
+            array( '0',   true ),
+
+            // Other truthy or string values should be rejected.
+            array( true,     false ),
+            array( 'string', false ),
+        );
+    }
 }