Backport: Object Storage retrieval: containerName and name missing (#… (php-opencloud#211)

* Backport: Object Storage retrieval: containerName and name missing (php-opencloud#210)

* Repopulate instance from input array

* Added unit tests

* Fixed authentication v2 bug

* Updated unit tests

# Conflicts:
#	tests/unit/ObjectStore/v1/Models/ContainerTest.php

* Fixed psr2

Author: haphan

.php_cs.dist

@@ -0,0 +1,20 @@
+
+<?php
+if (!file_exists(__DIR__.'/src')) {
+    exit(0);
+}
+return PhpCsFixer\Config::create()
+    ->setRules(
+        [
+            '@PSR2'                => true,
+            'array_syntax'         => ['syntax' => 'short'],
+            'protected_to_private' => false
+        ]
+    )
+    ->setUsingCache(false)
+    ->setRiskyAllowed(true)
+    ->setFinder(
+        PhpCsFixer\Finder::create()
+            ->in(__DIR__.'/src')
+            ->append([__FILE__, __DIR__.'/samples'])
+    );

.travis.yml

@@ -24,7 +24,7 @@ before_install:
 before_script:
   - composer install --prefer-source
   - vendor/bin/parallel-lint --exclude vendor .
-  - vendor/bin/php-cs-fixer fix --dry-run --diff --level psr2 .
+  - vendor/bin/php-cs-fixer fix --dry-run --diff
   - phpenv config-add ./xdebug.ini
 
 after_script:

composer.json

@@ -49,6 +49,6 @@
         "psr/log": "~1.0",
         "satooshi/php-coveralls": "~1.0",
         "jakub-onderka/php-parallel-lint": "0.*",
-        "friendsofphp/php-cs-fixer": "^1.0"
+        "friendsofphp/php-cs-fixer": "^2.9"
     }
 }

samples/identity/v2/authentication.php

@@ -5,22 +5,23 @@
 use GuzzleHttp\Client;
 use GuzzleHttp\HandlerStack;
 use OpenStack\Common\Transport\Utils as TransportUtils;
+use OpenStack\Identity\v2\Service;
 use OpenStack\OpenStack;
 
 $authUrl = 'https://keystone.example.com:5000/v2.0';
 
+$httpClient = new Client([
+    'base_uri' => TransportUtils::normalizeUrl($authUrl),
+    'handler'  => HandlerStack::create(),
+]);
+
 $options = [
     'authUrl'         => $authUrl,
     'region'          => 'RegionOne',
     'username'        => 'foo',
     'password'        => 'bar',
     'tenantName'      => 'baz',
-    'identityService' => new Client(
-        [
-            'base_uri' => TransportUtils::normalizeUrl($authUrl),
-            'handler'  => HandlerStack::create(),
-        ]
-    ),
+    'identityService' => Service::factory($httpClient),
 ];
 
 /** @var OpenStack $openstack */

src/Common/Api/Parameter.php

@@ -207,7 +207,9 @@ private function validateEnums($userValues)
     {
         if (!empty($this->enum) && $this->type == 'string' && !in_array($userValues, $this->enum)) {
             throw new \Exception(sprintf(
-                'The only permitted values are %s. You provided %s', implode(', ', $this->enum), print_r($userValues, true)
+                'The only permitted values are %s. You provided %s',
+                implode(', ', $this->enum),
+                print_r($userValues, true)
             ));
         }
     }
@@ -217,7 +219,10 @@ private function validateType($userValues)
         if (!$this->hasCorrectType($userValues)) {
             throw new \Exception(sprintf(
                 'The key provided "%s" has the wrong value type. You provided %s (%s) but was expecting %s',
-                $this->name, print_r($userValues, true), gettype($userValues), $this->type
+                $this->name,
+                print_r($userValues, true),
+                gettype($userValues),
+                $this->type
             ));
         }
     }

src/Common/Error/Builder.php

@@ -116,8 +116,11 @@ public function httpError(RequestInterface $request, ResponseInterface $response
     {
         $message = $this->header('HTTP Error');
 
-        $message .= sprintf("The remote server returned a \"%d %s\" error for the following transaction:\n\n",
-            $response->getStatusCode(), $response->getReasonPhrase());
+        $message .= sprintf(
+            "The remote server returned a \"%d %s\" error for the following transaction:\n\n",
+            $response->getStatusCode(),
+            $response->getReasonPhrase()
+        );
 
         $message .= $this->header('Request');
         $message .= trim($this->str($request)) . PHP_EOL . PHP_EOL;
@@ -163,8 +166,11 @@ public function userInputError(string $expectedType, $userValue, string $further
     {
         $message = $this->header('User Input Error');
 
-        $message .= sprintf("%s was expected, but the following value was passed in:\n\n%s\n",
-            $expectedType, print_r($userValue, true));
+        $message .= sprintf(
+            "%s was expected, but the following value was passed in:\n\n%s\n",
+            $expectedType,
+            print_r($userValue, true)
+        );
 
         $message .= "Please ensure that the value adheres to the expectation above. ";
 

src/Common/Service/Builder.php

@@ -166,7 +166,8 @@ private function mergeOptions(array $serviceOptions): array
 
         if (!isset($options['identityService']) || !($options['identityService'] instanceof IdentityService)) {
             throw new \InvalidArgumentException(sprintf(
-                '"identityService" must be specified and implement %s', IdentityService::class
+                '"identityService" must be specified and implement %s',
+                IdentityService::class
             ));
         }
 

src/Common/Transport/JsonSerializer.php

@@ -101,7 +101,8 @@ private function serializeObjectValue($value)
             } elseif (!($value instanceof \stdClass)) {
                 throw new \InvalidArgumentException(sprintf(
                     'When an object value is provided, it must either be \stdClass or implement the Serializable '
-                    . 'interface, you provided %s', print_r($value, true)
+                    . 'interface, you provided %s',
+                    print_r($value, true)
                 ));
             }
         }

src/Identity/v2/Models/Catalog.php

@@ -61,7 +61,10 @@ public function getServiceUrl(
 
         throw new \RuntimeException(sprintf(
             "Endpoint URL could not be found in the catalog for this service.\nName: %s\nType: %s\nRegion: %s\nURL type: %s",
-            $serviceName, $serviceType, $region, $urlType
+            $serviceName,
+            $serviceType,
+            $region,
+            $urlType
         ));
     }
 }

src/Identity/v3/Models/Catalog.php

@@ -56,7 +56,10 @@ public function getServiceUrl(string $name, string $type, string $region, string
 
         throw new \RuntimeException(sprintf(
             "Endpoint URL could not be found in the catalog for this service.\nName: %s\nType: %s\nRegion: %s\nURL type: %s",
-            $name, $type, $region, $urlType
+            $name,
+            $type,
+            $region,
+            $urlType
         ));
     }
 }