Delete articles

Author: joedixon

app/Http/Controllers/Articles/ArticlesController.php

@@ -7,6 +7,7 @@
 use App\Http\Middleware\RedirectIfUnconfirmed;
 use App\Http\Requests\ArticleRequest;
 use App\Jobs\CreateArticle;
+use App\Jobs\DeleteArticle;
 use App\Jobs\UpdateArticle;
 use App\Models\Article;
 use App\Models\Tag;
@@ -68,5 +69,12 @@ public function update(ArticleRequest $request, Article $article)
 
     public function delete(Article $article)
     {
+        $this->authorize(ArticlePolicy::DELETE, $article);
+
+        $this->dispatchNow(new DeleteArticle($article));
+
+        $this->success('articles.deleted');
+
+        return redirect()->route('articles');
     }
 }

app/Jobs/DeleteArticle.php

@@ -0,0 +1,20 @@
+<?php
+
+namespace App\Jobs;
+
+use App\Models\Article;
+
+final class DeleteArticle
+{
+    private $article;
+
+    public function __construct(Article $article)
+    {
+        $this->article = $article;
+    }
+
+    public function handle()
+    {
+        $this->article->delete();
+    }
+}

resources/lang/en/articles.php

@@ -4,5 +4,6 @@
 
     'created' => 'Article successfully created!',
     'updated' => 'Article successfully updated!',
+    'deleted' => 'Article successfully deleted!',
 
 ];

tests/Feature/ArticleTest.php

@@ -119,4 +119,31 @@ public function an_article_may_not_updated_to_contain_an_http_image_url()
         $response->assertSessionHas('error', 'Something went wrong. Please review the fields below.');
         $response->assertSessionHasErrors(['body' => 'The body field contains at least one image with an HTTP link.']);
     }
+
+    /** @test */
+    public function users_can_delete_their_own_articles()
+    {
+        $user = $this->createUser();
+        factory(Article::class)->create([
+            'author_id' => $user->id(),
+            'slug' => 'my-first-article',
+        ]);
+
+        $this->loginAs($user);
+
+        $this->delete('/articles/my-first-article')
+            ->assertRedirectedTo('/articles')
+            ->assertSessionHas('success', 'Article successfully deleted!');
+    }
+
+    /** @test */
+    public function users_cannot_delete_an_article_they_do_not_own()
+    {
+        factory(Article::class)->create(['slug' => 'my-first-thread']);
+
+        $this->login();
+
+        $this->delete('/articles/my-first-thread')
+            ->assertForbidden();
+    }
 }

tests/Integration/Jobs/DeleteArticleTest.php

@@ -0,0 +1,23 @@
+<?php
+
+namespace Tests\Integration\Jobs;
+
+use App\Jobs\DeleteArticle;
+use App\Models\Article;
+use Illuminate\Foundation\Testing\DatabaseMigrations;
+use Tests\TestCase;
+
+class DeleteArticleTest extends TestCase
+{
+    use DatabaseMigrations;
+
+    /** @test */
+    public function an_article_can_be_deleted()
+    {
+        $article = factory(Article::class)->create();
+
+        $this->dispatch(new DeleteArticle($article));
+
+        $this->assertDatabaseMissing('articles', ['id' => $article->id()]);
+    }
+}