WordPress.org
Get WordPress

Make WordPress Core

Opened 8 years ago

Closed 8 years ago

#40283 closed defect (bug) (fixed)

Add missing global cache groups in tests

Reported by: jeremyfelt's profile jeremyfelt Owned by: jeremyfelt's profile jeremyfelt
Milestone: 4.8 Priority: normal
Severity: normal Version:
Component: Cache API Keywords: has-patch
Focuses: multisite Cc:

Description

Several global cache groups are missing from WP_UnitTestCase::flush_cache(), which can cause some fun cross-test confusion when running multisite tests with object cache enabled.

Previously, [1332/tests]

Attachments (1)

40283.diff​ (1.7 KB) - added by jeremyfelt 8 years ago.

Download all attachments as: .zip

Change History (3)

@jeremyfelt
8 years ago

#1 @jeremyfelt
8 years ago

  • Owner set to jeremyfelt
  • Status changed from new to assigned

40283.diff​ adds useremail, userslugs, networks, sites, site-details groups to both WP_UnitTestCase::flush_cache() and Tests_Cache::init_cache(). It's less necessary in the cache tests, but doesn't hurt to be consistent.

#2 @jeremyfelt
8 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 40343:

Tests: Add missing global cache groups.

Add useremail, userslugs, networks, sites, and site-details as global cache groups when added with unit tests. This aligns the list with the core default.

Fixes #40283.

Note: See TracTickets for help on using tickets.