Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable WP_Query results cache in tests #453

Merged
merged 1 commit into from
Feb 28, 2023
Merged

Conversation

kovshenin
Copy link
Contributor

In most of these tests we're adding posts directly to the database which does not properly invalidate the WP_Query results cache introduced in WordPress 6.1. This change disables the WP_Query results cache.

See: https://make.wordpress.org/core/2022/10/07/improvements-to-wp_query-performance-in-6-1/

In most of these tests we're adding posts directly to the database
which does not properly invalidate the WP_Query results cache
introduced in WordPress 6.1. This commit disables the WP_Query
results cache.
Copy link
Contributor

@mikelittle mikelittle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch. Looks good.

@kovshenin kovshenin merged commit fcbd602 into master Feb 28, 2023
@kovshenin kovshenin deleted the tests-fix-query-cache branch February 28, 2023 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants