Better trim and add a test

Author: Nyholm

src/Subscriber/AutoUpdateTitleWithLabelSubscriber.php

@@ -61,7 +61,7 @@ public function onPullRequest(GitHubEvent $event)
         }
 
         // Add back labels
-        $prTitle = trim($prPrefix.' '.$prTitle);
+        $prTitle = trim($prPrefix.' '.trim($prTitle));
         if ($originalTitle === $prTitle) {
             return;
         }

tests/Subscriber/AutoUpdateTitleWithLabelSubscriberTest.php

@@ -42,7 +42,7 @@ protected function setUp()
         $this->dispatcher->addSubscriber($this->subscriber);
     }
 
-    public function testOnPullRequestOpen()
+    public function testOnPullRequestLabeled()
     {
         $event = new GitHubEvent([
             'action' => 'labeled',
@@ -63,4 +63,23 @@ public function testOnPullRequestOpen()
         $this->assertSame(1234, $responseData['pull_request']);
         $this->assertSame('[Console][FrameworkBundle] [bar] Foo', $responseData['new_title']);
     }
+
+    public function testOnPullRequestLabeledWithExisting()
+    {
+        $event = new GitHubEvent([
+            'action' => 'labeled',
+            'number' => 1234,
+            'pull_request' => [
+                'title' => '[Messenger] Fix JSON',
+                'labels' => [
+                    ['name' => 'Status: Needs Review', 'color' => 'abcabc'],
+                    ['name' => 'Messenger', 'color' => 'dddddd'],
+                ],
+            ],
+        ], $this->repository);
+
+        $this->dispatcher->dispatch($event, GitHubEvents::PULL_REQUEST);
+        $responseData = $event->getResponseData();
+        $this->assertEmpty($responseData);
+    }
 }