[11.x] Fix unique jobs that have a uniqueVia method (#54294)

* Adds test for uniqueVia job dispatch * Make getName public
laravel · Jan 22, 2025 · 71203fc · 71203fc
1 parent 1c2a6c1
commit 71203fc
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 1 deletion.
diff --git a/src/Illuminate/Cache/Repository.php b/src/Illuminate/Cache/Repository.php
@@ -638,7 +638,7 @@ protected function getSeconds($ttl)
      *
      * @return string|null
      */
-    protected function getName()
+    public function getName()
     {
         return $this->config['store'] ?? null;
     }

diff --git a/tests/Integration/Queue/UniqueJobTest.php b/tests/Integration/Queue/UniqueJobTest.php
@@ -4,6 +4,7 @@
 
 use Exception;
 use Illuminate\Bus\Queueable;
+use Illuminate\Container\Container;
 use Illuminate\Contracts\Cache\Repository as Cache;
 use Illuminate\Contracts\Queue\ShouldBeUnique;
 use Illuminate\Contracts\Queue\ShouldBeUniqueUntilProcessing;
@@ -51,6 +52,14 @@ public function testUniqueJobsAreNotDispatched()
         );
     }
 
+    public function testUniqueJobWithViaDispatched()
+    {
+        Bus::fake();
+
+        UniqueViaJob::dispatch();
+        Bus::assertDispatched(UniqueViaJob::class);
+    }
+
     public function testLockIsReleasedForSuccessfulJobs()
     {
         UniqueTestJob::$handled = false;
@@ -222,3 +231,11 @@ public function __construct(public User $user)
     {
     }
 }
+
+class UniqueViaJob extends UniqueTestJob
+{
+    public function uniqueVia(): Cache
+    {
+        return Container::getInstance()->make(Cache::class);
+    }
+}