diff --git a/libraries/ServerStatusData.class.php b/libraries/ServerStatusData.class.php index 17ed188dbf..18676bf42a 100644 --- a/libraries/ServerStatusData.class.php +++ b/libraries/ServerStatusData.class.php @@ -6,8 +6,10 @@ * * @package PhpMyAdmin */ - -require_once 'libraries/common.inc.php'; +if (!defined('TESTSUITE')) { + //the TESTSUITE has already included common.inc.php + require_once 'libraries/common.inc.php'; +} /** * This class provides data about the server status diff --git a/test/libraries/PMA_server_binlog_test.php b/test/libraries/PMA_server_binlog_test.php index 478fef3bbd..568c8ce600 100644 --- a/test/libraries/PMA_server_binlog_test.php +++ b/test/libraries/PMA_server_binlog_test.php @@ -20,6 +20,13 @@ require_once 'libraries/sanitizing.lib.php'; require_once 'libraries/sqlparser.lib.php'; require_once 'libraries/js_escape.lib.php'; +/** + * PMA_ServerBinlog_Test class + * + * this class is for testing server_bin_log.lib.php functions + * + * @package PhpMyAdmin-test + */ class PMA_ServerBinlog_Test extends PHPUnit_Framework_TestCase { /** diff --git a/test/libraries/PMA_server_collations_test.php b/test/libraries/PMA_server_collations_test.php index aad4eccb39..8c94411cb8 100644 --- a/test/libraries/PMA_server_collations_test.php +++ b/test/libraries/PMA_server_collations_test.php @@ -39,6 +39,13 @@ require_once 'libraries/server_common.inc.php'; require_once 'libraries/mysql_charsets.inc.php'; require_once 'libraries/server_collations.lib.php'; +/** + * PMA_ServerBinlog_Test class + * + * this class is for testing server_collations.lib.php functions + * + * @package PhpMyAdmin-test + */ class PMA_ServerCollations_Test extends PHPUnit_Framework_TestCase { /** diff --git a/test/libraries/PMA_server_common_test.php b/test/libraries/PMA_server_common_test.php index ce2c20788a..1c39b31dca 100644 --- a/test/libraries/PMA_server_common_test.php +++ b/test/libraries/PMA_server_common_test.php @@ -15,6 +15,13 @@ require_once 'libraries/url_generating.lib.php'; require_once 'libraries/server_common.lib.php'; require_once 'libraries/Theme.class.php'; +/** + * PMA_ServerCommon_Test class + * + * this class is for testing server_common.lib.php functions + * + * @package PhpMyAdmin-test + */ class PMA_ServerCommon_Test extends PHPUnit_Framework_TestCase { /** diff --git a/test/libraries/PMA_server_engines_test.php b/test/libraries/PMA_server_engines_test.php index 50702d585f..9943969962 100644 --- a/test/libraries/PMA_server_engines_test.php +++ b/test/libraries/PMA_server_engines_test.php @@ -22,6 +22,13 @@ require_once 'libraries/sqlparser.lib.php'; require_once 'libraries/js_escape.lib.php'; require_once 'libraries/StorageEngine.class.php'; +/** + * PMA_ServerEngines_Test class + * + * this class is for testing server_engines.lib.php functions + * + * @package PhpMyAdmin-test + */ class PMA_ServerEngines_Test extends PHPUnit_Framework_TestCase { /** diff --git a/test/libraries/PMA_server_plugins_test.php b/test/libraries/PMA_server_plugins_test.php index 2a35513118..195af93624 100644 --- a/test/libraries/PMA_server_plugins_test.php +++ b/test/libraries/PMA_server_plugins_test.php @@ -20,6 +20,13 @@ require_once 'libraries/sanitizing.lib.php'; require_once 'libraries/sqlparser.lib.php'; require_once 'libraries/js_escape.lib.php'; +/** + * PMA_ServerPlugins_Test class + * + * this class is for testing server_plugins.lib.php functions + * + * @package PhpMyAdmin-test + */ class PMA_ServerPlugins_Test extends PHPUnit_Framework_TestCase { /** diff --git a/test/libraries/PMA_server_status_monitor_test.php b/test/libraries/PMA_server_status_monitor_test.php new file mode 100644 index 0000000000..1edd2f9d3a --- /dev/null +++ b/test/libraries/PMA_server_status_monitor_test.php @@ -0,0 +1,220 @@ +getMockBuilder('PMA_DatabaseInterface') + ->disableOriginalConstructor() + ->getMock(); + + //this data is needed when PMA_ServerStatusData constructs + $server_status = array( + "Aborted_clients" => "0", + "Aborted_connects" => "0", + "Com_delete_multi" => "0", + "Com_create_function" => "0", + "Com_empty_query" => "0", + ); + + $server_variables= array( + "auto_increment_increment" => "1", + "auto_increment_offset" => "1", + "automatic_sp_privileges" => "ON", + "back_log" => "50", + "big_tables" => "OFF", + ); + + $dbi->expects($this->at(0))->method('fetchResult') + ->with('SHOW GLOBAL STATUS', 0, 1) + ->will($this->returnValue($server_status)); + + $server_variables = array(); + $dbi->expects($this->at(1))->method('fetchResult') + ->with('SHOW GLOBAL VARIABLES', 0, 1) + ->will($this->returnValue($server_variables)); + + $GLOBALS['dbi'] = $dbi; + + $this->ServerStatusData = new PMA_ServerStatusData(); + } + + /** + * Test for PMA_getHtmlForMonitor + * + * @return void + */ + public function testPMAGetHtmlForMonitor() + { + //Call the test function + $html = PMA_getHtmlForMonitor($this->ServerStatusData); + + //validate 1: PMA_getHtmlForTabLinks + $this->assertContains( + '