Uts for PMA_server_privileges and fix UT error #588

Merged
merged 1 commit into from Aug 12, 2013

Conversation

Projects
None yet
2 participants
Contributor

xmujay commented Aug 12, 2013

No description provided.

@xmujay xmujay commented on the diff Aug 12, 2013

test/libraries/PMA_server_privileges_test.php
@@ -98,7 +199,7 @@ public function testPMAGetHtmlForExportUserDefinition()
$GLOBALS['cfg']['TextareaRows'] = 'TextareaCols';
list($title, $export)
- = PMA_getHtmlForExportUserDefinition($username, $hostname);
+ = PMA_getListForExportUserDefinition($username, $hostname);
@xmujay

xmujay Aug 12, 2013

Contributor

Here fix UT error

lem9 was assigned Aug 12, 2013

Contributor

xmujay commented Aug 12, 2013

Hi Marc,

This pull request, I add more UT and fix UT error.

The root of error:
I changed a function name when refactoring code, but forget to change it on UT due to they are on my two pull requests. so the trival don't have errors

Sorry for the inconvenience.

Contributor

lem9 commented Aug 12, 2013

Thanks!

@lem9 lem9 added a commit that referenced this pull request Aug 12, 2013

@lem9 lem9 Merge pull request #588 from xmujay/server_privileges_UT
Uts for PMA_server_privileges and fix UT error
114ee14

@lem9 lem9 merged commit 114ee14 into phpmyadmin:master Aug 12, 2013

1 check passed

default The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment