[12.x] feat: --memory=0 should mean skip memory exceeded verification (Breaking Change) (#54393)

* [12.x] feat: MemoryExceeded 0 should mean infinite memory

* [12.x] feat: MemoryExceeded 0 should mean infinite memory

* [12.x] feat: MemoryExceeded 0 should mean infinite memory

Author: mathiasgrimm

src/Illuminate/Queue/Worker.php

@@ -748,7 +748,7 @@ protected function supportsAsyncSignals()
      */
     public function memoryExceeded($memoryLimit)
     {
-        return (memory_get_usage(true) / 1024 / 1024) >= $memoryLimit;
+        return $memoryLimit > 0 && (memory_get_usage(true) / 1024 / 1024) >= $memoryLimit;
     }
 
     /**

tests/Queue/QueueWorkerTest.php

@@ -100,6 +100,24 @@ public function testWorkerStopsWhenMemoryExceeded()
         $this->events->shouldHaveReceived('dispatch')->with(m::type(JobProcessed::class))->once();
     }
 
+    public function testWorkerMemoryExceededWhenMemoryIsZero()
+    {
+        $worker = new Worker(...$this->workerDependencies());
+        $this->assertFalse($worker->memoryExceeded(0));
+    }
+
+    public function testWorkerMemoryExceededWhenMemoryGreaterThanZero()
+    {
+        $worker = new Worker(...$this->workerDependencies());
+        $this->assertTrue($worker->memoryExceeded(1));
+    }
+
+    public function testWorkerMemoryExceededWhenMemoryIsNegative()
+    {
+        $worker = new Worker(...$this->workerDependencies());
+        $this->assertFalse($worker->memoryExceeded(-1));
+    }
+
     public function testJobCanBeFiredBasedOnPriority()
     {
         $worker = $this->getWorker('default', [