ICM: Timeout tests added.

Author: dmitry-ivanov

src/WithoutOverlapping.php

@@ -37,7 +37,7 @@ public function setMutexStrategy($strategy)
 
     public function getMutexTimeout()
     {
-        return (isset($this->mutexTimeout) ? $this->mutexTimeout : 0);
+        return (property_exists($this, 'mutexTimeout') ? $this->mutexTimeout : 0);
     }
 
     public function setMutexTimeout($timeout)

tests/WithoutOverlappingTraitTest.php

@@ -35,6 +35,12 @@ public function mutex_timeout_can_be_overloaded_by_protected_field()
         $this->assertEquals(3000, (new TimeoutCommand)->getMutexTimeout());
     }
 
+    /** @test */
+    public function mutex_timeout_can_be_set_to_null_by_protected_field()
+    {
+        $this->assertNull((new NullTimeoutCommand)->getMutexTimeout());
+    }
+
     /** @test */
     public function it_generates_mutex_name_based_on_the_command_name_and_arguments()
     {

tests/fixture/app/Console/Commands/NullTimeoutCommand.php

@@ -0,0 +1,17 @@
+<?php
+
+use Illuminate\Console\Command;
+use Illuminated\Console\WithoutOverlapping;
+
+class NullTimeoutCommand extends Command
+{
+    use WithoutOverlapping;
+
+    protected $signature = 'icm:null-timeout-command';
+    protected $mutexTimeout = null;
+
+    public function handle()
+    {
+        $this->info('Done!');
+    }
+}

tests/fixture/app/Console/Kernel.php

@@ -5,6 +5,7 @@ class Kernel extends \Orchestra\Testbench\Console\Kernel
     protected $commands = [
         GenericCommand::class,
         MysqlStrategyCommand::class,
+        NullTimeoutCommand::class,
         TimeoutCommand::class,
     ];
 }