Update Tests

Author: vcorre

tests/Pheanstalk/PheanstalkTest.php

@@ -9,6 +9,7 @@
 use Pheanstalk\Structure\Schedule;
 use Pheanstalk\Structure\Task;
 use Pheanstalk\Structure\Job;
+use Pheanstalk\Structure\TaskInstance;
 use Pheanstalk\Structure\TimeSchedule;
 use Pheanstalk\Structure\Tube;
 use Pheanstalk\Structure\Workflow;
@@ -184,6 +185,28 @@ public function testKill()
         $this->killFirstRunningInstance($workflow);
     }
 
+    /**
+     * @expectedException \Pheanstalk\Exception\ServerException
+     */
+    public function testShouldFailWithNoPid()
+    {
+        $workflow = $this->pheanstalk->createTask('testSleep', 'testGroup', '/bin/sleep 10');
+        $this->pheanstalk->put($workflow);
+        $instances = $this->pheanstalk->getWorkflowInstances($workflow, GetWorkflowInstancesCommand::FILTER_EXECUTING);
+        $this->assertFalse($instances->isEmpty());
+        /** @var WorkflowInstance $workflowInstance */
+        $workflowInstance = $instances->filter(function(WorkflowInstance $instance) {
+            return $instance->getStatus() === 'RUNNING';
+        })->first();
+        /** @var JobInstance $jobInstance */
+        $jobInstance = $workflowInstance->getJobInstances()->first();
+        $taskInstance = $jobInstance->getTaskInstances()->first();
+        /** @var$taskInstance TaskInstance */
+        $taskInstance->setPid(0);
+
+        $this->pheanstalk->kill($workflowInstance, $taskInstance);
+    }
+
     protected function killFirstRunningInstance(Workflow $workflow)
     {
         try{

tests/Pheanstalk/Structure/ScheduleTest.php

@@ -0,0 +1,77 @@
+<?php
+
+namespace Pheanstalk\Structure;
+
+use Doctrine\Common\Collections\ArrayCollection;
+use PHPUnit\Framework\TestCase;
+
+class ScheduleTest extends TestCase
+{
+    /** @var Schedule $schedule */
+    protected $schedule;
+
+    protected function setUp()
+    {
+        parent::setUp(); // TODO: Change the autogenerated stub
+        $this->schedule = new Schedule(1, new TimeSchedule());
+    }
+
+    public function testId()
+    {
+        $this->assertNull($this->schedule->getId());
+        $this->schedule->setId(1);
+        $this->assertSame(1, $this->schedule->getId());
+    }
+
+    public function testActive()
+    {
+        $this->schedule->setActive(true);
+        $this->assertTrue($this->schedule->isActive());
+    }
+
+    public function testComment()
+    {
+        $this->assertNull($this->schedule->getComment());
+        $this->schedule->setComment('1');
+        $this->assertSame('1', $this->schedule->getComment());
+    }
+
+    public function testHost()
+    {
+        $this->assertNull($this->schedule->getHost());
+        $this->schedule->setHost('1');
+        $this->assertSame('1', $this->schedule->getHost());
+    }
+
+    public function testNode()
+    {
+        $this->schedule->setNode('1');
+        $this->assertSame('1', $this->schedule->getNode());
+    }
+
+    public function testOnFailure()
+    {
+        $this->schedule->setOnFailure('1');
+        $this->assertSame('1', $this->schedule->getOnFailure());
+    }
+
+    public function testSchedule()
+    {
+        $timeSchedule = new TimeSchedule();
+        $this->schedule->setSchedule($timeSchedule);
+        $this->assertSame($timeSchedule, $this->schedule->getSchedule());
+    }
+
+    public function testUser()
+    {
+        $this->assertNull($this->schedule->getUser());
+        $this->schedule->setUser('1');
+        $this->assertSame('1', $this->schedule->getUser());
+    }
+
+    public function testWorkflow()
+    {
+        $this->schedule->setWorkflow(1);
+        $this->assertSame(1, $this->schedule->getWorkflow());
+    }
+}