slickframework / di

spec/ContainerBuilderSpec.php

Unused Use Statements +3 added lines, -3 removed lines

@@ -2,11 +2,11 @@
 
 namespace spec\Slick\Di;
 
-use Slick\Di\ContainerInterface;
-use Slick\Di\ContainerAwareInterface;
-use Slick\Di\ContainerBuilder;
 use PhpSpec\ObjectBehavior;
 use Prophecy\Argument;
+use Slick\Di\ContainerAwareInterface;
+use Slick\Di\ContainerBuilder;
+use Slick\Di\ContainerInterface;
 
 class ContainerBuilderSpec extends ObjectBehavior
 {

Spacing +2 added lines, -2 removed lines

@@ -41,7 +41,7 @@ 
     function it_hydrates_a_container_with_an_array_from_a_file(ContainerInterface $container)
     {
         $container->register(Argument::type('string'), Argument::any())->shouldBeCalledTimes(3);
-        $this->beConstructedWith(__DIR__ .'/services/services1.php');
+        $this->beConstructedWith(__DIR__.'/services/services1.php');
         $this->setContainer($container);
         $this->getContainer()->shouldBeAnInstanceOf(ContainerInterface::class);
     }
@@ -49,7 +49,7 @@ 
     function it_hydrates_a_container_with_all_arrays_form_the_files_within_a_directory(ContainerInterface $container)
     {
         $container->register(Argument::type('string'), Argument::any())->shouldBeCalledTimes(6);
-        $this->beConstructedWith(__DIR__ .'/services');
+        $this->beConstructedWith(__DIR__.'/services');
         $this->setContainer($container);
         $this->getContainer()->shouldBeAnInstanceOf(ContainerInterface::class);
     }

spec/ContainerSpec.php

Unused Use Statements +2 added lines, -2 removed lines

@@ -11,12 +11,12 @@
 
 use Interop\Container\ContainerInterface;
 use Interop\Container\Exception\NotFoundException;
-use Slick\Di\Container;
 use PhpSpec\ObjectBehavior;
 use Prophecy\Argument;
+use Slick\Di\Container;
 use Slick\Di\ContainerInjectionInterface;
-use Slick\Di\Definition\Scope;
 use Slick\Di\DefinitionInterface;
+use Slick\Di\Definition\Scope;
 use Slick\Di\ObjectHydratorAwareInterface;
 use Slick\Di\ObjectHydratorInterface;
 

Spacing +4 added lines, -4 removed lines

@@ -76,7 +76,7 @@ 
 
     function it_registers_a_callable_executing_it_when_resolving()
     {
-        $callable = function ($test) {
+        $callable = function($test) {
             return $test;
         };
         $this->register(
@@ -90,7 +90,7 @@ 
 
     function it_registers_an_alias_that_points_to_other_definition()
     {
-        $object = (object)[];
+        $object = (object) [];
         $this->register('test-object', $object);
         $this->register('alias', '@test-object');
         $this->get('alias')->shouldBe($object);
@@ -98,10 +98,10 @@ 
 
     function it_is_aware_of_definition_resolution_scope()
     {
-        $callable = function () {
+        $callable = function() {
             static $calls;
 
-            if (!$calls) { $calls = 0;}
+            if (!$calls) { $calls = 0; }
 
             return $calls++;
         };

spec/Definition/AliasSpec.php

Unused Use Statements +2 added lines, -3 removed lines

@@ -2,11 +2,10 @@
 
 namespace spec\Slick\Di\Definition;
 
-use Slick\Di\ContainerInterface;
-use Slick\Di\Definition\Alias;
 use PhpSpec\ObjectBehavior;
-use Prophecy\Argument;
+use Slick\Di\ContainerInterface;
 use Slick\Di\DefinitionInterface;
+use Slick\Di\Definition\Alias;
 use Slick\Di\Exception\ContainerNotSetException;
 
 class AliasSpec extends ObjectBehavior

spec/Definition/FactorySpec.php

Unused Use Statements +1 added lines, -2 removed lines

@@ -2,10 +2,9 @@
 
 namespace spec\Slick\Di\Definition;
 
-use Slick\Di\Definition\Factory;
 use PhpSpec\ObjectBehavior;
-use Prophecy\Argument;
 use Slick\Di\DefinitionInterface;
+use Slick\Di\Definition\Factory;
 
 class FactorySpec extends ObjectBehavior
 {

Spacing +2 added lines, -2 removed lines

@@ -12,7 +12,7 @@ 
     function let()
     {
         $this->beConstructedWith(
-            function (){
+            function() {
                 return 'Test Callable';
             }
         );
@@ -36,7 +36,7 @@ 
     function it_can_have_parameters_to_pass_to_callable_on_resolve()
     {
         $this->beConstructedWith(
-            function ($test) {
+            function($test) {
                 return $test;
             },
             ['test with parameters']

spec/Definition/Object/ResolverSpec.php

Unused Use Statements +1 added lines, -2 removed lines

@@ -2,12 +2,11 @@
 
 namespace spec\Slick\Di\Definition\Object;
 
+use PhpSpec\ObjectBehavior;
 use Slick\Di\ContainerInterface;
 use Slick\Di\Definition\Object\DefinitionData;
 use Slick\Di\Definition\Object\Resolver;
 use Slick\Di\Definition\Object\ResolverInterface;
-use PhpSpec\ObjectBehavior;
-use Prophecy\Argument;
 
 class ResolverSpec extends ObjectBehavior
 {

spec/Definition/ObjectDefinitionSpec.php

Unused Use Statements +2 added lines, -2 removed lines

@@ -5,11 +5,11 @@
 use PhpSpec\Exception\Example\FailureException;
 use PhpSpec\ObjectBehavior;
 use Slick\Di\ContainerInterface;
+use Slick\Di\DefinitionInterface;
 use Slick\Di\Definition\FluentObjectDefinitionInterface;
+use Slick\Di\Definition\ObjectDefinition;
 use Slick\Di\Definition\Object\DefinitionData;
 use Slick\Di\Definition\Object\ResolverInterface;
-use Slick\Di\Definition\ObjectDefinition;
-use Slick\Di\DefinitionInterface;
 use Slick\Di\Exception\ClassNotFoundException;
 use Slick\Di\Exception\MethodNotFoundException;
 use Slick\Di\Exception\PropertyNotFoundException;

Indentation +1 added lines, -1 removed lines

@@ -103,7 +103,7 @@
     {
         return [
             'haveArgumentsEquals' => function ($subject, $arguments) {
-                 return $subject->arguments == $arguments;
+                    return $subject->arguments == $arguments;
             },
             'haveACallEquals' => function (DefinitionData $subject, $expectedCall) {
                 $call = null;

Spacing +3 added lines, -3 removed lines

@@ -102,10 +102,10 @@ 
     public function getMatchers()
     {
         return [
-            'haveArgumentsEquals' => function ($subject, $arguments) {
+            'haveArgumentsEquals' => function($subject, $arguments) {
                  return $subject->arguments == $arguments;
             },
-            'haveACallEquals' => function (DefinitionData $subject, $expectedCall) {
+            'haveACallEquals' => function(DefinitionData $subject, $expectedCall) {
                 $call = null;
                 foreach ($subject->calls as $call) {
                     if ($call == $expectedCall) {
@@ -116,7 +116,7 @@ 
                 try {
                     \PHPUnit_Framework_Assert::assertEquals($expectedCall, $call);
                 } catch (\PHPUnit_Framework_ExpectationFailedException $caught) {
-                    $message .=  "\n". $caught->getComparisonFailure()->getDiff();
+                    $message .= "\n".$caught->getComparisonFailure()->getDiff();
 
                 }
 

spec/Definition/ValueSpec.php

Unused Use Statements +1 added lines, -2 removed lines

@@ -2,10 +2,9 @@
 
 namespace spec\Slick\Di\Definition;
 
-use Slick\Di\Definition\Value;
 use PhpSpec\ObjectBehavior;
-use Prophecy\Argument;
 use Slick\Di\DefinitionInterface;
+use Slick\Di\Definition\Value;
 
 class ValueSpec extends ObjectBehavior
 {

spec/ObjectHydratorSpec.php

Unused Use Statements +1 added lines, -3 removed lines

@@ -2,11 +2,9 @@
 
 namespace spec\Slick\Di;
 
-use Slick\Di\ContainerAwareInterface;
+use PhpSpec\ObjectBehavior;
 use Slick\Di\ContainerInterface;
 use Slick\Di\ObjectHydrator;
-use PhpSpec\ObjectBehavior;
-use Prophecy\Argument;
 use Slick\Di\ObjectHydratorInterface;
 
 require_once 'DryClass.php';

src/Definition/ObjectDefinition.php

Doc Comments +5 added lines, -5 removed lines

@@ -102,7 +102,7 @@ 
      *
      * @param array ...$arguments Arguments passed to object constructor
      *
-     * @return $this|Object|self
+     * @return ObjectDefinition
      */
     public function with(...$arguments)
     {
@@ -123,7 +123,7 @@ 
      *
      * @param array ...$arguments
      *
-     * @return self|Object
+     * @return ObjectDefinition
      */
     public function withConstructorArgument(...$arguments)
     {
@@ -217,7 +217,7 @@ 
      *
      * @param mixed $value
      *
-     * @return self|Object
+     * @return ObjectDefinition
      */
     public function assign($value)
     {
@@ -232,7 +232,7 @@ 
      *
      * @param string $property
      *
-     * @return self|Object
+     * @return ObjectDefinition
      */
     public function to($property)
     {
@@ -256,7 +256,7 @@ 
      * @param string $name
      * @param mixed  $value
      *
-     * @return self|Object
+     * @return ObjectDefinition
      */
     public function assignProperty($name, $value)
     {

Spacing +6 added lines, -6 removed lines

@@ -57,10 +57,10 @@ 
      */
     public function __construct($className)
     {
-        if (! class_exists($className)) {
+        if (!class_exists($className)) {
             throw new ClassNotFoundException(
-                "The class '{$className}' does not exists or it cannot be " .
-                "loaded. Object definition therefore cannot be " .
+                "The class '{$className}' does not exists or it cannot be ".
+                "loaded. Object definition therefore cannot be ".
                 "created."
             );
         }
@@ -197,7 +197,7 @@ 
     public function callMethod($methodName, ...$arguments)
     {
 
-        if (! method_exists($this->definitionData->className, $methodName)) {
+        if (!method_exists($this->definitionData->className, $methodName)) {
             throw new MethodNotFoundException(
                 "The method '{$methodName}' is not defined in the class ".
                 "{$this->definitionData->className}"
@@ -236,9 +236,9 @@ 
      */
     public function to($property)
     {
-        if (! property_exists($this->definitionData->className, $property)) {
+        if (!property_exists($this->definitionData->className, $property)) {
             throw new PropertyNotFoundException(
-                "The class '{$this->definitionData->className}' has no " .
+                "The class '{$this->definitionData->className}' has no ".
                 "property called '{$property}'."
             );
         }