Code coverage 100%

This commit encompasses the removal of the unused 'FakeFooProvider.php' test file. Additionally, annotations in 'Compiler.php' have been renamed and changes are implemented in 'CompilerTest.php' to reflect these changes. Also, refactor 'Scripts.php' class by removing redundant countable interface and its related methods. Fix indentation error in 'InstanceScript.php'.

src/Compiler.php

@@ -31,9 +31,9 @@ public function compile(AbstractModule $module, string $scriptDir): Scripts
         // Lock
         $fp = fopen($scriptDir . '/compile.lock', 'a+');
         if ($fp === false || ! flock($fp, LOCK_EX)) {
-            // @CoverageIgnoreStart
+            // @codeCoverageIgnoreStart
             throw new CompileLockFailed($scriptDir);
-            // @CoverageIgnoreEnd
+            // @codeCoverageIgnoreEnd
         }
 
         $scripts = new Scripts();

src/InstanceScript.php

@@ -99,7 +99,7 @@ private function addDependencyArg(bool $isSingleton, string $index, ReflectionPa
         /** @psalm-suppress PossiblyNullReference / The $parameter here can never be null */
         $ip = sprintf("['%s', '%s', '%s']", $parameter->getDeclaringClass()->getName(), $parameter->getDeclaringFunction()->getName(), $parameter->name); //@phpstan-ignore-line
         $func = $isSingleton ? '$singleton' : '$prototype';
-            $arg = sprintf("%s('%s', %s)", $func, $index, $ip);
+        $arg = sprintf("%s('%s', %s)", $func, $index, $ip);
         $this->args[] = $arg;
     }
 

src/Scripts.php

@@ -4,13 +4,10 @@
 
 namespace Ray\Compiler;
 
-use Countable;
-
-use function count;
 use function sprintf;
 use function str_replace;
 
-final class Scripts implements Countable
+final class Scripts
 {
     /** @var array<string, string> */
     private $scripts = [];
@@ -20,11 +17,6 @@ public function add(string $index, string $script): void
         $this->scripts[$index] = $script;
     }
 
-    public function count(): int
-    {
-        return count($this->scripts);
-    }
-
     public function save(string $scriptDir): void
     {
         $template = <<<'EOL'

tests/CompilerTest.php

@@ -12,9 +12,10 @@
 use Ray\Compiler\CompileVisitor\FakeBazProvider;
 use Ray\Compiler\CompileVisitor\FakeFoo;
 use Ray\Compiler\CompileVisitor\FakeFooInterface;
+use Ray\Compiler\CompileVisitor\FakeQux;
 use Ray\Compiler\Exception\InjectionPointUnbound;
-use Ray\Compiler\Exception\Unbound;
 use Ray\Di\AbstractModule;
+use Ray\Di\Exception\Unbound;
 use Ray\Di\Scope;
 
 use function get_class;
@@ -88,13 +89,13 @@ public function testCompileProvider(): void
         $module = new class () extends AbstractModule{
             protected function configure()
             {
-                $this->bind(FakeFooInterface::class)->toProvider(FakeFooProvider::class);
+                $this->bind(FakeBazInterface::class)->toProvider(FakeBazProvider::class);
             }
         };
 
         $this->compiler->compile($module, $this->scriptDir);
-        $instance = $this->injector->getInstance(FakeFooInterface::class);
-        $this->assertInstanceOf(FakeFoo::class, $instance);
+        $instance = $this->injector->getInstance(FakeBazInterface::class);
+        $this->assertInstanceOf(FakeBaz::class, $instance);
     }
 
     public function testCompileComplex(): void
@@ -197,7 +198,19 @@ protected function configure()
         $this->assertIsString(get_class($nullInstance));
     }
 
-    public function testNoInjectionPointInVeryFirstInject()
+    public function testUnbound(): void
+    {
+        $this->expectException(Unbound::class);
+        $module = new class () extends AbstractModule{
+            protected function configure()
+            {
+                $this->bind(FakeBar::class);
+            }
+        };
+        $this->compiler->compile($module, $this->scriptDir);
+    }
+
+    public function testNoInjectionPointInVeryFirstInject(): void
     {
         $this->expectException(InjectionPointUnbound::class);
         $module = new class () extends AbstractModule {
@@ -209,4 +222,21 @@ protected function configure()
         $this->compiler->compile($module, $this->scriptDir);
         $this->injector->getInstance(FakeBazInterface::class);
     }
+
+    public function testNullQualifer(): void
+    {
+        $module = new class () extends AbstractModule{
+            protected function configure()
+            {
+                $this->bind(FakeQux::class);
+                $this->bind(FakeBazInterface::class)->toProvider(FakeBazProvider::class)->in(Scope::SINGLETON);
+            }
+        };
+        $this->compiler->compile($module, $this->scriptDir);
+        $qux = $this->injector->getInstance(FakeQux::class);
+        $this->assertInstanceOf(FakeQux::class, $qux);
+        $this->assertSame([null], $qux->baz->qualifiers);
+        $this->assertInstanceOf(AirInjector::class, $qux->injector);
+        $this->assertInstanceOf(FakeBazInterface::class, $this->injector->getInstance(FakeBazInterface::class));
+    }
 }

tests/Fake/CompileVisitor/FakeBar.php

@@ -0,0 +1,10 @@
+<?php
+
+namespace Ray\Compiler\CompileVisitor;
+
+class FakeBar
+{
+    public function __construct(FakeFooInterface $foo)
+    {
+    }
+}

tests/Fake/CompileVisitor/FakeBaz.php

@@ -4,10 +4,10 @@
 
 class FakeBaz implements FakeBazInterface
 {
-    public $qualifers;
+    public $qualifiers;
 
-    public function __construct(?array $qualifers)
+    public function __construct(?array $qualifiers)
     {
-        $this->qualifers = $qualifers;
+        $this->qualifiers = $qualifiers;
     }
 }

tests/Fake/CompileVisitor/FakeQux.php

@@ -0,0 +1,24 @@
+<?php
+
+namespace Ray\Compiler\CompileVisitor;
+
+use Ray\Di\InjectorInterface;
+
+final class FakeQux
+{
+    /**
+     * @var FakeBaz
+     */
+    public $baz;
+
+    /**
+     * @var InjectorInterface
+     */
+    public $injector;
+
+    public function __construct(FakeBazInterface $baz, InjectorInterface $injector)
+    {
+        $this->baz = $baz;
+        $this->injector = $injector;
+    }
+}