fix(ignore-regexp-filter): bad string concatenation (refs #23)

src/Filter/FilterCollection.php

@@ -52,7 +52,7 @@ public function getDescription()
     {
         $parts = [];
         foreach ($this->filters as $filter) {
-            $parts[] = ' - '.$filter->getDescription();
+            $parts[] = '- '.$filter->getDescription();
         }
 
         return implode(PHP_EOL, $parts);

src/Filter/IgnoreRegexpFilter.php

@@ -28,7 +28,7 @@ public function __construct($ignoreRegexp)
      */
     public function getDescription()
     {
-        return 'project name should not match /'.$this->ignoreRegexp + '/';
+        return 'project name should not match /'.$this->ignoreRegexp.'/';
     }
 
     /**

tests/Filter/FilterCollectionTest.php

@@ -26,7 +26,7 @@ public function testEmpty()
      *
      * @return ProjectFilterInterface
      */
-    private function createMockFilter($accepted)
+    private function createMockFilter($accepted, $description = 'mock')
     {
         $filter = $this->getMockBuilder(ProjectFilterInterface::class)
             ->getMock()
@@ -35,6 +35,10 @@ private function createMockFilter($accepted)
             ->method('isAccepted')
             ->willReturn($accepted)
         ;
+        $filter->expects($this->any())
+            ->method('getDescription')
+            ->willReturn($description)
+        ;
 
         return $filter;
     }
@@ -61,10 +65,15 @@ public function testOneFalse()
     public function testTrueFalseTrue()
     {
         $filterCollection = new FilterCollection(new NullLogger());
-        $filterCollection->addFilter($this->createMockFilter(true));
-        $filterCollection->addFilter($this->createMockFilter(false));
-        $filterCollection->addFilter($this->createMockFilter(true));
+        $filterCollection->addFilter($this->createMockFilter(true, 'mock-1'));
+        $filterCollection->addFilter($this->createMockFilter(false, 'mock-2'));
+        $filterCollection->addFilter($this->createMockFilter(true, 'mock-3'));
         $project = $this->createMockProject('test');
         $this->assertFalse($filterCollection->isAccepted($project));
+
+        $this->assertEquals(
+            '- mock-1'.PHP_EOL.'- mock-2'.PHP_EOL.'- mock-3',
+            $filterCollection->getDescription()
+        );
     }
 }

tests/Filter/IgnoreRegexpFilterTest.php

@@ -14,6 +14,11 @@ public function testExample()
     {
         $filter = new IgnoreRegexpFilter('(^phpstorm|^typo3\/library)');
 
+        $this->assertEquals(
+            'project name should not match /(^phpstorm|^typo3\/library)/',
+            $filter->getDescription()
+        );
+
         $expectedResults = [
             'mborne/sample-project' => true,
             'something' => true,