facile-it / moka

src/Proxy/ProxyTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     private $__moka_methods = [];
 
     /**
-     * @return object
+     * @return \PHPUnit\Framework\MockObject\MockObject
      */
     public function __moka_getMock()
     {

Spacing +2 added lines, -2 removed lines

@@ -100,7 +100,7 @@ 
      */
     protected function doCall(string $name, array $arguments)
     {
-        if (!$this->__moka_mockingStrategy instanceof MockingStrategyInterface) {
+        if ( ! $this->__moka_mockingStrategy instanceof MockingStrategyInterface) {
             return null;
         }
 
@@ -117,7 +117,7 @@ 
      */
     protected function doGet(string $name)
     {
-        if (!$this->__moka_mockingStrategy instanceof MockingStrategyInterface) {
+        if ( ! $this->__moka_mockingStrategy instanceof MockingStrategyInterface) {
             return null;
         }
 

src/Strategy/AbstractMockingStrategy.php

Unused Use Statements -2 removed lines

@@ -8,11 +8,9 @@
 use Moka\Exception\MockNotCreatedException;
 use Moka\Exception\NotImplementedException;
 use function Moka\Factory\buildStubs;
-use Moka\Factory\StubFactory;
 use Moka\Stub\MethodStub;
 use Moka\Stub\PropertyStub;
 use Moka\Stub\StubInterface;
-use Moka\Stub\StubSet;
 
 /**
  * Class AbstractMockingStrategy

Spacing +3 added lines, -3 removed lines

@@ -33,7 +33,7 @@ 
      */
     final protected static function checkDependencies(string $dependencyClassName, string $dependencyPackageName)
     {
-        if (!class_exists($dependencyClassName)) {
+        if ( ! class_exists($dependencyClassName)) {
             throw new MissingDependencyException(
                 sprintf(
                     'Class "%s" does not exist, please install package "%s"',
@@ -151,7 +151,7 @@ 
     {
         $this->verifyMockType();
 
-        if (!is_a($mock, $this->mockType)) {
+        if ( ! is_a($mock, $this->mockType)) {
             throw new InvalidArgumentException(
                 sprintf(
                     'Mock object must be an instance of "%s", "%s" given',
@@ -169,7 +169,7 @@ 
      */
     private function verifyMockType(): void
     {
-        if (!$this->mockType) {
+        if ( ! $this->mockType) {
             throw new NotImplementedException('Mock type was not defined');
         }
     }

src/Proxy/ProxyContainer.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
      */
     public function get($id): ProxyInterface
     {
-        if (!$this->has($id)) {
+        if ( ! $this->has($id)) {
             throw new InvalidIdentifierException(
                 sprintf(
                     'Cannot find mock object with identifier "%s"',

src/Tests/IncompleteAbstractTestClass.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
 
     public function getCallable(): callable
     {
-        return function () {
+        return function() {
         };
     }
 

src/Tests/MokaMockingStrategyTestCase.php

Spacing +4 added lines, -4 removed lines

@@ -309,17 +309,17 @@ 
         ];
 
         $data = array_merge(
-            array_map(function (array $set) {
+            array_map(function(array $set) {
                 array_unshift($set, $required = true);
                 return $set;
             }, $required),
-            array_map(function (array $set) {
+            array_map(function(array $set) {
                 array_unshift($set, $required = false);
                 return $set;
             }, $optional)
         );
 
-        return array_reduce($data, function (array $data, array $set) {
+        return array_reduce($data, function(array $data, array $set) {
             $key = preg_replace('/^an? +/', '', $set[1]);
             $data[$key] = $set;
 
@@ -424,7 +424,7 @@ 
 
         $object = $this->strategy->get($mock);
         foreach ($fqcns as $fqcn) {
-            if (!is_a($object, $fqcn)) {
+            if ( ! is_a($object, $fqcn)) {
                 $this->markFeatureUnsupported(
                     sprintf(
                         'getting a valid mock from %s',

src/Generator/Template/ParameterTemplate.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
     protected static function doGenerate(\ReflectionParameter $parameter): string
     {
         try {
-            $defaultValue = !$parameter->allowsNull()
+            $defaultValue = ! $parameter->allowsNull()
                 ? var_export($parameter->getDefaultValue(), true)
                 : 'null';
 

src/Generator/Template/ClassTemplate.php

Spacing +2 added lines, -2 removed lines

@@ -84,8 +84,8 @@
 
         foreach ($methods as $method) {
             if (
-                !$method->isFinal() &&
-                !\in_array($method->name, self::UNSAFE_METHODS, $strict = true)
+                ! $method->isFinal() &&
+                ! \in_array($method->name, self::UNSAFE_METHODS, $strict = true)
             ) {
                 $methodsCode[] = MethodTemplate::generate($method);
             }

src/Generator/Template/VisibilityTrait.php

Spacing +2 added lines, -2 removed lines

@@ -21,8 +21,8 @@
     protected static function getVisibility(\Reflector $reflector): string
     {
         if (
-            !$reflector instanceof \ReflectionMethod &&
-            !$reflector instanceof \ReflectionProperty
+            ! $reflector instanceof \ReflectionMethod &&
+            ! $reflector instanceof \ReflectionProperty
         ) {
             throw new InvalidArgumentException(
                 sprintf(

src/Builder/ProxyBuilder.php

Spacing +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
             return $this->buildProxy($fqcnOrAlias);
         }
 
-        if (!$this->container->has($alias)) {
+        if ( ! $this->container->has($alias)) {
             $this->container->set($alias, $this->buildProxy($fqcnOrAlias));
         }