diff --git a/tests/Feature/Controllers/AttachmentControllerTest.php b/tests/Feature/Controllers/AttachmentControllerTest.php
index ba7eaa8de5..34c8282e13 100644
--- a/tests/Feature/Controllers/AttachmentControllerTest.php
+++ b/tests/Feature/Controllers/AttachmentControllerTest.php
@@ -119,23 +119,6 @@ class AttachmentControllerTest extends TestCase
         $response->assertSee('<ol class="breadcrumb">');
     }
 
-    /**
-     * @covers \FireflyIII\Http\Controllers\AttachmentController::preview
-     * @covers \FireflyIII\Http\Controllers\AttachmentController::__construct
-     */
-    public function testView()
-    {
-        $repository   = $this->mock(AttachmentRepositoryInterface::class);
-        $repository->shouldReceive('exists')->once()->andReturn(true);
-        $repository->shouldReceive('getContent')->once()->andReturn('This is attachment number one.');
-
-        $journalRepos = $this->mock(JournalRepositoryInterface::class);
-        $journalRepos->shouldReceive('first')->once()->andReturn(new TransactionJournal);
-        $this->be($this->user());
-        $response = $this->get(route('attachments.view', [3]));
-        $response->assertStatus(200);
-    }
-
     /**
      * @covers \FireflyIII\Http\Controllers\AttachmentController::update
      */
@@ -159,4 +142,37 @@ class AttachmentControllerTest extends TestCase
         $response->assertStatus(302);
         $response->assertSessionHas('success');
     }
+
+    /**
+     * @covers \FireflyIII\Http\Controllers\AttachmentController::view
+     * @covers \FireflyIII\Http\Controllers\AttachmentController::__construct
+     */
+    public function testView()
+    {
+        $repository = $this->mock(AttachmentRepositoryInterface::class);
+        $repository->shouldReceive('exists')->once()->andReturn(true);
+        $repository->shouldReceive('getContent')->once()->andReturn('This is attachment number one.');
+
+        $journalRepos = $this->mock(JournalRepositoryInterface::class);
+        $journalRepos->shouldReceive('first')->once()->andReturn(new TransactionJournal);
+        $this->be($this->user());
+        $response = $this->get(route('attachments.view', [3]));
+        $response->assertStatus(200);
+    }
+
+    /**
+     * @covers \FireflyIII\Http\Controllers\AttachmentController::view
+     * @covers \FireflyIII\Http\Controllers\AttachmentController::__construct
+     */
+    public function testViewFail()
+    {
+        $repository = $this->mock(AttachmentRepositoryInterface::class);
+        $repository->shouldReceive('exists')->once()->andReturn(false);
+
+        $journalRepos = $this->mock(JournalRepositoryInterface::class);
+        $journalRepos->shouldReceive('first')->once()->andReturn(new TransactionJournal);
+        $this->be($this->user());
+        $response = $this->get(route('attachments.view', [1]));
+        $response->assertStatus(500);
+    }
 }