KodingKittens / ChuckConsoleBundle

Command/ChuckCommand.php

Indentation +20 added lines, -20 removed lines

@@ -9,26 +9,26 @@
 
 class ChuckCommand extends ContainerAwareCommand
 {
-    protected function configure()
-    {
-        $this
-            ->setName('chuck:fact')
-            ->setDescription('A Chuck Norris Fact everyday keeps the doctor away')
-            ->addArgument('firstName', InputArgument::OPTIONAL, 'Replace Chuck by this string', 'Chuck')
-            ->addArgument('lastName', InputArgument::OPTIONAL, 'Replace Norris by this string', 'Norris')
-        ;
-    }
+	protected function configure()
+	{
+		$this
+			->setName('chuck:fact')
+			->setDescription('A Chuck Norris Fact everyday keeps the doctor away')
+			->addArgument('firstName', InputArgument::OPTIONAL, 'Replace Chuck by this string', 'Chuck')
+			->addArgument('lastName', InputArgument::OPTIONAL, 'Replace Norris by this string', 'Norris')
+		;
+	}
 
-    protected function execute(InputInterface $input, OutputInterface $output)
-    {
-        $fact = $this->getContainer()->get('kk_labs_chuck_console.fact_service')
-            ->setFirstName($input->getArgument('firstName'))
-            ->setLastName($input->getArgument('lastName'))
-            ->getFact()
-        ;
+	protected function execute(InputInterface $input, OutputInterface $output)
+	{
+		$fact = $this->getContainer()->get('kk_labs_chuck_console.fact_service')
+			->setFirstName($input->getArgument('firstName'))
+			->setLastName($input->getArgument('lastName'))
+			->getFact()
+		;
 
-        if ($fact) {
-            $output->writeln($fact);
-        }
-    }
+		if ($fact) {
+			$output->writeln($fact);
+		}
+	}
 }

Tests/ChuckFactService/ChuckFactServiceTest.php

Indentation +53 added lines, -53 removed lines

@@ -8,68 +8,68 @@
 
 class ChuckFactServiceTest extends \PHPUnit_Framework_TestCase
 {
-    /**
-     * @var ContainerInterface
-     */
-    private $container;
+	/**
+	 * @var ContainerInterface
+	 */
+	private $container;
 
-    protected function setUp()
-    {
-        $kernel = new AppKernel('test', true);
-        $kernel->boot();
+	protected function setUp()
+	{
+		$kernel = new AppKernel('test', true);
+		$kernel->boot();
 
-        $this->randomFirstName = substr(str_shuffle(MD5(microtime())), 0, 10);
-        $this->randomLastName = substr(str_shuffle(MD5(microtime())), 0, 10);
+		$this->randomFirstName = substr(str_shuffle(MD5(microtime())), 0, 10);
+		$this->randomLastName = substr(str_shuffle(MD5(microtime())), 0, 10);
 
-        $this->container = $kernel->getContainer();
-    }
+		$this->container = $kernel->getContainer();
+	}
 
-    public function testServiceIsDefinedInContainer()
-    {
-        $service = $this->container->get('kk_labs_chuck_console.fact_service');
+	public function testServiceIsDefinedInContainer()
+	{
+		$service = $this->container->get('kk_labs_chuck_console.fact_service');
 
-        $this->assertInstanceOf('KK\Labs\ChuckConsoleBundle\ChuckFactService\ChuckAPIService', $service);
-    }
+		$this->assertInstanceOf('KK\Labs\ChuckConsoleBundle\ChuckFactService\ChuckAPIService', $service);
+	}
 
-    public function testGetCustomFact()
-    {
-        $this->assertRegExp(
-            "/$this->randomFirstName|$this->randomLastName/",
-            $this->getFacts($this->randomFirstName, $this->randomLastName)
-        );
-    }
+	public function testGetCustomFact()
+	{
+		$this->assertRegExp(
+			"/$this->randomFirstName|$this->randomLastName/",
+			$this->getFacts($this->randomFirstName, $this->randomLastName)
+		);
+	}
 
-    public function testGetChuckFact()
-    {
-        $this->assertRegExp('/Chuck|Norris/', $this->getFacts());
-    }
+	public function testGetChuckFact()
+	{
+		$this->assertRegExp('/Chuck|Norris/', $this->getFacts());
+	}
 
-    /**
-     * @param string $firstName
-     * @param string $lastName
-     *
-     * @return string
-     */
-    private function getFacts($firstName = 'Chuck', $lastName = 'Norris')
-    {
-        $chuckService = $this->getChuckService($firstName, $lastName);
+	/**
+	 * @param string $firstName
+	 * @param string $lastName
+	 *
+	 * @return string
+	 */
+	private function getFacts($firstName = 'Chuck', $lastName = 'Norris')
+	{
+		$chuckService = $this->getChuckService($firstName, $lastName);
 
-        $facts = '';
-        for ($i = 0; $i < 5; ++$i) {
-            $facts .= $chuckService->getFact();
-        }
+		$facts = '';
+		for ($i = 0; $i < 5; ++$i) {
+			$facts .= $chuckService->getFact();
+		}
 
-        return $facts;
-    }
+		return $facts;
+	}
 
-    /**
-     * @param string $firstName
-     * @param string $lastName
-     *
-     * @return ChuckAPIService
-     */
-    private function getChuckService($firstName = 'Chuck', $lastName = 'Norris')
-    {
-        return new ChuckAPIService($firstName, $lastName, 5);
-    }
+	/**
+	 * @param string $firstName
+	 * @param string $lastName
+	 *
+	 * @return ChuckAPIService
+	 */
+	private function getChuckService($firstName = 'Chuck', $lastName = 'Norris')
+	{
+		return new ChuckAPIService($firstName, $lastName, 5);
+	}
 }

Tests/Command/ChuckCommandTest.php

Indentation +31 added lines, -31 removed lines

@@ -9,35 +9,35 @@
 
 class ChuckCommandTest extends \PHPUnit_Framework_TestCase
 {
-    public function testExecute()
-    {
-        $firstName = substr(str_shuffle(MD5(microtime())), 0, 10);
-        $lastName = substr(str_shuffle(MD5(microtime())), 0, 10);
-
-        $application = new Application();
-        $application->add(new ChuckCommand());
-
-        $command = $application->find('chuck:fact');
-        $command->setContainer($this->getMockContainer());
-        $commandTester = new CommandTester($command);
-        $commandTester->execute(array(
-            'command' => $command->getName(),
-            'firstName' => $firstName,
-            'lastName' => $lastName,
-        ));
-
-        $this->assertRegExp("/$firstName|$lastName/", $commandTester->getDisplay());
-    }
-
-    protected function getMockContainer()
-    {
-        $mockContainer = $this->getMock('Symfony\Component\DependencyInjection\Container');
-        $mockContainer
-            ->expects($this->once())
-            ->method('get')
-            ->with('kk_labs_chuck_console.fact_service')
-            ->willReturn(new ChuckAPIService());
-
-        return $mockContainer;
-    }
+	public function testExecute()
+	{
+		$firstName = substr(str_shuffle(MD5(microtime())), 0, 10);
+		$lastName = substr(str_shuffle(MD5(microtime())), 0, 10);
+
+		$application = new Application();
+		$application->add(new ChuckCommand());
+
+		$command = $application->find('chuck:fact');
+		$command->setContainer($this->getMockContainer());
+		$commandTester = new CommandTester($command);
+		$commandTester->execute(array(
+			'command' => $command->getName(),
+			'firstName' => $firstName,
+			'lastName' => $lastName,
+		));
+
+		$this->assertRegExp("/$firstName|$lastName/", $commandTester->getDisplay());
+	}
+
+	protected function getMockContainer()
+	{
+		$mockContainer = $this->getMock('Symfony\Component\DependencyInjection\Container');
+		$mockContainer
+			->expects($this->once())
+			->method('get')
+			->with('kk_labs_chuck_console.fact_service')
+			->willReturn(new ChuckAPIService());
+
+		return $mockContainer;
+	}
 }

Tests/Fixtures/app/AppKernel.php

Indentation +37 added lines, -37 removed lines

@@ -7,41 +7,41 @@
 
 class AppKernel extends Kernel
 {
-    public function registerBundles()
-    {
-        return array(
-            new \KK\Labs\ChuckConsoleBundle\KKLabsChuckConsoleBundle(),
-        );
-    }
-
-    public function registerContainerConfiguration(LoaderInterface $loader)
-    {
-        $loader->load(__DIR__.'/config/config_test.yml');
-    }
-
-    /**
-     * @return string
-     */
-    public function getCacheDir()
-    {
-        $cacheDir = sys_get_temp_dir().'/cache';
-        if (!is_dir($cacheDir)) {
-            mkdir($cacheDir, 0777, true);
-        }
-
-        return $cacheDir;
-    }
-
-    /**
-     * @return string
-     */
-    public function getLogDir()
-    {
-        $logDir = sys_get_temp_dir().'/logs';
-        if (!is_dir($logDir)) {
-            mkdir($logDir, 0777, true);
-        }
-
-        return $logDir;
-    }
+	public function registerBundles()
+	{
+		return array(
+			new \KK\Labs\ChuckConsoleBundle\KKLabsChuckConsoleBundle(),
+		);
+	}
+
+	public function registerContainerConfiguration(LoaderInterface $loader)
+	{
+		$loader->load(__DIR__.'/config/config_test.yml');
+	}
+
+	/**
+	 * @return string
+	 */
+	public function getCacheDir()
+	{
+		$cacheDir = sys_get_temp_dir().'/cache';
+		if (!is_dir($cacheDir)) {
+			mkdir($cacheDir, 0777, true);
+		}
+
+		return $cacheDir;
+	}
+
+	/**
+	 * @return string
+	 */
+	public function getLogDir()
+	{
+		$logDir = sys_get_temp_dir().'/logs';
+		if (!is_dir($logDir)) {
+			mkdir($logDir, 0777, true);
+		}
+
+		return $logDir;
+	}
 }

Tests/bootstrap.php

Indentation +1 added lines, -1 removed lines

@@ -1,7 +1,7 @@
 <?php
 
 if (!is_file($autoloadFile = __DIR__.'/../vendor/autoload.php')) {
-    throw new \LogicException('Could not find autoload.php in vendor/. Did you run "composer install --dev"?');
+	throw new \LogicException('Could not find autoload.php in vendor/. Did you run "composer install --dev"?');
 }
 
 require $autoloadFile;

DependencyInjection/KKLabsChuckConsoleExtension.php

Indentation +14 added lines, -14 removed lines

@@ -14,20 +14,20 @@
  */
 class KKLabsChuckConsoleExtension extends Extension
 {
-    /**
-     * {@inheritdoc}
-     */
-    public function load(array $configs, ContainerBuilder $container)
-    {
-        $configuration = new Configuration();
-        $config = $this->processConfiguration($configuration, $configs);
+	/**
+	 * {@inheritdoc}
+	 */
+	public function load(array $configs, ContainerBuilder $container)
+	{
+		$configuration = new Configuration();
+		$config = $this->processConfiguration($configuration, $configs);
 
-        $loader = new Loader\XmlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
-        $loader->load('services.xml');
+		$loader = new Loader\XmlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
+		$loader->load('services.xml');
 
-        $container->setParameter('chuck.who.first_name', $config['who']['first_name']);
-        $container->setParameter('chuck.who.last_name', $config['who']['last_name']);
-        $container->setParameter('chuck.timeout', $config['timeout']);
-        $container->setParameter('chuck.enabled_environments', $config['environments']);
-    }
+		$container->setParameter('chuck.who.first_name', $config['who']['first_name']);
+		$container->setParameter('chuck.who.last_name', $config['who']['last_name']);
+		$container->setParameter('chuck.timeout', $config['timeout']);
+		$container->setParameter('chuck.enabled_environments', $config['environments']);
+	}
 }

DependencyInjection/Configuration.php

Indentation +25 added lines, -25 removed lines

@@ -12,31 +12,31 @@
  */
 class Configuration implements ConfigurationInterface
 {
-    /**
-     * {@inheritdoc}
-     */
-    public function getConfigTreeBuilder()
-    {
-        $treeBuilder = new TreeBuilder();
-        $rootNode = $treeBuilder->root('kk_labs_chuck_console');
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getConfigTreeBuilder()
+	{
+		$treeBuilder = new TreeBuilder();
+		$rootNode = $treeBuilder->root('kk_labs_chuck_console');
 
-        $rootNode
-            ->children()
-                ->arrayNode('who')
-                    ->addDefaultsIfNotSet()
-                        ->children()
-                            ->scalarNode('first_name')->defaultValue('Chuck')->end()
-                            ->scalarNode('last_name')->defaultValue('Norris')->end()
-                        ->end()
-                    ->end()//who
-                ->scalarNode('timeout')
-                    ->defaultValue(10)
-                ->end()
-                ->arrayNode('environments')
-                    ->prototype('scalar')->end()
-                    ->defaultValue(['dev'])
-            ->end();
+		$rootNode
+			->children()
+				->arrayNode('who')
+					->addDefaultsIfNotSet()
+						->children()
+							->scalarNode('first_name')->defaultValue('Chuck')->end()
+							->scalarNode('last_name')->defaultValue('Norris')->end()
+						->end()
+					->end()//who
+				->scalarNode('timeout')
+					->defaultValue(10)
+				->end()
+				->arrayNode('environments')
+					->prototype('scalar')->end()
+					->defaultValue(['dev'])
+			->end();
 
-        return $treeBuilder;
-    }
+		return $treeBuilder;
+	}
 }

ChuckFactService/ChuckAPIService.php

Indentation +117 added lines, -117 removed lines

@@ -7,121 +7,121 @@
 
 class ChuckAPIService
 {
-    /**
-     * If user has not its own parameters set, value will be 'Chuck'.
-     *
-     * @var string
-     */
-    private $firstName;
-
-    /**
-     * If user has not its own parameters set, value will be 'Norris'.
-     *
-     * @var string
-     */
-    private $lastName;
-
-    /**
-     * After n seconds, cancel.
-     *
-     * @var int
-     */
-    private $timeout;
-
-    /**
-     * @param string $firstName
-     * @param string $lastName
-     * @param int    $timeout
-     */
-    public function __construct($firstName = 'Chuck', $lastName = 'Norris', $timeout = 2)
-    {
-        $this
-            ->setFirstName($firstName)
-            ->setLastName($lastName)
-            ->setTimeout($timeout)
-        ;
-    }
-
-    /**
-     * @param string $firstName
-     *
-     * @return $this
-     */
-    public function setFirstName($firstName)
-    {
-        $this->firstName = $firstName;
-
-        return $this;
-    }
-
-    /**
-     * @param string $lastName
-     *
-     * @return $this
-     */
-    public function setLastName($lastName)
-    {
-        $this->lastName = $lastName;
-
-        return $this;
-    }
-
-    /**
-     * @param int $timeout
-     *
-     * @return $this
-     */
-    public function setTimeout($timeout)
-    {
-        $this->timeout = $timeout;
-
-        return $this;
-    }
-
-    /**
-     * Get fact from Internet Chuck Norris Database.
-     *
-     * @return string|false
-     */
-    public function getFact()
-    {
-        try {
-            $response = $this->getResponse();
-            if ($response->getStatusCode() == 200) {
-                $datas = json_decode($response->getBody());
-
-                return html_entity_decode(stripslashes($datas->value->joke));
-            }
-        } catch (ConnectException $e) {
-            return false;
-        }
-    }
-
-    /**
-     * @return \Psr\Http\Message\ResponseInterface
-     */
-    private function getResponse()
-    {
-        $client = $this->getClient();
-
-	    $response = $client->get('random', [
-		    'query' => [
-			    'firstName' => $this->firstName,
-			    'lastName' => $this->lastName,
-		    ],
-	    ]);
-
-        return $response;
-    }
-
-    /**
-     * @return Client
-     */
-    private function getClient()
-    {
-        return new Client([
-            'base_uri' => 'http://api.icndb.com/jokes/',
-            'timeout' => $this->timeout,
-        ]);
-    }
+	/**
+	 * If user has not its own parameters set, value will be 'Chuck'.
+	 *
+	 * @var string
+	 */
+	private $firstName;
+
+	/**
+	 * If user has not its own parameters set, value will be 'Norris'.
+	 *
+	 * @var string
+	 */
+	private $lastName;
+
+	/**
+	 * After n seconds, cancel.
+	 *
+	 * @var int
+	 */
+	private $timeout;
+
+	/**
+	 * @param string $firstName
+	 * @param string $lastName
+	 * @param int    $timeout
+	 */
+	public function __construct($firstName = 'Chuck', $lastName = 'Norris', $timeout = 2)
+	{
+		$this
+			->setFirstName($firstName)
+			->setLastName($lastName)
+			->setTimeout($timeout)
+		;
+	}
+
+	/**
+	 * @param string $firstName
+	 *
+	 * @return $this
+	 */
+	public function setFirstName($firstName)
+	{
+		$this->firstName = $firstName;
+
+		return $this;
+	}
+
+	/**
+	 * @param string $lastName
+	 *
+	 * @return $this
+	 */
+	public function setLastName($lastName)
+	{
+		$this->lastName = $lastName;
+
+		return $this;
+	}
+
+	/**
+	 * @param int $timeout
+	 *
+	 * @return $this
+	 */
+	public function setTimeout($timeout)
+	{
+		$this->timeout = $timeout;
+
+		return $this;
+	}
+
+	/**
+	 * Get fact from Internet Chuck Norris Database.
+	 *
+	 * @return string|false
+	 */
+	public function getFact()
+	{
+		try {
+			$response = $this->getResponse();
+			if ($response->getStatusCode() == 200) {
+				$datas = json_decode($response->getBody());
+
+				return html_entity_decode(stripslashes($datas->value->joke));
+			}
+		} catch (ConnectException $e) {
+			return false;
+		}
+	}
+
+	/**
+	 * @return \Psr\Http\Message\ResponseInterface
+	 */
+	private function getResponse()
+	{
+		$client = $this->getClient();
+
+		$response = $client->get('random', [
+			'query' => [
+				'firstName' => $this->firstName,
+				'lastName' => $this->lastName,
+			],
+		]);
+
+		return $response;
+	}
+
+	/**
+	 * @return Client
+	 */
+	private function getClient()
+	{
+		return new Client([
+			'base_uri' => 'http://api.icndb.com/jokes/',
+			'timeout' => $this->timeout,
+		]);
+	}
 }

EventListener/ConsoleTerminateListener.php

Indentation +40 added lines, -40 removed lines

@@ -13,44 +13,44 @@
  */
 class ConsoleTerminateListener
 {
-    /**
-     * @var ChuckAPIService
-     */
-    private $chuckApiService;
-
-    /**
-     * @var array
-     */
-    private $enabledEnvironments;
-
-    /**
-     * @var string
-     */
-    private $env;
-
-    /**
-     * ConsoleTerminateListener constructor.
-     * @param ChuckAPIService $chuckAPIService
-     */
-    public function __construct(ChuckAPIService $chuckAPIService, $environments = [], $env)
-    {
-        $this->chuckApiService = $chuckAPIService;
-        $this->enabledEnvironments = $environments;
-        $this->env = $env;
-    }
-
-    /**
-     * @param ConsoleTerminateEvent $event
-     */
-    public function onConsoleTerminate(ConsoleTerminateEvent $event)
-    {
-        if (
-            !$event->getCommand() instanceof ChuckCommand
-            && in_array($this->env, $this->enabledEnvironments)
-            && $fact = $this->chuckApiService->getFact()
-        ) {
-
-            $event->getOutput()->writeln($fact);
-        }
-    }
+	/**
+	 * @var ChuckAPIService
+	 */
+	private $chuckApiService;
+
+	/**
+	 * @var array
+	 */
+	private $enabledEnvironments;
+
+	/**
+	 * @var string
+	 */
+	private $env;
+
+	/**
+	 * ConsoleTerminateListener constructor.
+	 * @param ChuckAPIService $chuckAPIService
+	 */
+	public function __construct(ChuckAPIService $chuckAPIService, $environments = [], $env)
+	{
+		$this->chuckApiService = $chuckAPIService;
+		$this->enabledEnvironments = $environments;
+		$this->env = $env;
+	}
+
+	/**
+	 * @param ConsoleTerminateEvent $event
+	 */
+	public function onConsoleTerminate(ConsoleTerminateEvent $event)
+	{
+		if (
+			!$event->getCommand() instanceof ChuckCommand
+			&& in_array($this->env, $this->enabledEnvironments)
+			&& $fact = $this->chuckApiService->getFact()
+		) {
+
+			$event->getOutput()->writeln($fact);
+		}
+	}
 }