bug #130 When a PR is open with "WIP", add status: needs work (Nyholm)

This PR was squashed before being merged into the master branch.

Discussion
----------

When a PR is open with "WIP", add status: needs work

Follow up from #123

Currently, an PR opened with "WIP:" in the title dont get any status.

Commits
-------

d006101 fixes
faf9bd7 When a PR is open with "WIP", add status: needs work

Author: weaverryan

src/Subscriber/NeedsReviewNewPRSubscriber.php

@@ -29,13 +29,12 @@ public function onPullRequest(GitHubEvent $event)
             return;
         }
 
+        $newStatus = Status::NEEDS_REVIEW;
         if (WipParser::matchTitle($data['pull_request']['title'])) {
-            return;
+            $newStatus = Status::NEEDS_WORK;
         }
 
         $pullRequestNumber = $data['pull_request']['number'];
-        $newStatus = Status::NEEDS_REVIEW;
-
         $this->statusApi->setIssueStatus($pullRequestNumber, $newStatus, $repository);
 
         $event->setResponseData([

tests/Subscriber/NeedsReviewNewPRSubscriberTest.php

@@ -56,9 +56,9 @@ public function testOnPullRequestOpen()
 
     public function testOnPullRequestOpenWIP()
     {
-        $this->statusApi->expects($this->never())
+        $this->statusApi->expects($this->once())
             ->method('setIssueStatus')
-            ->with(1234, Status::NEEDS_REVIEW);
+            ->with(1234, Status::NEEDS_WORK);
 
         $event = new GitHubEvent([
             'action' => 'opened',
@@ -68,7 +68,9 @@ public function testOnPullRequestOpenWIP()
         $this->dispatcher->dispatch($event, GitHubEvents::PULL_REQUEST);
 
         $responseData = $event->getResponseData();
-        $this->assertEmpty($responseData);
+        $this->assertCount(2, $responseData);
+        $this->assertSame(1234, $responseData['pull_request']);
+        $this->assertSame(Status::NEEDS_WORK, $responseData['status_change']);
     }
 
     public function testOnPullRequestNotOpen()