Hi,
On Monday, 1 July 2013 at 8:44 PM, Ayush Chaudhary wrote:
Hi,
On Monday, 1 July 2013 at 8:37 PM, Marc Delisle wrote:
adam a écrit :
Hi,
Dne Mon, 1 Jul 2013 22:17:44 +0800 adam <adamgsoc2013@gmail.com (mailto:adamgsoc2013@gmail.com)> napsal(a):
Hi Michal,
Hi
Dne Wed, 26 Jun 2013 10:05:36 -0400 Marc Delisle <marc@infomarc.info (mailto:marc@infomarc.info)> napsal(a):
> I don't understand why I have different results when running phpunit on > the ayushchd/unit_testing branch (5 failures) and on the master branch > (with Michal's merge 10 minutes ago), which gives me 16 failures. >
Most of the differences were most likely caused by different error reporting level in PHP. Since 68b10df I've enabled E_ALL error reporting in testsuite, so the number of failed tests should be same regardless this factor.
This also leads to 30! failed tests on Travis (the CI server did see these failures before).
As most of these tests were written by Adam, please look at them and fix them ASAP.
Look at the failures, https://travis-ci.org/phpmyadmin/phpmyadmin/jobs/8620957
It is from two files: /home/travis/build/phpmyadmin/phpmyadmin/test/libraries/PMA_ConfigFile_test.php /home/travis/build/phpmyadmin/phpmyadmin/test/libraries/PMA_Tracker_test.php
In fact, Asyush worked on these two files: https://github.com/phpmyadmin/phpmyadmin/commits/master/test/libraries/PMA_C... https://github.com/phpmyadmin/phpmyadmin/commits/master/test/libraries/PMA_T...
Sorry for overlooking (I mangled file names in my head between looking at Travis and Git). Indeed these were done by Ayush, so the complain should have gone to him :-).
I sent a pull request: https://github.com/phpmyadmin/phpmyadmin/pull/471
This can fix most errors. thanks
I am puzzled. With my current master at 864cf690d8a7d10e37a56113d396f7c26630a9bb, phpunit 3.7.21 and PHP 5.4.10, I still have 18 failures.
Here is the first example:
There were 18 failures:
- ImportCsv_Test::testDoImport
Failed asserting that ' RENAME TABLE `pma_test`.`table1` TO `pma_test`.`table3`;CREATE TABLE IF NOT EXISTS `pma_test`.`TABLE 1` (`COL 1` varchar(48)) DEFAULT CHARACTER SET utf8 COLLATE utf8_general_ci;INSERT INTO `pma_test`.`TABLE 1` (`COL 1`) VALUES ('1,dbbase,,ddd,SELECT * FROM `db_content` WHERE 1');' contains "CREATE DATABASE IF NOT EXISTS `CSV_DB` DEFAULT CHARACTER".
phpmyadmin/test/classes/plugin/import/ImportCsv_test.php:134 /usr/local/bin/phpunit:46
I am having similar results at my end too. Here[0] is the output on my machine when I run phpunit. I reckon this is something to do with the backup globals flag. Is it now set to true on the CI server? Additionally, I did not get the "Undefined index: server" error on my machine, but that could be because on my machine, $GLOBALS['server'] was being set somewhere else during phpunit's run while on the server it wasn't.
[0] https://gist.github.com/ayushchd/5901501
-- Marc Delisle http://infomarc.info
This SF.net (http://SF.net) email is sponsored by Windows:
Build for Windows Store.
http://p.sf.net/sfu/windows-dev2dev _______________________________________________ Phpmyadmin-devel mailing list Phpmyadmin-devel@lists.sourceforge.net (mailto:Phpmyadmin-devel@lists.sourceforge.net) https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel
I just set backupGlobals to true in phpunit.xml and indeed that seems to have been the cause of different errors showing up everywhere.