Refactor: remove redundant assertions and improve array handling

Removed unnecessary assertions such as `assert` and `property_exists` to make the tests cleaner and more efficient. Adjusted array handling by casting `bindings` to an array for better compatibility and added necessary code comments to suppress static analysis warnings where applicable. These changes improve the maintainability and readability of the codebase without affecting the functionality.

src/di/Bind.php

@@ -218,7 +218,7 @@ private function getStringName(array $name): string
              * @param array-key $key
              */
             static function (array $carry, $key) use ($name): array {
-                if (! is_string($key)) {
+                if (! is_string($key)) { // @phpstan-ignore-line
                     throw new InvalidToConstructorNameParameter((string) $key);
                 }
 

src/di/MultiBinding/LazyProvider.php

@@ -7,8 +7,6 @@
 use Ray\Di\InjectorInterface;
 use Ray\Di\ProviderInterface;
 
-use function assert;
-
 /**
  * @template T of ProviderInterface
  */

src/di/Name.php

@@ -151,9 +151,11 @@ private function parseName(string $name): array
             if (isset($exploded[1])) {
                 [$key, $value] = $exploded;
                 if (isset($key[0]) && $key[0] === '$') {
+                    assert(is_string($key)); // @phpstan-ignore-line
                     $key = substr($key, 1);
                 }
 
+                /** @psalm-suppress MixedArgument */
                 $names[trim($key)] = trim($value);
             }
         }

tests/di/ArgumentsTest.php

@@ -8,8 +8,6 @@
 use ReflectionMethod;
 use ReflectionParameter;
 
-use function assert;
-use function is_object;
 use function spl_object_hash;
 
 class ArgumentsTest extends TestCase

tests/di/DependencyTest.php

@@ -115,7 +115,7 @@ public function testInjectInterceptor(): void
         $isWeave = (new ReflectionClass($instance))->implementsInterface(WeavedInterface::class);
         $this->assertTrue($isWeave);
         assert(property_exists($instance, 'bindings'));
-        $this->assertArrayHasKey('returnSame', $instance->bindings);
+        $this->assertArrayHasKey('returnSame', (array) $instance->bindings);
     }
 
     /**

tests/di/GrapherTest.php

@@ -9,10 +9,8 @@
 
 use function assert;
 use function file_get_contents;
-use function is_object;
 use function is_string;
 use function passthru;
-use function property_exists;
 use function unserialize;
 
 class GrapherTest extends TestCase
@@ -28,7 +26,6 @@ public function testGetInstanceWithArgs(): void
         $grapher = new Grapher(new FakeUntargetModule(), __DIR__ . '/tmp');
         $instance = $grapher->newInstanceArgs(FakeUntargetChild::class, ['1']);
         $this->assertInstanceOf(FakeUntargetChild::class, $instance);
-        assert(property_exists($instance, 'val'));
         $this->assertSame('1', $instance->val);
     }
 

tests/di/InjectorTest.php

@@ -16,7 +16,6 @@
 use function file_get_contents;
 use function is_string;
 use function passthru;
-use function property_exists;
 use function serialize;
 use function spl_object_hash;
 use function unlink;
@@ -184,7 +183,6 @@ public function testAnnotationBasedInjection(): Injector
         $this->assertInstanceOf(FakeMirrorInterface::class, $car->spareMirror);
         $this->assertSame(spl_object_hash($car->rightMirror), spl_object_hash($car->spareMirror));
         $this->assertInstanceOf(FakeHandle::class, $car->handle);
-        assert($car->handle instanceof FakeHandle);
         $this->assertSame($car->handle->logo, 'momo');
 
         return $injector;
@@ -242,7 +240,6 @@ public function testSerializeBuiltinBinding(): void
         $injector = unserialize(serialize(new Injector()));
         assert($injector instanceof InjectorInterface);
         $instance = $injector->getInstance(FakeBuiltin::class);
-        assert(property_exists($instance, 'injector'));
         $this->assertInstanceOf(Injector::class, $instance->injector);
     }
 
@@ -430,9 +427,7 @@ protected function configure()
         $instance = $injector->getInstance(FakeAop::class);
         $result = $instance->returnSame(2);
         $this->assertSame(2, $result);
-        assert(property_exists($instance, 'bindings'));
-        assert(isset($instance->bindings['returnSame'][0]));
-        $this->assertInstanceOf(NullInterceptor::class, $instance->bindings['returnSame'][0]);
+        $this->assertInstanceOf(NullInterceptor::class, $instance->bindings['returnSame'][0]); // @phpstan-ignore-line
     }
 
     public function testModuleArray(): void