<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"

"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">

<html xmlns="http://www.w3.org/1999/xhtml">

<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />

<title>[54830] Tests: Resolve `WP_Query` test failures on MariaDB due to indeterminate sort order.</title>

</head>

<body>

<style type="text/css"><!--

#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }

#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }

#msg dt:after { content:':';}

#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt;  }

#msg dl a { font-weight: bold}

#msg dl a:link    { color:#fc3; }

#msg dl a:active  { color:#ff0; }

#msg dl a:visited { color:#cc6; }

h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }

#msg pre { white-space: pre-line; overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }

#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }

#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }

#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }

#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }

#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }

#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }

#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }

#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }

#logmsg pre { background: #eee; padding: 1em; }

#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}

#logmsg dl { margin: 0; }

#logmsg dt { font-weight: bold; }

#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }

#logmsg dd:before { content:'\00bb';}

#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }

#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }

#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }

#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }

#logmsg table th.Corner { text-align: left; }

#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }

#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }

#patch { width: 100%; }

#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}

#patch .propset h4, #patch .binary h4 {margin:0;}

#patch pre {padding:0;line-height:1.2em;margin:0;}

#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}

#patch .propset .diff, #patch .binary .diff  {padding:10px 0;}

#patch span {display:block;padding:0 10px;}

#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}

#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}

#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}

#patch .lines, .info {color:#888;background:#fff;}

--></style>

<div id="msg">

<dl class="meta" style="font-size: 105%">

<dt style="float: left; width: 6em; font-weight: bold">Revision</dt> <dd><a style="font-weight: bold" href="https://core.trac.wordpress.org/changeset/54830">54830</a><script type="application/ld+json">{"@context":"http://schema.org","@type":"EmailMessage","description":"Review this Commit","action":{"@type":"ViewAction","url":"https://core.trac.wordpress.org/changeset/54830","name":"Review Commit"}}</script></dd>

<dt style="float: left; width: 6em; font-weight: bold">Author</dt> <dd>SergeyBiryukov</dd>

<dt style="float: left; width: 6em; font-weight: bold">Date</dt> <dd>2022-11-12 15:25:57 +0000 (Sat, 12 Nov 2022)</dd>

</dl>

<pre style='padding-left: 1em; margin: 2em 0; border-left: 2px solid #ccc; line-height: 1.25; font-size: 105%; font-family: sans-serif'>Tests: Resolve `WP_Query` test failures on MariaDB due to indeterminate sort order.

<a href="https://core.trac.wordpress.org/changeset/54768">[54768]</a> added a few tests to verify that caching within `WP_Query` is bypassed when the `SELECT` clause has been modified via a filter, to avoid cache key collisions and the returning of incomplete or unexpected results.

However, creating several posts with the same date/time fields can result in inconsistent sort ordering between MySQL and MariaDB, as each engine refines the order further using a different index.

This commit aims to stabilize the tests by using `assertEqualSets()` instead of `assertEquals()`, since testing the order is out of their scope. Includes removing `array_unshift()` and `array_reverse()` calls as no longer needed.

This resolves a few test failures on MariaDB along the lines of:

{{{

Tests_Query_FieldsClause::test_should_limit_fields_to_id_and_parent_subset

Posts property for first query is not of expected form.

Failed asserting that two arrays are equal.

--- Expected

+++ Actual

@@ @@

 Array (

     0 => stdClass Object (

-        'ID' => 36019

+        'ID' => 36015

         'post_parent' => 0

     )

     1 => stdClass Object (

-        'ID' => 36018

+        'ID' => 36016

         'post_parent' => 0

     )

     2 => stdClass Object (...)

     3 => stdClass Object (

-        'ID' => 36016

+        'ID' => 36018

         'post_parent' => 0

     )

     4 => stdClass Object (

-        'ID' => 36015

+        'ID' => 36019

         'post_parent' => 0

     )

 )

/tmp/wp-test-runner/tests/phpunit/tests/query/fieldsClause.php:67

/tmp/wp-test-runner/phpunit-5.7.phar:598

}}}

Follow-up to <a href="https://core.trac.wordpress.org/changeset/54768">[54768]</a>.

Props peterwilsoncc, SergeyBiryukov.

Merges <a href="https://core.trac.wordpress.org/changeset/54829">[54829]</a> to the 6.1 branch.

Fixes <a href="https://core.trac.wordpress.org/ticket/57012">#57012</a>.</pre>

<h3>Modified Paths</h3>

<ul>

<li><a href="#branches61testsphpunittestsqueryfieldsClausephp">branches/6.1/tests/phpunit/tests/query/fieldsClause.php</a></li>

</ul>

<h3>Property Changed</h3>

<ul>

<li><a href="#branches61">branches/6.1/</a></li>

</ul>

</div>

<div id="patch">

<h3>Diff</h3>

<span class="cx" style="display: block; padding: 0 10px">Index: branches/6.1

</span><span class="cx" style="display: block; padding: 0 10px">===================================================================

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">--- branches/6.1 2022-11-12 14:49:25 UTC (rev 54829)

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+++ branches/6.1  2022-11-12 15:25:57 UTC (rev 54830)

</ins><a id="branches61"></a>

<div class="propset"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Property changes: branches/6.1</h4>

<pre class="diff"><span>

</span></pre></div>

<a id="svnmergeinfo"></a>

<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: svn:mergeinfo</h4></div>

<span class="cx" style="display: block; padding: 0 10px"> /branches/5.0:43681-43682,43684-43688,43719-43720,43723,43726-43727,43729-43731,43734-43744,43747,43751-43754,43758,43760-43765,43767-43770,43772,43774-43781,43783,43785,43790-43806,43808-43821,43825,43828,43830-43834,43836-43843,43846-43863,43867-43889,43891-43894,43897-43905,43908-43909,43911-43929,43931-43942,43946-43947,43949-43956,43959-43964,43967-43969,43988,43994,44014,44017,44047,44183,44185,44187-44206,44208-44213,44231-44232,44235,44248,44284,44287-44288

</span><span class="cx" style="display: block; padding: 0 10px"> /branches/5.5:49373-49379,49381

</span><span class="cx" style="display: block; padding: 0 10px"> /branches/5.8:51889

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-/trunk:54645-54646,54649,54652,54660,54662,54669,54674,54678,54686,54690,54693,54698,54706,54724,54729-54730,54733,54741,54745,54751,54766,54768-54771,54774,54777,54782,54785-54789,54797,54799,54801-54804,54811,54813,54815,54817,54819,54821,54823-54824

</del><span class="cx" style="display: block; padding: 0 10px">\ No newline at end of property

</span><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+/trunk:54645-54646,54649,54652,54660,54662,54669,54674,54678,54686,54690,54693,54698,54706,54724,54729-54730,54733,54741,54745,54751,54766,54768-54771,54774,54777,54782,54785-54789,54797,54799,54801-54804,54811,54813,54815,54817,54819,54821,54823-54824,54829

</ins><span class="cx" style="display: block; padding: 0 10px">\ No newline at end of property

</span><a id="branches61testsphpunittestsqueryfieldsClausephp"></a>

<div class="modfile"><h4 style="background-color: #eee; color: inherit; margin: 1em 0; padding: 1.3em; font-size: 115%">Modified: branches/6.1/tests/phpunit/tests/query/fieldsClause.php</h4>

<pre class="diff"><span>

<span class="info" style="display: block; padding: 0 10px; color: #888">--- branches/6.1/tests/phpunit/tests/query/fieldsClause.php   2022-11-12 14:49:25 UTC (rev 54829)

+++ branches/6.1/tests/phpunit/tests/query/fieldsClause.php     2022-11-12 15:25:57 UTC (rev 54830)

</span><span class="lines" style="display: block; padding: 0 10px; color: #888">@@ -54,23 +54,19 @@

</span><span class="cx" style="display: block; padding: 0 10px"> 

</span><span class="cx" style="display: block; padding: 0 10px">                $expected = array();

</span><span class="cx" style="display: block; padding: 0 10px">                foreach ( self::$post_ids as $post_id ) {

<del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                        // Use array_shift to populate in the reverse order.

-                       array_unshift(

-                               $expected,

-                               (object) array(

-                                       'ID'          => $post_id,

-                                       'post_parent' => 0,

-                               )

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+                 $expected[] = (object) array(

+                               'ID'          => $post_id,

+                               'post_parent' => 0,

</ins><span class="cx" style="display: block; padding: 0 10px">                         );

</span><span class="cx" style="display: block; padding: 0 10px">                }

</span><span class="cx" style="display: block; padding: 0 10px"> 

</span><del style="background-color: #fdd; text-decoration:none; display:block; padding: 0 10px">-                $this->assertEquals( $expected, $q->posts, 'Posts property for first query is not of expected form.' );

</del><ins style="background-color: #dfd; text-decoration:none; display:block; padding: 0 10px">+         $this->assertEqualSets( $expected, $q->posts, 'Posts property for first query is not of expected form.' );

</ins><span class="cx" style="display: block; padding: 0 10px">                 $this->assertSame( 5, $q->found_posts, 'Number of found posts is not five.' );

</span><span class="cx" style="display: block; padding: 0 10px">                $this->assertEquals( 1, $q->max_num_pages, 'Number of found pages is not one.' );

</span><span class="cx" style="display: block; padding: 0 10px"> 