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

Timestamp:

10/17/2025 06:46:42 PM (5 months ago)

Author:

johnjamesjacoby

Message:

Networks and Sites: add a general pre_site_option filter to get_network_option().

This change brings get_network_option() up-to-speed with get_option() by adding a more generalized way to short-circuit its return value.

It also introduces 2 new unit tests: one to mirror an existing pre_option test, and another to confirm that this new filter is working as intended.

Props audrasjb, Drivingralle, johnjamesjacoby, jorbin, nimeshatxecurify, rollybueno, shailu25, welcher, westonruter.

See #37930, r54145.

Fixes #56870.

File:

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

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

@@ -421 +421 @@
         $this->assertArrayHasKey( 'ticket_61730_notoption', $network_notoptions_cache_after, 'The option should be in the notoptions cache.' );
     }
+
+    /**
+     * Assert that the 'pre_site_option' hook is called once per call to get_network_option().
+     *
+     * @ticket 56870
+     *
+     * @group ms-required
+     *
+     * @covers ::get_network_option
+     */
+    public function test_get_network_option_should_call_pre_site_option_filter() {
+        $filter = new MockAction();
+
+        add_filter( 'pre_site_option', array( $filter, 'filter' ) );
+
+        get_network_option( get_current_network_id(), 'ignored' );
+
+        $this->assertSame( 1, $filter->get_call_count() );
+    }
+
+    /**
+     * Verifies that the global 'pre_site_option' filter short-circuits get_network_option().
+     *
+     * @ticket 56870
+     *
+     * @group ms-required
+     *
+     * @covers ::get_network_option
+     */
+    public function test_pre_site_option_filter_short_circuits_get_network_option() {
+        $option       = 'ticket_56870_pre_site_option_short_circuit';
+        $network_id   = get_current_network_id();
+        $default_val  = 'default-value';
+        $expected_val = 'filtered-value';
+
+        $callback = function ( $pre, $opt, $net_id ) use ( $option, $network_id, $expected_val ) {
+            // Ensure the filter is invoked for the requested option and network, then short-circuit.
+            if ( $opt === $option && (int) $net_id === (int) $network_id ) {
+                return $expected_val;
+            }
+            return $pre;
+        };
+
+        add_filter( 'pre_site_option', $callback, 10, 3 );
+
+        $actual_val = get_network_option( $network_id, $option, $default_val );
+
+        // The global pre filter should short-circuit and return $expected_val regardless of storage or default.
+        $this->assertSame( $expected_val, $actual_val );
+    }
 }