AliceMajere / wonderland-container

src/Service/ServiceInstanceInterface.php

Indentation +4 added lines, -4 removed lines

@@ -10,10 +10,10 @@
 interface ServiceInstanceInterface
 {
 
-	/** @return string */
-	public function getServiceName();
+    /** @return string */
+    public function getServiceName();
 
-	/** @return mixed */
-	public function getInstance();
+    /** @return mixed */
+    public function getInstance();
 
 }

src/Service/InstanceDefinition.php

Indentation +19 added lines, -19 removed lines

@@ -9,28 +9,28 @@
  */
 class InstanceDefinition implements ServiceInstanceInterface
 {
-	/** @var string */
-	private $serviceName;
+    /** @var string */
+    private $serviceName;
 
-	/** @var mixed */
-	private $instance;
+    /** @var mixed */
+    private $instance;
 
-	public function __construct(string $serviceName, $instance)
-	{
-		$this->serviceName = $serviceName;
-		$this->instance = $instance;
-	}
+    public function __construct(string $serviceName, $instance)
+    {
+        $this->serviceName = $serviceName;
+        $this->instance = $instance;
+    }
 
-	/** @return string */
-	public function getServiceName()
-	{
-		return $this->serviceName;
-	}
+    /** @return string */
+    public function getServiceName()
+    {
+        return $this->serviceName;
+    }
 
-	/** @return mixed */
-	public function getInstance()
-	{
-		return $this->instance;
-	}
+    /** @return mixed */
+    public function getInstance()
+    {
+        return $this->instance;
+    }
 
 }

src/Service/ServiceDefinitionInterface.php

Indentation +8 added lines, -8 removed lines

@@ -10,16 +10,16 @@
 interface ServiceDefinitionInterface
 {
 
-	/** @return string */
-	public function getServiceName();
+    /** @return string */
+    public function getServiceName();
 
-	/** @return string */
-	public function getClass();
+    /** @return string */
+    public function getClass();
 
-	/** @return array */
-	public function getConstructArgs();
+    /** @return array */
+    public function getConstructArgs();
 
-	/** @return array */
-	public function getCalls();
+    /** @return array */
+    public function getCalls();
 
 }

tests/bootstrap.php

Indentation +2 added lines, -2 removed lines

@@ -2,7 +2,7 @@ 
 declare( strict_types = 1 );
 
 if ( PHP_SAPI !== 'cli' ) {
-	die( 'Not an entry point' );
+    die( 'Not an entry point' );
 }
 
 error_reporting( -1 );
@@ -10,7 +10,7 @@ 
 ini_set( 'display_errors', '1' );
 
 if ( !is_readable( __DIR__ . '/../vendor/autoload.php' ) ) {
-	die( 'You need to install this package with Composer before you can run the tests' );
+    die( 'You need to install this package with Composer before you can run the tests' );
 }
 
 require_once __DIR__ . '/../vendor/autoload.php';

Spacing +6 added lines, -6 removed lines

@@ -1,16 +1,16 @@
 <?php
-declare( strict_types = 1 );
+declare(strict_types=1);
 
-if ( PHP_SAPI !== 'cli' ) {
-	die( 'Not an entry point' );
+if (PHP_SAPI !== 'cli') {
+	die('Not an entry point');
 }
 
 error_reporting( -1 );
 
-ini_set( 'display_errors', '1' );
+ini_set('display_errors', '1');
 
-if ( !is_readable( __DIR__ . '/../vendor/autoload.php' ) ) {
-	die( 'You need to install this package with Composer before you can run the tests' );
+if (!is_readable(__DIR__ . '/../vendor/autoload.php')) {
+	die('You need to install this package with Composer before you can run the tests');
 }
 
 require_once __DIR__ . '/../vendor/autoload.php';

src/Service/ServiceDefinition.php

Indentation +49 added lines, -49 removed lines

@@ -9,62 +9,62 @@
  */
 class ServiceDefinition implements ServiceDefinitionInterface
 {
-	/** @var string */
-	private $serviceName;
+    /** @var string */
+    private $serviceName;
 
-	/** @var string */
-	private $class;
+    /** @var string */
+    private $class;
 
-	/** @var array */
-	private $constructArgs;
+    /** @var array */
+    private $constructArgs;
 
-	/** @var array */
-	private $calls;
+    /** @var array */
+    private $calls;
 
-	/**
-	 * RepositoryFactory constructor.
-	 *
-	 * @param string $serviceName
-	 * @param string $class
-	 * @param array $args
-	 * @param array $calls
-	 */
-	public function __construct(string $serviceName, string $class, array $args = [], array $calls = [])
-	{
-		$this->serviceName = $serviceName;
-		$this->class = $class;
-		$this->constructArgs = $args;
-		$this->calls = $calls;
-	}
+    /**
+     * RepositoryFactory constructor.
+     *
+     * @param string $serviceName
+     * @param string $class
+     * @param array $args
+     * @param array $calls
+     */
+    public function __construct(string $serviceName, string $class, array $args = [], array $calls = [])
+    {
+        $this->serviceName = $serviceName;
+        $this->class = $class;
+        $this->constructArgs = $args;
+        $this->calls = $calls;
+    }
 
-	/**
-	 * @return string
-	 */
-	public function getClass()
-	{
-		return $this->class;
-	}
+    /**
+     * @return string
+     */
+    public function getClass()
+    {
+        return $this->class;
+    }
 
-	/** @return string */
-	public function getServiceName()
-	{
-		return $this->serviceName;
-	}
+    /** @return string */
+    public function getServiceName()
+    {
+        return $this->serviceName;
+    }
 
-	/**
-	 * @return array
-	 */
-	public function getConstructArgs()
-	{
-		return $this->constructArgs;
-	}
+    /**
+     * @return array
+     */
+    public function getConstructArgs()
+    {
+        return $this->constructArgs;
+    }
 
-	/**
-	 * @return array
-	 */
-	public function getCalls()
-	{
-		return $this->calls;
-	}
+    /**
+     * @return array
+     */
+    public function getCalls()
+    {
+        return $this->calls;
+    }
 
 }

src/ServiceContainer.php

Indentation +153 added lines, -153 removed lines

@@ -14,158 +14,158 @@
  */
 class ServiceContainer implements ContainerInterface
 {
-	/** @var ServiceDefinitionInterface[] */
-	private $services;
-
-	/** @var array */
-	private $serviceInstances;
-
-	/**
-	 * ServiceContainer constructor.
-	 */
-	public function __construct()
-	{
-		$this->serviceInstances = [];
-	}
-
-	/**
-	 * @param ServiceDefinitionInterface $serviceDefinition
-	 * @return ServiceContainer
-	 */
-	public function addService(ServiceDefinitionInterface $serviceDefinition)
-	{
-		$this->services[$serviceDefinition->getServiceName()] = $serviceDefinition;
-
-		return $this;
-	}
-
-	/**
-	 * @param ServiceInstanceInterface $definition
-	 * @return ServiceContainer
-	 * @throws DuplicatedServiceException
-	 */
-	public function addServiceInstance(ServiceInstanceInterface $definition)
-	{
-		if (true === array_key_exists($definition->getServiceName(), $this->serviceInstances)) {
-			throw new DuplicatedServiceException(
-				'The service' . $definition->getServiceName() . ' is already registered in the container'
-			);
-		}
-
-		$this->serviceInstances[$definition->getServiceName()] = $definition->getInstance();
-
-		return $this;
-	}
-
-	/**
-	 * @param string $index
-	 * @param bool $new
-	 * @return mixed|null
-	 */
-	public function get($index, $new = false)
-	{
-		// if service is shared true and already exists we return it
-		if (isset($this->serviceInstances[$index]) && false === $new) {
-			return $this->serviceInstances[$index];
-		}
-
-		// is service not defined return null
-		if (!isset($this->services[$index])) {
-			return null;
-		}
-
-		// we create a new instance
-		$instance = $this->create($this->services[$index]);
-
-		// if shared true we set it in the current instances
-		if (false === $new) {
-			$this->serviceInstances[$index] = $instance;
-		}
-
-		return $instance;
-	}
-
-	/**
-	 * Returns true if the container can return an entry for the given identifier.
-	 * Returns false otherwise.
-	 *
-	 * `has($id)` returning true does not mean that `get($id)` will not throw an exception.
-	 * It does however mean that `get($id)` will not throw a `NotFoundExceptionInterface`.
-	 *
-	 * @param string $index Identifier of the entry to look for.
-	 *
-	 * @return bool
-	 */
-	public function has($index)
-	{
-		// if service is shared true and already exists we return it
-		if (isset($this->serviceInstances[$index])) {
-			return true;
-		}
-
-		// is service not defined return null
-		if (isset($this->services[$index])) {
-			return true;
-		}
-
-		return false;
-	}
-
-	/**
-	 * @param ServiceDefinitionInterface $serviceDefinition
-	 * @return mixed
-	 */
-	private function create(ServiceDefinitionInterface $serviceDefinition)
-	{
-		// we get the class name in a var for the new later
-		$newClass = $serviceDefinition->getClass();
-
-		// we get the construct args
-		$args = $this->checkArgsServices($serviceDefinition->getConstructArgs());
-		$instance = new $newClass(...$args);
-
-		// we call the injection methods after we instance the object
-		$calls = $this->checkCallsServices($serviceDefinition->getCalls());
-		foreach ($calls as $call => $params) {
-			call_user_func_array([$instance, $call], $params);
-		}
-
-		// we create the new instance
-		return $instance;
-	}
-
-	/**
-	 * @param array $args
-	 * @return array
-	 */
-	private function checkArgsServices(array $args)
-	{
-		// we check if any of the construct args are services themself and we create them
-		foreach ($args as $k => $arg) {
-			if (true === $this->has($arg)) {
-				$args[$k] = $this->get($arg);
-			}
-		}
-
-		return $args;
-	}
-
-	/**
-	 * @param array $calls
-	 * @return array
-	 */
-	private function checkCallsServices(array $calls)
-	{
-		// we check if any of the calls args are services themself and we create them
-		foreach ($calls as $callArgs) {
-			foreach ($callArgs as $k => $arg) {
-				if (true === $this->has($arg)) {
-					$args[$k] = $this->get($arg);
-				}
-			}
-		}
-
-		return $calls;
-	}
+    /** @var ServiceDefinitionInterface[] */
+    private $services;
+
+    /** @var array */
+    private $serviceInstances;
+
+    /**
+     * ServiceContainer constructor.
+     */
+    public function __construct()
+    {
+        $this->serviceInstances = [];
+    }
+
+    /**
+     * @param ServiceDefinitionInterface $serviceDefinition
+     * @return ServiceContainer
+     */
+    public function addService(ServiceDefinitionInterface $serviceDefinition)
+    {
+        $this->services[$serviceDefinition->getServiceName()] = $serviceDefinition;
+
+        return $this;
+    }
+
+    /**
+     * @param ServiceInstanceInterface $definition
+     * @return ServiceContainer
+     * @throws DuplicatedServiceException
+     */
+    public function addServiceInstance(ServiceInstanceInterface $definition)
+    {
+        if (true === array_key_exists($definition->getServiceName(), $this->serviceInstances)) {
+            throw new DuplicatedServiceException(
+                'The service' . $definition->getServiceName() . ' is already registered in the container'
+            );
+        }
+
+        $this->serviceInstances[$definition->getServiceName()] = $definition->getInstance();
+
+        return $this;
+    }
+
+    /**
+     * @param string $index
+     * @param bool $new
+     * @return mixed|null
+     */
+    public function get($index, $new = false)
+    {
+        // if service is shared true and already exists we return it
+        if (isset($this->serviceInstances[$index]) && false === $new) {
+            return $this->serviceInstances[$index];
+        }
+
+        // is service not defined return null
+        if (!isset($this->services[$index])) {
+            return null;
+        }
+
+        // we create a new instance
+        $instance = $this->create($this->services[$index]);
+
+        // if shared true we set it in the current instances
+        if (false === $new) {
+            $this->serviceInstances[$index] = $instance;
+        }
+
+        return $instance;
+    }
+
+    /**
+     * Returns true if the container can return an entry for the given identifier.
+     * Returns false otherwise.
+     *
+     * `has($id)` returning true does not mean that `get($id)` will not throw an exception.
+     * It does however mean that `get($id)` will not throw a `NotFoundExceptionInterface`.
+     *
+     * @param string $index Identifier of the entry to look for.
+     *
+     * @return bool
+     */
+    public function has($index)
+    {
+        // if service is shared true and already exists we return it
+        if (isset($this->serviceInstances[$index])) {
+            return true;
+        }
+
+        // is service not defined return null
+        if (isset($this->services[$index])) {
+            return true;
+        }
+
+        return false;
+    }
+
+    /**
+     * @param ServiceDefinitionInterface $serviceDefinition
+     * @return mixed
+     */
+    private function create(ServiceDefinitionInterface $serviceDefinition)
+    {
+        // we get the class name in a var for the new later
+        $newClass = $serviceDefinition->getClass();
+
+        // we get the construct args
+        $args = $this->checkArgsServices($serviceDefinition->getConstructArgs());
+        $instance = new $newClass(...$args);
+
+        // we call the injection methods after we instance the object
+        $calls = $this->checkCallsServices($serviceDefinition->getCalls());
+        foreach ($calls as $call => $params) {
+            call_user_func_array([$instance, $call], $params);
+        }
+
+        // we create the new instance
+        return $instance;
+    }
+
+    /**
+     * @param array $args
+     * @return array
+     */
+    private function checkArgsServices(array $args)
+    {
+        // we check if any of the construct args are services themself and we create them
+        foreach ($args as $k => $arg) {
+            if (true === $this->has($arg)) {
+                $args[$k] = $this->get($arg);
+            }
+        }
+
+        return $args;
+    }
+
+    /**
+     * @param array $calls
+     * @return array
+     */
+    private function checkCallsServices(array $calls)
+    {
+        // we check if any of the calls args are services themself and we create them
+        foreach ($calls as $callArgs) {
+            foreach ($callArgs as $k => $arg) {
+                if (true === $this->has($arg)) {
+                    $args[$k] = $this->get($arg);
+                }
+            }
+        }
+
+        return $calls;
+    }
 
 }

tests/unit/ServiceContainerTest.php

Indentation +114 added lines, -114 removed lines

@@ -18,119 +18,119 @@
  */
 class ServiceContainerTest extends TestCase
 {
-	private const SERVICE_NAME = 'service.name';
-	private const DEFAULT_SERVICE_NANE = 'default.service.name';
-
-	/** @var ServiceContainer|MockObject */
-	private $container;
-
-	/** @var ServiceContainer */
-	private $realContainer;
-
-	protected function setUp()
-	{
-		$this->container = $this->getMockBuilder(ServiceContainer::class)->getMock();
-		$this->realContainer = new ServiceContainer();
-		$this->realContainer->addService(new ServiceDefinition(self::DEFAULT_SERVICE_NANE, \DateTime::class));
-	}
-
-	public function testConstructor()
-	{
-		$this->assertAttributeEquals([], 'serviceInstances', $this->container);
-	}
-
-	/**
-	 * @covers \Wonderland\Container\ServiceContainer::addService
-	 */
-	public function test_AddService()
-	{
-		$definition = $this->getMockBuilder(ServiceDefinition::class)
-			->setConstructorArgs([self::SERVICE_NAME, \DateTime::class])
-			->getMock();
-		;
-
-		$definition->expects($this->once())
-			->method('getServiceName')
-			->willReturn(self::SERVICE_NAME);
-
-		$this->container->expects($this->once())
-			->method('addService')
-			->willReturn($this->container);
-
-		$this->assertSame($this->container, $this->container->addService($definition));
-		$this->assertSame($this->realContainer, $this->realContainer->addService($definition));
-	}
-
-	/**
-	 * @throws DuplicatedServiceException
-	 */
-	public function test_AddServiceInstance()
-	{
-		$definition = $this->getMockBuilder(InstanceDefinition::class)
-			->setConstructorArgs([self::SERVICE_NAME, new \DateTime()])
-			->getMock();
-
-		$definition->expects($this->atLeast(2))
-			->method('getServiceName')
-			->willReturn(self::SERVICE_NAME);
-
-		$this->container->expects($this->once())
-			->method('addServiceInstance')
-			->willReturn($this->container);
-
-		$this->assertSame($this->container, $this->container->addServiceInstance($definition));
-		$this->assertSame($this->realContainer, $this->realContainer->addServiceInstance($definition));
-	}
-
-	/**
-	 * @throws DuplicatedServiceException
-	 */
-	public function test_AddServiceInstanceException()
-	{
-		$this->expectException(DuplicatedServiceException::class);
-
-		$definition = $this->getMockBuilder(InstanceDefinition::class)
-			->setConstructorArgs([self::SERVICE_NAME, new \DateTime()])
-			->getMock();
-
-		$definition->expects($this->atLeast(2))
-			->method('getServiceName')
-			->willReturn(self::SERVICE_NAME);
-
-		$this->realContainer->addServiceInstance($definition);
-		$this->realContainer->addServiceInstance($definition);
-	}
-
-	public function test_get()
-	{
-		$this->assertSame(null, $this->realContainer->get('test'));
-		$this->assertSame(\DateTime::class, get_class($this->realContainer->get(self::DEFAULT_SERVICE_NANE)));
-		$instance = $this->realContainer->get(self::DEFAULT_SERVICE_NANE);
-		$this->assertSame(\DateTime::class, get_class($instance));
-		$this->assertSame($instance, $this->realContainer->get(self::DEFAULT_SERVICE_NANE));
-
-		$this->realContainer->addServiceInstance(
-			new InstanceDefinition('call.format', 'd-m-Y')
-		);
-
-		$this->realContainer->addService(
-			new ServiceDefinition(
-				'service.call',
-				\DateTime::class,
-				['1970-01-01'],
-				['format' => ['call.format']]
-			)
-		);
-
-		$date = new \DateTime('1970-01-01');
-		$this->assertSame($date->format('m-d-Y'), $this->realContainer->get('service.call')->format('m-d-Y'));
-	}
-
-	public function test_has()
-	{
-		$this->assertSame(false, $this->realContainer->has('fake'));
-		$this->assertSame(true, $this->realContainer->has(self::DEFAULT_SERVICE_NANE));
-		$this->assertSame(true, $this->realContainer->has(self::DEFAULT_SERVICE_NANE));
-	}
+    private const SERVICE_NAME = 'service.name';
+    private const DEFAULT_SERVICE_NANE = 'default.service.name';
+
+    /** @var ServiceContainer|MockObject */
+    private $container;
+
+    /** @var ServiceContainer */
+    private $realContainer;
+
+    protected function setUp()
+    {
+        $this->container = $this->getMockBuilder(ServiceContainer::class)->getMock();
+        $this->realContainer = new ServiceContainer();
+        $this->realContainer->addService(new ServiceDefinition(self::DEFAULT_SERVICE_NANE, \DateTime::class));
+    }
+
+    public function testConstructor()
+    {
+        $this->assertAttributeEquals([], 'serviceInstances', $this->container);
+    }
+
+    /**
+     * @covers \Wonderland\Container\ServiceContainer::addService
+     */
+    public function test_AddService()
+    {
+        $definition = $this->getMockBuilder(ServiceDefinition::class)
+            ->setConstructorArgs([self::SERVICE_NAME, \DateTime::class])
+            ->getMock();
+        ;
+
+        $definition->expects($this->once())
+            ->method('getServiceName')
+            ->willReturn(self::SERVICE_NAME);
+
+        $this->container->expects($this->once())
+            ->method('addService')
+            ->willReturn($this->container);
+
+        $this->assertSame($this->container, $this->container->addService($definition));
+        $this->assertSame($this->realContainer, $this->realContainer->addService($definition));
+    }
+
+    /**
+     * @throws DuplicatedServiceException
+     */
+    public function test_AddServiceInstance()
+    {
+        $definition = $this->getMockBuilder(InstanceDefinition::class)
+            ->setConstructorArgs([self::SERVICE_NAME, new \DateTime()])
+            ->getMock();
+
+        $definition->expects($this->atLeast(2))
+            ->method('getServiceName')
+            ->willReturn(self::SERVICE_NAME);
+
+        $this->container->expects($this->once())
+            ->method('addServiceInstance')
+            ->willReturn($this->container);
+
+        $this->assertSame($this->container, $this->container->addServiceInstance($definition));
+        $this->assertSame($this->realContainer, $this->realContainer->addServiceInstance($definition));
+    }
+
+    /**
+     * @throws DuplicatedServiceException
+     */
+    public function test_AddServiceInstanceException()
+    {
+        $this->expectException(DuplicatedServiceException::class);
+
+        $definition = $this->getMockBuilder(InstanceDefinition::class)
+            ->setConstructorArgs([self::SERVICE_NAME, new \DateTime()])
+            ->getMock();
+
+        $definition->expects($this->atLeast(2))
+            ->method('getServiceName')
+            ->willReturn(self::SERVICE_NAME);
+
+        $this->realContainer->addServiceInstance($definition);
+        $this->realContainer->addServiceInstance($definition);
+    }
+
+    public function test_get()
+    {
+        $this->assertSame(null, $this->realContainer->get('test'));
+        $this->assertSame(\DateTime::class, get_class($this->realContainer->get(self::DEFAULT_SERVICE_NANE)));
+        $instance = $this->realContainer->get(self::DEFAULT_SERVICE_NANE);
+        $this->assertSame(\DateTime::class, get_class($instance));
+        $this->assertSame($instance, $this->realContainer->get(self::DEFAULT_SERVICE_NANE));
+
+        $this->realContainer->addServiceInstance(
+            new InstanceDefinition('call.format', 'd-m-Y')
+        );
+
+        $this->realContainer->addService(
+            new ServiceDefinition(
+                'service.call',
+                \DateTime::class,
+                ['1970-01-01'],
+                ['format' => ['call.format']]
+            )
+        );
+
+        $date = new \DateTime('1970-01-01');
+        $this->assertSame($date->format('m-d-Y'), $this->realContainer->get('service.call')->format('m-d-Y'));
+    }
+
+    public function test_has()
+    {
+        $this->assertSame(false, $this->realContainer->has('fake'));
+        $this->assertSame(true, $this->realContainer->has(self::DEFAULT_SERVICE_NANE));
+        $this->assertSame(true, $this->realContainer->has(self::DEFAULT_SERVICE_NANE));
+    }
 
 }