[wp-trac] [WordPress Trac] #61530: Test tool and unit test improvements for 6.7

WordPress Trac noreply at wordpress.org
Tue Aug 20 20:05:38 UTC 2024


#61530: Test tool and unit test improvements for 6.7
--------------------------------------+---------------------
 Reporter:  jonsurrell                |       Owner:  (none)
     Type:  task (blessed)            |      Status:  new
 Priority:  normal                    |   Milestone:  6.7
Component:  Build/Test Tools          |     Version:
 Severity:  normal                    |  Resolution:
 Keywords:  has-patch has-unit-tests  |     Focuses:
--------------------------------------+---------------------

Comment (by hellofromTonya):

 In [changeset:"58919" 58919]:
 {{{
 #!CommitTicketReference repository="" revision="58919"
 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.
 }}}

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/61530#comment:28>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list