Fix tests

Author: joedixon

app/Jobs/LikeReply.php

@@ -39,10 +39,10 @@ public function __construct(Reply $reply, User $user)
      */
     public function handle()
     {
-        try {
-            $this->reply->likedBy($this->user);
-        } catch (QueryException $exception) {
+        if($this->reply->isLikedBy($this->user)) {
             throw new CannotLikeReplyMultipleTimes();
         }
+
+        $this->reply->likedBy($this->user);
     }
 }

tests/Components/Jobs/LikeReplyTest.php

@@ -34,7 +34,7 @@ public function we_cannot_like_a_reply_twice()
 
         $this->assertTrue($reply->fresh()->isLikedBy($user));
 
-        $this->expectException(CannotLikeReplyTwice::class);
+        $this->expectException(CannotLikeReplyMultipleTimes::class);
 
         $this->dispatch(new LikeReply($reply, $user));
     }

tests/Components/Jobs/DislikeReplyTest.php renamed to tests/Components/Jobs/UnlikeReplyTest.php

@@ -2,13 +2,13 @@
 
 namespace Tests\Components\Jobs;
 
-use App\User;
-use Tests\TestCase;
+use App\Jobs\UnlikeReply;
 use App\Models\Reply;
-use App\Jobs\DislikeReply;
+use App\User;
 use Illuminate\Foundation\Testing\DatabaseMigrations;
+use Tests\TestCase;
 
-class DislikeReplyTest extends TestCase
+class UnlikeReplyTest extends TestCase
 {
     use DatabaseMigrations;
 
@@ -21,7 +21,7 @@ public function we_can_dislike_a_reply()
         $reply->likedBy($user);
         $this->assertTrue($reply->fresh()->isLikedBy($user));
 
-        $this->dispatch(new DislikeReply($reply, $user));
+        $this->dispatch(new UnlikeReply($reply, $user));
 
         $this->assertFalse($reply->fresh()->isLikedBy($user));
     }
@@ -35,9 +35,9 @@ public function we_cannot_dislike_a_reply()
         $reply->likedBy($user);
         $this->assertTrue($reply->fresh()->isLikedBy($user));
 
-        $this->dispatch(new DislikeReply($reply, $user));
+        $this->dispatch(new UnlikeReply($reply, $user));
 
-        $this->dispatch(new DislikeReply($reply, $user));
+        $this->dispatch(new UnlikeReply($reply, $user));
 
         //$this->assertFalse($reply->fresh()->isLikedBy($user));
     }

tests/Feature/ReplyTest.php

@@ -117,14 +117,15 @@ public function unconfirmed_users_cannot_see_the_reply_input()
             );
     }
 
+    /** @test */
     public function user_can_like_a_reply()
     {
         $user = factory(User::class)->create();
 
         $thread = factory(Thread::class)->create(['author_id' => $user->id(), 'slug' => 'the-first-thread']);
         $reply = factory(Reply::class)->create(['replyable_id' => $thread->id()]);
 
-        $this->put('/forum/the-first-thread/'.$reply->id().'/like');
-            //->assertForbidden();
+        $this->put('/forum/the-first-thread/'.$reply->id().'/like')
+            ->assertOk();
     }
 }