From b144cf5619687b0282cbf3f18cb9591274acd74a Mon Sep 17 00:00:00 2001
From: Adam Wathan <adam.wathan@gmail.com>
Date: Tue, 4 Jul 2017 10:30:13 -0400
Subject: [PATCH] 118 - Updating the Other Tests

---
 app/Http/Controllers/Backstage/ConcertsController.php | 7 +++----
 database/factories/ModelFactory.php                   | 5 +++--
 tests/Feature/Backstage/EditConcertTest.php           | 7 +++++++
 3 files changed, 13 insertions(+), 6 deletions(-)

diff --git a/app/Http/Controllers/Backstage/ConcertsController.php b/app/Http/Controllers/Backstage/ConcertsController.php
index 53855fe..acebcca 100644
--- a/app/Http/Controllers/Backstage/ConcertsController.php
+++ b/app/Http/Controllers/Backstage/ConcertsController.php
@@ -69,6 +69,9 @@ class ConcertsController extends Controller
 
     public function update($id)
     {
+        $concert = Auth::user()->concerts()->findOrFail($id);
+        abort_if($concert->isPublished(), 403);
+
         $this->validate(request(), [
             'title' => ['required'],
             'date' => ['required', 'date'],
@@ -82,10 +85,6 @@ class ConcertsController extends Controller
             'ticket_quantity' => ['required', 'integer', 'min:1'],
         ]);
 
-        $concert = Auth::user()->concerts()->findOrFail($id);
-
-        abort_if($concert->isPublished(), 403);
-
         $concert->update([
             'title' => request('title'),
             'subtitle' => request('subtitle'),
diff --git a/database/factories/ModelFactory.php b/database/factories/ModelFactory.php
index 65c9018..0eb450b 100644
--- a/database/factories/ModelFactory.php
+++ b/database/factories/ModelFactory.php
@@ -31,14 +31,15 @@ $factory->define(App\Concert::class, function (Faker\Generator $faker) {
         },
         'title' => 'Example Band',
         'subtitle' => 'with The Fake Openers',
+        'additional_information' => 'Some sample additional information.',
         'date' => Carbon::parse('+2 weeks'),
-        'ticket_price' => 2000,
         'venue' => 'The Example Theatre',
         'venue_address' => '123 Example Lane',
         'city' => 'Fakeville',
         'state' => 'ON',
         'zip' => '90210',
-        'additional_information' => 'Some sample additional information.',
+        'ticket_price' => 2000,
+        'ticket_quantity' => 5,
     ];
 });
 
diff --git a/tests/Feature/Backstage/EditConcertTest.php b/tests/Feature/Backstage/EditConcertTest.php
index 0606b7e..8241b3b 100644
--- a/tests/Feature/Backstage/EditConcertTest.php
+++ b/tests/Feature/Backstage/EditConcertTest.php
@@ -26,6 +26,7 @@ class EditConcertTest extends TestCase
             'state' => 'New state',
             'zip' => '99999',
             'ticket_price' => '72.50',
+            'ticket_quantity' => '10',
         ], $overrides);
     }
 
@@ -169,6 +170,7 @@ class EditConcertTest extends TestCase
             'state' => 'Old state',
             'zip' => '00000',
             'ticket_price' => 2000,
+            'ticket_quantity' => 5,
         ]);
         $this->assertFalse($concert->isPublished());
 
@@ -184,6 +186,7 @@ class EditConcertTest extends TestCase
             'state' => 'New state',
             'zip' => '99999',
             'ticket_price' => '72.50',
+            'ticket_quantity' => '10',
         ]);
 
         $response->assertStatus(404);
@@ -198,6 +201,7 @@ class EditConcertTest extends TestCase
             $this->assertEquals('Old state', $concert->state);
             $this->assertEquals('00000', $concert->zip);
             $this->assertEquals(2000, $concert->ticket_price);
+            $this->assertEquals(5, $concert->ticket_quantity);
         });
     }
 
@@ -217,6 +221,7 @@ class EditConcertTest extends TestCase
             'state' => 'Old state',
             'zip' => '00000',
             'ticket_price' => 2000,
+            'ticket_quantity' => 5,
         ]);
         $this->assertTrue($concert->isPublished());
 
@@ -232,6 +237,7 @@ class EditConcertTest extends TestCase
             'state' => 'New state',
             'zip' => '99999',
             'ticket_price' => '72.50',
+            'ticket_quantity' => '10',
         ]);
 
         $response->assertStatus(403);
@@ -246,6 +252,7 @@ class EditConcertTest extends TestCase
             $this->assertEquals('Old state', $concert->state);
             $this->assertEquals('00000', $concert->zip);
             $this->assertEquals(2000, $concert->ticket_price);
+            $this->assertEquals(5, $concert->ticket_quantity);
         });
     }