added tests, missing endpoint/functions

Author: nickurt

.editorconfig

@@ -0,0 +1,15 @@
+; This file is for unifying the coding style for different editors and IDEs.
+; More information at https://editorconfig.org
+
+root = true
+
+[*]
+charset = utf-8
+indent_size = 4
+indent_style = space
+end_of_line = lf
+insert_final_newline = true
+trim_trailing_whitespace = true
+
+[*.md]
+trim_trailing_whitespace = false

.gitattributes

@@ -0,0 +1,11 @@
+# Path-based git attributes
+# https://www.kernel.org/pub/software/scm/git/docs/gitattributes.html
+
+# Ignore all test and documentation with "export-ignore".
+/.gitattributes     export-ignore
+/.gitignore         export-ignore
+/.travis.yml        export-ignore
+/phpunit.xml.dist   export-ignore
+/.scrutinizer.yml   export-ignore
+/tests              export-ignore
+/.editorconfig      export-ignore

.gitignore

@@ -1,4 +1,5 @@
-/vendor
-composer.phar
+build
 composer.lock
-.DS_Store
+docs
+vendor
+coverage

.travis.yml

@@ -0,0 +1,23 @@
+language: php
+
+cache:
+  directories:
+    - $HOME/.composer/cache
+
+matrix:
+  fast_finish: true
+  include:
+    - php: 7.3
+      env: LARAVEL='5.8.*' TESTBENCH='3.8.*' COMPOSER_FLAGS='--prefer-lowest'
+    - php: 7.3
+      env: LARAVEL='5.8.*' TESTBENCH='3.8.*' COMPOSER_FLAGS='--prefer-stable'
+
+before_install:
+  - travis_retry composer self-update
+  - travis_retry composer require --no-update --no-interaction "illuminate/support:${LARAVEL}" "orchestra/testbench:${TESTBENCH}"
+
+install:
+  - travis_retry composer update ${COMPOSER_FLAGS} --prefer-dist --no-interaction --no-suggest
+
+script:
+  - vendor/bin/phpunit

README.md

@@ -1,12 +1,15 @@
 ## Laravel Plesk Onyx 17.8
-
+[![Latest Stable Version](https://poser.pugx.org/nickurt/laravel-plesk/v/stable?format=flat-square)](https://packagist.org/packages/nickurt/laravel-plesk)
+[![MIT Licensed](https://img.shields.io/badge/license-MIT-brightgreen.svg?style=flat-square)](LICENSE.md)
+[![Build Status](https://img.shields.io/travis/nickurt/laravel-plesk/master.svg?style=flat-square)](https://travis-ci.org/nickurt/laravel-plesk)
+[![Total Downloads](https://img.shields.io/packagist/dt/nickurt/laravel-plesk.svg?style=flat-square)](https://packagist.org/packages/nickurt/laravel-plesk)
 ### Installation
 Install this package with composer:
 ```
 composer require nickurt/laravel-plesk
 ```
 
-Add the provider to config/app.php file
+Add the provider to `config/app.php` file
 
 ```php
 'nickurt\Plesk\ServiceProvider',
@@ -25,6 +28,6 @@ php artisan vendor:publish --provider="nickurt\Plesk\ServiceProvider" --tag="con
 ```
 ### Tests
 ```sh
-phpunit
+composer test
 ```
 - - - 

composer.json

@@ -9,7 +9,8 @@
         "guzzlehttp/guzzle": "6.*"
     },
     "require-dev": {
-        "phpunit/phpunit" : "^7.5"
+        "phpunit/phpunit" : "^7.5",
+        "orchestra/testbench": "^3.8"
     },
     "autoload": {
         "psr-4": {
@@ -19,6 +20,15 @@
             "src/helpers.php"
         ]
     },
+    "autoload-dev": {
+        "psr-4": {
+            "nickurt\\Plesk\\tests\\": "tests"
+        }
+    },
+    "scripts": {
+        "test": "vendor/bin/phpunit",
+        "test-coverage": "vendor/bin/phpunit --coverage-html coverage"
+    },
     "minimum-stability": "dev",
     "prefer-stable": true,
     "extra": {

phpunit.xml.dist

@@ -19,4 +19,11 @@
             <directory suffix=".php">src/</directory>
         </whitelist>
     </filter>
+    <logging>
+        <log type="tap" target="build/report.tap"/>
+        <log type="junit" target="build/report.junit.xml"/>
+        <log type="coverage-html" target="build/coverage"/>
+        <log type="coverage-text" target="build/coverage.txt"/>
+        <log type="coverage-clover" target="build/logs/clover.xml"/>
+    </logging>
 </phpunit>

src/Api/AbstractApi.php

@@ -6,9 +6,7 @@
 
 abstract class AbstractApi implements ApiInterface
 {
-    /**
-     * @var Client
-     */
+    /** @var \nickurt\Plesk\Client */
     public $client;
 
     /**
@@ -20,40 +18,71 @@ public function __construct(Client $client)
         $this->client = $client;
     }
 
-    public function delete($path, $body, array $headers = [])
+    /**
+     * @param string $path
+     * @param array $parameters
+     * @param array $headers
+     * @return mixed
+     * @throws \GuzzleHttp\Exception\GuzzleException
+     */
+    public function delete($path, array $parameters = [], array $headers = [])
     {
-        //
+        return $this->client->getHttpClient()->delete(
+            $path,
+            $parameters,
+            $headers
+        );
     }
 
     /**
-     * @param $path
+     * @param string $path
      * @param array $parameters
      * @param array $headers
      * @return mixed
      */
     public function get($path, array $parameters = [], array $headers = [])
     {
-        $response = $this->client->getHttpClient()->get(
+        return $this->client->getHttpClient()->get(
             $path,
             $parameters,
             $headers
         );
-
-        return $response;
     }
 
     public function path($path, $body, array $headers = [])
     {
         //
     }
 
+    /**
+     * @param string $path
+     * @param array $body
+     * @param array $headers
+     * @return mixed
+     * @throws \GuzzleHttp\Exception\GuzzleException
+     */
     public function post($path, $body, array $headers = [])
     {
-        //
+        return $this->client->getHttpClient()->post(
+            $path,
+            $body,
+            $headers
+        );
     }
 
+    /**
+     * @param string $path
+     * @param array $body
+     * @param array $headers
+     * @return mixed
+     * @throws \GuzzleHttp\Exception\GuzzleException
+     */
     public function put($path, $body, array $headers = [])
     {
-        //
+        return $this->client->getHttpClient()->put(
+            $path,
+            $body,
+            $headers
+        );
     }
 }

src/Api/Authentication.php

@@ -5,11 +5,12 @@
 class Authentication extends AbstractApi
 {
     /**
-     * @param $params
+     * @param array $params
      * @return mixed
+     * @throws \GuzzleHttp\Exception\GuzzleException
      */
     public function keys($params)
     {
-        //
+        return $this->post('auth/keys', $params);
     }
 }

src/Api/Cli.php

@@ -5,12 +5,14 @@
 class Cli extends AbstractApi
 {
     /**
-     * @param $id
-     * @param $body
+     * @param int $id
+     * @param array $params
+     * @return mixed
+     * @throws \GuzzleHttp\Exception\GuzzleException
      */
-    public function call($id, $body)
+    public function call($id, $params)
     {
-        //
+        return $this->post('cli/' . $id . '/call', $params);
     }
 
     /**
@@ -22,7 +24,7 @@ public function commands()
     }
 
     /**
-     * @param $id
+     * @param int $id
      * @return mixed
      */
     public function ref($id)

src/Api/Clients.php

@@ -5,19 +5,50 @@
 class Clients extends AbstractApi
 {
     /**
+     * @param int $id
      * @return mixed
+     * @throws \GuzzleHttp\Exception\GuzzleException
      */
-    public function all()
+    public function destroy($id)
+    {
+        return $this->delete('clients/' . $id);
+    }
+
+    /**
+     * @return mixed
+     */
+    public function index()
     {
         return $this->get('clients');
     }
 
     /**
-     * @param $id
+     * @param int $id
+     * @return mixed
+     */
+    public function show($id)
+    {
+        return $this->get('clients/' . $id);
+    }
+
+    /**
+     * @param array $params
+     * @return mixed
+     * @throws \GuzzleHttp\Exception\GuzzleException
+     */
+    public function store($params)
+    {
+        return $this->post('clients', $params);
+    }
+
+    /**
+     * @param int $id
+     * @param array $params
      * @return mixed
+     * @throws \GuzzleHttp\Exception\GuzzleException
      */
-    public function read($id)
+    public function update($id, $params)
     {
-        return $this->get('clients/'.$id);
+        return $this->put('clients/' . $id, $params);
     }
 }

src/Api/Domains.php

@@ -5,19 +5,69 @@
 class Domains extends AbstractApi
 {
     /**
+     * @param int $id
      * @return mixed
      */
-    public function all()
+    public function client($id)
+    {
+        return $this->get('domains/' . $id . '/client');
+    }
+
+    /**
+     * @param int $id
+     * @return mixed
+     * @throws \GuzzleHttp\Exception\GuzzleException
+     */
+    public function destroy($id)
+    {
+        return $this->delete('domains/' . $id);
+    }
+
+    /**
+     * @return mixed
+     */
+    public function index()
     {
         return $this->get('domains');
     }
 
     /**
-     * @param $id
+     * @param int $id
+     * @return mixed
+     */
+    public function show($id)
+    {
+        return $this->get('domains/' . $id);
+    }
+
+    /**
+     * @param int $id
+     * @return mixed
+     * @throws \GuzzleHttp\Exception\GuzzleException
+     */
+    public function status($id)
+    {
+        return $this->get('domains/' . $id . '/status');
+    }
+
+    /**
+     * @param array $params
+     * @return mixed
+     * @throws \GuzzleHttp\Exception\GuzzleException
+     */
+    public function store($params)
+    {
+        return $this->post('domains', $params);
+    }
+
+    /**
+     * @param int $id
+     * @param array $params
      * @return mixed
+     * @throws \GuzzleHttp\Exception\GuzzleException
      */
-    public function read($id)
+    public function update($id, $params)
     {
-        return $this->get('domains/'.$id);
+        return $this->put('domains/' . $id, $params);
     }
 }