Workflow tests (#16)

Author: fzia

.github/workflows/pull-requests.yml

@@ -0,0 +1,51 @@
+name: Flagsmith PHP Pull Request
+
+on:
+    pull_request:
+        types: [opened, synchronize, reopened, ready_for_review]
+        branches:
+            - main
+            - release**
+
+jobs:
+    test:
+        runs-on: ubuntu-latest
+        name: Flag engine Unit tests
+
+        strategy:
+            max-parallel: 4
+            matrix:
+                php-version: ['7.4', '8.0', '8.1'] # TODO add '8.2' once available
+
+        steps:
+            - name: Cloning repo
+              uses: actions/checkout@v2
+#              with:
+#                  fetch-depth: 0
+#                  submodules: recursive
+
+            - name: Setup PHP
+              uses: shivammathur/setup-php@v2
+              with:
+                  version: '{{ matrix.php-version }}'
+                  tools: php-cs-fixer, phpunit
+
+            - name: Get composer cache directory
+              id: composer-cache
+              run: echo "::set-output name=dir::$(composer config cache-files-dir)"
+
+            - name: Cache dependencies
+              uses: actions/cache@v2
+              with:
+                  path: ${{ steps.composer-cache.outputs.dir }}
+                  key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.json') }}
+                  restore-keys: ${{ runner.os }}-composer-
+
+            - name: Install dependencies
+              run: composer install --prefer-dist
+
+            - name: Check coding style
+              run: ./vendor/bin/php-cs-fixer fix -vvv --config=.php-cs-fixer.php --dry-run --diff
+
+            - name: Running tests
+              run: ./vendor/bin/phpunit tests/

.php-cs-fixer.php

@@ -0,0 +1,23 @@
+<?php
+
+$finder = PhpCsFixer\Finder::create()
+    ->exclude(
+        [
+            'vendor',
+        ]
+    )
+    ->in(__DIR__);
+
+$config = new PhpCsFixer\Config();
+
+return $config
+    ->setRules(
+        [
+            '@PSR12' => true,
+            'single_quote' => true,
+            'no_trailing_comma_in_singleline_array' => true,
+            'array_indentation' => true,
+            'phpdoc_indent' => true,
+        ]
+    )
+    ->setFinder($finder);

composer.json

@@ -18,7 +18,9 @@
     },
     "require-dev": {
         "guzzlehttp/guzzle": "^7.3",
-        "guzzlehttp/psr7": "^2.1.0"
+        "guzzlehttp/psr7": "^2.1.0",
+        "phpunit/phpunit": "^9.5",
+        "friendsofphp/php-cs-fixer": "^3.6"
     },
     "autoload": {
         "psr-4": {

examples/index.php

@@ -1,7 +1,8 @@
 <?php
+
 require_once './vendor/autoload.php';
 
-use \Flagsmith\Flagsmith;
+use Flagsmith\Flagsmith;
 use Http\Discovery\Psr17FactoryDiscovery;
 use Http\Discovery\Psr18ClientDiscovery;
 
@@ -30,8 +31,3 @@
 var_dump($response->getBody()->getContents());
 
 // TODO - write more API examples once they are implemented.
-
-
-
-
-

src/Cache.php

@@ -123,7 +123,7 @@ public function setMultiple(array $values): bool
     public function getMultiple(array $keys, $default = null)
     {
         return $this->cache->getMultiple(
-            array_map(fn($key) => $this->getKeyWithPrefix($key), $keys),
+            array_map(fn ($key) => $this->getKeyWithPrefix($key), $keys),
             $default
         );
     }

src/Flagsmith.php

@@ -206,7 +206,7 @@ public function getIdentityByIndentity(Identity $identity): Identity
                 [
                     'identifier' => $identity->getId(),
                     'traits' => array_map(
-                        fn(IdentityTrait $trait) => [
+                        fn (IdentityTrait $trait) => [
                             'trait_key' => $trait->getKey(),
                             'trait_value' => $trait->getValue(),
                         ],
@@ -351,7 +351,7 @@ public function setTraitsByIdentity(Identity $identity): bool
             'PUT',
             'traits/bulk/',
             array_map(
-                fn(IdentityTrait $trait) => [
+                fn (IdentityTrait $trait) => [
                     'identity' => ['identifier' => $identity->getId()],
                     'trait_key' => $trait->getKey(),
                     'trait_value' => $trait->getValue(),
@@ -526,7 +526,7 @@ private function call(string $method, string $uri, array $body = []): array
         if ($method !== 'GET') {
             $request = $request->withBody($stream);
         }
-        
+
         try {
             $response = $this->client->sendRequest($request);
         } catch (RequestExceptionInterface $e) {

tests/Engine/EngineTest.php

@@ -0,0 +1,11 @@
+<?php
+
+use PHPUnit\Framework\TestCase;
+
+class EngineTest extends TestCase
+{
+    public function testAssertion()
+    {
+        $this->assertTrue(true);
+    }
+}