Changeset 33110 for trunk/tests/phpunit/tests/option/transient.php

Timestamp:

07/07/2015 04:44:08 PM (11 years ago)

Author:

ocean90

Message:

Transients: If get_option( $transient_timeout ) returns false, don't bother trying to delete the transient in get_transient().

props jamesgol, ericmann.
fixes #30380.

File:

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

trunk/tests/phpunit/tests/option/transient.php

@@ -84 +84 @@
                 $this->assertFalse( get_transient( $key ) );
         }
+
+        /**
+         * If get_option( $transient_timeout ) returns false, don't bother trying to delete the transient.
+         *
+         * @ticket 30380
+         */
+        function test_nonexistent_key_dont_delete_if_false() {
+                // Create a bogus a transient
+                $key = 'test_transient';
+                set_transient( $key, 'test', 60 * 10 );
+                $this->assertEquals( 'test', get_transient( $key ) );
+
+                // Useful variables for tracking
+                $transient_timeout = '_transient_timeout_' . $key;
+
+                // Mock an action for tracking action calls
+                $a = new MockAction();
+
+                // Make sure the timeout option returns false
+                add_filter( 'option_' . $transient_timeout, '__return_false' );
+
+                // Add some actions to make sure options are _not_ deleted
+                add_action( 'delete_option', array( $a, 'action' ) );
+
+                // Act
+                get_transient( $key );
+
+                // Make sure delete option was not called for both the transient and the timeout
+                $this->assertEquals( 0, $a->get_call_count() );
+        }
+
+        /**
+         * @ticket 30380
+         */
+        function test_nonexistent_key_old_timeout() {
+                // Create a transient
+                $key = 'test_transient';
+                set_transient( $key, 'test', 60 * 10 );
+                $this->assertEquals( 'test', get_transient( $key ) );
+
+                // Make sure the timeout option returns false
+                $timeout = '_transient_timeout_' . $key;
+                $transient_option = '_transient_' . $key;
+                add_filter( 'option_' . $timeout, '__return_zero' );
+
+                // Mock an action for tracking action calls
+                $a = new MockAction();
+
+                // Add some actions to make sure options are deleted
+                add_action( 'delete_option', array( $a, 'action' ) );
+
+                // Act
+                get_transient( $key );
+
+                // Make sure delete option was called for both the transient and the timeout
+                $this->assertEquals( 2, $a->get_call_count() );
+
+                $expected = array(
+                        array(
+                                'action' => 'action',
+                                'tag'    => 'delete_option',
+                                'args'   => array( $transient_option ),
+                        ),
+                        array(
+                                'action' => 'action',
+                                'tag'    => 'delete_option',
+                                'args'   => array( $timeout ),
+                        ),
+                );
+                $this->assertEquals( $expected, $a->get_events() );
+        }
 }