From 66807421bd286ca782ff0f2bdb98a69e101c7dcd Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Michal=20=C4=8Ciha=C5=99?= Date: Wed, 12 Jul 2017 17:38:14 +0200 Subject: [PATCH] Remove no longer needed code from test MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Apparenty nothing really needs PMA_USR_BROWSER_AGENT here, so avoid setting it. This removes need for runkit here. Issue #13484 Signed-off-by: Michal Čihař --- .../classes/plugin/auth/AuthenticationConfigTest.php | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/test/classes/plugin/auth/AuthenticationConfigTest.php b/test/classes/plugin/auth/AuthenticationConfigTest.php index 924bc576ae..5feb64d6b6 100644 --- a/test/classes/plugin/auth/AuthenticationConfigTest.php +++ b/test/classes/plugin/auth/AuthenticationConfigTest.php @@ -93,15 +93,6 @@ class AuthenticationConfigTest extends PMATestCase $GLOBALS['cfg']['Servers'] = array(1); $GLOBALS['allowDeny_forbidden'] = false; $GLOBALS['collation_connection'] = 'utf-8'; - if (!defined('PMA_USR_BROWSER_AGENT')) { - define('PMA_USR_BROWSER_AGENT', 'chrome'); - - $removeConstant = true; - - if (! PMA_HAS_RUNKIT) { - $this->markTestSkipped('Cannot remove constant'); - } - } $dbi = $this->getMockBuilder('PhpMyAdmin\DatabaseInterface') ->disableOriginalConstructor() @@ -142,8 +133,5 @@ class AuthenticationConfigTest extends PMATestCase . 'class="button disableAjax">Retry to connect', $html ); - if ($removeConstant) { - runkit_constant_remove('PMA_USR_BROWSER_AGENT'); - } } }