Refactor test classes and update dependencies

Adjusted test class syntax for consistency and readability. Removed unused helper methods and improved inline documentation. Updated PHP CS Fixer and made adjustments to composer.json scripts for better environment handling.

Author: Pavlusha311245

composer.json

@@ -19,17 +19,17 @@
   ],
   "scripts": {
     "test": [
-      "composer test:unit",
-      "composer test:phpstan"
+      "@test:unit",
+      "@test:phpstan"
     ],
     "test:unit": "./vendor/bin/pest",
     "test:coverage": "./vendor/bin/pest --coverage --coverage-clover coverage.xml",
     "test:phpstan": "./vendor/bin/phpstan analyse -c phpstan.neon",
-    "lint": "./vendor/bin/php-cs-fixer fix --rules=@PSR12 ."
+    "lint": "PHP_CS_FIXER_IGNORE_ENV=1 ./vendor/bin/php-cs-fixer fix --rules=@PSR12 ."
   },
   "require": {
     "php": "^8.3|^8.4",
-    "friendsofphp/php-cs-fixer": "^3.65",
+    "friendsofphp/php-cs-fixer": "^3.68",
     "guzzlehttp/guzzle": "^7.9",
     "nesbot/carbon": "^3.8",
     "doctrine/inflector": "^2.0",

src/Enums/ApiPathEnum.php

@@ -0,0 +1,22 @@
+<?php
+
+namespace UnitPhpSdk\Enums;
+
+enum ApiPathEnum: string
+{
+    case UNIT = '/unit';
+    case CONFIG = '/config';
+    case ROUTES = '/config/routes';
+    case APPLICATIONS = '/config/applications';
+    case LISTENERS = '/config/listeners';
+    case UPSTREAMS = '/config/upstreams';
+    case ROUTE = '/config/routes/{route}';
+    case APPLICATION = '/config/applications/{application}';
+    case LISTENER = '/config/listeners/{listener}';
+    case UPSTREAM = '/config/upstreams/{upstream}';
+    case CERTIFICATES = '/certificates';
+    case CERTIFICATE = '/certificates/{certificate}';
+    case STATUS = '/status';
+
+    case ACCESS_LOG = '/config/access_log';
+}

src/Http/UnitHttpClient.php

@@ -0,0 +1,53 @@
+<?php
+
+namespace UnitPhpSdk\Http;
+
+use GuzzleHttp\Client;
+use GuzzleHttp\ClientInterface;
+
+class UnitHttpClient
+{
+    private ClientInterface $client;
+
+    public function __construct(
+        /**
+         * Base URL for requests
+         *
+         * @var string
+         */
+        private readonly string  $baseUrl,
+        private readonly ?string $socket = null,
+        ?ClientInterface         $client = null
+    ) {
+        $this->client = $client ?? new Client([
+            'base_uri' => $this->baseUrl,
+            'curl' => $this->socket ? [
+                CURLOPT_UNIX_SOCKET_PATH => $this->socket
+            ] : []
+        ]);
+    }
+
+    public function request(string $method, string $uri, array $options = [])
+    {
+
+    }
+
+    //    /**
+    //     * @param string $address
+    //     * @return string
+    //     */
+    //    private function parseAddress(string $address): string
+    //    {
+    //        $scheme = parse_url($address, PHP_URL_SCHEME);
+    //        $host = parse_url($address, PHP_URL_HOST);
+    //        $port = parse_url($address, PHP_URL_PORT);
+    //
+    //        $address = "$scheme://$host";
+    //
+    //        if ($port) {
+    //            $address .= ":$port";
+    //        }
+    //
+    //        return $address;
+    //    }
+}

src/helpers.php

@@ -34,18 +34,3 @@ function parse_listener_pass(string $string): array
         }
     }
 }
-
-//if (!function_exists('get_unit_endpoint')) {
-//    /**
-//     * Get the Unit endpoint from the configuration.
-//     *
-//     * @return string The Unit endpoint.
-//     */
-//    function get_unit_endpoint($class): string
-//    {
-//        return match ($class) {
-//            Unit::class => '/',
-//            Config::class => '/config'
-//        };
-//    }
-//}

tests/Unit/Applications/Targets/PhpTargetTest.php

@@ -28,4 +28,4 @@
     $phpTarget = new PhpTarget($data);
 
     expect($phpTarget->toArray())->toBe($data);
-});
+});

tests/Unit/Applications/Targets/PythonTargetTest.php

@@ -37,4 +37,4 @@
     $pythonTarget = new PythonTarget($data);
 
     expect($pythonTarget->toArray())->toBe($data);
-});
+});

tests/Unit/ProcessIsolation/ApplicationProcessTest.php

@@ -1,7 +1,6 @@
 <?php
 
 use PHPUnit\Framework\TestCase;
-
 use UnitPhpSdk\Config\Application\ProcessManagement\ApplicationProcess;
 
 use function PHPUnit\Framework\assertEquals;

tests/Unit/Settings/TelemetryTest.php

@@ -45,4 +45,4 @@
 
     expect($telemetry->getEndpoint())->toBe($newEndpoint)
         ->and($telemetry->getProtocol())->toBe($newProtocol);
-});
+});

tests/Unit/Statistics/ModuleStatisticsTest.php

@@ -22,9 +22,9 @@
 });
 
 it(
-/**
- * @throws InvalidArgumentException
- */
+    /**
+     * @throws InvalidArgumentException
+     */
     'throws InvalidArgumentException when version is missing',
     function () {
         $data = [
@@ -39,9 +39,9 @@ function () {
 );
 
 it(
-/**
- * @throws InvalidArgumentException
- */
+    /**
+     * @throws InvalidArgumentException
+     */
     'throws InvalidArgumentException when lib is missing',
     function () {
         $data = [

tests/Unit/Traits/CanUploadTest.php

@@ -14,7 +14,9 @@
 });
 
 it('sets and gets API endpoint', function () {
-    $upload = new class { use CanUpload; };
+    $upload = new class () {
+        use CanUpload;
+    };
 
     $apiEndpoint = $this->faker->url();
     $upload->setApiEndpoint($apiEndpoint);
@@ -46,9 +48,10 @@
 //});
 
 it('throws exception during upload', function () {
-    $upload = new class {
+    $upload = new class () {
         use CanUpload;
-        public function toArray() {
+        public function toArray()
+        {
             return ['key' => 'value'];
         }
     };
@@ -60,11 +63,11 @@ public function toArray() {
     $apiEndpoint = $this->faker->url();
     $upload->setApiEndpoint($apiEndpoint);
 
-    expect(fn() => $upload->upload($request))->toThrow(UnitException::class, 'Upload failed');
+    expect(fn () => $upload->upload($request))->toThrow(UnitException::class, 'Upload failed');
 });
 
 it('removes data successfully', function () {
-    $upload = new class {
+    $upload = new class () {
         use CanUpload;
     };
 
@@ -84,7 +87,7 @@ public function toArray() {
 });
 
 it('throws exception during remove', function () {
-    $upload = new class {
+    $upload = new class () {
         use CanUpload;
     };
 
@@ -95,5 +98,5 @@ public function toArray() {
     $apiEndpoint = $this->faker->url();
     $upload->setApiEndpoint($apiEndpoint);
 
-    expect(fn() => $upload->remove($request))->toThrow(UnitException::class, 'Remove failed');
-});
+    expect(fn () => $upload->remove($request))->toThrow(UnitException::class, 'Remove failed');
+});