Hi, config.inc.php does not exist in pMA by default so what's the purpose of testing for the same?
If we replace it by config.sample.inc.php which exists, we can test for file permissions! behavior of config.sample.inc.php will be similar to how config.inc.php will behave in real scenario!
Minhaz cistoner.org
On Wed, Mar 5, 2014 at 6:43 PM, Michal Čihař michal@cihar.com wrote:
Hi
Dne Wed, 5 Mar 2014 17:49:54 +0530 Minhaz A V minhazav@gmail.com napsal(a):
In the setUp() of the test class *PMA_ConfigTest *( ./test/classes/PMA_ConfigTest.php ) two objects are created one for
testing
purpose ($object variable of class) and other as a global variable
the test object is initialized with no parameter, while the global one is initialized with "*config.inc.php*" as a parameter, Although it should be initialized with *"config.sample.inc.php" *as this file exist in codebase while former does not.
Why do you think it should be initialized with existing file? It should be IMHO working configuration, not the sample one.
But there is a problem I'm facing here, for the global version the assertion for permissions is failing, while it should not .... I can't understand this. Can anyone help?
I think that doing assertions on global version is simply wrong.
-- Michal Čihař | http://cihar.com | http://phpmyadmin.net
Subversion Kills Productivity. Get off Subversion & Make the Move to Perforce. With Perforce, you get hassle-free workflows. Merge that actually works. Faster operations. Version large binaries. Built-in WAN optimization and the freedom to use Git, Perforce or both. Make the move to Perforce.
http://pubads.g.doubleclick.net/gampad/clk?id=122218951&iu=/4140/ostg.cl... _______________________________________________ Phpmyadmin-devel mailing list Phpmyadmin-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel