Add exclution

Add exclution

Fix README

Update Travis

Author: PedroTroller

.travis.yml

@@ -8,4 +8,4 @@ before_script:
 
 script:
     - bin/phpspec status
-    - bin/phpspec -fpretty
+    - bin/phpspec run -fpretty

README.md

@@ -23,3 +23,23 @@ extensions:
 ```bash
 ./bin/phpspec status
 ```
+
+**Add exclusion (via phpspec.yml)**
+
+```yml
+#phpspec.yml
+knp.welldone.exclusion:
+    - "*Controller"
+    - "App\Entity\*"
+
+extensions:
+    - Knp\PhpSpec\WellDone\Extension
+```
+
+**Add exclusion (via command)**
+
+```bash
+./bin/phpspec status -e "*Controller, App\Entity\*"
+```
+
+With command, yml parameter will be overwrite.

phpspec.yml

@@ -1,2 +1,3 @@
+knp.welldone.exclusion: ~
 extensions:
     - Knp\PhpSpec\WellDone\Extension

spec/Knp/PhpSpec/WellDone/Locator/ResourceManagerSpec.php

@@ -0,0 +1,14 @@
+<?php
+
+namespace spec\Knp\PhpSpec\WellDone\Locator;
+
+use PhpSpec\ObjectBehavior;
+use Prophecy\Argument;
+
+class ResourceManagerSpec extends ObjectBehavior
+{
+    function it_is_initializable()
+    {
+        $this->shouldHaveType('Knp\PhpSpec\WellDone\Locator\ResourceManager');
+    }
+}

src/Knp/PhpSpec/WellDone/Console/Command/StatusCommand.php

@@ -7,6 +7,7 @@
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 use Knp\PhpSpec\WellDone\Formater\ProgressFormater;
+use Symfony\Component\Console\Input\InputOption;
 
 class StatusCommand extends Command
 {
@@ -21,12 +22,32 @@ public function __construct(Filesystem $filesystem, ProgressFormater $formater)
         $this->formater   = $formater;
     }
 
+    public function configure()
+    {
+        $this
+            ->setName('status')
+            ->setDescription('Say which class has spec or not.')
+            ->setDefinition(array(
+                new InputOption('exclude', 'e', InputOption::VALUE_REQUIRED, 'File exclusion pattern (ex : "*Controller, App\Entity\*")', null)
+            ))
+        ;
+    }
+
     protected function execute(InputInterface $input, OutputInterface $output)
     {
         $container = $this->getApplication()->getContainer();
         $container->configure();
 
-        $resources = $container->get('locator.resource_manager')->locateResources('');
+        $exclusion = $container->getParam('knp.welldone.exclusion', '');
+        if (null !== $input->getOption('exclude')) {
+            $exclusion = $input->getOption('exclude');
+        }
+
+        if (is_array($exclusion)) {
+            $exclusion = implode(', ', $exclusion);
+        }
+
+        $resources = $container->get('locator.resource_manager')->locateResourcesWithExclusion($exclusion);
 
         foreach ($this->buildMessages($resources) as $message) {
             $output->writeln($message);

src/Knp/PhpSpec/WellDone/Extension.php

@@ -5,6 +5,7 @@
 use PhpSpec\Extension\ExtensionInterface;
 use PhpSpec\ServiceContainer;
 use Symfony\Component\Console\Input\InputArgument;
+use Knp\PhpSpec\WellDone\Locator\ResourceManager;
 use Knp\PhpSpec\WellDone\Locator\NoSpecLocator;
 use Knp\PhpSpec\WellDone\Locator\ResourceInspector;
 use Knp\PhpSpec\WellDone\Util\Filesystem;
@@ -41,6 +42,17 @@ protected function setupCommands(ServiceContainer $container)
 
     protected function setupLocators(ServiceContainer $container)
     {
+        $container->setShared('locator.resource_manager', function ($c) {
+            $manager = new ResourceManager();
+
+            array_map(
+                array($manager, 'registerLocator'),
+                $c->getByPrefix('locator.locators')
+            );
+
+            return $manager;
+        });
+
         $container->addConfigurator(function ($c) {
             $suites = $c->getParam('suites', array('main' => ''));
 

src/Knp/PhpSpec/WellDone/Formater/ProgressFormater.php

@@ -19,7 +19,7 @@ public function buildProgressBar(array $resources)
         $notDone = $this->filter($resources, false);
 
         $length = count($done) + count($notDone);
-        $state = (count($done) * $this->max) / $length;
+        $state = $length > 0 ? (count($done) * $this->max) / $length : $this->max;
 
         $progress = sprintf(
             '<%s>%s</%s>',

src/Knp/PhpSpec/WellDone/Locator/ExclusionLocatorInterface.php

@@ -0,0 +1,9 @@
+<?php
+
+namespace Knp\PhpSpec\WellDone\Locator;
+
+interface ExclusionLocatorInterface
+{
+    public function findResourcesWithExclusion($query);
+    public function supportsExclusionQuery($query);
+}

src/Knp/PhpSpec/WellDone/Locator/NoSpecLocator.php

@@ -6,7 +6,7 @@
 use PhpSpec\Locator\PSR0\PSR0Resource;
 use Knp\PhpSpec\WellDone\Locator\ResourceInspector;
 
-class NoSpecLocator extends PSR0Locator
+class NoSpecLocator extends PSR0Locator implements ExclusionLocatorInterface
 {
     public function __construct(ResourceInspector $inspector, $srcNamespace = '', $specNamespacePrefix = 'spec', $srcPath = 'src', $specPath = '.')
     {
@@ -20,7 +20,18 @@ public function getAllResources()
         return $this->findNotSpecResources($this->getFullSrcPath());
     }
 
-    protected function findNotSpecResources($path)
+    public function findResourcesWithExclusion($query)
+    {
+        return $this->findNotSpecResources($this->getFullSrcPath(), $query);
+    }
+
+    public function supportsExclusionQuery($query)
+    {
+        $query = preg_replace('/[A-Za-z_\\*]/', '', $query);
+        return empty($query);
+    }
+
+    protected function findNotSpecResources($path, $query = null)
     {
         if (!$this->getFilesystem()->pathExists($path)) {
             return array();
@@ -32,7 +43,9 @@ protected function findNotSpecResources($path)
             if ($this->inspector->isClass($resource)
                 && !$this->inspector->isAbstract($resource)
                 && !$this->inspector->hasSpec($resource)
+                && !$this->inspector->matchQueries($resource, $query)
             ) {
+
                 $resources[] = $resource;
             }
         }

src/Knp/PhpSpec/WellDone/Locator/ResourceInspector.php

@@ -34,6 +34,28 @@ public function isAbstract(PSR0Resource $resource)
         return $this->is($resource, T_ABSTRACT);
     }
 
+    public function matchQueries(PSR0Resource $resource, $queries, $delim = ',')
+    {
+        foreach (explode($delim, $queries) as $query) {
+            if (true === $this->matchQuery($resource, trim($query))) {
+
+                return true;
+            }
+        }
+
+        return false;
+    }
+
+    public function matchQuery(PSR0Resource $resource, $query)
+    {
+        $query = str_replace('/', '\\', $query);
+        $query = str_replace('\\', '\\\\', $query);
+        $query = str_replace('*', '(.*)', $query);
+        $query = sprintf('/^%s$/', $query);
+
+        return 1 == preg_match($query, $resource->getSrcClassname());
+    }
+
     protected function is(PSR0Resource $resource, $tag)
     {
         $tokens = $this->filesystem->getTokens($resource->getSrcFilename());

src/Knp/PhpSpec/WellDone/Locator/ResourceManager.php

@@ -0,0 +1,70 @@
+<?php
+
+namespace Knp\PhpSpec\WellDone\Locator;
+
+use PhpSpec\Locator\ResourceManager as BaseManager;
+use PhpSpec\Locator\ResourceLocatorInterface;
+
+class ResourceManager extends BaseManager
+{
+    protected $locators = array();
+
+    public function registerLocator(ResourceLocatorInterface $locator)
+    {
+        $this->locators[] = $locator;
+
+        @usort($this->locators, function ($locator1, $locator2) {
+            return $locator2->getPriority() - $locator1->getPriority();
+        });
+    }
+
+    public function locateResources($query)
+    {
+        $resources = array();
+        foreach ($this->locators as $locator) {
+            if (empty($query)) {
+                $resources = array_merge($resources, $locator->getAllResources());
+                continue;
+            }
+
+            if (!$locator->supportsQuery($query)) {
+                continue;
+            }
+
+            $resources = array_merge($resources, $locator->findResources($query));
+        }
+
+        return array_values($resources);
+    }
+
+    public function locateResourcesWithExclusion($query)
+    {
+        $resources = array();
+        foreach ($this->locators as $locator) {
+            if ($locator instanceof ExclusionLocatorInterface) {
+                if (!$locator->supportsExclusionQuery($query)) {
+                    continue;
+                }
+
+                $resources = array_merge($resources, $locator->findResourcesWithExclusion($query));
+            } else {
+                $resources = array_merge($resources, $locator->getAllResources());
+            }
+        }
+
+        return array_values($resources);
+    }
+
+    public function createResource($classname)
+    {
+        foreach ($this->locators as $locator) {
+            if ($locator->supportsClass($classname)) {
+                return $locator->createResource($classname);
+            }
+        }
+
+        throw new RuntimeException(sprintf(
+            'Can not find appropriate suite scope for class `%s`.', $classname
+        ));
+    }
+}