cs and test fix

Author: Nyholm

Command/DeleteObsoleteCommand.php

@@ -11,7 +11,6 @@
 
 namespace Translation\Bundle\Command;
 
-use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Helper\ProgressBar;
 use Symfony\Component\Console\Input\InputArgument;
@@ -54,11 +53,10 @@ class DeleteObsoleteCommand extends Command
     private $catalogueFetcher;
 
     /**
-     *
-     * @param StorageManager $storageManager
+     * @param StorageManager       $storageManager
      * @param ConfigurationManager $configurationManager
-     * @param CatalogueManager $catalogueManager
-     * @param CatalogueFetcher $catalogueFetcher
+     * @param CatalogueManager     $catalogueManager
+     * @param CatalogueFetcher     $catalogueFetcher
      */
     public function __construct(
         StorageManager $storageManager,
@@ -73,7 +71,6 @@ public function __construct(
         parent::__construct();
     }
 
-
     protected function configure()
     {
         $this

Command/DownloadCommand.php

@@ -11,7 +11,6 @@
 
 namespace Translation\Bundle\Command;
 
-use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputOption;
@@ -21,7 +20,6 @@
 use Translation\Bundle\Service\CacheClearer;
 use Translation\Bundle\Service\ConfigurationManager;
 use Translation\Bundle\Service\StorageManager;
-use Translation\Bundle\Service\StorageService;
 use Translation\Bundle\Model\Configuration;
 
 /**
@@ -49,10 +47,9 @@ class DownloadCommand extends Command
     private $cacheCleaner;
 
     /**
-     *
-     * @param StorageManager $storageManager
+     * @param StorageManager       $storageManager
      * @param ConfigurationManager $configurationManager
-     * @param CacheClearer $cacheCleaner
+     * @param CacheClearer         $cacheCleaner
      */
     public function __construct(
         StorageManager $storageManager,
@@ -65,7 +62,6 @@ public function __construct(
         parent::__construct();
     }
 
-
     protected function configure()
     {
         $this

Command/ExtractCommand.php

@@ -11,7 +11,6 @@
 
 namespace Translation\Bundle\Command;
 
-use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
@@ -62,11 +61,10 @@ class ExtractCommand extends Command
     private $configurationManager;
 
     /**
-     *
-     * @param CatalogueFetcher $catalogueFetcher
-     * @param CatalogueWriter $catalogueWriter
-     * @param CatalogueCounter $catalogueCounter
-     * @param Importer $importer
+     * @param CatalogueFetcher     $catalogueFetcher
+     * @param CatalogueWriter      $catalogueWriter
+     * @param CatalogueCounter     $catalogueCounter
+     * @param Importer             $importer
      * @param ConfigurationManager $configurationManager
      */
     public function __construct(
@@ -85,7 +83,6 @@ public function __construct(
         parent::__construct();
     }
 
-
     protected function configure()
     {
         $this
@@ -99,7 +96,6 @@ protected function configure()
 
     protected function execute(InputInterface $input, OutputInterface $output)
     {
-
         $config = $this->configurationManager->getConfiguration($input->getArgument('configuration'));
 
         $locales = [];

Command/StatusCommand.php

@@ -11,7 +11,6 @@
 
 namespace Translation\Bundle\Command;
 
-use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
@@ -47,10 +46,9 @@ class StatusCommand extends Command
     private $catalogueFetcher;
 
     /**
-     *
-     * @param CatalogueCounter $catalogueCounter
+     * @param CatalogueCounter     $catalogueCounter
      * @param ConfigurationManager $configurationManager
-     * @param CatalogueFetcher $catalogueFetcher
+     * @param CatalogueFetcher     $catalogueFetcher
      */
     public function __construct(
         CatalogueCounter $catalogueCounter,
@@ -64,7 +62,6 @@ public function __construct(
         parent::__construct();
     }
 
-
     protected function configure()
     {
         $this

Command/SyncCommand.php

@@ -11,13 +11,11 @@
 
 namespace Translation\Bundle\Command;
 
-use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 use Translation\Bundle\Service\StorageManager;
-use Translation\Bundle\Service\StorageService;
 
 /**
  * @author Tobias Nyholm <tobias.nyholm@gmail.com>
@@ -32,7 +30,6 @@ class SyncCommand extends Command
     private $storageManager;
 
     /**
-     *
      * @param StorageManager $storageManager
      */
     public function __construct(StorageManager $storageManager)

Tests/Functional/Command/ExtractCommandTest.php

@@ -67,7 +67,8 @@ public function testExecute()
         $this->bootKernel();
         $application = new Application($this->kernel);
 
-        $application->add(new ExtractCommand());
+        $container = $this->getContainer();
+        $application->add($container->get('php_translator.console.extract'));
 
         $command = $application->find('translation:extract');
         $commandTester = new CommandTester($command);

Tests/Functional/Command/StatusCommandTest.php

@@ -29,7 +29,8 @@ public function testExecute()
         $this->bootKernel();
         $application = new Application($this->kernel);
 
-        $application->add(new StatusCommand());
+        $container = $this->getContainer();
+        $application->add($container->get('php_translator.console.status'));
 
         $command = $application->find('translation:status');
         $commandTester = new CommandTester($command);