Fix: Add tests for Container

test/Faker/Extension/ContainerTest.php

@@ -5,24 +5,43 @@
 namespace Faker\Test\Extension;
 
 use Faker\Container\Container;
+use Faker\Container\ContainerException;
 use Faker\Core\File;
 use Faker\Extension\Extension;
+use Faker\Test;
 use PHPUnit\Framework\TestCase;
 use Psr\Container\ContainerExceptionInterface;
-use Psr\Container\NotFoundExceptionInterface;
 
 /**
  * @covers \Faker\Container\Container
  */
 final class ContainerTest extends TestCase
 {
+    public function testHasThrowsInvalidArgumentExceptionWhenIdentifierIsNotAString(): void
+    {
+        $container = new Container([]);
+
+        $this->expectException(\InvalidArgumentException::class);
+
+        $container->has(false);
+    }
+
     public function testHasReturnsFalseWhenContainerDoesNotHaveDefinitionForService(): void
     {
         $container = new Container([]);
 
         self::assertFalse($container->has('foo'));
     }
 
+    public function testGetThrowsInvalidArgumentExceptionWhenIdentifierIsNotAString(): void
+    {
+        $container = new Container([]);
+
+        $this->expectException(\InvalidArgumentException::class);
+
+        $container->get(false);
+    }
+
     public function testGetThrowsNotFoundExceptionWhenContainerDoesNotHaveDefinitionForService(): void
     {
         $container = new Container([]);
@@ -43,6 +62,42 @@ public function testGetFromString(): void
         self::assertInstanceOf(File::class, $object);
     }
 
+    public function testGetThrowsRuntimeExceptionWhenServiceCouldNotBeResolvedFromCallable(): void
+    {
+        $id = 'foo';
+
+        $container = new Container([
+            $id => static function (): void {
+                throw new \RuntimeException();
+            },
+        ]);
+
+        $this->expectException(ContainerException::class);
+        $this->expectExceptionMessage(sprintf(
+            'Error while invoking callable for "%s"',
+            $id,
+        ));
+
+        $container->get($id);
+    }
+
+    public function testGetThrowsRuntimeExceptionWhenServiceCouldNotBeResolvedFromClass(): void
+    {
+        $id = 'foo';
+
+        $container = new Container([
+            $id => Test\Fixture\Container\UnconstructableClass::class,
+        ]);
+
+        $this->expectException(ContainerException::class);
+        $this->expectExceptionMessage(sprintf(
+            'Could not instantiate class "%s"',
+            $id,
+        ));
+
+        $container->get($id);
+    }
+
     /**
      * @dataProvider provideDefinitionThatDoesNotResolveToExtension
      */

test/Fixture/Container/UnconstructableClass.php

@@ -0,0 +1,15 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Faker\Test\Fixture\Container;
+
+use Faker\Extension;
+
+final class UnconstructableClass implements Extension\Extension
+{
+    public function __construct()
+    {
+        throw new \RuntimeException('Sorry, not sorry');
+    }
+}