fix: ensure debounce logic respects entity callability during cache initialization and testing

Author: mostafaznv

src/Cache.php

@@ -60,15 +60,17 @@ public function refresh(CacheEvent $event): void
 
             foreach ($this->model::cacheEntities() as $entity) {
                 if ($entity->debounce) {
-                    $this->initCache($entity, $entity->getTtl());
-
-                    RefreshDebouncer::dispatch(
-                        model: $this->model,
-                        name: $entity->name,
-                        queueConnection: $entity->queueConnection,
-                        queueName: $entity->queueName,
-                        wait: $entity->debounceWaitTime
-                    );
+                    if ($this->entityIsCallable($entity, $event)) {
+                        $this->initCache($entity, $entity->getTtl());
+
+                        RefreshDebouncer::dispatch(
+                            model: $this->model,
+                            name: $entity->name,
+                            queueConnection: $entity->queueConnection,
+                            queueName: $entity->queueName,
+                            wait: $entity->debounceWaitTime
+                        );
+                    }
                 }
                 else if ($entity->isQueueable) {
                     if ($this->entityIsCallable($entity, $event)) {

tests/Feature/DebounceCacheTest.php

@@ -3,6 +3,7 @@
 use Illuminate\Support\Facades\Artisan;
 use Illuminate\Support\Facades\Bus;
 use Illuminate\Support\Facades\Queue;
+use Mostafaznv\LaraCache\DTOs\CacheData;
 use Mostafaznv\LaraCache\Enums\CacheStatus;
 use Mostafaznv\LaraCache\Jobs\DebounceRefresh;
 use Mostafaznv\LaraCache\Jobs\UpdateLaraCacheModelsList;
@@ -201,3 +202,69 @@
         ->and($cache->value)
         ->toBeInstanceOf(DebounceTestModel::class);
 });
+
+it('will respect cache creation rules', function () {
+    # prepare
+    $worksOnCreation = 'debounce-test-model.latest.debounce';
+    $doesNotWorkOnCreation = 'debounce-test-model.latest.no-creation';
+
+    createDebounceModel();
+
+
+    # test 1 - after creation, before running the queue
+    $cache = Cache::get($worksOnCreation);
+    expect($cache)
+        ->toBeInstanceOf(CacheData::class)
+        ->and($cache->status)
+        ->toBe(CacheStatus::CREATING);
+
+    $cache = Cache::get($doesNotWorkOnCreation);
+    expect($cache)->toBeNull();
+
+
+    # waiting for debouncing
+    testTime()->freeze(
+        now()->addSeconds($this->waitTime - 1)
+    );
+
+    Artisan::call('queue:work --once --sleep=0');
+
+
+
+    # test 2 - after running the queue; before debouncing time is over
+    $cache = Cache::get($worksOnCreation);
+    expect($cache)
+        ->toBeInstanceOf(CacheData::class)
+        ->and($cache->status)
+        ->toBe(CacheStatus::CREATING);
+
+    $cache = Cache::get($doesNotWorkOnCreation);
+    expect($cache)->toBeNull();
+
+
+
+    # waiting for debouncing
+    testTime()->freeze(
+        now()->addSeconds($this->waitTime + 1)
+    );
+
+
+    # test 3 - after debouncing time is over
+    Artisan::call('queue:work --once --sleep=0');
+
+    $cache = Cache::get($worksOnCreation);
+    expect($cache)
+        ->toBeInstanceOf(CacheData::class)
+        ->and($cache->status)
+        ->toBe(CacheStatus::CREATED);
+
+    $cache = Cache::get($doesNotWorkOnCreation);
+    expect($cache)->toBeNull();
+
+    $cache = DebounceTestModel::cache()->get('latest.no-creation', true);
+    expect($cache)
+        ->toBeInstanceOf(CacheData::class)
+        ->and($cache->status)
+        ->toBe(CacheStatus::CREATED);
+
+});

tests/TestSupport/TestModels/DebounceTestModel.php

@@ -29,6 +29,16 @@ public static function cacheEntities(): array
             CacheEntity::make('latest')
                 ->validForRestOfDay()
                 ->setDefault(-1)
+                ->cache(function() {
+                    return DebounceTestModel::query()->latest()->first();
+                }),
+
+            CacheEntity::make('latest.no-creation')
+                ->validForRestOfDay()
+                ->debounce()
+                ->refreshAfterCreate(false)
+                ->refreshAfterUpdate(false)
+                ->setDefault(-1)
                 ->cache(function() {
                     return DebounceTestModel::query()->latest()->first();
                 })