Fix input and args defaultValue

`defaultValue` should be set only if value different than null

Author: Jeremiah VALERIE

.styleci.yml

@@ -5,3 +5,4 @@ enabled:
 
 disabled:
     - unalign_equals
+    - psr0

src/Generator/AbstractTypeGenerator.php

@@ -153,10 +153,12 @@ protected function varExport($var, $default = null, array $compilerNames = [])
                 if (count($lines) > 1) {
                     $firstLine = array_shift($lines) . "'" . ' . "\n"';
                     $lastLine = "'" . array_pop($lines);
-                    $lines = array_map(function($s) { return "'" . $s . "'" . ' . "\n"'; }, $lines);
+                    $lines = array_map(function ($s) {
+                        return "'" . $s . "'" . ' . "\n"';
+                    }, $lines);
                     array_unshift($lines, $firstLine);
                     array_push($lines, $lastLine);
-                    $string = implode(" . \n", $lines);
+                    $string = implode(" . ", $lines);
                 }
 
                 return $string;

src/Generator/TypeGenerator.php

@@ -55,7 +55,12 @@ protected function generateDeprecationReason(array $value)
 
     protected function generateDefaultValue(array $value)
     {
-        return $this->varExportFromArrayValue($value, 'defaultValue');
+        $defaultValue = $this->varExportFromArrayValue($value, 'defaultValue');
+        if ('null' === $defaultValue) {
+            return '';
+        }
+
+        return "\n".'<spaces>\'defaultValue\' => '.$defaultValue.',';
     }
 
     protected function generateType(array $value)

src/Resources/skeleton/ArgConfig.php.skeleton

@@ -1,6 +1,5 @@
 [
 <spaces>'name' => <name>,
 <spaces>'type' => <type>,
-<spaces>'description' => <description>,
-<spaces>'defaultValue' => <defaultValue>,
+<spaces>'description' => <description>,<defaultValue>
 ],

src/Resources/skeleton/InputFieldConfig.php.skeleton

@@ -1,5 +1,4 @@
 <name> => [
 <spaces>'type' => <type>,
-<spaces>'defaultValue' => <defaultValue>,
-<spaces>'description' => <description>,
+<spaces>'description' => <description>,<defaultValue>
 ],

tests/StarWarsIntrospectionTest.php

@@ -311,7 +311,7 @@ public function testAllowsQueryingTheSchemaForFieldArgs()
                             'name' => 'hero',
                             'args' => [
                                 [
-                                    'defaultValue' =>  'null',
+                                    'defaultValue' =>  null,
                                     'description' => "If omitted, returns the hero of the whole saga.\nIf provided, returns the hero of that particular episode.\n",
                                     'name' => 'episode',
                                     'type' => [
@@ -370,7 +370,7 @@ public function testAllowsQueryingTheSchemaForFieldArgs()
                                         'kind' => 'SCALAR',
                                         'ofType' => null,
                                     ],
-                                    'defaultValue' => 'null',
+                                    'defaultValue' => null,
                                 ]
                             ],
                         ],

tests/TestCase.php

@@ -19,7 +19,7 @@ public function assertCodeStandard($pathToCode, $level = null, $fixers = null)
     {
         // Run linter in dry-run mode so it changes nothing.
         $csBuilder = new ProcessBuilder([
-            'php-cs-fixer',
+            __DIR__ . '/../bin/php-cs-fixer',
             'fix',
             '--dry-run',
             '--diff',