Merge pull request #2 from Fred07/feature/implement-test

Feature/implement test

.travis.yml

@@ -0,0 +1,7 @@
+language: php
+php:
+  - '7.2'
+install:
+  - composer install
+script: vendor/bin/phpunit
+sudo: false

composer.json

@@ -10,12 +10,13 @@
     ],
     "minimum-stability": "stable",
     "require": {
-        "php": ">=7.1",
+        "php": ">=7.2",
         "guzzlehttp/guzzle": "^6.3",
         "ext-json": "*"
     },
     "require-dev": {
-        "orchestra/testbench": "^3.8"
+        "orchestra/testbench": "^3.8",
+        "phpunit/phpunit": "^8.2"
     },
     "autoload": {
         "psr-4": {

phpunit.xml

@@ -0,0 +1,7 @@
+<phpunit bootstrap="vendor/autoload.php">
+    <testsuites>
+        <testsuite name="phpunit test">
+            <directory>tests</directory>
+        </testsuite>
+    </testsuites>
+</phpunit>

readme.md

@@ -1,3 +1,5 @@
+[![Build Status](https://travis-ci.com/Fred07/laravel-slack.svg?branch=master)](https://travis-ci.com/Fred07/laravel-slack)
+
 # slack service provider
 
 本專案以 service provider 方式

src/Client.php

@@ -33,22 +33,35 @@ class Client
 
     /**
      * Client constructor.
+     * @param Guzzle|null $client
      * @param string $endpoint
      * @param string|null $channel
      * @param string $username
      */
-    public function __construct(string $endpoint, string $channel = null, $username = '')
+    public function __construct(string $endpoint, string $channel = null, $username = '', Guzzle $client = null)
     {
-        $this->http = new Guzzle([
-            'headers' => [
-                'Content-type: application/json'
-            ],
-        ]);
+        if ($client === null) {
+            $this->http = $this->makeHttpClient();
+        } else {
+            $this->http = $client;
+        }
+
         $this->channel = $channel;
         $this->username = $username;
         $this->endpoint = $endpoint;
     }
 
+    /**
+     * @param array $headers
+     * @return Guzzle
+     */
+    private function makeHttpClient(array $headers = []): Guzzle
+    {
+        return new Guzzle([
+            'headers' => !empty($headers) ? $headers : ['Content-type: application/json'],
+        ]);
+    }
+
     /**
      * @param callable $callback
      *

tests/ClientTest.php

@@ -0,0 +1,43 @@
+<?php
+
+use Fred\SlackService\Message;
+use PHPUnit\Framework\TestCase;
+
+class ClientTest extends TestCase
+{
+    /**
+     * @test
+     */
+    public function it_should_send_correct_message_to_slack(): void
+    {
+        $expected = [
+            'text' => 'hello',
+            'channel' => 't-ch',
+            'mrkdwn' => true,
+            'username' => 't-name',
+        ];
+
+        $container = [];
+        $history = GuzzleHttp\Middleware::history($container);
+        $mock = new GuzzleHttp\Handler\MockHandler([
+            new GuzzleHttp\Psr7\Response(200, ['test' => true]),
+        ]);
+        $handler = GuzzleHttp\HandlerStack::create($mock);
+        $handler->push($history);
+        $client = new GuzzleHttp\Client(['handler' => $handler]);
+
+        $slackClient = new Fred\SlackService\Client('/mock-server', 't-ch', 't-name', $client);
+        $slackClient->compose(function (Message $message) {
+            $message->setText('hello');
+            return $message;
+        })->send();
+
+        $transaction = $container[0];
+        /** @var \GuzzleHttp\Psr7\Request $request */
+        $request = $transaction['request'];
+
+        $actual = $request->getBody()->getContents();
+
+        $this->assertEquals($expected, json_decode($actual, true));
+    }
+}