adjust cs-fixer config

.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);

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

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] = [];
-//        EventManager::$events[CQCommand::class] = [];
+        MessageUtil::addShortCommand('test', 'test');
-//        MessageUtil::addShortCommand('test', 'test');
+        $this->assertCount(1, EventManager::$events[CQCommand::class]);
-//        $this->assertCount(1, EventManager::$events[CQCommand::class]);
-        $this->assertTrue(true);
     }
 
     /**
      * @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));
     }

tests/bootstrap.php

@@ -1,20 +1,5 @@
 <?php
 
-use ZM\Config\ZMConfig;
+declare(strict_types=1);
-use ZM\Console\Console;
-use ZM\Store\LightCacheInside;
-use ZM\Store\ZMAtomic;
-use ZM\Utils\DataProvider;
-use ZM\Utils\Terminal;
 
 require_once __DIR__ . '/../vendor/autoload.php';
-
-//ZMConfig::setDirectory(WORKING_DIR . '/config/');
-//ZMConfig::setEnv();
-//if (ZMConfig::get('global') === false) {
-//    die (zm_internal_errcode('E00007') . 'Global config load failed: ' . ZMConfig::$last_error . "\nError path: " . DataProvider::getSourceRootDir() . "\nPlease init first!\nSee: https://github.com/zhamao-robot/zhamao-framework/issues/37\n");
-//}
-//LightCacheInside::init();
-//ZMAtomic::init();
-//Terminal::init();
-//Console::setLevel(4);