Delete series

Author: joedixon

app/Http/Controllers/Articles/SeriesController.php

@@ -7,6 +7,7 @@
 use App\Http\Middleware\RedirectIfUnconfirmed;
 use App\Http\Requests\SeriesRequest;
 use App\Jobs\CreateSeries;
+use App\Jobs\DeleteSeries;
 use App\Jobs\UpdateSeries;
 use App\Models\Series;
 use App\Models\Tag;
@@ -19,6 +20,11 @@ public function __construct()
         $this->middleware([Authenticate::class, RedirectIfUnconfirmed::class], ['except' => ['index', 'show']]);
     }
 
+    public function index()
+    {
+        
+    }
+
     public function show(Series $series)
     {
 
@@ -59,4 +65,15 @@ public function update(SeriesRequest $request, Series $series)
 
         return redirect()->route('series.show', $series->slug());
     }
+
+    public function delete(Series $series)
+    {
+        $this->authorize(SeriesPolicy::DELETE, $series);
+        
+        $this->dispatchNow(new DeleteSeries($series));
+        
+        $this->success('series.deleted');
+
+        return redirect()->route('series');
+    }
 }

app/Jobs/DeleteSeries.php

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

database/factories/SeriesFactory.php

@@ -1,6 +1,7 @@
 <?php
 
 use App\Models\Series;
+use App\User;
 use Faker\Generator as Faker;
 
 $factory->define(Series::class, function (Faker $faker) {
@@ -9,5 +10,6 @@
             return factory(User::class)->create()->id;
         },
         'title' => $faker->word,
+        'slug' => $faker->unique()->slug,
     ];
 });

resources/lang/en/series.php

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

routes/web.php

@@ -90,11 +90,13 @@
 
 // Series
 Route::prefix('series')->namespace('Articles')->group(function () {
+    Route::get('/', 'SeriesController@index')->name('series');
     Route::get('/create', 'SeriesController@create')->name('series.create');
     Route::post('/', 'SeriesController@store')->name('series.store');
     Route::get('{series}', 'SeriesController@show')->name('series.show');
     Route::get('{series}/edit', 'SeriesController@edit')->name('series.edit');
     Route::put('{series}', 'SeriesController@update')->name('series.update');
+    Route::delete('{series}', 'SeriesController@delete')->name('series.delete');
 });
 
 // Admin

tests/Feature/SeriesTest.php

@@ -83,4 +83,31 @@ public function users_cannot_edit_a_series_with_a_title_that_is_too_long()
         $response->assertSessionHas('error', 'Something went wrong. Please review the fields below.');
         $response->assertSessionHasErrors(['title' => 'The title may not be greater than 100 characters.']);
     }
+
+    /** @test */
+    public function users_can_delete_their_own_series()
+    {
+        $user = $this->createUser();
+        factory(Series::class)->create([
+            'author_id' => $user->id(),
+            'slug' => 'my-first-series',
+        ]);
+
+        $this->loginAs($user);
+
+        $this->delete('/series/my-first-series')
+            ->assertRedirectedTo('/series')
+            ->assertSessionHas('success', 'Series successfully deleted!');
+    }
+
+    /** @test */
+    public function users_cannot_delete_a_series_they_do_not_own()
+    {
+        factory(Series::class)->create(['slug' => 'my-first-thread']);
+
+        $this->login();
+
+        $this->delete('/series/my-first-thread')
+            ->assertForbidden();
+    }
 }

tests/Integration/Jobs/DeleteSeriesTest.php

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