LinioIT / ParameterHandler

Tests/ProcessorTest.php

Doc Comments +4 added lines

@@ -167,6 +167,10 @@
         $this->assertFileEquals($dataDir.'/expected.' . $fileType, $workingDir.'/'.$testCase['config']['file'], $testCase['title']);
     }
 
+    /**
+     * @param string $dataDir
+     * @param string $workingDir
+     */
     private function initializeTestCase(array $testCase, $dataDir, $workingDir, $fileType)
     {
         $fs = new Filesystem();

Spacing +8 added lines, -8 removed lines

@@ -138,23 +138,23 @@ 
      */
     public function testParameterHandling($testCaseName, $fileType)
     {
-        $dataDir = __DIR__.'/fixtures/testcases/'. $fileType . '/' .$testCaseName;
+        $dataDir = __DIR__.'/fixtures/testcases/'.$fileType.'/'.$testCaseName;
 
         $testCase = array_replace_recursive(
             array(
                 'title' => 'unknown test',
                 'config' => array(
-                    'file' => 'parameters.' . $fileType,
+                    'file' => 'parameters.'.$fileType,
                     'file-type' => $fileType,
                 ),
-                'dist-file' => 'parameters.'. $fileType .'.dist',
+                'dist-file' => 'parameters.'.$fileType.'.dist',
                 'environment' => array(),
                 'interactive' => false,
             ),
             (array) Yaml::parse(file_get_contents($dataDir.'/setup.yml'))
         );
 
-        $workingDir = sys_get_temp_dir() . '/incenteev_parameter_handler';
+        $workingDir = sys_get_temp_dir().'/incenteev_parameter_handler';
         $exists = $this->initializeTestCase($testCase, $dataDir, $workingDir, $fileType);
 
         $message = sprintf('<info>%s the "%s" file</info>', $exists ? 'Updating' : 'Creating', $testCase['config']['file']);
@@ -164,7 +164,7 @@ 
 
         $this->processor->processFile($testCase['config']);
 
-        $this->assertFileEquals($dataDir.'/expected.' . $fileType, $workingDir.'/'.$testCase['config']['file'], $testCase['title']);
+        $this->assertFileEquals($dataDir.'/expected.'.$fileType, $workingDir.'/'.$testCase['config']['file'], $testCase['title']);
     }
 
     private function initializeTestCase(array $testCase, $dataDir, $workingDir, $fileType)
@@ -175,10 +175,10 @@ 
             $fs->remove($workingDir);
         }
 
-        $fs->copy($dataDir.'/dist.' . $fileType, $workingDir.'/'. $testCase['dist-file']);
+        $fs->copy($dataDir.'/dist.'.$fileType, $workingDir.'/'.$testCase['dist-file']);
 
-        if ($exists = file_exists($dataDir.'/existing.' . $fileType)) {
-            $fs->copy($dataDir.'/existing.' . $fileType, $workingDir.'/'.$testCase['config']['file']);
+        if ($exists = file_exists($dataDir.'/existing.'.$fileType)) {
+            $fs->copy($dataDir.'/existing.'.$fileType, $workingDir.'/'.$testCase['config']['file']);
         }
 
         foreach ($testCase['environment'] as $var => $value) {

FileHandler/YamlHandler.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
      */
     public function save($filePath, $data)
     {
-        return file_put_contents($filePath, "# This file is auto-generated during the composer install\n" . Yaml::dump($data, 99));
+        return file_put_contents($filePath, "# This file is auto-generated during the composer install\n".Yaml::dump($data, 99));
     }
 
     /**

FileHandlerFactory.php

Spacing +2 added lines, -2 removed lines

@@ -20,10 +20,10 @@
     public static function createFileHandler($fileType)
     {
         if (!isset(self::$handlers[$fileType])) {
-            throw new \RuntimeException('Unsupported file type: ' . $fileType);
+            throw new \RuntimeException('Unsupported file type: '.$fileType);
         }
 
-        $className = '\\Incenteev\\ParameterHandler\\FileHandler\\' . self::$handlers[$fileType];
+        $className = '\\Incenteev\\ParameterHandler\\FileHandler\\'.self::$handlers[$fileType];
 
         return new $className();
     }

Processor.php

Spacing +1 added lines, -1 removed lines

@@ -78,7 +78,7 @@
         }
 
         if (empty($config['dist-file'])) {
-            $config['dist-file'] = $config['file'] . '.dist';
+            $config['dist-file'] = $config['file'].'.dist';
         }
 
         if (!is_file($config['dist-file'])) {

Tests/fixtures/testcases/php/custom_dist_file/existing.php

Indentation +3 added lines, -3 removed lines

@@ -1,9 +1,9 @@
 <?php
 // This file is auto-generated during the composer install
 return array (
-  'parameters' =>
-  array (
+    'parameters' =>
+    array (
     'foo' => 'bar',
     'boolean' => false,
-  ),
+    ),
 );

Spacing +2 added lines, -2 removed lines

@@ -1,8 +1,8 @@
 <?php
 // This file is auto-generated during the composer install
-return array (
+return array(
   'parameters' =>
-  array (
+  array(
     'foo' => 'bar',
     'boolean' => false,
   ),

Tests/fixtures/testcases/php/custom_dist_file/expected.php

Indentation +3 added lines, -3 removed lines

@@ -1,9 +1,9 @@
 <?php
 // This file is auto-generated during the composer install
 return array (
-  'parameters' => 
-  array (
+    'parameters' => 
+    array (
     'foo' => 'bar',
     'boolean' => false,
-  ),
+    ),
 );

Spacing +2 added lines, -2 removed lines

@@ -1,8 +1,8 @@
 <?php
 // This file is auto-generated during the composer install
-return array (
+return array(
   'parameters' => 
-  array (
+  array(
     'foo' => 'bar',
     'boolean' => false,
   ),

Tests/fixtures/testcases/php/existent/expected.php

Indentation +5 added lines, -5 removed lines

@@ -1,15 +1,15 @@
 <?php
 // This file is auto-generated during the composer install
 return array (
-  'parameters' => 
-  array (
+    'parameters' => 
+    array (
     'foo' => 'existing_foo',
     'boolean' => false,
     'another' => NULL,
     'nested' => 
     array (
-      'foo' => 'bar',
-      'bar' => 'baz',
+        'foo' => 'bar',
+        'bar' => 'baz',
+    ),
     ),
-  ),
 );

Spacing +3 added lines, -3 removed lines

@@ -1,13 +1,13 @@
 <?php
 // This file is auto-generated during the composer install
-return array (
+return array(
   'parameters' => 
-  array (
+  array(
     'foo' => 'existing_foo',
     'boolean' => false,
     'another' => NULL,
     'nested' => 
-    array (
+    array(
       'foo' => 'bar',
       'bar' => 'baz',
     ),

Tests/fixtures/testcases/php/existent_empty/expected.php

Indentation +3 added lines, -3 removed lines

@@ -1,8 +1,8 @@
 <?php
 // This file is auto-generated during the composer install
 return array (
-  'parameters' => 
-  array (
+    'parameters' => 
+    array (
     'foo' => 'bar',
-  ),
+    ),
 );

Spacing +2 added lines, -2 removed lines

@@ -1,8 +1,8 @@
 <?php
 // This file is auto-generated during the composer install
-return array (
+return array(
   'parameters' => 
-  array (
+  array(
     'foo' => 'bar',
   ),
 );