Merge pull request #44 from coderflexx/add_down_migration

remove down migration

Author: ousid

database/migrations/create_tickets_table.php.stub

@@ -25,14 +25,4 @@ return new class extends Migration
             $table->timestamps();
         });
     }
-
-    /**
-     * Reverse the migrations.
-     *
-     * @return void
-     */
-    public function down()
-    {
-        Schema::dropIfExists('laravel_tickets');
-    }
 };

src/Models/Ticket.php

@@ -25,10 +25,10 @@
  */
 class Ticket extends Model
 {
+    use Concerns\InteractsWithTicketRelations;
+    use Concerns\InteractsWithTickets;
     use HasFactory;
     use TicketScope;
-    use Concerns\InteractsWithTickets;
-    use Concerns\InteractsWithTicketRelations;
 
     /**
      * The attributes that aren't mass assignable.

tests/Feature/TicketTest.php

@@ -5,22 +5,22 @@
 
 it('filters tickets by status', function () {
     Ticket::factory()
-            ->times(3)
-            ->create([
-                'status' => 'open',
-            ]);
+        ->times(3)
+        ->create([
+            'status' => 'open',
+        ]);
 
     Ticket::factory()
-            ->times(7)
-            ->create([
-                'status' => 'closed',
-            ]);
+        ->times(7)
+        ->create([
+            'status' => 'closed',
+        ]);
 
     Ticket::factory()
-            ->times(6)
-            ->create([
-                'status' => 'archived',
-            ]);
+        ->times(6)
+        ->create([
+            'status' => 'archived',
+        ]);
 
     $this->assertEquals(Ticket::count(), 16);
     $this->assertEquals(Ticket::opened()->count(), 3);
@@ -31,16 +31,16 @@
 
 it('filters tickets by resolved status', function () {
     Ticket::factory()
-            ->times(2)
-            ->create([
-                'is_resolved' => true,
-            ]);
+        ->times(2)
+        ->create([
+            'is_resolved' => true,
+        ]);
 
     Ticket::factory()
-            ->times(10)
-            ->create([
-                'is_resolved' => false,
-            ]);
+        ->times(10)
+        ->create([
+            'is_resolved' => false,
+        ]);
 
     $this->assertEquals(Ticket::count(), 12);
     $this->assertEquals(Ticket::resolved()->count(), 2);
@@ -49,16 +49,16 @@
 
 it('filters tickets by locked status', function () {
     Ticket::factory()
-            ->times(3)
-            ->create([
-                'is_locked' => true,
-            ]);
+        ->times(3)
+        ->create([
+            'is_locked' => true,
+        ]);
 
     Ticket::factory()
-            ->times(9)
-            ->create([
-                'is_locked' => false,
-            ]);
+        ->times(9)
+        ->create([
+            'is_locked' => false,
+        ]);
 
     $this->assertEquals(Ticket::count(), 12);
     $this->assertEquals(Ticket::locked()->count(), 3);
@@ -67,28 +67,28 @@
 
 it('filters tickets by priority status', function () {
     Ticket::factory()
-            ->times(7)
-            ->create([
-                'priority' => 'low',
-            ]);
+        ->times(7)
+        ->create([
+            'priority' => 'low',
+        ]);
 
     Ticket::factory()
-            ->times(5)
-            ->create([
-                'priority' => 'normal',
-            ]);
+        ->times(5)
+        ->create([
+            'priority' => 'normal',
+        ]);
 
     Ticket::factory()
-            ->times(15)
-            ->create([
-                'priority' => 'high',
-            ]);
+        ->times(15)
+        ->create([
+            'priority' => 'high',
+        ]);
 
     Ticket::factory()
-            ->times(15)
-            ->create([
-                'priority' => 'something else',
-            ]);
+        ->times(15)
+        ->create([
+            'priority' => 'something else',
+        ]);
 
     $this->assertEquals(Ticket::count(), 42);
     $this->assertEquals(Ticket::withLowPriority()->count(), 7);
@@ -246,7 +246,7 @@
     ]);
 
     $ticket->archive()
-            ->markAsLocked();
+        ->markAsLocked();
 
     $this->assertTrue($ticket->isArchived());
     $this->assertTrue($ticket->isLocked());

tests/Models/User.php

@@ -14,14 +14,14 @@
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Foundation\Auth\Access\Authorizable;
 
-class User extends Model implements CanUseTickets, AuthenticatableContract, AuthorizableContract, CanResetPasswordContract
+class User extends Model implements AuthenticatableContract, AuthorizableContract, CanResetPasswordContract, CanUseTickets
 {
     use Authenticatable,
         Authorizable,
         CanResetPassword,
-        MustVerifyEmail,
         HasFactory,
-        HasTickets;
+        HasTickets,
+        MustVerifyEmail;
 
     protected $guarded = [];
 }

tests/Unit/LabelTest.php

@@ -7,10 +7,10 @@
     $ticket = Ticket::factory()->create();
 
     $label = Label::factory()
-                ->create([
-                    'name' => 'Support',
-                    'slug' => 'supoort',
-                ]);
+        ->create([
+            'name' => 'Support',
+            'slug' => 'supoort',
+        ]);
 
     $tableName = config(
         'laravel_ticket.table_names.labels',

tests/Unit/MessageTest.php

@@ -14,10 +14,10 @@
     );
 
     $message = Message::factory()
-                ->create([
-                    $tableName['columns']['ticket_foreing_id'] => $ticket->id,
-                    'message' => 'Message from a ticket',
-                ]);
+        ->create([
+            $tableName['columns']['ticket_foreing_id'] => $ticket->id,
+            'message' => 'Message from a ticket',
+        ]);
 
     $this->assertDatabaseHas($tableName['table'], [
         $tableName['columns']['ticket_foreing_id'] => $ticket->id,