Fix the tests and dependencies.

Author: jsacksick

README.md

@@ -13,7 +13,7 @@ Shop at your favorite stores today and experience the freedom to pay later with
 SDK covers all of Klarna API: https://developers.klarna.com/api/
 
 ### Prerequisites
-* PHP 7.1 or above
+* PHP 8.1 or above
 * [API credentials](#api-credentials)
 * [Composer](https://getcomposer.org/) (optional)
 

composer.json

@@ -21,9 +21,8 @@
     },
     "require-dev": {
         "phpunit/phpunit": "~10.0",
-        "squizlabs/php_codesniffer": "1.5.*",
+        "squizlabs/php_codesniffer": "^3.0",
         "phpmd/phpmd": "2.1.*",
-        "phploc/phploc": "2.0.*",
         "php-coveralls/php-coveralls": "^2.1",
         "apigen/apigen": "4.1.*",
         "klarna/apigen-theme": "2.1.0"
@@ -38,12 +37,7 @@
         ],
         "analyze": [
             "mkdir -p build/logs",
-            "vendor/bin/phploc --log-csv build/logs/phploc.csv src/ tests/",
-            "mkdir -p build/pdepend",
-            "vendor/bin/pdepend --jdepend-xml=build/logs/jdepend.xml --jdepend-chart=build/pdepend/dependencies.svg --overview-pyramid=build/pdepend/overview-pyramid.svg src/",
-            "vendor/bin/phpmd src/,tests/ xml phpmd.xml --reportfile build/logs/pmd.xml || true",
-            "vendor/bin/phpcs --standard=PSR2 --report=checkstyle --report-file=build/logs/checkstyle.xml --extensions=php src/ tests/",
-            "vendor/bin/phpcpd --log-pmd build/logs/pmd-cpd.xml src/ tests/ || true"
+            "vendor/bin/phpcs --standard=PSR2 --report=checkstyle --report-file=build/logs/checkstyle.xml --extensions=php src/ tests/"
         ]
     }
 }

src/Klarna/Rest/Resource.php

@@ -106,7 +106,7 @@ public function setLocation($url)
     /**
      * Overrides: Stores the ID KEY field in order to restore it after exchanging the array without
      * the ID field.
-     * 
+     *
      * @param array $array Data to be exchanged
      */
     public function exchangeArray($array): array

src/Klarna/Rest/Transport/ApiResponse.php

@@ -104,8 +104,9 @@ public function setHeaders(array $headers)
         $this->headerNames = $this->headers = [];
         foreach ($headers as $header => $value) {
             if (is_int($header)) {
-                // Numeric array keys are converted to int by PHP but having a header name '123' is not forbidden by the spec
-                // and also allowed in withHeader(). So we need to cast it to string again for the following assertion to pass.
+                // Numeric array keys are converted to int by PHP but having a header name '123' is not forbidden
+                // by the spec and also allowed in withHeader().
+                // So we need to cast it to string again for the following assertion to pass.
                 $header = (string) $header;
             }
             $this->assertHeader($header);

src/Klarna/Rest/Transport/GuzzleConnector.php

@@ -88,7 +88,8 @@ public function __construct(
 
         if ($userAgent === null) {
             $class = new \ReflectionClass($client);
-            $version = $class->hasConstant('MAJOR_VERSION') ? $class->getConstant('MAJOR_VERSION') : $class->getConstant('VERSION');
+            $version = $class->hasConstant('MAJOR_VERSION') ? $class->getConstant('MAJOR_VERSION') :
+              $class->getConstant('VERSION');
             $userAgent = UserAgent::createDefault(['Guzzle/' . $version]);
         }
         $this->userAgent = $userAgent;

tests/Integration/Checkout/OrderTest.php

@@ -117,6 +117,6 @@ public function testHandlingExceptionsUnauthorized()
 
         putenv('PASSWORD=wrong_password');
         $output = $this->execFile($this->rootPath . '/docs/examples/CheckoutAPI/handling_exceptions.php');
-        $this->assertTrue($this->isTextPresents('Unexpected response HTTP status 401', $output));
+        $this->assertTrue($this->isTextPresents('Code: 401', $output));
     }
 }

tests/Unit/Checkout/OrderTest.php

@@ -74,12 +74,12 @@ public function testCreate()
             ->will($this->returnValue('{}'));
 
 
-      $this->response->expects($this->exactly(2))
-          ->method('getHeader')
-          ->willReturnMap([
-              ['Content-Type', ['application/json']],
-              ['Location', ['http://somewhere/a-path']]
-          ]);
+        $this->response->expects($this->exactly(2))
+            ->method('getHeader')
+            ->willReturnMap([
+                ['Content-Type', ['application/json']],
+                ['Location', ['http://somewhere/a-path']]
+            ]);
 
         $order = new Order($this->connector);
         $location = $order->create($data)
@@ -126,12 +126,12 @@ public function testCreateNoContentType()
             ->method('getStatus')
             ->will($this->returnValue('201'));
 
-      $this->response
-          ->expects($this->exactly(1))
-          ->method('getHeader')
-          ->willReturnMap([
-              ['Content-Type', []]
-        ]);
+        $this->response
+            ->expects($this->exactly(1))
+            ->method('getHeader')
+            ->willReturnMap([
+                ['Content-Type', []]
+          ]);
 
         $order = new Order($this->connector);