diff --git a/tests/php-unit-tests/src/BaseTestCase/ItopTestCase.php b/tests/php-unit-tests/src/BaseTestCase/ItopTestCase.php index 6e30f5ed4..59bcf8810 100644 --- a/tests/php-unit-tests/src/BaseTestCase/ItopTestCase.php +++ b/tests/php-unit-tests/src/BaseTestCase/ItopTestCase.php @@ -8,7 +8,6 @@ namespace Combodo\iTop\Test\UnitTest; use CMDBSource; -use DateTime; use DeprecatedCallsLog; use MySQLTransactionNotClosedException; use ReflectionMethod; @@ -649,7 +648,7 @@ abstract class ItopTestCase extends KernelTestCase * @param $bXDebugEnabled * @return string */ - protected function CallItopUrl($sUrl, ?array $aPostFields = [], ?array $aCurlOptions = [], $bXDebugEnabled = false): string + protected function CallUrl($sUrl, ?array $aPostFields = [], ?array $aCurlOptions = [], $bXDebugEnabled = false): string { $ch = curl_init(); if ($bXDebugEnabled) { @@ -658,20 +657,14 @@ abstract class ItopTestCase extends KernelTestCase curl_setopt($ch, CURLOPT_URL, $sUrl); curl_setopt($ch, CURLOPT_POST, 1);// set post data to true - if (!is_array($aPostFields)) { - var_dump($aPostFields); - } - curl_setopt($ch, CURLOPT_POSTFIELDS, $aPostFields); curl_setopt($ch, CURLOPT_RETURNTRANSFER, true); // Force disable of certificate check as most of dev / test env have a self-signed certificate curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, false); curl_setopt($ch, CURLOPT_SSL_VERIFYHOST, 0); - - var_dump($aCurlOptions); curl_setopt_array($ch, $aCurlOptions); + curl_setopt($ch, CURLOPT_POSTFIELDS, $aPostFields); $sOutput = curl_exec($ch); - //\IssueLog::Info("$sUrl error code:", null, ['error' => curl_error($ch)]); $info = curl_getinfo($ch); $this->aLastCurlGetInfo = $info; @@ -687,6 +680,7 @@ abstract class ItopTestCase extends KernelTestCase protected function CallItopUri(string $sUri, ?array $aPostFields = [], ?array $aCurlOptions = [], $bXDebugEnabled = false): string { $sUrl = \MetaModel::GetConfig()->Get('app_root_url')."/$sUri"; - return $this->CallItopUrl($sUrl, $aPostFields, $aCurlOptions, $bXDebugEnabled); + + return $this->CallUrl($sUrl, $aPostFields, $aCurlOptions, $bXDebugEnabled); } } diff --git a/tests/php-unit-tests/unitary-tests/application/LoginTest.php b/tests/php-unit-tests/unitary-tests/application/LoginTest.php deleted file mode 100644 index cea8785c7..000000000 --- a/tests/php-unit-tests/unitary-tests/application/LoginTest.php +++ /dev/null @@ -1,62 +0,0 @@ -sConfigPath = MetaModel::GetConfig()->GetLoadedFile(); - $this->sConfigTmpBackupFile = tempnam(sys_get_temp_dir(), "config_"); - file_put_contents($this->sConfigTmpBackupFile, file_get_contents($this->sConfigPath)); - - $oConfig = new \Config($this->sConfigPath); - $this->sLoginMode = "unimplemented_loginmode"; - $oConfig->AddAllowedLoginTypes($this->sLoginMode); - - @chmod($this->sConfigPath, 0770); - $oConfig->WriteToFile(); - @chmod($this->sConfigPath, 0444); - } - - protected function tearDown(): void - { - if (! is_null($this->sConfigTmpBackupFile) && is_file($this->sConfigTmpBackupFile)) { - //put config back - @chmod($this->sConfigPath, 0770); - file_put_contents($this->sConfigPath, file_get_contents($this->sConfigTmpBackupFile)); - @chmod($this->sConfigPath, 0444); - @unlink($this->sConfigTmpBackupFile); - } - parent::tearDown(); - } - - protected function CallItopUrlByCurl($sUri, ?array $aPostFields = []) - { - $ch = curl_init(); - - $sUrl = MetaModel::GetConfig()->Get('app_root_url')."/$sUri"; - curl_setopt($ch, CURLOPT_URL, $sUrl); - if (0 !== sizeof($aPostFields)) { - curl_setopt($ch, CURLOPT_POST, 1);// set post data to true - curl_setopt($ch, CURLOPT_POSTFIELDS, $aPostFields); - } - curl_setopt($ch, CURLOPT_RETURNTRANSFER, true); - $sOutput = curl_exec($ch); - curl_close($ch); - - return $sOutput; - } -} diff --git a/tests/php-unit-tests/unitary-tests/application/query/QueryTest.php b/tests/php-unit-tests/unitary-tests/application/query/QueryTest.php index 63521d74d..efbe9f57d 100644 --- a/tests/php-unit-tests/unitary-tests/application/query/QueryTest.php +++ b/tests/php-unit-tests/unitary-tests/application/query/QueryTest.php @@ -147,7 +147,8 @@ class QueryTest extends ItopDataTestCase CURLOPT_HTTPAUTH => CURLAUTH_BASIC, CURLOPT_USERPWD => self::USER.':'.self::PASSWORD, ]; - return $this->CallItopUrl($url, [], $aCurlOptions); + + return $this->CallUrl($url, [], $aCurlOptions); } /** @inheritDoc */