Merge pull request #3 from cocur/stop

Added stop method to BackgroundProcess

Author: Florian Eckerstorfer

src/BackgroundProcess.php

@@ -69,6 +69,23 @@ public function isRunning()
         return false;
     }
 
+    /**
+     * Stops the process.
+     *
+     * @return boolean `true` if the processes was stopped, `false` otherwise.
+     */
+    public function stop()
+    {
+        try {
+            $result = shell_exec(sprintf('kill %d 2>&1', $this->pid));
+            if (!preg_match('/No such process/', $result)) {
+                return true;
+            }
+        } catch (Exception $e) {}
+
+        return false;
+    }
+
     /**
      * Returns the ID of the process.
      *

tests/BackgroundProcessTest.php

@@ -10,7 +10,7 @@
 /**
  * BackgroundProcessTest
  *
- * @category  Test
+ * @category  test
  * @package   cocur/background-process
  * @author    Florian Eckerstorfer <[email protected]>
  * @copyright 2013-2104 Florian Eckerstorfer
@@ -26,13 +26,17 @@ class BackgroundProcessTest extends \PHPUnit_Framework_TestCase
      * @covers Cocur\BackgroundProcess\BackgroundProcess::run()
      * @covers Cocur\BackgroundProcess\BackgroundProcess::isRunning()
      * @covers Cocur\BackgroundProcess\BackgroundProcess::getPid()
+     * @covers Cocur\BackgroundProcess\BackgroundProcess::stop()
      */
     public function testRun()
     {
-        $process = new BackgroundProcess('sleep 1');
-        $this->assertFalse($process->isRunning());
+        $process = new BackgroundProcess('sleep 5');
+        $this->assertFalse($process->isRunning(), 'process should not run');
         $process->run();
-        $this->assertNotNull($process->getPid());
-        $this->assertTrue($process->isRunning());
+        $this->assertNotNull($process->getPid(), 'process should have a pid');
+        $this->assertTrue($process->isRunning(), 'process should run');
+        $this->assertTrue($process->stop(), 'stop process');
+        $this->assertFalse($process->isRunning(), 'processes should not run anymore');
+        $this->assertFalse($process->stop(), 'cannot stop process that is not running');
     }
 }