diff --git a/test/classes/PMA_DisplayResults_test.php b/test/classes/PMA_DisplayResults_test.php index 948add1f9e..657836056c 100644 --- a/test/classes/PMA_DisplayResults_test.php +++ b/test/classes/PMA_DisplayResults_test.php @@ -44,7 +44,7 @@ class PMA_DisplayResults_Test extends PHPUnit_Framework_TestCase $GLOBALS['lang'] = 'en'; $GLOBALS['server'] = 0; $GLOBALS['text_dir'] = 'ltr'; - require_once 'libraries/Response.class.php'; + include_once 'libraries/Response.class.php'; } /** @@ -78,8 +78,8 @@ class PMA_DisplayResults_Test extends PHPUnit_Framework_TestCase /** * Test for setting display mode * - * @param string $the_disp_mode the synthetic value for display_mode (see a - * few lines above for explanations) + * @param string $the_disp_mode the synthetic value for display_mode (see + * a few lines above for explanations) * @param integer $the_total the total number of rows returned by the SQL * query without any programmatically appended * LIMIT clause @@ -554,11 +554,11 @@ class PMA_DisplayResults_Test extends PHPUnit_Framework_TestCase } /** - * Provide data for testGetOperationLinksForVerticleTableCase1 + * Provide data for testGetOperationLinksForVerticleTable * * @return array parameters and output */ - public function dataProviderForTestGetOperationLinksForVerticleTableCase1() + public function dataProviderForTestGetOperationLinksForVerticleTable() { return array( array( @@ -566,146 +566,37 @@ class PMA_DisplayResults_Test extends PHPUnit_Framework_TestCase ' ' - ) - ); - } - - /** - * Test for _getOperationLinksForVerticleTable - case 1 - * - * @param string $operation edit/copy/delete - * @param string $output output of _getOperationLinksForVerticleTable - * - * @return void - * - * @dataProvider dataProviderForTestGetOperationLinksForVerticleTableCase1 - */ - public function testGetOperationLinksForVerticleTableCase1( - $operation, $output - ) { - $vertical_display = array( - 'row_delete' => array(), - 'textbtn' => '\n \n \n', - 'edit' => array(), - 'copy' => array( - '\nCopy Copy\n\nCopy Copy\n array( - '\n\n\nCopy Copy\n\nCopy Copy\n -' - ) - ); - } - - /** - * Test for _getOperationLinksForVerticleTable - case 2 - * - * @param string $operation edit/copy/delete - * @param string $output output of _getOperationLinksForVerticleTable - * - * @return void - * - * @dataProvider dataProviderForTestGetOperationLinksForVerticleTableCase2 - */ - public function testGetOperationLinksForVerticleTableCase2( - $operation, $output - ) { - $vertical_display = array( - 'row_delete' => array(), - 'textbtn' => '\n \n \n', - 'edit' => array(), - 'copy' => array( - '\nCopy Copy\n\nCopy Copy\n\nCOPY1COPY2\n" ), - 'delete' => array( - '\n\n\n\n\nDELETE1DELETE2\n" + ), ); } /** - * Test for _getOperationLinksForVerticleTable - case 3 + * Test for _getOperationLinksForVerticleTable * * @param string $operation edit/copy/delete * @param string $output output of _getOperationLinksForVerticleTable * * @return void * - * @dataProvider dataProviderForTestGetOperationLinksForVerticleTableCase3 + * @dataProvider dataProviderForTestGetOperationLinksForVerticleTable */ - public function testGetOperationLinksForVerticleTableCase3( + public function testGetOperationLinksForVerticleTable( $operation, $output ) { $vertical_display = array( 'row_delete' => array(), 'textbtn' => '\n \n \n', 'edit' => array(), - 'copy' => array( - '\nCopy Copy\n\nCopy Copy\n array( - '\n\n Delete Delete
DELETE FROM `data`.`new` WHERE `new`.`id` = 1
' - ) - ); - } - - /** - * Test for _getCheckboxAndLinks - case 1 - * - * @param string $position the position of the checkbox and links - * @param string $del_url delete url - * @param array $is_display array with explicit indexes for all the - * display elements - * @param string $row_no row number - * @param string $where_clause where clause - * @param string $where_clause_html url encoded where clause - * @param array $condition_array array of conditions in the where clause - * @param string $del_query delete query - * @param string $id_suffix suffix for the id - * @param string $edit_url edit url - * @param string $copy_url copy url - * @param string $class css classes for the td elements - * @param string $edit_str text for the edit link - * @param string $copy_str text for the copy link - * @param string $del_str text for the delete link - * @param string $js_conf text for the JS confirmation - * @param string $output output of _getCheckboxAndLinks - * - * @return void - * - * @dataProvider dataProviderForGetCheckboxAndLinksCase1 - */ - public function testGetCheckboxAndLinksCase1( - $position, $del_url, $is_display, $row_no, $where_clause, - $where_clause_html, $condition_array, $del_query, $id_suffix, $edit_url, - $copy_url, $class, $edit_str, $copy_str, $del_str, $js_conf, $output - ) { - $this->assertEquals( - $output, - $this->_callPrivateFunction( - '_getCheckboxAndLinks', - array( - $position, $del_url, $is_display, $row_no, $where_clause, - $where_clause_html, $condition_array, $del_query, - $id_suffix, $edit_url, $copy_url, $class, $edit_str, - $copy_str, $del_str, $js_conf - ) - ) - ); - } - - /** - * Data provider for testGetCheckboxAndLinksCase2 - * - * @return array parameters and output - */ - public function dataProviderForGetCheckboxAndLinksCase2() - { - return array( + ), array( PMA_DisplayResults::POSITION_RIGHT, 'sql.php?db=data&table=new&sql_query=DELETE+FROM+%60data%60.%60new%60+WHERE+%60new%60.%60id%60+%3D+1&message_to_show=The+row+has+been+deleted&goto=sql.php%3Fdb%3Ddata%26table%3Dnew%26sql_query%3DSELECT%2B%252A%2BFROM%2B%2560new%2560%26message_to_show%3DThe%2Brow%2Bhas%2Bbeen%2Bdeleted%26goto%3Dtbl_structure.php%26token%3Dae4c6d18375f446dfa068420c1f6a4e8&token=ae4c6d18375f446dfa068420c1f6a4e8', @@ -1220,63 +1024,7 @@ class PMA_DisplayResults_Test extends PHPUnit_Framework_TestCase Edit Edit ' - ) - ); - } - - /** - * Test for _getCheckboxAndLinks - case 2 - * - * @param string $position the position of the checkbox and links - * @param string $del_url delete url - * @param array $is_display array with explicit indexes for all the - * display elements - * @param string $row_no row number - * @param string $where_clause where clause - * @param string $where_clause_html url encoded where clause - * @param array $condition_array array of conditions in the where clause - * @param string $del_query delete query - * @param string $id_suffix suffix for the id - * @param string $edit_url edit url - * @param string $copy_url copy url - * @param string $class css classes for the td elements - * @param string $edit_str text for the edit link - * @param string $copy_str text for the copy link - * @param string $del_str text for the delete link - * @param string $js_conf text for the JS confirmation - * @param string $output output of _getCheckboxAndLinks - * - * @return void - * - * @dataProvider dataProviderForGetCheckboxAndLinksCase2 - */ - public function testGetCheckboxAndLinksCase2( - $position, $del_url, $is_display, $row_no, $where_clause, - $where_clause_html, $condition_array, $del_query, $id_suffix, $edit_url, - $copy_url, $class, $edit_str, $copy_str, $del_str, $js_conf, $output - ) { - $this->assertEquals( - $output, - $this->_callPrivateFunction( - '_getCheckboxAndLinks', - array( - $position, $del_url, $is_display, $row_no, $where_clause, - $where_clause_html, $condition_array, $del_query, - $id_suffix, $edit_url, $copy_url, $class, $edit_str, - $copy_str, $del_str, $js_conf - ) - ) - ); - } - - /** - * Data provider for testGetCheckboxAndLinksCase3 - * - * @return array parameters and output - */ - public function dataProviderForGetCheckboxAndLinksCase3() - { - return array( + ), array( PMA_DisplayResults::POSITION_NONE, 'sql.php?db=data&table=new&sql_query=DELETE+FROM+%60data%60.%60new%60+WHERE+%60new%60.%60id%60+%3D+1&message_to_show=The+row+has+been+deleted&goto=sql.php%3Fdb%3Ddata%26table%3Dnew%26sql_query%3DSELECT%2B%252A%2BFROM%2B%2560new%2560%26message_to_show%3DThe%2Brow%2Bhas%2Bbeen%2Bdeleted%26goto%3Dtbl_structure.php%26token%3Dae4c6d18375f446dfa068420c1f6a4e8&token=ae4c6d18375f446dfa068420c1f6a4e8', @@ -1306,12 +1054,12 @@ class PMA_DisplayResults_Test extends PHPUnit_Framework_TestCase 'Delete Delete', 'DELETE FROM `data`.`new` WHERE `new`.`id` = 1', ' ' - ) + ), ); } /** - * Test for _getCheckboxAndLinks - case 3 + * Test for _getCheckboxAndLinks * * @param string $position the position of the checkbox and links * @param string $del_url delete url @@ -1334,9 +1082,9 @@ class PMA_DisplayResults_Test extends PHPUnit_Framework_TestCase * * @return void * - * @dataProvider dataProviderForGetCheckboxAndLinksCase3 + * @dataProvider dataProviderForGetCheckboxAndLinks */ - public function testGetCheckboxAndLinksCase3( + public function testGetCheckboxAndLinks( $position, $del_url, $is_display, $row_no, $where_clause, $where_clause_html, $condition_array, $del_query, $id_suffix, $edit_url, $copy_url, $class, $edit_str, $copy_str, $del_str, $js_conf, $output