From 0237d78f613a66f41037472396e2af1666204898 Mon Sep 17 00:00:00 2001
From: James Cole <thegrumpydictator@gmail.com>
Date: Mon, 9 Jul 2018 19:42:53 +0200
Subject: [PATCH] Fix some tests.

---
 app/Http/Controllers/Admin/ConfigurationController.php | 1 +
 tests/Feature/Controllers/Admin/UserControllerTest.php | 7 ++++++-
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/app/Http/Controllers/Admin/ConfigurationController.php b/app/Http/Controllers/Admin/ConfigurationController.php
index ed3097c9b8..65ae4cbc24 100644
--- a/app/Http/Controllers/Admin/ConfigurationController.php
+++ b/app/Http/Controllers/Admin/ConfigurationController.php
@@ -28,6 +28,7 @@ use FireflyIII\Http\Middleware\IsDemoUser;
 use FireflyIII\Http\Middleware\IsSandStormUser;
 use FireflyIII\Http\Requests\ConfigurationRequest;
 use FireflyIII\Support\Facades\FireflyConfig;
+use Illuminate\Http\RedirectResponse;
 
 /**
  * Class ConfigurationController.
diff --git a/tests/Feature/Controllers/Admin/UserControllerTest.php b/tests/Feature/Controllers/Admin/UserControllerTest.php
index 9b034f16d8..3835627969 100644
--- a/tests/Feature/Controllers/Admin/UserControllerTest.php
+++ b/tests/Feature/Controllers/Admin/UserControllerTest.php
@@ -35,7 +35,7 @@ class UserControllerTest extends TestCase
     /**
      *
      */
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         Log::debug(sprintf('Now in %s.', \get_class($this)));
@@ -46,6 +46,8 @@ class UserControllerTest extends TestCase
      */
     public function testDelete(): void
     {
+        $repository = $this->mock(UserRepositoryInterface::class);
+        $repository->shouldReceive('hasRole')->once()->andReturn(true);
         $this->be($this->user());
         $response = $this->get(route('admin.users.delete', [1]));
         $response->assertStatus(200);
@@ -60,6 +62,8 @@ class UserControllerTest extends TestCase
     {
         $repository = $this->mock(UserRepositoryInterface::class);
         $repository->shouldReceive('destroy')->once();
+        $repository->shouldReceive('hasRole')->once()->andReturn(true);
+
         $this->be($this->user());
         $response = $this->post(route('admin.users.destroy', ['2']));
         $response->assertStatus(302);
@@ -71,6 +75,7 @@ class UserControllerTest extends TestCase
      */
     public function testEdit(): void
     {
+        $repository = $this->mock(UserRepositoryInterface::class);
         $this->be($this->user());
         $response = $this->get(route('admin.users.edit', [1]));
         $response->assertStatus(200);