[Phpmyadmin-devel] Unit tests failing
Hi, I recently merged with upstream. However, a lot of unit tests seem to be failing on my machine. All of the failures seem to be related to database. I do have a database named 'pma_test'. It seems I need to import some SQL and also, set me database authentication details somewhere. It would be great if someone could point out those settings (phpunit.xml?) -- Ayush Chaudhary
Le 2013-06-15 10:36, Ayush Chaudhary a écrit :
Hi,
I recently merged with upstream. However, a lot of unit tests seem to be failing on my machine. All of the failures seem to be related to database. I do have a database named 'pma_test'. It seems I need to import some SQL and also, set me database authentication details somewhere.
It would be great if someone could point out those settings (phpunit.xml?)
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. -- Marc Delisle http://infomarc.info
Le 2013-06-15 11:20, Marc Delisle a écrit :
Le 2013-06-15 10:36, Ayush Chaudhary a écrit :
Hi,
I recently merged with upstream. However, a lot of unit tests seem to be failing on my machine. All of the failures seem to be related to database. I do have a database named 'pma_test'. It seems I need to import some SQL and also, set me database authentication details somewhere.
It would be great if someone could point out those settings (phpunit.xml?)
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.
Git bisect tells me that this commit is the one that produced 12 failures: https://github.com/phpmyadmin/phpmyadmin/commit/0f4e573b1e0bcb743334e4818102... Note that when bisecting, I marked as good other commits that were producing a smaller number of failures. -- Marc Delisle http://infomarc.info
On Sat, Jun 15, 2013 at 9:04 PM, Marc Delisle <marc@infomarc.info> wrote:
Le 2013-06-15 11:20, Marc Delisle a écrit :
Le 2013-06-15 10:36, Ayush Chaudhary a écrit :
Hi,
I recently merged with upstream. However, a lot of unit tests seem to be failing on my machine. All of the failures seem to be related to database. I do have a database named 'pma_test'. It seems I need to import some SQL and also, set me database authentication details somewhere.
It would be great if someone could point out those settings (phpunit.xml?)
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.
Git bisect tells me that this commit is the one that produced 12 failures:
https://github.com/phpmyadmin/phpmyadmin/commit/0f4e573b1e0bcb743334e4818102...
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. -- Thanks and Regards, Madhura Jayaratne
Le 2013-06-16 12:27, Madhura Jayaratne a écrit :
On Sat, Jun 15, 2013 at 9:04 PM, Marc Delisle <marc@infomarc.info <mailto:marc@infomarc.info>> wrote:
Le 2013-06-15 11:20, Marc Delisle a écrit : > Le 2013-06-15 10:36, Ayush Chaudhary a écrit : >> Hi, >> >> I recently merged with upstream. However, a lot of unit tests seem to be >> failing on my machine. All of the failures seem to be related to >> database. I do have a database named 'pma_test'. It seems I need to >> import some SQL and also, set me database authentication details somewhere. >> >> It would be great if someone could point out those settings (phpunit.xml?) > > 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. >
Git bisect tells me that this commit is the one that produced 12 failures:
https://github.com/phpmyadmin/phpmyadmin/commit/0f4e573b1e0bcb743334e4818102...
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.
Thanks Madhura, I am now down to three failures (with PHP 5.4.10). Am I alone? -- Marc Delisle http://infomarc.info
2013/6/16 Marc Delisle <marc@infomarc.info>:
Le 2013-06-16 12:27, Madhura Jayaratne a écrit :
On Sat, Jun 15, 2013 at 9:04 PM, Marc Delisle <marc@infomarc.info <mailto:marc@infomarc.info>> wrote:
Le 2013-06-15 11:20, Marc Delisle a écrit : > Le 2013-06-15 10:36, Ayush Chaudhary a écrit : >> Hi, >> >> I recently merged with upstream. However, a lot of unit tests seem to be >> failing on my machine. All of the failures seem to be related to >> database. I do have a database named 'pma_test'. It seems I need to >> import some SQL and also, set me database authentication details somewhere. >> >> It would be great if someone could point out those settings (phpunit.xml?) > > 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. >
Git bisect tells me that this commit is the one that produced 12 failures:
https://github.com/phpmyadmin/phpmyadmin/commit/0f4e573b1e0bcb743334e4818102...
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.
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
Hi On Monday, 17 June 2013 at 1:16 AM, Dieter Adriaenssens wrote:
2013/6/16 Marc Delisle <marc@infomarc.info (mailto:marc@infomarc.info)>:
Le 2013-06-16 12:27, Madhura Jayaratne a écrit :
On Sat, Jun 15, 2013 at 9:04 PM, Marc Delisle <marc@infomarc.info (mailto:marc@infomarc.info) <mailto:marc@infomarc.info>> wrote:
Le 2013-06-15 11:20, Marc Delisle a écrit :
Le 2013-06-15 10:36, Ayush Chaudhary a écrit :
Hi,
I recently merged with upstream. However, a lot of unit tests seem to be failing on my machine. All of the failures seem to be related to database. I do have a database named 'pma_test'. It seems I need to import some SQL and also, set me database authentication details somewhere.
It would be great if someone could point out those settings (phpunit.xml?)
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.
Git bisect tells me that this commit is the one that produced 12 failures:
https://github.com/phpmyadmin/phpmyadmin/commit/0f4e573b1e0bcb743334e4818102...
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.
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 (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
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. Also, I am now down to the following 2 failures now: There were 2 failures: 1) ImportShp_Test::testGetProperties Failed asserting that null matches expected 'text/plain'. /Users/ayushchd/Sites/htdocs/phpmyadmin/test/classes/plugin/import/ImportShp_test.php:75 /usr/local/php5-20130103-080303/bin/phpunit:46 2) PMA_SQLParser_Test::testParser with data set #3 ('SELECT * from `aaa;', array('SELECT * from `aaa`;', array('alpha_reservedWord', 'SELECT', 6, true), array('punct', '*', 0), array('alpha_reservedWord', 'from', 13, true), array('quote_backtick', '`aaa`', 0), array('punct_queryend', ';', 0), 5), '<div class="notice"><img src="theme/s_notice.png" title="" alt="" /> Automatically appended backtick to the end of query!</div>') Failed asserting that two strings are equal. --- Expected +++ Actual @@ @@ -'<div class="notice"><img src="theme/s_notice.png" title="" alt="" /> Automatically appended backtick to the end of query!</div>' +'' /usr/local/php5-20130103-080303/bin/phpunit:46 -- Please advice. Thanks, Ayush
Hi,
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.
Also, I am now down to the following 2 failures now:
There were 2 failures:
1) ImportShp_Test::testGetProperties Failed asserting that null matches expected 'text/plain'.
I have fixed this error in the latest pull request: https://github.com/adamgsoc2013/phpmyadmin/commit/336d18cdbc95c322e1ab176cd7... still wait for review
/Users/ayushchd/Sites/htdocs/phpmyadmin/test/classes/plugin/import/ImportShp_test.php:75 /usr/local/php5-20130103-080303/bin/phpunit:46
2) PMA_SQLParser_Test::testParser with data set #3 ('SELECT * from `aaa;', array('SELECT * from `aaa`;', array('alpha_reservedWord', 'SELECT', 6, true), array('punct', '*', 0), array('alpha_reservedWord', 'from', 13, true), array('quote_backtick', '`aaa`', 0), array('punct_queryend', ';', 0), 5), '<div class="notice"><img src="theme/s_notice.png" title="" alt="" /> Automatically appended backtick to the end of query!</div>') Failed asserting that two strings are equal. --- Expected +++ Actual @@ @@ -'<div class="notice"><img src="theme/s_notice.png" title="" alt="" /> Automatically appended backtick to the end of query!</div>' +''
/usr/local/php5-20130103-080303/bin/phpunit:46
--
Please advice.
Adam
On Mon, Jun 17, 2013 at 11:25 AM, Ayush Chaudhary <ayushchd@gmail.com>wrote:
Hi
On Monday, 17 June 2013 at 1:16 AM, Dieter Adriaenssens wrote:
2013/6/16 Marc Delisle <marc@infomarc.info>:
Le 2013-06-16 12:27, Madhura Jayaratne a écrit :
On Sat, Jun 15, 2013 at 9:04 PM, Marc Delisle <marc@infomarc.info <mailto:marc@infomarc.info <marc@infomarc.info>>> wrote:
Le 2013-06-15 11:20, Marc Delisle a écrit :
Le 2013-06-15 10:36, Ayush Chaudhary a écrit :
Hi,
I recently merged with upstream. However, a lot of unit tests seem to be failing on my machine. All of the failures seem to be related to database. I do have a database named 'pma_test'. It seems I need to import some SQL and also, set me database authentication details somewhere.
It would be great if someone could point out those settings (phpunit.xml?)
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.
Git bisect tells me that this commit is the one that produced 12 failures:
https://github.com/phpmyadmin/phpmyadmin/commit/0f4e573b1e0bcb743334e4818102...
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.
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@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. -- Thanks and Regards, Madhura Jayaratne
Hi, On Monday, 17 June 2013 at 2:35 PM, Madhura Jayaratne wrote:
On Mon, Jun 17, 2013 at 11:25 AM, Ayush Chaudhary <ayushchd@gmail.com (mailto:ayushchd@gmail.com)> wrote:
Hi
On Monday, 17 June 2013 at 1:16 AM, Dieter Adriaenssens wrote:
2013/6/16 Marc Delisle <marc@infomarc.info (mailto:marc@infomarc.info)>:
Le 2013-06-16 12:27, Madhura Jayaratne a écrit :
On Sat, Jun 15, 2013 at 9:04 PM, Marc Delisle <marc@infomarc.info (mailto:marc@infomarc.info) <mailto:marc@infomarc.info>> wrote:
Le 2013-06-15 11:20, Marc Delisle a écrit :
Le 2013-06-15 10:36, Ayush Chaudhary a écrit : > Hi, > > I recently merged with upstream. However, a lot of unit tests seem to be > failing on my machine. All of the failures seem to be related to > database. I do have a database named 'pma_test'. It seems I need to > import some SQL and also, set me database authentication details somewhere. > > It would be great if someone could point out those settings (phpunit.xml?)
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.
Git bisect tells me that this commit is the one that produced 12 failures:
https://github.com/phpmyadmin/phpmyadmin/commit/0f4e573b1e0bcb743334e4818102...
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.
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 (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
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?
-- Thanks and Regards,
Madhura Jayaratne
------------------------------------------------------------------------------ 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
Thanks, Ayush
On Mon, Jun 17, 2013 at 4:07 PM, Ayush Chaudhary <ayushchd@gmail.com> wrote:
Hi,
On Monday, 17 June 2013 at 2:35 PM, Madhura Jayaratne wrote:
On Mon, Jun 17, 2013 at 11:25 AM, Ayush Chaudhary <ayushchd@gmail.com>wrote:
Hi
On Monday, 17 June 2013 at 1:16 AM, Dieter Adriaenssens wrote:
2013/6/16 Marc Delisle <marc@infomarc.info>:
Le 2013-06-16 12:27, Madhura Jayaratne a écrit :
On Sat, Jun 15, 2013 at 9:04 PM, Marc Delisle <marc@infomarc.info <mailto:marc@infomarc.info <marc@infomarc.info>>> wrote:
Le 2013-06-15 11:20, Marc Delisle a écrit :
Le 2013-06-15 10:36, Ayush Chaudhary a écrit :
Hi,
I recently merged with upstream. However, a lot of unit tests seem to be failing on my machine. All of the failures seem to be related to database. I do have a database named 'pma_test'. It seems I need to import some SQL and also, set me database authentication details somewhere.
It would be great if someone could point out those settings (phpunit.xml?)
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.
Git bisect tells me that this commit is the one that produced 12 failures:
https://github.com/phpmyadmin/phpmyadmin/commit/0f4e573b1e0bcb743334e4818102...
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.
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@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?
Exactly. See https://github.com/phpmyadmin/phpmyadmin/pull/432 -- Thanks and Regards, Madhura Jayaratne
participants (5)
-
adam -
Ayush Chaudhary -
Dieter Adriaenssens -
Madhura Jayaratne -
Marc Delisle