<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 25, 2015 at 6:50 AM, Marc Delisle <span dir="ltr"><<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class="">Le 2015-08-24 15:10, Hugues Peccatte a écrit :<br>
> Le lun. 24 août 2015 à 16:00, Marc Delisle <<a href="mailto:marc@infomarc.info">marc@infomarc.info</a><br>
</span>> <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>>> a écrit :<br>
<span class="">><br>
>     Le 2015-08-24 03:00, Michal Čihař a écrit :<br>
>     > Hi<br>
>     ><br>
>     > Dne Sat, 22 Aug 2015 14:11:00 -0400<br>
</span>>     > Marc Delisle <<a href="mailto:marc@infomarc.info">marc@infomarc.info</a> <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>>><br>
<div><div class="h5">>     napsal(a):<br>
>     ><br>
>     >> Hi,<br>
>     >> does someone have a clue about these? [0] I cannot reproduce these<br>
>     >> problems on my phpunit.<br>
>     >><br>
>     >> [0] <a href="https://travis-ci.org/phpmyadmin/phpmyadmin/jobs/76699535" rel="noreferrer" target="_blank">https://travis-ci.org/phpmyadmin/phpmyadmin/jobs/76699535</a><br>
>     >><br>
>     ><br>
>     ><br>
>     > The problems seem to be there at least since this build:<br>
>     ><br>
>     > <a href="https://travis-ci.org/phpmyadmin/phpmyadmin/jobs/76402734#L452" rel="noreferrer" target="_blank">https://travis-ci.org/phpmyadmin/phpmyadmin/jobs/76402734#L452</a><br>
><br>
>     About this failing test on Travis:<br>
>     ===========<br>
>     PMA_ExportSql_Test::testExportDBCreate<br>
><br>
>     Failed asserting that 'DROP DATABASE `db`;<br>
>     CREATE DATABASE IF NOT EXISTS `db` COLLATE utf8_general_ci;<br>
>     USE `db`;'<br>
>     contains<br>
><br>
>     "CREATE DATABASE IF NOT EXISTS `db` DEFAULT CHARACTER SET utf8 COLLATE<br>
>     utf8_general_ci;"<br>
>     =========<br>
><br>
>     looking at exportDBCreate() the only possible reason is that<br>
>     if (PMA_DRIZZLE)<br>
><br>
>     is true<br>
><br>
>     --<br>
>     Marc Delisle | phpMyAdmin<br>
><br>
><br>
> Hi,<br>
><br>
> There are too many implied global config used by functions. So it's<br>
> quite hard to generate a generic unit test.<br>
<br>
</div></div>However there must be a reason why this test recently started to fail on<br>
Travis.<br>
<span class="im"><br>
> But… If Drizzle is removed, we will simplify the code and avoid this<br>
> specific error. :)<br>
><br>
> H.<br>
<br>
<br>
<br>
</span><div class=""><div class="h5">--<br>
Marc Delisle | phpMyAdmin<br>
<br></div></div></blockquote><div> </div><div>Sorry about the noise and the failures should be fixed with <a href="https://github.com/phpmyadmin/phpmyadmin/commit/e29ba1df2297716cba1909c64399ebec233306b0">https://github.com/phpmyadmin/phpmyadmin/commit/e29ba1df2297716cba1909c64399ebec233306b0</a>.</div><div><br></div><div>Looks like the failure of <span style="color:rgba(0,0,0,0.298039);font-family:Consolas,'Liberation Mono',Menlo,Courier,monospace;font-size:12px;line-height:16.7999992370605px;white-space:pre;background-color:rgb(244,247,251)">testSetPropertiesWithDrizzle() </span>prevented constants being restored and led to other test failures. May be we should move these tests to a new class and do the redefining and restoring of constants in setup() and teardown() methods respectively?</div></div><div><br></div>-- <br><div class="gmail_signature">Thanks and Regards,<div><br></div><div>Madhura Jayaratne<br><div><br></div></div></div>
</div></div>