hogosha / monitor

src/Configuration/ConfigurationDumper.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
     {
         $content = '';
         foreach ($config as $name => $section) {
-            $content .= Yaml::dump([$name => $section], 4).PHP_EOL;
+            $content .= Yaml::dump([$name => $section], 4) . PHP_EOL;
         }
 
         return $content;

src/Configuration/ConfigurationLoader.php

Spacing +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
      */
     public function getConfigurationFilepath()
     {
-        return $this->configFilepath ?: $this->rootDirectory.DIRECTORY_SEPARATOR.$this->filename;
+        return $this->configFilepath ?: $this->rootDirectory . DIRECTORY_SEPARATOR . $this->filename;
     }
 
     /**

src/Console/MonitorApplicationConfig.php

Spacing +3 added lines, -3 removed lines

@@ -38,20 +38,20 @@
 SIGNATURE;
 
         $this
-            ->setDisplayName($signature."\n".'Hogosha Monitoring Tool')
+            ->setDisplayName($signature . "\n" . 'Hogosha Monitoring Tool')
             ->setName('monitor')
             ->setVersion(Monitor::VERSION)
                 ->addOption('config', 'c', Option::OPTIONAL_VALUE, 'Config file path', rtrim(getcwd(), DIRECTORY_SEPARATOR))
                 ->beginCommand('init')
                     ->setDescription('Create a default configuration file if you do not have one')
-                    ->setHandler(function () use ($configurationLoader, $configurationDumper, $filesystem) {
+                    ->setHandler(function() use ($configurationLoader, $configurationDumper, $filesystem) {
                         return new InitHandler($configurationLoader, $configurationDumper, $filesystem);
                     })
                     ->setHelp('php <info>bin/monitor</info> init')
                 ->end()
                 ->beginCommand('run')
                     ->setDescription('Launch the monitor process')
-                    ->setHandler(function () use ($configurationLoader) {
+                    ->setHandler(function() use ($configurationLoader) {
                         return new RunHandler($configurationLoader);
                     })
                     ->setHelp('php <info>bin/monitor</info> init')

src/Renderer/ListRenderer.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
                     $result->getStatusCode(),
                     $result->getName(),
                     $result->getReponseTime()
-                )."\n"
+                ) . "\n"
             );
         }
     }

tests/Handler/InitHandlerTest.php

Spacing +3 added lines, -3 removed lines

@@ -24,7 +24,7 @@ 
      */
     public function testExistingConfigurationFile()
     {
-        $file = sys_get_temp_dir().DIRECTORY_SEPARATOR.Monitor::CONFIG_FILENAME;
+        $file = sys_get_temp_dir() . DIRECTORY_SEPARATOR . Monitor::CONFIG_FILENAME;
 
         // Dump the configuration set by the enduser
         $configuration = [
@@ -41,7 +41,7 @@ 
 
         $content = '';
         foreach ($configuration as $name => $section) {
-            $content .= Yaml::dump([$name => $section], 4).PHP_EOL;
+            $content .= Yaml::dump([$name => $section], 4) . PHP_EOL;
         }
 
         file_put_contents($file, $content);
@@ -108,7 +108,7 @@ 
      */
     public function testDefaultConfigurationFile()
     {
-        $testFile = sys_get_temp_dir().DIRECTORY_SEPARATOR.Monitor::CONFIG_FILENAME;
+        $testFile = sys_get_temp_dir() . DIRECTORY_SEPARATOR . Monitor::CONFIG_FILENAME;
 
         $configurationLoader = new ConfigurationLoader();
         $configurationDumper = new ConfigurationDumper();

src/bootstrap.php

Spacing +4 added lines, -4 removed lines

@@ -18,10 +18,10 @@
     return file_exists($file) ? include $file : false;
 }
 
-if ((!$loader = includeIfExists(__DIR__.'/../vendor/autoload.php')) && (!$loader = includeIfExists(__DIR__.'/../../../autoload.php'))) {
-    echo 'You must set up the project dependencies, run the following commands:'.PHP_EOL.
-        'curl -sS https://getcomposer.org/installer | php'.PHP_EOL.
-        'php composer.phar install'.PHP_EOL;
+if ((!$loader = includeIfExists(__DIR__ . '/../vendor/autoload.php')) && (!$loader = includeIfExists(__DIR__ . '/../../../autoload.php'))) {
+    echo 'You must set up the project dependencies, run the following commands:' . PHP_EOL .
+        'curl -sS https://getcomposer.org/installer | php' . PHP_EOL .
+        'php composer.phar install' . PHP_EOL;
     exit(1);
 }
 

tests/Guesser/StatusGuesserTest.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Hogosha\Monitor\Guesser\StatusGuesser;
 use Hogosha\Monitor\Model\Result;
-use Hogosha\Monitor\Model\ResultCollection;
 use Hogosha\Monitor\Model\UrlInfo;
 
 /**

src/Runner/Runner.php

Doc Comments +2 added lines, -2 removed lines

@@ -39,7 +39,7 @@ 
      * Constructor.
      *
      * @param UrlProvider  $urlProvider
-     * @param GuzzleClient $client
+     * @param Client $client
      */
     public function __construct(UrlProvider $urlProvider, Client $client)
     {
@@ -50,7 +50,7 @@ 
     /**
      * run.
      *
-     * @return array
+     * @return ResultCollection
      */
     public function run()
     {

Unused Use Statements -2 removed lines

@@ -16,9 +16,7 @@
 namespace Hogosha\Monitor\Runner;
 
 use GuzzleHttp\Client;
-use GuzzleHttp\Exception\ConnectException;
 use GuzzleHttp\Pool;
-use GuzzleHttp\Promise;
 use GuzzleHttp\Psr7\Request;
 use GuzzleHttp\TransferStats;
 use Hogosha\Monitor\Client\GuzzleClient;

Spacing +3 added lines, -3 removed lines

@@ -61,9 +61,9 @@ 
 
         $resultCollection = new ResultCollection();
 
-        $requests = function () use ($urls, $client, $resultCollection) {
+        $requests = function() use ($urls, $client, $resultCollection) {
             foreach ($urls as $url) {
-                yield function () use ($client, $url, $resultCollection) {
+                yield function() use ($client, $url, $resultCollection) {
                     return $client->sendAsync(
                         new Request(
                             $url->getMethod(),
@@ -72,7 +72,7 @@ 
                         ),
                         [
                             'timeout' => $url->getTimeout(),
-                            'on_stats' => function (TransferStats $tranferStats) use ($url, $resultCollection) {
+                            'on_stats' => function(TransferStats $tranferStats) use ($url, $resultCollection) {
 
                                 if ($tranferStats->hasResponse()) {
                                     $statusCode = $tranferStats->getResponse()->getStatusCode();

src/Compiler/Compiler.php

Spacing +22 added lines, -22 removed lines

@@ -30,7 +30,7 @@ 
      */
     public function compile()
     {
-        $pharFilePath = dirname(__FILE__).'/../../build/monitor.phar';
+        $pharFilePath = dirname(__FILE__) . '/../../build/monitor.phar';
         if (file_exists($pharFilePath)) {
             unlink($pharFilePath);
         }
@@ -41,7 +41,7 @@ 
         $phar->setSignatureAlgorithm(\Phar::SHA1);
 
         $phar->startBuffering();
-        $root = __DIR__.'/../..';
+        $root = __DIR__ . '/../..';
 
         $finder = new Finder();
         $finder->files()
@@ -52,33 +52,33 @@ 
             ->exclude('Tests')
             ->exclude('tests')
             ->exclude('docs')
-            ->in($root.'/src')
-            ->in($root.'/vendor/guzzlehttp')
-            ->in($root.'/vendor/eljam')
-            ->in($root.'/vendor/hogosha')
-            ->in($root.'/vendor/webmozart')
-            ->in($root.'/vendor/psr')
-            ->in($root.'/vendor/guzzle')
-            ->in($root.'/vendor/symfony')
+            ->in($root . '/src')
+            ->in($root . '/vendor/guzzlehttp')
+            ->in($root . '/vendor/eljam')
+            ->in($root . '/vendor/hogosha')
+            ->in($root . '/vendor/webmozart')
+            ->in($root . '/vendor/psr')
+            ->in($root . '/vendor/guzzle')
+            ->in($root . '/vendor/symfony')
         ;
 
         foreach ($finder as $file) {
             $this->addFile($phar, $file);
         }
 
-        $this->addFile($phar, new \SplFileInfo($root.'/vendor/autoload.php'));
-        $this->addFile($phar, new \SplFileInfo($root.'/vendor/composer/autoload_namespaces.php'));
-        $this->addFile($phar, new \SplFileInfo($root.'/vendor/composer/autoload_psr4.php'));
-        $this->addFile($phar, new \SplFileInfo($root.'/vendor/composer/autoload_classmap.php'));
-        $this->addFile($phar, new \SplFileInfo($root.'/vendor/composer/autoload_files.php'));
-        $this->addFile($phar, new \SplFileInfo($root.'/vendor/composer/autoload_real.php'));
+        $this->addFile($phar, new \SplFileInfo($root . '/vendor/autoload.php'));
+        $this->addFile($phar, new \SplFileInfo($root . '/vendor/composer/autoload_namespaces.php'));
+        $this->addFile($phar, new \SplFileInfo($root . '/vendor/composer/autoload_psr4.php'));
+        $this->addFile($phar, new \SplFileInfo($root . '/vendor/composer/autoload_classmap.php'));
+        $this->addFile($phar, new \SplFileInfo($root . '/vendor/composer/autoload_files.php'));
+        $this->addFile($phar, new \SplFileInfo($root . '/vendor/composer/autoload_real.php'));
 
-        if (file_exists($root.'/vendor/composer/include_paths.php')) {
-            $this->addFile($phar, new \SplFileInfo($root.'/vendor/composer/include_paths.php'));
+        if (file_exists($root . '/vendor/composer/include_paths.php')) {
+            $this->addFile($phar, new \SplFileInfo($root . '/vendor/composer/include_paths.php'));
         }
-        $this->addFile($phar, new \SplFileInfo($root.'/vendor/composer/ClassLoader.php'));
+        $this->addFile($phar, new \SplFileInfo($root . '/vendor/composer/ClassLoader.php'));
 
-        $binContent = file_get_contents($root.'/bin/monitor');
+        $binContent = file_get_contents($root . '/bin/monitor');
         $binContent = preg_replace('{^#!/usr/bin/env php\s*}', '', $binContent);
         $phar->addFromString('bin/monitor', $binContent);
 
@@ -90,12 +90,12 @@ 
 
     protected function addFile(\Phar $phar, \SplFileInfo $file, $strip = true)
     {
-        $path = str_replace(dirname(dirname(__DIR__)).DIRECTORY_SEPARATOR, '', $file->getRealPath());
+        $path = str_replace(dirname(dirname(__DIR__)) . DIRECTORY_SEPARATOR, '', $file->getRealPath());
         $content = file_get_contents($file);
         if ($strip) {
             $content = self::stripWhitespace($content);
         } elseif ('LICENSE' === basename($file)) {
-            $content = "\n".$content."\n";
+            $content = "\n" . $content . "\n";
         }
 
         if ($path === 'src/Monitor.php') {