Added option to disable tracking

Author: Peter Klooster

src/JobStatus.php

@@ -87,7 +87,7 @@ public function getIsQueuedAttribute()
         return $this->status === self::STATUS_QUEUED;
     }
 
-    public function getIsRetrying()
+    public function getIsRetryingAttribute()
     {
         return $this->status === self::STATUS_RETRYING;
     }

src/Trackable.php

@@ -14,6 +14,7 @@ trait Trackable
     protected $statusId;
     protected $progressNow = 0;
     protected $progressMax = 0;
+    protected $shouldTrack = true;
 
     protected function setProgressMax($value)
     {
@@ -76,7 +77,7 @@ public function getJobStatusId()
 
     public function __sleep()
     {
-        if (!$this->statusId) {
+        if (!$this->statusId && $this->shouldTrack) {
             $this->prepareStatus();
         }
 

tests/Feature/TrackableTest.php

@@ -10,6 +10,7 @@
 use Imtigger\LaravelJobStatus\Tests\Data\TestJobWithDatabase;
 use Imtigger\LaravelJobStatus\Tests\Data\TestJobWithException;
 use Imtigger\LaravelJobStatus\Tests\Data\TestJobWithoutConstruct;
+use Imtigger\LaravelJobStatus\Tests\Data\TestJobWithoutTracking;
 
 class TrackableTest extends TestCase
 {
@@ -75,4 +76,17 @@ public function testWithoutPrepareStatus()
 
         $this->assertEquals(1, JobStatus::query()->count());
     }
+
+    public function testWithoutPrepareStatusAndTrackingDisabled()
+    {
+        $job = new TestJobWithoutTracking();
+
+        $this->assertNull($job->getJobStatusId());
+
+        $this->assertEquals(0, JobStatus::query()->count());
+
+        app(Dispatcher::class)->dispatch($job);
+
+        $this->assertEquals(0, JobStatus::query()->count());
+    }
 }

tests/_data/TestJob.php

@@ -6,7 +6,6 @@
 use Illuminate\Contracts\Queue\ShouldQueue;
 use Illuminate\Foundation\Bus\Dispatchable;
 use Illuminate\Queue\InteractsWithQueue;
-use Illuminate\Queue\SerializesModels;
 use Imtigger\LaravelJobStatus\Trackable;
 use Imtigger\LaravelJobStatus\TrackableJob;
 

tests/_data/TestJobWithDatabase.php

@@ -8,7 +8,6 @@
 use Illuminate\Foundation\Testing\Concerns\InteractsWithDatabase;
 use Illuminate\Foundation\Testing\Constraints\HasInDatabase;
 use Illuminate\Queue\InteractsWithQueue;
-use Illuminate\Queue\SerializesModels;
 use Imtigger\LaravelJobStatus\Tests\Feature\TestCase;
 use Imtigger\LaravelJobStatus\Trackable;
 use Imtigger\LaravelJobStatus\TrackableJob;

tests/_data/TestJobWithException.php

@@ -6,7 +6,6 @@
 use Illuminate\Contracts\Queue\ShouldQueue;
 use Illuminate\Foundation\Bus\Dispatchable;
 use Illuminate\Queue\InteractsWithQueue;
-use Illuminate\Queue\SerializesModels;
 use Imtigger\LaravelJobStatus\Trackable;
 use Imtigger\LaravelJobStatus\TrackableJob;
 

tests/_data/TestJobWithoutConstruct.php

@@ -6,7 +6,6 @@
 use Illuminate\Contracts\Queue\ShouldQueue;
 use Illuminate\Foundation\Bus\Dispatchable;
 use Illuminate\Queue\InteractsWithQueue;
-use Illuminate\Queue\SerializesModels;
 use Imtigger\LaravelJobStatus\Trackable;
 use Imtigger\LaravelJobStatus\TrackableJob;
 

tests/_data/TestJobWithoutTracking.php

@@ -0,0 +1,27 @@
+<?php
+
+namespace Imtigger\LaravelJobStatus\Tests\Data;
+
+use Illuminate\Bus\Queueable;
+use Illuminate\Contracts\Queue\ShouldQueue;
+use Illuminate\Foundation\Bus\Dispatchable;
+use Illuminate\Queue\InteractsWithQueue;
+use Imtigger\LaravelJobStatus\Trackable;
+use Imtigger\LaravelJobStatus\TrackableJob;
+
+class TestJobWithoutTracking implements ShouldQueue, TrackableJob
+{
+    use InteractsWithQueue;
+    use Queueable;
+    use Dispatchable;
+    use Trackable;
+
+    public function __construct()
+    {
+        $this->shouldTrack = false;
+    }
+
+    public function handle()
+    {
+    }
+}