diff --git a/.php-cs-fixer.php b/.php-cs-fixer.php index 4ff477c3..7f9f3f20 100644 --- a/.php-cs-fixer.php +++ b/.php-cs-fixer.php @@ -65,11 +65,11 @@ return (new PhpCsFixer\Config()) 'standardize_not_equals' => true, 'multiline_comment_opening_closing' => true, 'phpdoc_summary' => false, + 'php_unit_test_class_requires_covers' => false, ]) ->setFinder( PhpCsFixer\Finder::create() - ->exclude('vendor') - ->exclude('docs') ->in(__DIR__ . '/src') + ->in(__DIR__ . '/tests') ) ->setUsingCache(false); diff --git a/tests/ZM/Utils/DataProviderTest.php b/tests/ZM/Utils/DataProviderTest.php index fcce5bdb..84b8178a 100644 --- a/tests/ZM/Utils/DataProviderTest.php +++ b/tests/ZM/Utils/DataProviderTest.php @@ -4,9 +4,12 @@ declare(strict_types=1); namespace Tests\ZM\Utils; -use ZM\Utils\DataProvider; use PHPUnit\Framework\TestCase; +use ZM\Utils\DataProvider; +/** + * @internal + */ class DataProviderTest extends TestCase { public function testGetSourceRootDir(): void diff --git a/tests/ZM/Utils/MessageUtilTest.php b/tests/ZM/Utils/MessageUtilTest.php index 3cf45548..e4a07dd0 100644 --- a/tests/ZM/Utils/MessageUtilTest.php +++ b/tests/ZM/Utils/MessageUtilTest.php @@ -12,25 +12,20 @@ use ZM\Utils\MessageUtil; /** * @internal - * @coversNothing */ class MessageUtilTest extends TestCase { public function testAddShortCommand(): void { - // 此处需要进行 Worker 间通信,无法测试 -// EventManager::$events[CQCommand::class] = []; -// MessageUtil::addShortCommand('test', 'test'); -// $this->assertCount(1, EventManager::$events[CQCommand::class]); - $this->assertTrue(true); + EventManager::$events[CQCommand::class] = []; + MessageUtil::addShortCommand('test', 'test'); + $this->assertCount(1, EventManager::$events[CQCommand::class]); } /** * @dataProvider providerTestContainsImage - * @param mixed $msg - * @param mixed $expected */ - public function testContainsImage($msg, $expected): void + public function testContainsImage(string $msg, bool $expected): void { $this->assertEquals($expected, MessageUtil::containsImage($msg)); } @@ -61,10 +56,8 @@ class MessageUtilTest extends TestCase /** * @dataProvider providerTestArrayToStr - * @param mixed $array - * @param mixed $expected */ - public function testArrayToStr($array, $expected): void + public function testArrayToStr(array $array, string $expected): void { $this->assertEquals($expected, MessageUtil::arrayToStr($array)); } @@ -94,10 +87,8 @@ class MessageUtilTest extends TestCase /** * @dataProvider providerTestIsAtMe - * @param mixed $msg - * @param mixed $expected */ - public function testIsAtMe($msg, $expected): void + public function testIsAtMe(string $msg, bool $expected): void { $this->assertEquals($expected, MessageUtil::isAtMe($msg, 123456789)); } @@ -125,10 +116,8 @@ class MessageUtilTest extends TestCase /** * @dataProvider providerTestStrToArray - * @param mixed $str - * @param mixed $expected */ - public function testStrToArray($str, $expected): void + public function testStrToArray(string $str, array $expected): void { $this->assertEquals($expected, MessageUtil::strToArray($str)); } @@ -150,10 +139,8 @@ class MessageUtilTest extends TestCase /** * @dataProvider providerTestSplitCommand - * @param mixed $msg - * @param mixed $expected */ - public function testSplitCommand($msg, $expected): void + public function testSplitCommand(string $msg, array $expected): void { $this->assertEquals($expected, MessageUtil::splitCommand($msg)); } diff --git a/tests/bootstrap.php b/tests/bootstrap.php index 8efe956d..a075e1e8 100644 --- a/tests/bootstrap.php +++ b/tests/bootstrap.php @@ -1,20 +1,5 @@