From 02b2e4c2141da5e970aadfba53ba2be527dbaa8c Mon Sep 17 00:00:00 2001
From: James Brooks <jbrooksuk@me.com>
Date: Fri, 2 Dec 2016 15:12:01 +0000
Subject: [PATCH] Added missing command tests

---
 .../System/Config/UpdateConfigCommand.php     |  2 +-
 .../Commands/System/Mail/TestMailCommand.php  |  2 +-
 .../System/Config/UpdateConfigCommandTest.php | 49 +++++++++++++++++++
 .../System/Mail/TestMailCommandTest.php       | 46 +++++++++++++++++
 4 files changed, 97 insertions(+), 2 deletions(-)
 create mode 100644 tests/Bus/Commands/System/Config/UpdateConfigCommandTest.php
 create mode 100644 tests/Bus/Commands/System/Mail/TestMailCommandTest.php

diff --git a/app/Bus/Commands/System/Config/UpdateConfigCommand.php b/app/Bus/Commands/System/Config/UpdateConfigCommand.php
index c0c9b12b..9fb21610 100644
--- a/app/Bus/Commands/System/Config/UpdateConfigCommand.php
+++ b/app/Bus/Commands/System/Config/UpdateConfigCommand.php
@@ -16,7 +16,7 @@ namespace CachetHQ\Cachet\Bus\Commands\System\Config;
  *
  * @author James Brooks <james@alt-three.com>
  */
-class UpdateConfigCommand
+final class UpdateConfigCommand
 {
     /**
      * This is the config key/values array.
diff --git a/app/Bus/Commands/System/Mail/TestMailCommand.php b/app/Bus/Commands/System/Mail/TestMailCommand.php
index 131d864e..782ea873 100644
--- a/app/Bus/Commands/System/Mail/TestMailCommand.php
+++ b/app/Bus/Commands/System/Mail/TestMailCommand.php
@@ -18,7 +18,7 @@ use CachetHQ\Cachet\Models\User;
  *
  * @author James Brooks <james@alt-three.com>
  */
-class TestMailCommand
+final class TestMailCommand
 {
     /**
      * The user to send the notification to.
diff --git a/tests/Bus/Commands/System/Config/UpdateConfigCommandTest.php b/tests/Bus/Commands/System/Config/UpdateConfigCommandTest.php
new file mode 100644
index 00000000..6d0f5009
--- /dev/null
+++ b/tests/Bus/Commands/System/Config/UpdateConfigCommandTest.php
@@ -0,0 +1,49 @@
+<?php
+
+/*
+ * This file is part of Cachet.
+ *
+ * (c) Alt Three Services Limited
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace CachetHQ\Tests\Cachet\Bus\Commands\System\Config;
+
+use AltThree\TestBench\CommandTrait;
+use CachetHQ\Cachet\Bus\Commands\System\Config\UpdateConfigCommand;
+use CachetHQ\Cachet\Bus\Handlers\Commands\System\Config\UpdateConfigCommandHandler;
+use CachetHQ\Tests\Cachet\AbstractTestCase;
+
+/**
+ * This is the update config command test class.
+ *
+ * @author James Brooks <james@alt-three.com>
+ */
+class UpdateConfigCommandTest extends AbstractTestCase
+{
+    use CommandTrait;
+
+    protected function getObjectAndParams()
+    {
+        $params = [
+            'values' => [
+                'FOO' => 'bar',
+            ],
+        ];
+        $object = new UpdateConfigCommand($params['values']);
+
+        return compact('params', 'object');
+    }
+
+    protected function objectHasRules()
+    {
+        return false;
+    }
+
+    protected function getHandlerClass()
+    {
+        return UpdateConfigCommandHandler::class;
+    }
+}
diff --git a/tests/Bus/Commands/System/Mail/TestMailCommandTest.php b/tests/Bus/Commands/System/Mail/TestMailCommandTest.php
new file mode 100644
index 00000000..1e92f46e
--- /dev/null
+++ b/tests/Bus/Commands/System/Mail/TestMailCommandTest.php
@@ -0,0 +1,46 @@
+<?php
+
+/*
+ * This file is part of Cachet.
+ *
+ * (c) Alt Three Services Limited
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace CachetHQ\Tests\Cachet\Bus\Commands\System\Mail;
+
+use AltThree\TestBench\CommandTrait;
+use CachetHQ\Cachet\Bus\Commands\System\Mail\TestMailCommand;
+use CachetHQ\Cachet\Bus\Handlers\Commands\System\Mail\TestMailCommandHandler;
+use CachetHQ\Cachet\Models\User;
+use CachetHQ\Tests\Cachet\AbstractTestCase;
+
+/**
+ * This is the test mail command test class.
+ *
+ * @author James Brooks <james@alt-three.com>
+ */
+class TestMailCommandTest extends AbstractTestCase
+{
+    use CommandTrait;
+
+    protected function getObjectAndParams()
+    {
+        $params = ['user' => new User()];
+        $object = new TestMailCommand($params['user']);
+
+        return compact('params', 'object');
+    }
+
+    protected function objectHasRules()
+    {
+        return false;
+    }
+
+    protected function getHandlerClass()
+    {
+        return TestMailCommandHandler::class;
+    }
+}