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,6 +12,7 @@
 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;
@@ -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,6 +198,18 @@ protected function configure()
         $this->assertIsString(get_class($nullInstance));
     }
 
+    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()
     {
         $this->expectException(InjectionPointUnbound::class);
@@ -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->qualifers);
+        $this->assertInstanceOf(AirInjector::class, $qux->injector);
+        $this->assertInstanceOf(FakeBazInterface::class, $this->injector->getInstance(FakeBazInterface::class));
+    }
 }