Finalize StorageInterface constants + Fix tests (#260)

composer.json

@@ -58,7 +58,7 @@
         "rector/rector": "^1.0.0",
         "roave/infection-static-analysis-plugin": "^1.16",
         "spatie/phpunit-watcher": "^1.23",
-        "vimeo/psalm": "^5.18",
+        "vimeo/psalm": "^5.25",
         "yiisoft/error-handler": "^3.0",
         "yiisoft/event-dispatcher": "^1.0",
         "yiisoft/log": "^2.0",

src/DebugServiceProvider.php

@@ -23,7 +23,6 @@ public function getDefinitions(): array
 
     public function getExtensions(): array
     {
-        return [
-        ];
+        return [];
     }
 }

src/Storage/StorageInterface.php

@@ -11,9 +11,20 @@
  */
 interface StorageInterface
 {
-    public const TYPE_SUMMARY = 'summary';
-    public const TYPE_DATA = 'data';
-    public const TYPE_OBJECTS = 'objects';
+    /**
+     * @psalm-suppress MissingClassConstType
+     */
+    final public const TYPE_SUMMARY = 'summary';
+
+    /**
+     * @psalm-suppress MissingClassConstType
+     */
+    final public const TYPE_DATA = 'data';
+
+    /**
+     * @psalm-suppress MissingClassConstType
+     */
+    final public const TYPE_OBJECTS = 'objects';
 
     /**
      * Add collector to get debug data from

tests/Unit/DebugServiceProviderTest.php

@@ -5,34 +5,21 @@
 namespace Yiisoft\Yii\Debug\Tests\Unit;
 
 use PHPUnit\Framework\TestCase;
-use Psr\EventDispatcher\EventDispatcherInterface;
-use Psr\EventDispatcher\ListenerProviderInterface;
-use Psr\Log\LoggerInterface;
-use Psr\Log\NullLogger;
+use Psr\Container\ContainerInterface;
 use Yiisoft\Di\Container;
 use Yiisoft\Di\ContainerConfig;
-use Yiisoft\EventDispatcher\Dispatcher\Dispatcher;
-use Yiisoft\EventDispatcher\Provider\Provider;
+use Yiisoft\Yii\Debug\Collector\ContainerInterfaceProxy;
 use Yiisoft\Yii\Debug\DebugServiceProvider;
-use Yiisoft\Yii\Debug\Collector\EventDispatcherInterfaceProxy;
-use Yiisoft\Yii\Debug\Collector\LoggerInterfaceProxy;
 
 final class DebugServiceProviderTest extends TestCase
 {
     public function testRegister(): void
     {
-        $config = ContainerConfig::create()
-            ->withDefinitions([
-                LoggerInterface::class => NullLogger::class,
-                EventDispatcherInterface::class => Dispatcher::class,
-                ListenerProviderInterface::class => Provider::class,
-            ])
-            ->withProviders([
-                new DebugServiceProvider(),
-            ]);
+        $config = ContainerConfig::create()->withProviders([
+            new DebugServiceProvider(),
+        ]);
         $container = new Container($config);
 
-        $this->assertInstanceOf(LoggerInterfaceProxy::class, $container->get(LoggerInterface::class));
-        $this->assertInstanceOf(EventDispatcherInterfaceProxy::class, $container->get(EventDispatcherInterface::class));
+        $this->assertInstanceOf(ContainerInterfaceProxy::class, $container->get(ContainerInterface::class));
     }
 }