Le 2013-06-15 10:36, Ayush Chaudhary a écrit : Hi Ayush, with current master, I also see failures (15) here. A while ago, I also reported tests failures (some of them seemingly pointing to database not connecting) and I remember that someone (Madhura?) replied that the tests should be able to run without any db connection. It was another test failing that gave this impression.

On Sat, Jun 15, 2013 at 9:04 PM, Marc Delisle &lt;marc(a)infomarc.info <mailto:marc@infomarc.info>> wrote: Le 2013-06-15 11:20, Marc Delisle a écrit : seem to be somewhere. (phpunit.xml?) any db Git bisect tells me that this commit is the one that produced 12 failures: https://github.com/phpmyadmin/phpmyadmin/commit/0f4e573b1e0bcb743334e481810… Note that when bisecting, I marked as good other commits that were producing a smaller number of failures. Looks like the failures are due to some problems in backing up globals. I made the commit 0f4e573b1e <https://github.com/phpmyadmin/phpmyadmin/commit/0f4e573b1e0bcb743334e4818102b428bd023d2e> as there should not be any need to manually backup and restore GLOBALS['dbi'] as backupGlobals directive is now active. Somehow failing to restore the GLOBALS['dbi'] would result in other tests using the mocked database interface leading to test failures related to database interface. I will revert the commit until the cause of this is properly investigated.

Hi On Monday, 17 June 2013 at 1:16 AM, Dieter Adriaenssens wrote: 2013/6/16 Marc Delisle &lt;marc(a)infomarc.info&gt;fo>: Le 2013-06-16 12:27, Madhura Jayaratne a écrit : On Sat, Jun 15, 2013 at 9:04 PM, Marc Delisle &lt;marc(a)infomarc.info <mailto:marc@infomarc.info &lt;marc(a)infomarc.info&gt;&gt;&gt; wrote: Le 2013-06-15 11:20, Marc Delisle a écrit : seem to be somewhere. (phpunit.xml?) any db Git bisect tells me that this commit is the one that produced 12 failures: https://github.com/phpmyadmin/phpmyadmin/commit/0f4e573b1e0bcb743334e481810… Note that when bisecting, I marked as good other commits that were producing a smaller number of failures. Looks like the failures are due to some problems in backing up globals. I made the commit 0f4e573b1e < https://github.com/phpmyadmin/phpmyadmin/commit/0f4e573b1e0bcb743334e481810… as there should not be any need to manually backup and restore GLOBALS['dbi'] as backupGlobals directive is now active. Somehow failing to restore the GLOBALS['dbi'] would result in other tests using the mocked database interface leading to test failures related to database interface. I will revert the commit until the cause of this is properly investigated. Thanks Madhura, I am now down to three failures (with PHP 5.4.10). Am I alone? I've included the errors (3) and failures (1) I get with PHP 5.3.5 and PHPUnit (3.7.13) below. But when running the unit tests I also get 3 PHP Notices that variable $cfg is not defined in libraries/charset_conversion.lib.php (lines 18, 32, 39) [exec] There were 3 errors: [exec] [exec] 1) PMA_Charset_Conversion_Test::testCharsetConversion [exec] iconv(): Detected an illegal character in input string [exec] [exec] /var/www/phpmyadmin/libraries/iconv_wrapper.lib.php:101 [exec] /var/www/phpmyadmin/libraries/charset_conversion.lib.php:91 [exec] /var/www/phpmyadmin/test/libraries/PMA_charset_conversion_test.php:68 [exec] [exec] 2) PMA_Iconv_Wrapper_Test::testIconvWrapper with data set #0 ('UTF-8', 'ISO-8859-1//IGNORE', 'UTF-8', 'ISO-8859-1//IGNORE', 'Euro Symbol: €') [exec] iconv(): Detected an illegal character in input string [exec] [exec] /var/www/phpmyadmin/test/libraries/PMA_iconv_wrapper_test.php:39 [exec] [exec] 3) PMA_Iconv_Wrapper_Test::testIconvWrapper with data set #1 ('UTF-8', 'ISO-8859-1//IGNORE//TRANSLIT', 'UTF-8', 'ISO-8859-1//IGNORE', 'Euro Symbol: €') [exec] iconv(): Detected an illegal character in input string [exec] [exec] /var/www/phpmyadmin/test/libraries/PMA_iconv_wrapper_test.php:39 [exec] [exec] -- [exec] [exec] There was 1 failure: [exec] [exec] 1) ImportShp_Test::testGetProperties [exec] Failed asserting that null matches expected 'text/plain'. [exec] [exec] /var/www/phpmyadmin/test/classes/plugin/import/ImportShp_test.php:75 -- Kind regards, Dieter Adriaenssens ------------------------------------------------------------------------------ This SF.net email is sponsored by Windows: Build for Windows Store. http://p.sf.net/sfu/windows-dev2dev _______________________________________________ Phpmyadmin-devel mailing list Phpmyadmin-devel(a)lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel Two of those tests, which are throwing the iconv error were written by me. I tested them on OS X and they seem to work fine here. I am wondering what could be the issue. It looks like IBM AIX is using some different charset names and setting

Hi, On Monday, 17 June 2013 at 2:35 PM, Madhura Jayaratne wrote: On Mon, Jun 17, 2013 at 11:25 AM, Ayush Chaudhary &lt;ayushchd(a)gmail.com&gt;wrote;wrote: Hi On Monday, 17 June 2013 at 1:16 AM, Dieter Adriaenssens wrote: 2013/6/16 Marc Delisle &lt;marc(a)infomarc.info&gt;fo>: Le 2013-06-16 12:27, Madhura Jayaratne a écrit : On Sat, Jun 15, 2013 at 9:04 PM, Marc Delisle &lt;marc(a)infomarc.info <mailto:marc@infomarc.info &lt;marc(a)infomarc.info&gt;&gt;&gt; wrote: Le 2013-06-15 11:20, Marc Delisle a écrit : seem to be somewhere. (phpunit.xml?) any db Git bisect tells me that this commit is the one that produced 12 failures: https://github.com/phpmyadmin/phpmyadmin/commit/0f4e573b1e0bcb743334e481810… Note that when bisecting, I marked as good other commits that were producing a smaller number of failures. Looks like the failures are due to some problems in backing up globals. I made the commit 0f4e573b1e < https://github.com/phpmyadmin/phpmyadmin/commit/0f4e573b1e0bcb743334e481810… as there should not be any need to manually backup and restore GLOBALS['dbi'] as backupGlobals directive is now active. Somehow failing to restore the GLOBALS['dbi'] would result in other tests using the mocked database interface leading to test failures related to database interface. I will revert the commit until the cause of this is properly investigated. Thanks Madhura, I am now down to three failures (with PHP 5.4.10). Am I alone? I've included the errors (3) and failures (1) I get with PHP 5.3.5 and PHPUnit (3.7.13) below. But when running the unit tests I also get 3 PHP Notices that variable $cfg is not defined in libraries/charset_conversion.lib.php (lines 18, 32, 39) [exec] There were 3 errors: [exec] [exec] 1) PMA_Charset_Conversion_Test::testCharsetConversion [exec] iconv(): Detected an illegal character in input string [exec] [exec] /var/www/phpmyadmin/libraries/iconv_wrapper.lib.php:101 [exec] /var/www/phpmyadmin/libraries/charset_conversion.lib.php:91 [exec] /var/www/phpmyadmin/test/libraries/PMA_charset_conversion_test.php:68 [exec] [exec] 2) PMA_Iconv_Wrapper_Test::testIconvWrapper with data set #0 ('UTF-8', 'ISO-8859-1//IGNORE', 'UTF-8', 'ISO-8859-1//IGNORE', 'Euro Symbol: €') [exec] iconv(): Detected an illegal character in input string [exec] [exec] /var/www/phpmyadmin/test/libraries/PMA_iconv_wrapper_test.php:39 [exec] [exec] 3) PMA_Iconv_Wrapper_Test::testIconvWrapper with data set #1 ('UTF-8', 'ISO-8859-1//IGNORE//TRANSLIT', 'UTF-8', 'ISO-8859-1//IGNORE', 'Euro Symbol: €') [exec] iconv(): Detected an illegal character in input string [exec] [exec] /var/www/phpmyadmin/test/libraries/PMA_iconv_wrapper_test.php:39 [exec] [exec] -- [exec] [exec] There was 1 failure: [exec] [exec] 1) ImportShp_Test::testGetProperties [exec] Failed asserting that null matches expected 'text/plain'. [exec] [exec] /var/www/phpmyadmin/test/classes/plugin/import/ImportShp_test.php:75 -- Kind regards, Dieter Adriaenssens ------------------------------------------------------------------------------ This SF.net email is sponsored by Windows: Build for Windows Store. http://p.sf.net/sfu/windows-dev2dev _______________________________________________ Phpmyadmin-devel mailing list Phpmyadmin-devel(a)lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel Two of those tests, which are throwing the iconv error were written by me. I tested them on OS X and they seem to work fine here. I am wondering what could be the issue. It looks like IBM AIX is using some different charset names and setting $GLOBALS['PMA_recoding_engine'] to PMA_CHARSET_ICONV_AIX in testCharsetConversion() would result in the character set names being mapped to IBM AIX compliant formats. So it's possible that those names would fail in some other environment. Same applies for PMA_aix_iconv_wrapper function. However to test the logic in there is suggest to split the charset name mapping part to some other function. You mean for IBM AIX compliance, we should check only the mapping logic and leave out the actual execution as that is machine dependent?