[TESTS] Add test annotations to core classes

This commit is contained in:
Hugo Sales 2021-08-08 00:37:02 +00:00 committed by Diogo Peralta Cordeiro
parent 7cace2051f
commit 8e627f2c18
Signed by: diogo
GPG Key ID: 18D2D35001FBFAB0
5 changed files with 28 additions and 13 deletions

View File

@ -25,7 +25,6 @@ use App\Core\DB\DB;
use function App\Core\I18n\_m;
use App\Entity\Attachment;
use App\Util\Common;
use App\Util\Exception\ClientException;
use App\Util\Exception\DuplicateFoundException;
use App\Util\Exception\NoSuchFileException;
use App\Util\Exception\NotFoundException;
@ -161,7 +160,9 @@ class GSFile
autoLastModified: true
);
if (Common::config('site', 'x_static_delivery')) {
// @codeCoverageIgnoreStart
$response->trustXSendfileTypeHeader();
// @codeCoverageIgnoreEnd
}
return $response;
} else {
@ -184,8 +185,10 @@ class GSFile
case 1:
return $res[0];
default:
// @codeCoverageIgnoreStart
Log::error('Media query returned more than one result for identifier: \"' . $id . '\"');
throw new ClientException(_m('Internal server error'));
throw new ServerException(_m('Internal server error'));
// @codeCoverageIgnoreEnd
}
}

View File

@ -43,6 +43,12 @@ abstract class Log
self::$logger = $l;
}
/**
* Log a critical error when a really unexpected exception occured. This indicates a bug in the software
*
* @throws ServerException
* @codeCoverageIgnore
*/
public static function unexpected_exception(\Exception $e)
{
$backtrace = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS, 2);

View File

@ -48,6 +48,7 @@ abstract class Queue
* @param string $queue
* @param bool $high
* @param array $stamps
* @codeCoverageIgnore
*/
public static function enqueue($message, string $queue, bool $high = false, array $stamps = [])
{

View File

@ -35,6 +35,9 @@ abstract class FormFields
];
}
/**
* @codeCoverageIgnore
*/
public static function password(array $options = []): array
{
['password', PasswordType::class, [

View File

@ -146,18 +146,20 @@ class Nickname
throw new NicknameReservedException();
} elseif ($check_already_used) {
switch ($which) {
case self::CHECK_LOCAL_USER:
$lu = LocalUser::findByNicknameOrEmail($nickname, email: '');
if ($lu !== null) {
throw new NicknameTakenException($lu->getActor());
case self::CHECK_LOCAL_USER:
$lu = LocalUser::findByNicknameOrEmail($nickname, email: '');
if ($lu !== null) {
throw new NicknameTakenException($lu->getActor());
}
break;
// @codeCoverageIgnoreStart
case self::CHECK_LOCAL_GROUP:
throw new \NotImplementedException();
break;
default:
throw new \InvalidArgumentException();
// @codeCoverageIgnoreEnd
}
break;
case self::CHECK_LOCAL_GROUP:
throw new \NotImplementedException();
break;
default:
throw new \InvalidArgumentException();
}
}
}