ICM: RuntimeException instead of simple code setting.

Author: dmitry-ivanov

src/WithoutOverlapping.php

@@ -2,6 +2,7 @@
 
 namespace Illuminated\Console;
 
+use RuntimeException;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 
@@ -18,9 +19,7 @@ protected function initializeMutex()
     {
         $mutex = new Mutex($this);
         if (!$mutex->acquireLock($this->getMutexTimeout())) {
-            $this->setCode(function () {
-                $this->info('Command is running now!');
-            });
+            throw new RuntimeException('Command is running now!');
         }
 
         register_shutdown_function([$this, 'releaseMutexLock'], $mutex);

tests/ConsoleMutex/WithoutOverlappingTraitTest.php

@@ -6,6 +6,7 @@
 use Mockery;
 use MysqlStrategyCommand;
 use NullTimeoutCommand;
+use RuntimeException;
 use TimeoutCommand;
 
 class WithoutOverlappingTraitTest extends TestCase
@@ -101,11 +102,11 @@ public function it_blocks_if_trying_to_run_another_instance_of_the_command()
     {
         $mutex = Mockery::mock('overload:Illuminated\Console\Mutex');
         $mutex->shouldReceive('acquireLock')->with(0)->once()->andReturn(false);
-        $mutex->shouldReceive('releaseLock')->withNoArgs();
 
-        $this->artisan('icm:generic');
+        $this->expectException(RuntimeException::class);
+        $this->expectExceptionMessage('Command is running now!');
 
-        $this->seeArtisanOutput('Command is running now!');
+        $this->artisan('icm:generic');
     }
 
     /**