mapado / rest-client-sdk

Tests/Units/Mapping.php

Doc Comments +1 added lines, -1 removed lines

@@ -202,7 +202,7 @@
      * getMappingArray
      *
      * @access private
-     * @return void
+     * @return ClassMetadata[]
      */
     private function getMappingArray()
     {

Spacing +7 added lines, -7 removed lines

@@ -24,7 +24,7 @@ 
             ->given($this->newTestedInstance)
                 ->and($this->testedInstance->setMapping([new ClassMetadata('foo', null, null)]))
             ->then($testedInstance = $this->testedInstance)
-            ->exception(function () use ($testedInstance) {
+            ->exception(function() use ($testedInstance) {
                 @$testedInstance->getModelName();
             })
                 ->isInstanceOf('Mapado\RestClientSdk\Exception\MappingException')
@@ -34,7 +34,7 @@ 
             ->given($this->newTestedInstance)
                 ->and($this->testedInstance->setMapping([new ClassMetadata('foo', null, null)]))
             ->then($testedInstance = $this->testedInstance)
-            ->exception(function () use ($testedInstance) {
+            ->exception(function() use ($testedInstance) {
                 $testedInstance->getModelName('orders');
             })
                 ->isInstanceOf('Mapado\RestClientSdk\Exception\MappingException')
@@ -44,7 +44,7 @@ 
             ->given($this->newTestedInstance)
             ->and($this->testedInstance->setMapping([new ClassMetadata('orders', null, null)]))
             ->then($testedInstance = $this->testedInstance)
-            ->exception(function () use ($testedInstance) {
+            ->exception(function() use ($testedInstance) {
                 $testedInstance->getModelName('orders');
             })
                 ->isInstanceOf('Mapado\RestClientSdk\Exception\MappingException')
@@ -92,7 +92,7 @@ 
             // no mappings
             ->given($this->newTestedInstance)
             ->then($testedInstance = $this->testedInstance)
-                ->exception(function () use ($testedInstance) {
+                ->exception(function() use ($testedInstance) {
                     $testedInstance->getKeyFromId('/orders/8');
                 })
                 ->isInstanceOf('Mapado\RestClientSdk\Exception\MappingException')
@@ -141,7 +141,7 @@ 
                     ->isEqualTo('order_items')
 
             ->then($testedInstance = $this->testedInstance)
-            ->exception(function () use ($testedInstance) {
+            ->exception(function() use ($testedInstance) {
                 $testedInstance->getKeyFromModel('\Not\Viable\Classname');
             })
                 ->isInstanceOf('Mapado\RestClientSdk\Exception\MappingException')
@@ -167,7 +167,7 @@ 
                     ->isInstanceOf('Mapado\RestClientSdk\Mapping\ClassMetadata')
 
             ->then
-            ->exception(function () use ($testedInstance) {
+            ->exception(function() use ($testedInstance) {
                 $testedInstance->getClassMetadata('Foo\Bar');
             })
                     ->isInstanceOf('Mapado\RestClientSdk\Exception\MappingException')
@@ -224,6 +224,6 @@ 
             'Foo\Bar\Client\ClientClient'
         );
 
-        return [ $order, $orderItem, $client, ];
+        return [$order, $orderItem, $client, ];
     }
 }

Tests/Model/Cart.php

Spacing +1 added lines, -1 removed lines

@@ -154,7 +154,7 @@
 
     public function addCartItemList($cartItem)
     {
-        $this->cartItemList[] =  $cartItem;
+        $this->cartItemList[] = $cartItem;
     }
 
     /**

Tests/Units/RestClient.php

Indentation +2 added lines, -2 removed lines

@@ -71,7 +71,7 @@ 
             ->then
                 ->object($this->testedInstance->get('/not-json'))
                     ->isInstanceOf('\GuzzleHttp\Psr7\Response')
-           ;
+            ;
     }
 
     /**
@@ -110,7 +110,7 @@ 
                     ->isInstanceOf('Mapado\RestClientSdk\Exception\RestException')
                     ->hasMessage('Error while deleting resource')
                     ->hasCode(2)
-           ;
+            ;
     }
 
     /**

Spacing +9 added lines, -9 removed lines

@@ -60,7 +60,7 @@ 
                     ->isNull()
 
             ->then($testedInstance = $this->testedInstance)
-                ->exception(function () use ($testedInstance) {
+                ->exception(function() use ($testedInstance) {
                     $testedInstance->get('/error');
                 })
                     ->isInstanceOf('Mapado\RestClientSdk\Exception\RestException')
@@ -104,7 +104,7 @@ 
                     ->isNull()
 
             ->then($testedInstance = $this->testedInstance)
-                ->exception(function () use ($testedInstance) {
+                ->exception(function() use ($testedInstance) {
                     $testedInstance->delete('/error');
                 })
                     ->isInstanceOf('Mapado\RestClientSdk\Exception\RestException')
@@ -149,7 +149,7 @@ 
                     ->string($data['activityUuid'])
                         ->isEqualTo('63d108be-629c-11e5-b5ce-f153631dac50')
             ->then($testedInstance = $this->testedInstance)
-                ->exception(function () use ($testedInstance, $params) {
+                ->exception(function() use ($testedInstance, $params) {
                     $testedInstance->post('/not-found', $params);
                 })
                     ->isInstanceOf('Mapado\RestClientSdk\Exception\RestClientException')
@@ -157,7 +157,7 @@ 
                     ->hasCode(3)
 
             ->then
-                ->exception(function () use ($testedInstance, $params) {
+                ->exception(function() use ($testedInstance, $params) {
                     $testedInstance->post('/error', $params);
                 })
                     ->isInstanceOf('Mapado\RestClientSdk\Exception\RestException')
@@ -202,7 +202,7 @@ 
                     ->string($data['activityUuid'])
                         ->isEqualTo('a9e82f60-629e-11e5-8903-0d978bd11e5d')
             ->then($testedInstance = $this->testedInstance)
-                ->exception(function () use ($testedInstance, $params) {
+                ->exception(function() use ($testedInstance, $params) {
                     $testedInstance->put('/not-found', $params);
                 })
                     ->isInstanceOf('Mapado\RestClientSdk\Exception\RestClientException')
@@ -210,7 +210,7 @@ 
                     ->hasCode(5)
 
             ->then
-                ->exception(function () use ($testedInstance, $params) {
+                ->exception(function() use ($testedInstance, $params) {
                     $testedInstance->put('/error', $params);
                 })
                     ->isInstanceOf('Mapado\RestClientSdk\Exception\RestException')
@@ -275,7 +275,7 @@ 
 
             ->if($testedInstance = $this->testedInstance)
             ->and
-                ->exception(function () use ($testedInstance) {
+                ->exception(function() use ($testedInstance) {
                     $testedInstance->get('/');
                 })
             ->then
@@ -290,13 +290,13 @@ 
             [
                 new Response(
                     200,
-                    [ 'Content-Type' => 'application/ld+json' ],
+                    ['Content-Type' => 'application/ld+json'],
                     file_get_contents(__DIR__ . '/../data/ticketing.list.no_result.json')
                 ),
 
                 new Response(
                     200,
-                    [ 'Content-Type' => 'application/ld+json' ],
+                    ['Content-Type' => 'application/ld+json'],
                     file_get_contents(__DIR__ . '/../data/ticketing.list.no_result.json')
                 ),
             ]

Tests/Units/SdkClient.php

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
                 ->object($testedInstance->getRepository('orders'))
                     ->isInstanceOf('Mapado\RestClientSdk\Tests\Model\ModelRepository')
 
-                ->exception(function () use ($testedInstance) {
+                ->exception(function() use ($testedInstance) {
                     $testedInstance->getRepository('foo');
                 })
                     ->isInstanceOf('Mapado\RestClientSdk\Exception\MappingException')

src/Mapping/Driver/AnnotationDriver.php

Spacing +2 added lines, -2 removed lines

@@ -76,7 +76,7 @@ 
 
         foreach ($iterator as $file) {
             $sourceFile = $file[0];
-            if (! preg_match('(^phar:)i', $sourceFile)) {
+            if (!preg_match('(^phar:)i', $sourceFile)) {
                 $sourceFile = realpath($sourceFile);
             }
 
@@ -114,7 +114,7 @@ 
     {
         $metadata = $this->getClassMetadataForClassname($classname);
 
-        return $metadata ? [$metadata,] : [];
+        return $metadata ? [$metadata, ] : [];
     }
 
     /**

src/Mapping.php

Spacing +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
     public function getMappingKeys()
     {
         return array_map(
-            function ($mapping) {
+            function($mapping) {
                 return $mapping->getKey();
             },
             $this->mapping

src/Model/ModelHydrator.php

Spacing +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
     public function deserializeAll($data, $modelName)
     {
         $data['hydra:member'] = array_map(
-            function ($member) use ($modelName) {
+            function($member) use ($modelName) {
                 return $this->deserialize($member, $modelName);
             },
             $data['hydra:member']

Indentation +7 added lines, -7 removed lines

@@ -33,13 +33,13 @@
     }
 
     /**
-      * convertId
-      *
-      * @param string $id
-      * @param string $modelName
-      * @access public
-      * @return string
-      */
+     * convertId
+     *
+     * @param string $id
+     * @param string $modelName
+     * @access public
+     * @return string
+     */
     public function convertId($id, $modelName)
     {
         $id = (string) $id;

src/EntityRepository.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 namespace Mapado\RestClientSdk;
 
 use Mapado\RestClientSdk\Exception\SdkException;
-use Symfony\Component\Cache\CacheItem;
 
 class EntityRepository
 {

Spacing +1 added lines, -1 removed lines

@@ -262,7 +262,7 @@
         $mapping = $this->sdk->getMapping();
 
         return array_map(
-            function ($item) use ($mapping) {
+            function($item) use ($mapping) {
                 if (is_object($item)) {
                     $classname = get_class($item);
 

Tests/Units/EntityRepository.php

Doc Comments -1 removed lines

@@ -125,7 +125,6 @@
     /**
      * testFindWithCache
      *
-     * @param mixed $method
      * @access public
      * @return void
      */

Spacing +6 added lines, -6 removed lines

@@ -114,7 +114,7 @@ 
         $this->calling($this->mockedRestClient)->get = [];
 
         $this
-            ->if($this->repository->find('1', [ 'foo' => 'bar', 'bar'  => 'baz' ]))
+            ->if($this->repository->find('1', ['foo' => 'bar', 'bar'  => 'baz']))
             ->then
                 ->mock($this->mockedRestClient)
                     ->call('get')
@@ -177,8 +177,8 @@ 
             ->given($this->resetMock($this->mockedRestClient))
                 ->and($this->mockedSdk->getCacheItemPool()->clear())
 
-            ->if($this->repository->findBy([ 'foo' => 'bar', 'bar'  => 'baz' ]))
-            ->and($this->repository->findBy([ 'foo' => 'bar', 'bar'  => 'baz' ]))
+            ->if($this->repository->findBy(['foo' => 'bar', 'bar'  => 'baz']))
+            ->and($this->repository->findBy(['foo' => 'bar', 'bar'  => 'baz']))
             ->if($this->repository->find(1))
             ->then
                 ->mock($this->mockedRestClient)
@@ -203,8 +203,8 @@ 
             // find one by
             ->given($this->resetMock($this->mockedRestClient))
 
-            ->if($this->repository->findOneBy([ 'foo' => 'baz', 'bar'  => 'bar' ]))
-            ->and($this->repository->findOneBy([ 'foo' => 'baz', 'bar'  => 'bar' ]))
+            ->if($this->repository->findOneBy(['foo' => 'baz', 'bar'  => 'bar']))
+            ->and($this->repository->findOneBy(['foo' => 'baz', 'bar'  => 'bar']))
             ->then
                 ->mock($this->mockedRestClient)
                     ->call('get')
@@ -222,7 +222,7 @@ 
 
             // find one by with data already in cache
             ->given($this->resetMock($this->mockedRestClient))
-            ->if($this->repository->findOneBy([ 'foo' => 'bar', 'bar'  => 'baz' ]))
+            ->if($this->repository->findOneBy(['foo' => 'bar', 'bar'  => 'baz']))
             ->then
                 ->mock($this->mockedRestClient)
                     ->call('get')

Indentation +9 added lines, -9 removed lines

@@ -340,10 +340,10 @@ 
         $this
             ->if($repository->findBy(['section' => $section1]))
             // ->then(ldd(iterator_to_array($arrayAdapter->getItems())))
-           ->then
-               ->mock($this->mockedRestClient)
-                   ->call('get')
-                       ->withArguments('/sections?section=%2Fsections%2F1')->once()
+            ->then
+                ->mock($this->mockedRestClient)
+                    ->call('get')
+                        ->withArguments('/sections?section=%2Fsections%2F1')->once()
 
             ->if($repository->findAll())
             // ->then(ldd(iterator_to_array($arrayAdapter->getItems())))
@@ -351,11 +351,11 @@ 
                 ->boolean($arrayAdapter->hasItem('test_prefix__sections_1'))
                     ->isTrue()
 
-           ->if($repository->find(1))
-           ->then
-               ->mock($this->mockedRestClient)
-                   ->call('get')
-                       ->withArguments('/sections/1')->never()
+            ->if($repository->find(1))
+            ->then
+                ->mock($this->mockedRestClient)
+                    ->call('get')
+                        ->withArguments('/sections/1')->never()
 
             // after update
             ->if($repository->update($section1))