Unit test (#360)

* fix tests for new Guzzle

* unit tests workflow

Author: k0ka

.github/workflows/unit_tests.yml

@@ -0,0 +1,36 @@
+name: Run Unit Tests
+on:
+  push:
+  pull_request:
+jobs:
+  tests:
+    runs-on: ubuntu-22.04
+    if: "!contains(github.event.head_commit.message, 'skip ci') && !contains(github.event.head_commit.message, 'ci skip')"
+    strategy:
+      fail-fast: false
+      matrix:
+         php: [ 7.2, 7.3, 7.4, 8.0, 8.1 ]
+    name: "php-${{ matrix.php }}"
+    steps:
+      - uses: actions/checkout@v2
+      - name: get cache directory
+        id: composer-cache
+        run: |
+          echo "::set-output name=dir::$(composer config cache-files-dir)"
+      - uses: actions/cache@v2
+        with:
+          path: |
+            ~/.php_cs.cache
+            ${{ steps.composer-cache.outputs.dir }}
+          key: ${{ runner.os }}-cache-${{ matrix.php }}-${{ hashFiles('**.composer.lock') }}
+      - uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php }}
+          extensions: curl
+          tools: composer:v2
+          coverage: none
+      - run: composer install --prefer-dist --no-interaction --no-progress
+      - run: vendor/bin/parallel-lint --exclude vendor .
+      - name: execute unit tests
+        run: vendor/bin/phpunit --configuration phpunit.xml.dist
+

.gitignore

@@ -4,6 +4,7 @@ coverage/
 vendor/
 *.pyc
 phpunit.xml
+.phpunit.result.cache
 coverage.xml
 composer.lock
 env_test.sh

composer.json

@@ -43,7 +43,8 @@
         "psr/log": "^1.0",
         "php-coveralls/php-coveralls": "^2.0",
         "friendsofphp/php-cs-fixer": "^2.18",
-        "php-parallel-lint/php-parallel-lint": "^1.2"
+        "php-parallel-lint/php-parallel-lint": "^1.2",
+        "phpspec/prophecy": "^1.17"
     },
     "extra": {
         "branch-alias": {

samples/images/v2/images/upload_binary_data.php

@@ -12,5 +12,9 @@
 $service = $openstack->imagesV2();
 
 $image  = $service->getImage('{imageId}');
-$stream = \GuzzleHttp\Psr7\stream_for(fopen('{fileName}', 'r'));
+
+$stream = function_exists('\GuzzleHttp\Psr7\stream_for')
+    ? \GuzzleHttp\Psr7\stream_for(fopen('{fileName}', 'r'))
+    : \GuzzleHttp\Psr7\Utils::streamFor(fopen('{fileName}', 'r'));
+
 $image->uploadData($stream);

tests/unit/Common/Resource/OperatorResourceTest.php

@@ -28,7 +28,9 @@ public function setUp(): void
 
     public function test_it_retrieves_base_http_url()
     {
-        $returnedUri = \GuzzleHttp\Psr7\uri_for('http://foo.com');
+        $returnedUri = function_exists('\GuzzleHttp\Psr7\uri_for')
+            ? \GuzzleHttp\Psr7\uri_for('http://foo.com')
+            : \GuzzleHttp\Psr7\Utils::uriFor('http://foo.com');
         $this->client->getConfig('base_uri')->shouldBeCalled()->willReturn($returnedUri);
 
         $uri = $this->resource->testBaseUri();

tests/unit/Images/v2/Models/ImageTest.php

@@ -30,7 +30,11 @@ public function setUp(): void
 
     public function test_it_retrieves()
     {
-        $this->client->getConfig('base_uri')->shouldBeCalled()->willReturn(\GuzzleHttp\Psr7\uri_for(''));
+        $returnedUri = function_exists('\GuzzleHttp\Psr7\uri_for')
+            ? \GuzzleHttp\Psr7\uri_for('')
+            : \GuzzleHttp\Psr7\Utils::uriFor('');
+
+        $this->client->getConfig('base_uri')->shouldBeCalled()->willReturn($returnedUri);
 
         $this->setupMock('GET', $this->path, null, [], 'GET_image');
 
@@ -123,7 +127,10 @@ public function test_it_deactivates()
 
     public function test_it_uploads_data_stream()
     {
-        $stream  = \GuzzleHttp\Psr7\stream_for('data');
+        $stream = function_exists('\GuzzleHttp\Psr7\stream_for')
+            ? \GuzzleHttp\Psr7\stream_for('data')
+            : \GuzzleHttp\Psr7\Utils::streamFor('data');
+
         $headers = ['Content-Type' => 'application/octet-stream'];
 
         $this->setupMock('PUT', $this->path . '/file', $stream, $headers, new Response(204));
@@ -133,7 +140,10 @@ public function test_it_uploads_data_stream()
 
     public function test_it_downloads_data()
     {
-        $stream  = \GuzzleHttp\Psr7\stream_for('data');
+        $stream = function_exists('\GuzzleHttp\Psr7\stream_for')
+            ? \GuzzleHttp\Psr7\stream_for('data')
+            : \GuzzleHttp\Psr7\Utils::streamFor('data');
+
         $headers = ['Content-Type' => 'application/octet-stream'];
         $response = new Response(200, $headers, $stream);
 

tests/unit/Images/v2/ServiceTest.php

@@ -23,10 +23,14 @@ public function setUp(): void
 
     public function test_it_creates_image()
     {
+        $returnedUri = function_exists('\GuzzleHttp\Psr7\uri_for')
+            ? \GuzzleHttp\Psr7\uri_for('')
+            : \GuzzleHttp\Psr7\Utils::uriFor('');
+
         $this->client
             ->getConfig('base_uri')
             ->shouldBeCalled()
-            ->willReturn(\GuzzleHttp\Psr7\uri_for(''));
+            ->willReturn($returnedUri);
 
         $expectedJson = [
             "name" => "Ubuntu 12.10",
@@ -58,10 +62,14 @@ public function test_it_creates_image()
 
     public function test_it_lists_images()
     {
+        $returnedUri = function_exists('\GuzzleHttp\Psr7\uri_for')
+            ? \GuzzleHttp\Psr7\uri_for('')
+            : \GuzzleHttp\Psr7\Utils::uriFor('');
+
         $this->client
             ->getConfig('base_uri')
             ->shouldBeCalled()
-            ->willReturn(\GuzzleHttp\Psr7\uri_for(''));
+            ->willReturn($returnedUri);
 
         $this->client
             ->request('GET', 'v2/images', ['query' => ['limit' => 5], 'headers' => []])

tests/unit/ObjectStore/v1/Models/ContainerTest.php

@@ -210,8 +210,9 @@ public function test_other_exceptions_are_thrown()
 
     public function test_it_chunks_according_to_provided_segment_size()
     {
-        /** @var \GuzzleHttp\Psr7\Stream $stream */
-        $stream = \GuzzleHttp\Psr7\stream_for(implode('', range('A', 'X')));
+        $stream = function_exists('\GuzzleHttp\Psr7\stream_for')
+            ? \GuzzleHttp\Psr7\stream_for(implode('', range('A', 'X')))
+            : \GuzzleHttp\Psr7\Utils::streamFor(implode('', range('A', 'X')));
 
         $data = [
             'name' => 'object',