Skip to content

Commit 5c19fa1

Browse files
Tests: Remove WP_Term::$filter property unset() within term tests.
Removes the `unset()` of the `WP_Term::$filter` property within the term tests. Why? Prior to the introduction of WP_Term, the term was added to the cache when its filter property was empty. To test the cache, the tests unset this property to trigger `wp_cache_add()` in `get_term()`. [34997] changed that behavior to trigger `wp_cache_add()` when the term was not found after `wp_cache_get()` (i.e. happened in `WP_Term::get_instance()`). Unsetting the filter property is and was not needed. Prior to `WP_Term`, the condition was an empty value. With `WP_Term`, the filter property is no longer part of the conditional logic for caching. Follow-up to [34997], [30954], [34035]. See #61890, #61530. git-svn-id: https://develop.svn.wordpress.org/trunk@58919 602fd350-edb4-49c9-b593-d223f7449a82
1 parent 6189d05 commit 5c19fa1

File tree

2 files changed

+0
-4
lines changed

2 files changed

+0
-4
lines changed

tests/phpunit/tests/term/cache.php

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -116,9 +116,6 @@ public function test_get_term_should_update_term_cache_when_passed_an_object() {
116116

117117
$num_queries = get_num_queries();
118118

119-
// get_term() will only be update the cache if the 'filter' prop is unset.
120-
unset( $term_object->filter );
121-
122119
$term_object_2 = get_term( $term_object, 'wptests_tax' );
123120

124121
// No new queries should have fired.

tests/phpunit/tests/term/getTerm.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,6 @@ public function test_passing_term_object_should_skip_database_query_when_filter_
9898

9999
$num_queries = get_num_queries();
100100

101-
unset( $term->filter );
102101
$term_a = get_term( $term, 'wptests_tax' );
103102

104103
$this->assertSame( $num_queries, get_num_queries() );

0 commit comments

Comments
 (0)