AliceMajere / wonderland-thread

tests/unit/Event/EventTest.php

Indentation +38 added lines, -38 removed lines

@@ -14,43 +14,43 @@
  */
 class EventTest extends TestCase
 {
-	/** @var PoolEvent */
-	private $event;
-
-	/** @var AbstractThread */
-	private $thread;
-
-	public function setUp()
-	{
-		$this->thread = new SuccessThread('');
-		$this->event = new PoolEvent('unit-test');
-		$this->event->setThread($this->thread);
-	}
-
-	public function test_event()
-	{
-		$this->assertSame('unit-test', $this->event->getEventName());
-		$this->assertSame(0, $this->event->getMaxRunningThreadNb());
-		$this->assertSame(0, $this->event->getRunningThreadNb());
-		$this->assertSame(0, $this->event->getThreadDoneNb());
-		$this->assertSame(0, $this->event->getThreadLeftNb());
-		$this->assertSame(0, $this->event->getThreadNb());
-		$this->assertSame($this->thread, $this->event->getThread());
-
-		$this->assertSame($this->event, $this->event->setThread($this->thread));
-		$this->assertSame($this->event, $this->event->setMaxRunningThreadNb(100));
-		$this->assertSame($this->event, $this->event->setEventName('fake'));
-		$this->assertSame($this->event, $this->event->setRunningThreadNb(10));
-		$this->assertSame($this->event, $this->event->setThreadDoneNb(90));
-		$this->assertSame($this->event, $this->event->setThreadLeftNb(10));
-		$this->assertSame($this->event, $this->event->setThreadNb(10));
-
-		$this->assertSame('fake', $this->event->getEventName());
-		$this->assertSame(100, $this->event->getMaxRunningThreadNb());
-		$this->assertSame(10, $this->event->getRunningThreadNb());
-		$this->assertSame(90, $this->event->getThreadDoneNb());
-		$this->assertSame(10, $this->event->getThreadLeftNb());
-		$this->assertSame(10, $this->event->getThreadNb());
-	}
+    /** @var PoolEvent */
+    private $event;
+
+    /** @var AbstractThread */
+    private $thread;
+
+    public function setUp()
+    {
+        $this->thread = new SuccessThread('');
+        $this->event = new PoolEvent('unit-test');
+        $this->event->setThread($this->thread);
+    }
+
+    public function test_event()
+    {
+        $this->assertSame('unit-test', $this->event->getEventName());
+        $this->assertSame(0, $this->event->getMaxRunningThreadNb());
+        $this->assertSame(0, $this->event->getRunningThreadNb());
+        $this->assertSame(0, $this->event->getThreadDoneNb());
+        $this->assertSame(0, $this->event->getThreadLeftNb());
+        $this->assertSame(0, $this->event->getThreadNb());
+        $this->assertSame($this->thread, $this->event->getThread());
+
+        $this->assertSame($this->event, $this->event->setThread($this->thread));
+        $this->assertSame($this->event, $this->event->setMaxRunningThreadNb(100));
+        $this->assertSame($this->event, $this->event->setEventName('fake'));
+        $this->assertSame($this->event, $this->event->setRunningThreadNb(10));
+        $this->assertSame($this->event, $this->event->setThreadDoneNb(90));
+        $this->assertSame($this->event, $this->event->setThreadLeftNb(10));
+        $this->assertSame($this->event, $this->event->setThreadNb(10));
+
+        $this->assertSame('fake', $this->event->getEventName());
+        $this->assertSame(100, $this->event->getMaxRunningThreadNb());
+        $this->assertSame(10, $this->event->getRunningThreadNb());
+        $this->assertSame(90, $this->event->getThreadDoneNb());
+        $this->assertSame(10, $this->event->getThreadLeftNb());
+        $this->assertSame(10, $this->event->getThreadNb());
+    }
 
 }

tests/unit/ThreadTest.php

Indentation +43 added lines, -43 removed lines

@@ -17,55 +17,55 @@
  */
 class ThreadTest extends TestCase
 {
-	/** @var AbstractThread */
-	private $thread;
+    /** @var AbstractThread */
+    private $thread;
 
-	public function setUp()
-	{
-		$this->thread = new SuccessThread('unit-test');
-		$this->thread->setMediator(new Mediator());
-		$this->thread->setPid(1);
-	}
+    public function setUp()
+    {
+        $this->thread = new SuccessThread('unit-test');
+        $this->thread->setMediator(new Mediator());
+        $this->thread->setPid(1);
+    }
 
-	public function test_pid()
-	{
-		$this->assertSame(1, $this->thread->getPid());
-		$this->assertSame($this->thread, $this->thread->setPid(100));
-		$this->assertSame(100, $this->thread->getPid());
-	}
+    public function test_pid()
+    {
+        $this->assertSame(1, $this->thread->getPid());
+        $this->assertSame($this->thread, $this->thread->setPid(100));
+        $this->assertSame(100, $this->thread->getPid());
+    }
 
-	public function test_processName()
-	{
-		$this->assertSame('unit-test', $this->thread->getProcessName());
-	}
+    public function test_processName()
+    {
+        $this->assertSame('unit-test', $this->thread->getProcessName());
+    }
 
-	/**
-	 * @throws ThreadException
-	 */
-	public function test_run()
-	{
-		$this->assertSame(0, $this->thread->run());
-	}
+    /**
+     * @throws ThreadException
+     */
+    public function test_run()
+    {
+        $this->assertSame(0, $this->thread->run());
+    }
 
-	/**
-	 * @throws ThreadException
-	 */
-	public function test_run_exception()
-	{
-		$this->expectException(ThreadException::class);
-		$thread = new ExceptionThread('unit-test');
-		$thread->run();
+    /**
+     * @throws ThreadException
+     */
+    public function test_run_exception()
+    {
+        $this->expectException(ThreadException::class);
+        $thread = new ExceptionThread('unit-test');
+        $thread->run();
 
-	}
+    }
 
-	/**
-	 * @throws ThreadException
-	 */
-	public function test_run_exception_status()
-	{
-		$this->expectException(ThreadException::class);
-		$thread = new ErrorThread('unit-test');
-		$thread->run();
-	}
+    /**
+     * @throws ThreadException
+     */
+    public function test_run_exception_status()
+    {
+        $this->expectException(ThreadException::class);
+        $thread = new ErrorThread('unit-test');
+        $thread->run();
+    }
 
 }

tests/unit/ThreadPoolTest.php

Indentation +84 added lines, -84 removed lines

@@ -19,89 +19,89 @@
  */
 class ThreadPoolTest extends TestCase
 {
-	/** @var ThreadPool */
-	private $threadPool;
-
-	protected function setUp()
-	{
-		$this->threadPool = new ThreadPool();
-	}
-
-	public function test_constructor_destructor()
-	{
-		$this->assertAttributeEquals([], 'threads', $this->threadPool);
-		$this->assertAttributeEquals([], 'runningThreads', $this->threadPool);
-		$this->assertAttributeEquals([], 'toRunThreads', $this->threadPool);
-		$this->assertAttributeEquals(false, 'isRunning', $this->threadPool);
-		$this->assertAttributeEquals(0, 'maxRunningThreadNb', $this->threadPool);
-
-		unset($this->threadPool);
-	}
-
-	public function test_getThreads()
-	{
-		$threads = [new SuccessThread('')];
-		$this->assertSame($this->threadPool, $this->threadPool->setThreads($threads));
-		$this->assertSame($threads, $this->threadPool->getThreads());
-	}
-
-	public function test_addThread()
-	{
-		$thread = new SuccessThread('');
-		$this->assertSame($this->threadPool, $this->threadPool->addThread($thread));
-		$this->assertContains($thread, $this->threadPool->getThreads());
-	}
-
-	public function test_getMaxRunningThreadNb()
-	{
-		$this->assertSame(0, $this->threadPool->getMaxRunningThreadNb());
-	}
-
-	public function test_setMaxRunningThreadNb()
-	{
-		$this->assertSame($this->threadPool, $this->threadPool->setMaxRunningThreadNb(10));
-		$this->assertSame(10, $this->threadPool->getMaxRunningThreadNb());
-	}
-
-	public function test_getToRunThreads()
-	{
-		$this->assertSame([], $this->threadPool->getToRunThreads());
-	}
-
-	public function test_getRunningThreads()
-	{
-		$this->assertSame([], $this->threadPool->getRunningThreads());
-	}
-
-	/**
-	 * @throws \Wonderland\Thread\Exception\ThreadException
-	 */
-	public function test_run()
-	{
-		$thread = new SuccessThread('');
-		$threads = [];
-		$listener = new TestListener();
-
-		$this->threadPool->setMaxRunningThreadNb(10);
-		$this->assertSame($this->threadPool, $this->threadPool->addListener($listener));
-		for ($i = 0; $i < 20; $i++) {
-			$threads[] = clone $thread;
-		}
-		$this->assertSame($this->threadPool, $this->threadPool->addThread($thread));
-		$this->assertSame($this->threadPool, $this->threadPool->setThreads($threads));
-
-		$this->threadPool->run();
-
-		$this->assertSame($this->threadPool, $this->threadPool->removeListener($listener));
-	}
-
-	/**
-	 * @throws ThreadException
-	 */
-	public function test_run_exception()
-	{
-		$this->expectException(ThreadException::class);
-		$this->threadPool->run();
-	}
+    /** @var ThreadPool */
+    private $threadPool;
+
+    protected function setUp()
+    {
+        $this->threadPool = new ThreadPool();
+    }
+
+    public function test_constructor_destructor()
+    {
+        $this->assertAttributeEquals([], 'threads', $this->threadPool);
+        $this->assertAttributeEquals([], 'runningThreads', $this->threadPool);
+        $this->assertAttributeEquals([], 'toRunThreads', $this->threadPool);
+        $this->assertAttributeEquals(false, 'isRunning', $this->threadPool);
+        $this->assertAttributeEquals(0, 'maxRunningThreadNb', $this->threadPool);
+
+        unset($this->threadPool);
+    }
+
+    public function test_getThreads()
+    {
+        $threads = [new SuccessThread('')];
+        $this->assertSame($this->threadPool, $this->threadPool->setThreads($threads));
+        $this->assertSame($threads, $this->threadPool->getThreads());
+    }
+
+    public function test_addThread()
+    {
+        $thread = new SuccessThread('');
+        $this->assertSame($this->threadPool, $this->threadPool->addThread($thread));
+        $this->assertContains($thread, $this->threadPool->getThreads());
+    }
+
+    public function test_getMaxRunningThreadNb()
+    {
+        $this->assertSame(0, $this->threadPool->getMaxRunningThreadNb());
+    }
+
+    public function test_setMaxRunningThreadNb()
+    {
+        $this->assertSame($this->threadPool, $this->threadPool->setMaxRunningThreadNb(10));
+        $this->assertSame(10, $this->threadPool->getMaxRunningThreadNb());
+    }
+
+    public function test_getToRunThreads()
+    {
+        $this->assertSame([], $this->threadPool->getToRunThreads());
+    }
+
+    public function test_getRunningThreads()
+    {
+        $this->assertSame([], $this->threadPool->getRunningThreads());
+    }
+
+    /**
+     * @throws \Wonderland\Thread\Exception\ThreadException
+     */
+    public function test_run()
+    {
+        $thread = new SuccessThread('');
+        $threads = [];
+        $listener = new TestListener();
+
+        $this->threadPool->setMaxRunningThreadNb(10);
+        $this->assertSame($this->threadPool, $this->threadPool->addListener($listener));
+        for ($i = 0; $i < 20; $i++) {
+            $threads[] = clone $thread;
+        }
+        $this->assertSame($this->threadPool, $this->threadPool->addThread($thread));
+        $this->assertSame($this->threadPool, $this->threadPool->setThreads($threads));
+
+        $this->threadPool->run();
+
+        $this->assertSame($this->threadPool, $this->threadPool->removeListener($listener));
+    }
+
+    /**
+     * @throws ThreadException
+     */
+    public function test_run_exception()
+    {
+        $this->expectException(ThreadException::class);
+        $this->threadPool->run();
+    }
 
 }

src/Factory/EventPoolFactory.php

Indentation +21 added lines, -21 removed lines

@@ -10,27 +10,27 @@
 class EventPoolFactory
 {
 
-	/**
-	 * @param string                                $eventName
-	 * @param ThreadPool|AbstractThreadPoolMediator $pool
-	 * @param AbstractThread|null                   $thread
-	 * @return PoolEvent
-	 */
-	public static function create(
-		string $eventName,
-		AbstractThreadPoolMediator $pool,
-		?AbstractThread $thread = null
-	): PoolEvent
-	{
-		$event = new PoolEvent($eventName);
-		$event->setThreadNb(count($pool->getThreads()));
-		$event->setThreadDoneNb(count($pool->getThreads()) - count($pool->getToRunThreads()));
-		$event->setMaxRunningThreadNb($pool->getMaxRunningThreadNb());
-		$event->setThreadLeftNb(count($pool->getToRunThreads()));
-		$event->setRunningThreadNb(count($pool->getRunningThreads()));
-		$event->setThread($thread);
+    /**
+     * @param string                                $eventName
+     * @param ThreadPool|AbstractThreadPoolMediator $pool
+     * @param AbstractThread|null                   $thread
+     * @return PoolEvent
+     */
+    public static function create(
+        string $eventName,
+        AbstractThreadPoolMediator $pool,
+        ?AbstractThread $thread = null
+    ): PoolEvent
+    {
+        $event = new PoolEvent($eventName);
+        $event->setThreadNb(count($pool->getThreads()));
+        $event->setThreadDoneNb(count($pool->getThreads()) - count($pool->getToRunThreads()));
+        $event->setMaxRunningThreadNb($pool->getMaxRunningThreadNb());
+        $event->setThreadLeftNb(count($pool->getToRunThreads()));
+        $event->setRunningThreadNb(count($pool->getRunningThreads()));
+        $event->setThread($thread);
 
-		return $event;
-	}
+        return $event;
+    }
 
 }

src/Mediator/Mediator.php

Indentation +45 added lines, -45 removed lines

@@ -7,50 +7,50 @@
 
 class Mediator
 {
-	/** @var ListenerInterface[] */
-	private $listeners = [];
-
-	/**
-	 * @return ListenerInterface[]
-	 */
-	public function getListeners(): array
-	{
-		return $this->listeners;
-	}
-
-	/**
-	 * @param ListenerInterface $listener
-	 * @return Mediator
-	 */
-	public function addListener(ListenerInterface $listener): self
-	{
-		$this->listeners[] = $listener;
-
-		return $this;
-	}
-
-	/**
-	 * @param ListenerInterface $listener
-	 * @return Mediator
-	 */
-	public function removeListener(ListenerInterface $listener): self
-	{
-		$key = array_search($listener, $this->listeners, true);
-		if (false !== $key) {
-			unset($this->listeners[$key]);
-		}
-
-		return $this;
-	}
-
-	/**
-	 * @param EventInterface $event
-	 */
-	public function notify(EventInterface $event)
-	{
-		foreach ($this->listeners as $listener) {
-			$listener->notify($event);
-		}
-	}
+    /** @var ListenerInterface[] */
+    private $listeners = [];
+
+    /**
+     * @return ListenerInterface[]
+     */
+    public function getListeners(): array
+    {
+        return $this->listeners;
+    }
+
+    /**
+     * @param ListenerInterface $listener
+     * @return Mediator
+     */
+    public function addListener(ListenerInterface $listener): self
+    {
+        $this->listeners[] = $listener;
+
+        return $this;
+    }
+
+    /**
+     * @param ListenerInterface $listener
+     * @return Mediator
+     */
+    public function removeListener(ListenerInterface $listener): self
+    {
+        $key = array_search($listener, $this->listeners, true);
+        if (false !== $key) {
+            unset($this->listeners[$key]);
+        }
+
+        return $this;
+    }
+
+    /**
+     * @param EventInterface $event
+     */
+    public function notify(EventInterface $event)
+    {
+        foreach ($this->listeners as $listener) {
+            $listener->notify($event);
+        }
+    }
 
 }

src/Mediator/Listener/AbstractListener.php

Indentation +14 added lines, -14 removed lines

@@ -7,20 +7,20 @@
 abstract class AbstractListener implements ListenerInterface
 {
 
-	/**
-	 * @param EventInterface $event
-	 */
-	public function notify(EventInterface $event)
-	{
-		$mapping = $this->getEventMapping();
-		if (false === isset($mapping[$event->getEventName()])) {
-			return;
-		}
+    /**
+     * @param EventInterface $event
+     */
+    public function notify(EventInterface $event)
+    {
+        $mapping = $this->getEventMapping();
+        if (false === isset($mapping[$event->getEventName()])) {
+            return;
+        }
 
-		call_user_func(
-			[$this, $mapping[$event->getEventName()]],
-			$event
-		);
-	}
+        call_user_func(
+            [$this, $mapping[$event->getEventName()]],
+            $event
+        );
+    }
 
 }

src/AbstractThread.php

Indentation +83 added lines, -83 removed lines

@@ -6,88 +6,88 @@
 
 abstract class AbstractThread extends AbstractThreadMediator
 {
-	const EXIT_STATUS_SUCCESS = 0;
-	const EXIT_STATUS_ERROR = 1;
-
-	/** @var int $pid */
-	private $pid;
-
-	/** @var string $processName */
-	private $processName;
-
-	/**
-	 * AbstractThread constructor.
-	 *
-	 * @param string $processName
-	 */
-	public function __construct(string $processName)
-	{
-		$this->processName = $processName;
-	}
-
-	/**
-	 * Return the name of the method to process during the thread
-	 *
-	 * @return string
-	 */
-	abstract protected function getMethodName(): string;
-
-	/**
-	 * Return the list of dependencies that will be passed as parameters of the method referenced by getMethodName
-	 *
-	 * @return array
-	 */
-	abstract protected function getDependencies(): array;
-
-	/**
-	 * @return int|null
-	 */
-	public function getPid(): ?int
-	{
-		return $this->pid;
-	}
-
-	/**
-	 * @param int $pid
-	 * @return AbstractThread
-	 */
-	public function setPid(int $pid): self
-	{
-		$this->pid = $pid;
-
-		return $this;
-	}
-
-	/**
-	 * @return string
-	 */
-	public function getProcessName(): ?string
-	{
-		return $this->processName;
-	}
-
-	/**
-	 * @return int
-	 * @throws ThreadException
-	 */
-	public function run(): int
-	{
-		if (false === method_exists($this, $this->getMethodName())) {
-			throw new ThreadException('No proper method defined for the thread');
-		}
-
-		$status = call_user_func_array(
-			[$this, $this->getMethodName()],
-			array_merge([$this->getProcessName()], $this->getDependencies())
-		);
-
-		if (null === $status) {
-			throw new ThreadException(
-				'Error. You must return a process status in '.get_class($this).':'.$this->getMethodName()
-			);
-		}
-
-		return $status;
-	}
+    const EXIT_STATUS_SUCCESS = 0;
+    const EXIT_STATUS_ERROR = 1;
+
+    /** @var int $pid */
+    private $pid;
+
+    /** @var string $processName */
+    private $processName;
+
+    /**
+     * AbstractThread constructor.
+     *
+     * @param string $processName
+     */
+    public function __construct(string $processName)
+    {
+        $this->processName = $processName;
+    }
+
+    /**
+     * Return the name of the method to process during the thread
+     *
+     * @return string
+     */
+    abstract protected function getMethodName(): string;
+
+    /**
+     * Return the list of dependencies that will be passed as parameters of the method referenced by getMethodName
+     *
+     * @return array
+     */
+    abstract protected function getDependencies(): array;
+
+    /**
+     * @return int|null
+     */
+    public function getPid(): ?int
+    {
+        return $this->pid;
+    }
+
+    /**
+     * @param int $pid
+     * @return AbstractThread
+     */
+    public function setPid(int $pid): self
+    {
+        $this->pid = $pid;
+
+        return $this;
+    }
+
+    /**
+     * @return string
+     */
+    public function getProcessName(): ?string
+    {
+        return $this->processName;
+    }
+
+    /**
+     * @return int
+     * @throws ThreadException
+     */
+    public function run(): int
+    {
+        if (false === method_exists($this, $this->getMethodName())) {
+            throw new ThreadException('No proper method defined for the thread');
+        }
+
+        $status = call_user_func_array(
+            [$this, $this->getMethodName()],
+            array_merge([$this->getProcessName()], $this->getDependencies())
+        );
+
+        if (null === $status) {
+            throw new ThreadException(
+                'Error. You must return a process status in '.get_class($this).':'.$this->getMethodName()
+            );
+        }
+
+        return $status;
+    }
 
 }

src/AbstractThreadMediator.php

Indentation +20 added lines, -20 removed lines

@@ -7,28 +7,28 @@
 
 class AbstractThreadMediator
 {
-	/** @var Mediator */
-	private $mediator;
+    /** @var Mediator */
+    private $mediator;
 
-	/**
-	 * @return Mediator
-	 */
-	protected function getMediator()
-	{
-		return $this->mediator;
-	}
+    /**
+     * @return Mediator
+     */
+    protected function getMediator()
+    {
+        return $this->mediator;
+    }
 
-	public function setMediator(Mediator $mediator)
-	{
-		$this->mediator = $mediator;
-	}
+    public function setMediator(Mediator $mediator)
+    {
+        $this->mediator = $mediator;
+    }
 
-	/**
-	 * @param EventInterface $event
-	 */
-	protected function notify(EventInterface $event)
-	{
-		$this->getMediator()->notify($event);
-	}
+    /**
+     * @param EventInterface $event
+     */
+    protected function notify(EventInterface $event)
+    {
+        $this->getMediator()->notify($event);
+    }
 
 }

src/ThreadPool.php

Indentation +268 added lines, -268 removed lines

@@ -7,280 +7,280 @@
 
 class ThreadPool extends AbstractThreadPoolMediator
 {
-	// 0.2s
-	private const SLEEP_TIME_MS = 50000;
-
-	/** @var AbstractThread[] $childs */
-	private $threads;
-
-	/** @var AbstractThread[] $toRunThreads */
-	private $toRunThreads;
-
-	/** @var AbstractThread[] $runningChilds */
-	private $runningThreads;
-
-	/** @var bool $isRunning */
-	private $isRunning;
-
-	/** @var int $maxRunningThreadNb */
-	private $maxRunningThreadNb;
-
-	/**
-	 * ThreadPool constructor.
-	 */
-	public function __construct()
-	{
-		parent::__construct();
-		$this->threads = [];
-		$this->runningThreads = [];
-		$this->toRunThreads = [];
-		$this->isRunning = false;
-		$this->maxRunningThreadNb = 0;
-	}
-
-	/**
-	 *
-	 */
-	public function __destruct()
-	{
-		pcntl_waitpid(-1, $status, WNOHANG);
-	}
-
-	/**
-	 * @return AbstractThread[]
-	 */
-	public function getThreads(): array
-	{
-		return $this->threads;
-	}
-
-	/**
-	 * @param AbstractThread[] $threads
-	 * @return ThreadPool
-	 */
-	public function setThreads(array $threads): self
-	{
-		$this->threads = $threads;
-
-		return $this;
-	}
-
-	/**
-	 * @param AbstractThread $thread
-	 * @return ThreadPool
-	 */
-	public function addThread(AbstractThread $thread): self
-	{
-		$this->threads[] = $thread;
-
-		return $this;
-	}
-
-	public function addLiveThread(AbstractThread $thread): self
+    // 0.2s
+    private const SLEEP_TIME_MS = 50000;
+
+    /** @var AbstractThread[] $childs */
+    private $threads;
+
+    /** @var AbstractThread[] $toRunThreads */
+    private $toRunThreads;
+
+    /** @var AbstractThread[] $runningChilds */
+    private $runningThreads;
+
+    /** @var bool $isRunning */
+    private $isRunning;
+
+    /** @var int $maxRunningThreadNb */
+    private $maxRunningThreadNb;
+
+    /**
+     * ThreadPool constructor.
+     */
+    public function __construct()
+    {
+        parent::__construct();
+        $this->threads = [];
+        $this->runningThreads = [];
+        $this->toRunThreads = [];
+        $this->isRunning = false;
+        $this->maxRunningThreadNb = 0;
+    }
+
+    /**
+     *
+     */
+    public function __destruct()
+    {
+        pcntl_waitpid(-1, $status, WNOHANG);
+    }
+
+    /**
+     * @return AbstractThread[]
+     */
+    public function getThreads(): array
+    {
+        return $this->threads;
+    }
+
+    /**
+     * @param AbstractThread[] $threads
+     * @return ThreadPool
+     */
+    public function setThreads(array $threads): self
+    {
+        $this->threads = $threads;
+
+        return $this;
+    }
+
+    /**
+     * @param AbstractThread $thread
+     * @return ThreadPool
+     */
+    public function addThread(AbstractThread $thread): self
+    {
+        $this->threads[] = $thread;
+
+        return $this;
+    }
+
+    public function addLiveThread(AbstractThread $thread): self
     {
         $this->toRunThreads[] = $thread;
 
         return $this;
     }
 
-	/**
-	 * @return int
-	 */
-	public function getMaxRunningThreadNb(): int
-	{
-		return $this->maxRunningThreadNb;
-	}
-
-	/**
-	 * @param int $maxRunningThreadNb
-	 * @return ThreadPool
-	 */
-	public function setMaxRunningThreadNb(int $maxRunningThreadNb): self
-	{
-		$this->maxRunningThreadNb = $maxRunningThreadNb;
-
-		return $this;
-	}
-
-	/**
-	 * @return AbstractThread[]
-	 */
-	public function getToRunThreads(): array
-	{
-		return $this->toRunThreads;
-	}
-
-	/**
-	 * @return AbstractThread[]
-	 */
-	public function getRunningThreads(): array
-	{
-		return $this->runningThreads;
-	}
-
-	/**
-	 * @throws ThreadException
-	 */
-	public function run()
-	{
-		$this->checkEnv();
-		$this->initRun();
-
-		while ($this->isRunningThreads()) {
-			$this->waitOnThreads();
-		}
-
-		$this->resetRun();
-	}
-
-	/**
-	 * @return bool
-	 * @throws ThreadException
-	 */
-	private function isRunningThreads(): bool
-	{
+    /**
+     * @return int
+     */
+    public function getMaxRunningThreadNb(): int
+    {
+        return $this->maxRunningThreadNb;
+    }
+
+    /**
+     * @param int $maxRunningThreadNb
+     * @return ThreadPool
+     */
+    public function setMaxRunningThreadNb(int $maxRunningThreadNb): self
+    {
+        $this->maxRunningThreadNb = $maxRunningThreadNb;
+
+        return $this;
+    }
+
+    /**
+     * @return AbstractThread[]
+     */
+    public function getToRunThreads(): array
+    {
+        return $this->toRunThreads;
+    }
+
+    /**
+     * @return AbstractThread[]
+     */
+    public function getRunningThreads(): array
+    {
+        return $this->runningThreads;
+    }
+
+    /**
+     * @throws ThreadException
+     */
+    public function run()
+    {
+        $this->checkEnv();
+        $this->initRun();
+
+        while ($this->isRunningThreads()) {
+            $this->waitOnThreads();
+        }
+
+        $this->resetRun();
+    }
+
+    /**
+     * @return bool
+     * @throws ThreadException
+     */
+    private function isRunningThreads(): bool
+    {
 //	    echo count($this->toRunThreads) . "\t|\t" . count($this->runningThreads) . PHP_EOL;
-		if (count($this->toRunThreads) > 0) {
-			while (count($this->runningThreads) < $this->maxRunningThreadNb && count($this->toRunThreads) > 0) {
-				$this->createThreadProcess(array_shift($this->toRunThreads));
-			}
-		}
-
-		return count($this->runningThreads) > 0;
-	}
-
-	/**
-	 * can't test some part of it this since we can't unit-test in web and we're never in a child
-	 * process when pid 0 when unit-testing since the coverage is done by the parent thread
-	 *
-	 * @param AbstractThread $thread
-	 * @throws ThreadException
-	 */
-	private function createThreadProcess(AbstractThread $thread)
-	{
-		$pid = pcntl_fork();
-
-		switch ($pid) {
-			case -1: //error forking
-				// @codeCoverageIgnoreStart
-				throw new ThreadException('Error while trying to fork. Check your server installation');
-				// @codeCoverageIgnoreEnd
-			case 0: // child
-				// @codeCoverageIgnoreStart
-				$thread->setMediator($this->getMediator());
-				$this->processThread($thread);
-				break;
-				// @codeCoverageIgnoreEnd
-			default: //parent
-				$thread->setPid($pid);
-				$this->runningThreads[] = $thread;
-				$this->notify(PoolEvent::POOL_NEW_THREAD, $thread);
-				$this->startRunStatus();
-		}
-	}
-
-	/**
-	 *
-	 */
-	private function waitOnThreads()
-	{
-		$this->notify(PoolEvent::POOL_PRE_WAIT_TICK);
-		foreach ($this->runningThreads as $k => $thread) {
-
-			$res = pcntl_waitpid($thread->getPid(), $status, WNOHANG);
-			$this->notify(PoolEvent::POOL_WAIT_TICK_PID);
-
-			if ($res === -1 || $res > 0) {
-				$this->notify(PoolEvent::POOL_WAIT_TICK_PID_REMOVED, $thread);
-				unset($this->runningThreads[$k]);
-			}
-
-		}
-		$this->notify(PoolEvent::POOL_POST_WAIT_TICK);
-
-		usleep(self::SLEEP_TIME_MS);
-	}
-
-	/**
-	 * @codeCoverageIgnore Can't test since this is only run in a child thread.. which doesnt' go throug the
-	 * unit-test coverage which is only done in the main process
-	 * @param AbstractThread $thread
-	 * @throws ThreadException
-	 */
-	private function processThread(AbstractThread $thread)
-	{
-		$this->notify(PoolEvent::THREAD_PRE_PROCESS, $thread);
-		$response = $thread->run();
-		$this->notify(PoolEvent::THREAD_POST_PROCESS, $thread);
-
-		switch ($response) {
-			case AbstractThread::EXIT_STATUS_SUCCESS:
-				$this->notify(PoolEvent::THREAD_EXIT_SUCCESS, $thread);
-				break;
-			case AbstractThread::EXIT_STATUS_ERROR:
-				$this->notify(PoolEvent::THREAD_EXIT_ERROR, $thread);
-				break;
-			default:
-				$this->notify(PoolEvent::THREAD_EXIT_UNKNOWN, $thread);
-		}
-
-		exit($response);
-	}
-
-	/**
-	 * Can't test the exception is not in cli since php-unit is only run in cli environment
-	 * @throws ThreadException
-	 */
-	private function checkEnv()
-	{
-		if (false === $this->isCli()) {
-			// @codeCoverageIgnoreStart
-			throw new ThreadException('Error. It is not safe to use process forking in other way than php-cli');
-			// @codeCoverageIgnoreEnd
-		}
-		if (0 === count($this->threads)) {
-			throw new ThreadException('Error. Can\'t run child threads processes without any added in the Pool');
-		}
-	}
-
-	/**
-	 *
-	 */
-	private function initRun()
-	{
-		$this->resetRun();
-	}
-
-	/**
-	 * @return bool
-	 */
-	private function isCli(): bool
-	{
-		return PHP_SAPI === 'cli';
-	}
-
-	/**
-	 *
-	 */
-	private function startRunStatus()
-	{
-		if (false === $this->isRunning) {
-			$this->notify(PoolEvent::POOL_RUN_START);
-			$this->isRunning = true;
-		}
-	}
-
-	/**
-	 *
-	 */
-	private function resetRun()
-	{
-		if (true === $this->isRunning) {
-			$this->notify(PoolEvent::POOL_RUN_STOP);
-		}
-		$this->isRunning = false;
-		$this->toRunThreads = $this->threads;
-		$this->runningThreads = [];
-	}
+        if (count($this->toRunThreads) > 0) {
+            while (count($this->runningThreads) < $this->maxRunningThreadNb && count($this->toRunThreads) > 0) {
+                $this->createThreadProcess(array_shift($this->toRunThreads));
+            }
+        }
+
+        return count($this->runningThreads) > 0;
+    }
+
+    /**
+     * can't test some part of it this since we can't unit-test in web and we're never in a child
+     * process when pid 0 when unit-testing since the coverage is done by the parent thread
+     *
+     * @param AbstractThread $thread
+     * @throws ThreadException
+     */
+    private function createThreadProcess(AbstractThread $thread)
+    {
+        $pid = pcntl_fork();
+
+        switch ($pid) {
+            case -1: //error forking
+                // @codeCoverageIgnoreStart
+                throw new ThreadException('Error while trying to fork. Check your server installation');
+                // @codeCoverageIgnoreEnd
+            case 0: // child
+                // @codeCoverageIgnoreStart
+                $thread->setMediator($this->getMediator());
+                $this->processThread($thread);
+                break;
+                // @codeCoverageIgnoreEnd
+            default: //parent
+                $thread->setPid($pid);
+                $this->runningThreads[] = $thread;
+                $this->notify(PoolEvent::POOL_NEW_THREAD, $thread);
+                $this->startRunStatus();
+        }
+    }
+
+    /**
+     *
+     */
+    private function waitOnThreads()
+    {
+        $this->notify(PoolEvent::POOL_PRE_WAIT_TICK);
+        foreach ($this->runningThreads as $k => $thread) {
+
+            $res = pcntl_waitpid($thread->getPid(), $status, WNOHANG);
+            $this->notify(PoolEvent::POOL_WAIT_TICK_PID);
+
+            if ($res === -1 || $res > 0) {
+                $this->notify(PoolEvent::POOL_WAIT_TICK_PID_REMOVED, $thread);
+                unset($this->runningThreads[$k]);
+            }
+
+        }
+        $this->notify(PoolEvent::POOL_POST_WAIT_TICK);
+
+        usleep(self::SLEEP_TIME_MS);
+    }
+
+    /**
+     * @codeCoverageIgnore Can't test since this is only run in a child thread.. which doesnt' go throug the
+     * unit-test coverage which is only done in the main process
+     * @param AbstractThread $thread
+     * @throws ThreadException
+     */
+    private function processThread(AbstractThread $thread)
+    {
+        $this->notify(PoolEvent::THREAD_PRE_PROCESS, $thread);
+        $response = $thread->run();
+        $this->notify(PoolEvent::THREAD_POST_PROCESS, $thread);
+
+        switch ($response) {
+            case AbstractThread::EXIT_STATUS_SUCCESS:
+                $this->notify(PoolEvent::THREAD_EXIT_SUCCESS, $thread);
+                break;
+            case AbstractThread::EXIT_STATUS_ERROR:
+                $this->notify(PoolEvent::THREAD_EXIT_ERROR, $thread);
+                break;
+            default:
+                $this->notify(PoolEvent::THREAD_EXIT_UNKNOWN, $thread);
+        }
+
+        exit($response);
+    }
+
+    /**
+     * Can't test the exception is not in cli since php-unit is only run in cli environment
+     * @throws ThreadException
+     */
+    private function checkEnv()
+    {
+        if (false === $this->isCli()) {
+            // @codeCoverageIgnoreStart
+            throw new ThreadException('Error. It is not safe to use process forking in other way than php-cli');
+            // @codeCoverageIgnoreEnd
+        }
+        if (0 === count($this->threads)) {
+            throw new ThreadException('Error. Can\'t run child threads processes without any added in the Pool');
+        }
+    }
+
+    /**
+     *
+     */
+    private function initRun()
+    {
+        $this->resetRun();
+    }
+
+    /**
+     * @return bool
+     */
+    private function isCli(): bool
+    {
+        return PHP_SAPI === 'cli';
+    }
+
+    /**
+     *
+     */
+    private function startRunStatus()
+    {
+        if (false === $this->isRunning) {
+            $this->notify(PoolEvent::POOL_RUN_START);
+            $this->isRunning = true;
+        }
+    }
+
+    /**
+     *
+     */
+    private function resetRun()
+    {
+        if (true === $this->isRunning) {
+            $this->notify(PoolEvent::POOL_RUN_STOP);
+        }
+        $this->isRunning = false;
+        $this->toRunThreads = $this->threads;
+        $this->runningThreads = [];
+    }
 
 }