add FileSystemTest

composer.json

@@ -32,6 +32,7 @@
         "brainmaestro/composer-git-hooks": "^3.0",
         "friendsofphp/php-cs-fixer": "^3.2 != 3.7.0",
         "jetbrains/phpstorm-attributes": "^1.0",
+        "mikey179/vfsstream": "^1.6",
         "phpspec/prophecy": "1.x-dev",
         "phpstan/extension-installer": "^1.1",
         "phpstan/phpstan": "^1.1",

src/ZM/Store/FileSystem.php

@@ -129,9 +129,6 @@ class FileSystem
                 $tokens = \PhpToken::tokenize(file_get_contents($path));
                 $found = false;
                 foreach ($tokens as $token) {
-                    if (!$token instanceof \PhpToken) {
-                        continue;
-                    }
                     if ($token->is(T_CLASS)) {
                         $found = true;
                         break;

tests/Trait/HasLogger.php

@@ -4,8 +4,10 @@ declare(strict_types=1);
 
 namespace Tests\Trait;
 
+use Prophecy\Argument;
 use Prophecy\Prophet;
 use Psr\Log\AbstractLogger;
+use Psr\Log\LogLevel;
 
 /**
  * 模拟 Logger 行为
@@ -36,9 +38,21 @@ trait HasLogger
     private function startMockLogger(): void
     {
         $logger = $this->prophet->prophesize(AbstractLogger::class);
-        $logger->log()->will(function ($args) {
+        $levels = [
-            $this->mockLog(...$args);
+            LogLevel::EMERGENCY,
-        });
+            LogLevel::ALERT,
+            LogLevel::CRITICAL,
+            LogLevel::ERROR,
+            LogLevel::WARNING,
+            LogLevel::NOTICE,
+            LogLevel::INFO,
+            LogLevel::DEBUG,
+        ];
+        $log_it = fn (...$args) => $this->mockLog(...$args);
+        foreach ($levels as $level) {
+            $logger->{$level}(Argument::type('string'), Argument::any())->will(fn ($args) => $log_it($level, ...$args));
+        }
+        $logger->log(Argument::in($levels), Argument::type('string'), Argument::any())->will(fn ($args) => $log_it(...$args));
         ob_logger_register($logger->reveal());
     }
 }

tests/Trait/HasVirtualFileSystem.php

@@ -0,0 +1,18 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\Trait;
+
+use org\bovigo\vfs\vfsStream;
+use org\bovigo\vfs\vfsStreamDirectory;
+
+trait HasVirtualFileSystem
+{
+    private vfsStreamDirectory $vfs;
+
+    private function setUpVfs(string $dir = 'root'): void
+    {
+        $this->vfs = vfsStream::setup($dir);
+    }
+}

tests/ZM/Store/FileSystemTest.php

@@ -0,0 +1,124 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\ZM\Store;
+
+use org\bovigo\vfs\vfsStream;
+use Tests\TestCase;
+use Tests\Trait\HasLogger;
+use Tests\Trait\HasVirtualFileSystem;
+use ZM\Store\FileSystem;
+
+/**
+ * @internal
+ */
+class FileSystemTest extends TestCase
+{
+    use HasVirtualFileSystem;
+    use HasLogger;
+
+    protected function setUp(): void
+    {
+        parent::setUp();
+        $this->setUpVfs();
+        $this->startMockLogger();
+    }
+
+    /**
+     * @dataProvider provideTestDetermineRelativePath
+     */
+    public function testDetermineRelativePath(string $path, bool $expected): void
+    {
+        $this->assertSame($expected, FileSystem::isRelativePath($path));
+    }
+
+    public function provideTestDetermineRelativePath(): array
+    {
+        return [
+            'relative' => ['relative/path', true],
+            'absolute' => ['/absolute/path', false],
+            'windows' => ['C:\Windows', !(DIRECTORY_SEPARATOR === '\\')],
+        ];
+    }
+
+    public function testCreateDirectoryWithNoPerm(): void
+    {
+        $old_perm = $this->vfs->getPermissions();
+        $this->vfs->chmod(0000);
+        $this->assertFalse($this->vfs->hasChild('test'));
+        $this->expectExceptionMessageMatches('/无法建立目录/');
+        FileSystem::createDir($this->vfs->url() . '/test');
+        $this->vfs->chmod($old_perm);
+    }
+
+    public function testCreateDirectory(): void
+    {
+        $this->assertFalse($this->vfs->hasChild('test'));
+        FileSystem::createDir($this->vfs->url() . '/test');
+        $this->assertTrue($this->vfs->hasChild('test'));
+    }
+
+    public function testGetReloadableFiles(): void
+    {
+        $files = FileSystem::getReloadableFiles();
+        $this->assertIsArray($files);
+        $this->assertNotEmpty($files);
+    }
+
+    public function testGetClassesPsr4(): void
+    {
+        vfsStream::create([
+            'Foo' => [
+                'Bar.php' => '<?php namespace Foo; class Bar {}',
+                'Baz.php' => '<?php namespace Foo; class Baz {}',
+                'Qux' => [
+                    'Quux.php' => '<?php namespace Bar\Qux; class Quux {}',
+                ],
+            ],
+            'Chore' => [
+                'global.php' => '<?php function global_function() {}',
+                'global_classes.php' => '<?php class GlobalClass {}',
+            ],
+            'Baz.php.ignore' => '',
+        ], $this->vfs);
+        $classes = FileSystem::getClassesPsr4($this->vfs->url(), '');
+        $this->assertSame([
+            '\Foo\Bar',
+            '\Foo\Qux\Quux',
+        ], $classes);
+    }
+
+    public function testGetClassesPsr4WithCustomRule(): void
+    {
+        vfsStream::create([
+            'Foo' => [
+                'Bar.php' => '<?php namespace Foo; class Bar {}',
+                'Baz.php' => '<?php namespace Foo; class Baz {}',
+            ],
+        ], $this->vfs);
+        $classes = FileSystem::getClassesPsr4($this->vfs->url(), '', fn (string $dir, array $pathinfo) => $pathinfo['filename'] === 'Bar');
+        $this->assertSame(['\Foo\Bar'], $classes);
+    }
+
+    public function testGetClassesPsr4WithReturnPath(): void
+    {
+        vfsStream::create([
+            'Foo' => [
+                'Bar.php' => '<?php namespace Foo; class Bar {}',
+                'Baz.php' => '<?php namespace Foo; class Baz {}',
+            ],
+        ], $this->vfs);
+        $classes = FileSystem::getClassesPsr4($this->vfs->url(), '', return_path_value: 'my_path');
+        $this->assertSame([
+            '\Foo\Bar' => 'my_path/Foo/Bar.php',
+            '\Foo\Baz' => 'my_path/Foo/Baz.php',
+        ], $classes);
+    }
+
+    public function testScanDirFilesWithNotExistsDir(): void
+    {
+        FileSystem::scanDirFiles($this->vfs->url() . '/not_exists');
+        $this->assertLogged('warning', zm_internal_errcode('E00080') . '扫描目录失败，目录不存在');
+    }
+}

tests/ZM/Utils/CatCodeTest.php

@@ -40,7 +40,6 @@ class CatCodeTest extends TestCase
                 ],
                 '',
             ],
-            'non-string, non-segment, non-array' => [123, ''],
         ];
     }