shmax / graphql-php-validation-toolkit

tests/Type/ValidatedFieldDefinition/NonNullScalarValidationTest.php

Spacing +5 added lines, -5 removed lines

@@ -55,13 +55,13 @@ 
             'fields' => [
                 'title' => [
                     'type' => Type::string(),
-                    'resolve' => static function ($book) {
+                    'resolve' => static function($book) {
                         return $book['title'];
                     },
                 ],
                 'author' => [
                     'type' => $this->personType,
-                    'resolve' => static function ($book) {
+                    'resolve' => static function($book) {
                         return $book['author'];
                     },
                 ],
@@ -71,7 +71,7 @@ 
         $this->schema = new Schema([
             'mutation' => new ObjectType([
                 'name' => 'Mutation',
-                'fields' => function () {
+                'fields' => function() {
                     return [
                         'updateBook' => new ValidatedFieldDefinition([
                             'name' => 'updateBook',
@@ -80,7 +80,7 @@ 
                                 'bookId' => [
                                     'type' => Type::nonNull(Type::id()),
                                     'errorCodes' => ['bookNotFound'],
-                                    'validate' => function ($bookId) {
+                                    'validate' => function($bookId) {
                                         if (isset($this->data['books'][$bookId])) {
                                             return 0;
                                         }
@@ -89,7 +89,7 @@ 
                                     },
                                 ],
                             ],
-                            'resolve' => function ($value, $args) : array {
+                            'resolve' => function($value, $args) : array {
                                 return $this->data['books'][$args['bookId']];
                             },
                         ]),

tests/Type/ValidatedFieldDefinition/ScalarValidationTest.php

Spacing +5 added lines, -5 removed lines

@@ -58,13 +58,13 @@ 
             'fields' => [
                 'title' => [
                     'type' => Type::string(),
-                    'resolve' => static function ($book) {
+                    'resolve' => static function($book) {
                         return $book['title'];
                     },
                 ],
                 'author' => [
                     'type' => $this->personType,
-                    'resolve' => static function ($book) {
+                    'resolve' => static function($book) {
                         return $book['author'];
                     },
                 ],
@@ -77,7 +77,7 @@ 
             'query' => $this->query,
             'mutation' => new ObjectType([
                 'name' => 'Mutation',
-                'fields' => function () {
+                'fields' => function() {
                     return [
                         'updateBook' => new ValidatedFieldDefinition([
                             'name' => 'updateBook',
@@ -86,7 +86,7 @@ 
                                 'bookId' => [
                                     'type' => Type::id(),
                                     'errorCodes' => ['bookNotFound'],
-                                    'validate' => function ($bookId) {
+                                    'validate' => function($bookId) {
                                         if (isset($this->data['books'][$bookId])) {
                                             return 0;
                                         }
@@ -95,7 +95,7 @@ 
                                     },
                                 ],
                             ],
-                            'resolve' => static function ($value, $args) : bool {
+                            'resolve' => static function($value, $args) : bool {
                                 return true;
                             },
                         ]),

tests/Type/ValidatedFieldDefinition/InputObjectValidationTest.php

Spacing +5 added lines, -5 removed lines

@@ -60,7 +60,7 @@ 
                 'title' => [
                     'type' => Type::string(),
                     'description' => 'Enter a book title, no more than 10 characters in length',
-                    'validate' => static function (string $title) {
+                    'validate' => static function(string $title) {
                         if (strlen($title) > 10) {
                             return [1, 'book title must be less than 10 chaacters'];
                         }
@@ -74,8 +74,8 @@ 
                         'unknownAuthor',
                         'authorDeceased',
                     ],
-                    'validate' => function (string $authorId) {
-                        if (! isset($this->data['people'][$authorId])) {
+                    'validate' => function(string $authorId) {
+                        if (!isset($this->data['people'][$authorId])) {
                             return ['unknownAuthor', 'We have no record of that author'];
                         }
                         return 0;
@@ -90,7 +90,7 @@ 
             'query' => $this->query,
             'mutation' => new ObjectType([
                 'name' => 'Mutation',
-                'fields' => function () {
+                'fields' => function() {
                     return [
                         'updateBook' => new ValidatedFieldDefinition([
                             'name' => 'updateBook',
@@ -100,7 +100,7 @@ 
                                     'type' => $this->bookAttributesInputType,
                                 ],
                             ],
-                            'resolve' => static function ($value, $args) : bool {
+                            'resolve' => static function($value, $args) : bool {
                                 // ...
                                 // do update
                                 // ...

tests/Type/UserErrorsType/BasicTest.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
     public function testValidationWithNoErrorCodes()
     {
         $type = UserErrorsType::create([
-            'validate' => static function ($value) {
+            'validate' => static function($value) {
             },
             'type' => Type::id(),
         ], ['upsertSku']);

tests/Type/ErrorCodeTypeGenerationTest.php

Spacing +3 added lines, -3 removed lines

@@ -23,7 +23,7 @@ 
                 'unknownUser',
                 'userIsMinor',
             ],
-            'typeSetter' => static function ($type) use (&$types) {
+            'typeSetter' => static function($type) use (&$types) {
                 $types[$type->name] = $type;
             },
             'type' => new IDType(['name' => 'User']),
@@ -56,7 +56,7 @@ 
                     ],
                 ],
             ]),
-            'typeSetter' => static function ($type) use (&$types) {
+            'typeSetter' => static function($type) use (&$types) {
                 $types[$type->name] = $type;
             },
         ], ['updateBook']);
@@ -84,7 +84,7 @@ 
                     ],
                 ],
             ]),
-            'typeSetter' => static function ($type) use (&$types) {
+            'typeSetter' => static function($type) use (&$types) {
                 $types[$type->name] = $type;
             },
         ], ['updateBook']);

Indentation +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
     {
         $types = [];
         new UserErrorsType([
-        	'validate' => static function($val) {},
+            'validate' => static function($val) {},
             'errorCodes' => [
                 'unknownUser',
                 'userIsMinor',
@@ -79,7 +79,7 @@ 
                 'name' => 'bookInput',
                 'fields' => [
                     'authorId' => [
-	                    'validate' => static function($authorId) {},
+                        'validate' => static function($authorId) {},
                         'errorCodes' => ['unknownAuthor'],
                         'type' => Type::id(),
                         'description' => 'An author Id',

src/Type/Definition/UserErrorsType.php

Indentation +27 added lines, -27 removed lines

@@ -26,33 +26,33 @@
         GraphQL\Utils\Utils::invariant($config['type'] instanceof Type, 'Must provide type.');
 
         if (isset($config['errorCodes'])) {
-        	if(isset($config['validate'])) {
-		        /** code property */
-		        $finalFields['code'] = [
-			        'type' => $this->_set(new EnumType([
-				        'name' => $this->_nameFromPath(array_merge($path)) . 'ErrorCode',
-				        'description' => 'Error code',
-				        'values' => $config['errorCodes'],
-			        ]), $config),
-			        'description' => 'An error code',
-			        'resolve' => static function ($value) {
-				        return $value['error'][0] ?? null;
-			        },
-		        ];
-
-		        /**
-		         * msg property
-		         */
-		        $finalFields['msg'] = [
-			        'type' => Type::string(),
-			        'description' => 'A natural language description of the issue',
-			        'resolve' => static function ($value) {
-				        return $value['error'][1] ?? null;
-			        },
-		        ];
-	        } else {
-        		throw new \Exception("If you specify errorCodes, you must also provide a validate callback");
-	        }
+            if(isset($config['validate'])) {
+                /** code property */
+                $finalFields['code'] = [
+                    'type' => $this->_set(new EnumType([
+                        'name' => $this->_nameFromPath(array_merge($path)) . 'ErrorCode',
+                        'description' => 'Error code',
+                        'values' => $config['errorCodes'],
+                    ]), $config),
+                    'description' => 'An error code',
+                    'resolve' => static function ($value) {
+                        return $value['error'][0] ?? null;
+                    },
+                ];
+
+                /**
+                 * msg property
+                 */
+                $finalFields['msg'] = [
+                    'type' => Type::string(),
+                    'description' => 'A natural language description of the issue',
+                    'resolve' => static function ($value) {
+                        return $value['error'][1] ?? null;
+                    },
+                ];
+            } else {
+                throw new \Exception("If you specify errorCodes, you must also provide a validate callback");
+            }
         }
 
         $type = $config['type'];

Spacing +11 added lines, -11 removed lines

@@ -26,7 +26,7 @@ 
         GraphQL\Utils\Utils::invariant($config['type'] instanceof Type, 'Must provide type.');
 
         if (isset($config['errorCodes'])) {
-        	if(isset($config['validate'])) {
+        	if (isset($config['validate'])) {
 		        /** code property */
 		        $finalFields['code'] = [
 			        'type' => $this->_set(new EnumType([
@@ -35,7 +35,7 @@ 
 				        'values' => $config['errorCodes'],
 			        ]), $config),
 			        'description' => 'An error code',
-			        'resolve' => static function ($value) {
+			        'resolve' => static function($value) {
 				        return $value['error'][0] ?? null;
 			        },
 		        ];
@@ -46,7 +46,7 @@ 
 		        $finalFields['msg'] = [
 			        'type' => Type::string(),
 			        'description' => 'A natural language description of the issue',
-			        'resolve' => static function ($value) {
+			        'resolve' => static function($value) {
 				        return $value['error'][1] ?? null;
 			        },
 		        ];
@@ -60,18 +60,18 @@ 
             $fields = [];
             foreach ($type->getFields() as $key => $field) {
                 $newType = static::create(
-                    $field->config + ['typeSetter' => $config['typeSetter'] ?? null],
+                    $field->config+['typeSetter' => $config['typeSetter'] ?? null],
                     array_merge($path, [$key])
                 );
 
-                if (! $newType) {
+                if (!$newType) {
                     continue;
                 }
 
                 $fields[$key] = [
                     'description' => 'Error for ' . $key,
                     'type' => $newType,
-                    'resolve' => static function ($value) use ($key) {
+                    'resolve' => static function($value) use ($key) {
                         return $value[$key] ?? null;
                     },
                 ];
@@ -88,7 +88,7 @@ 
                         'fields' => $fields,
                     ]), $config),
                     'description' => 'Suberrors for ' . ucfirst($path[count($path)-1]),
-                    'resolve' => static function (array $value) {
+                    'resolve' => static function(array $value) {
                         return $value['suberrors'] ?? null;
                     },
                 ];
@@ -112,7 +112,7 @@ 
                 $finalFields['suberrors'] = [
                     'description' => 'Suberrors for the list of ' . $type->ofType . ' items',
                     'type' => $newType,
-                    'resolve' => static function ($value) {
+                    'resolve' => static function($value) {
                         return $value['suberrors'] ?? null;
                     },
                 ];
@@ -126,7 +126,7 @@ 
             $finalFields['path'] = [
                 'type' => Type::listOf(Type::int()),
                 'description' => 'A path describing this items\'s location in the nested array',
-                'resolve' => static function ($value) {
+                'resolve' => static function($value) {
                     return $value['path'];
                 },
             ];
@@ -163,7 +163,7 @@ 
             $config['fields']['code'] = $config['fields']['code'] ?? [
                 'type' => Type::int(),
                 'description' => 'A numeric error code. 0 on success, non-zero on failure.',
-                'resolve' => static function ($value) {
+                'resolve' => static function($value) {
                     $error = $value['error'] ?? null;
                     switch (gettype($error)) {
                         case 'integer':
@@ -176,7 +176,7 @@ 
             $config['fields']['msg'] = $config['fields']['msg'] ?? [
                 'type' => Type::string(),
                 'description' => 'An error message.',
-                'resolve' => static function ($value) {
+                'resolve' => static function($value) {
                     $error = $value['error'] ?? null;
                     switch (gettype($error)) {
                         case 'integer':

src/Type/Definition/ValidatedFieldDefinition.php

Indentation +5 added lines, -5 removed lines

@@ -90,11 +90,11 @@
         ]);
     }
 
-	protected function _isAssoc(array $arr)
-	{
-		if (array() === $arr) return false;
-		return array_keys($arr) !== range(0, count($arr) - 1);
-	}
+    protected function _isAssoc(array $arr)
+    {
+        if (array() === $arr) return false;
+        return array_keys($arr) !== range(0, count($arr) - 1);
+    }
 
     protected function _validateItems($value, array $path, callable $validate) {
         foreach ($value as $idx => $subValue) {

Spacing +12 added lines, -12 removed lines

@@ -37,7 +37,7 @@ 
 
         $this->typeSetter = $config['typeSetter'] ?? null;
 
-        if (! isset($config['type'])) {
+        if (!isset($config['type'])) {
             throw new Exception('You must specify a type for your field');
         }
 
@@ -47,14 +47,14 @@ 
                 'result' => [
                     'type' => $config['type'],
                     'description' => 'The payload, if any',
-                    'resolve' => static function ($value) {
+                    'resolve' => static function($value) {
                         return $value['result'] ?? null;
                     },
                 ],
                 'valid' => [
                     'type' => Type::nonNull(Type::boolean()),
                     'description' => 'Whether all validation passed. True for yes, false for no.',
-                    'resolve' => static function ($value) {
+                    'resolve' => static function($value) {
                         return $value['valid'];
                     },
                 ],
@@ -71,17 +71,17 @@ 
             'type' => $type,
             'args' => $args,
             'name' => $name,
-            'resolve' => function ($value, $args1, $context, $info) use ($config, $args) {
+            'resolve' => function($value, $args1, $context, $info) use ($config, $args) {
                 // validate inputs
-                $config['type']  = new InputObjectType([
+                $config['type'] = new InputObjectType([
                     'name'=>'',
                     'fields' => $args,
                 ]);
                 $errors          = $this->_validate($config, $args1);
                 $result          = $errors;
-                $result['valid'] = ! $errors;
+                $result['valid'] = !$errors;
 
-                if (! isset($result['error']) && ! isset($result['suberrors'])) {
+                if (!isset($result['error']) && !isset($result['suberrors'])) {
                     $result['result'] = $config['resolve']($value, $args1, $context, $info);
                 }
 
@@ -93,19 +93,19 @@ 
 	protected function _isAssoc(array $arr)
 	{
 		if (array() === $arr) return false;
-		return array_keys($arr) !== range(0, count($arr) - 1);
+		return array_keys($arr) !== range(0, count($arr)-1);
 	}
 
     protected function _validateItems($value, array $path, callable $validate) {
         foreach ($value as $idx => $subValue) {
-            if(is_array($subValue) && !$this->_isAssoc($subValue)) {
+            if (is_array($subValue) && !$this->_isAssoc($subValue)) {
                 $path[count($path)-1] = $idx;
                 $newPath = $path;
                 $newPath[] = 0;
                 $this->_validateItems($subValue, $newPath, $validate);
             }
             else {
-                $path[count($path) - 1] = $idx;
+                $path[count($path)-1] = $idx;
                 $err = $validate($subValue);
 
                 if ($err) {
@@ -133,11 +133,11 @@ 
                     }
                 }
 
-                if(isset($arg['validateItem'])) {
+                if (isset($arg['validateItem'])) {
                     try {
                         $this->_validateItems($value, [0], $arg['validateItem']);
                     }
-                    catch(ValidateItemsError $e) {
+                    catch (ValidateItemsError $e) {
                         $res['suberrors'] = [
                             'error' => $e->error,
                             'path' => $e->path

Braces +5 added lines, -5 removed lines

@@ -92,7 +92,9 @@ 
 
 	protected function _isAssoc(array $arr)
 	{
-		if (array() === $arr) return false;
+		if (array() === $arr) {
+		    return false;
+		}
 		return array_keys($arr) !== range(0, count($arr) - 1);
 	}
 
@@ -103,8 +105,7 @@ 
                 $newPath = $path;
                 $newPath[] = 0;
                 $this->_validateItems($subValue, $newPath, $validate);
-            }
-            else {
+            } else {
                 $path[count($path) - 1] = $idx;
                 $err = $validate($subValue);
 
@@ -136,8 +137,7 @@ 
                 if(isset($arg['validateItem'])) {
                     try {
                         $this->_validateItems($value, [0], $arg['validateItem']);
-                    }
-                    catch(ValidateItemsError $e) {
+                    } catch(ValidateItemsError $e) {
                         $res['suberrors'] = [
                             'error' => $e->error,
                             'path' => $e->path

tests/Utils.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
         }
 
         // Toss out the first and last lines.
-        $lines = array_slice($lines, 1, count($lines) - 2);
+        $lines = array_slice($lines, 1, count($lines)-2);
 
         // take the tabs from the first line, and subtract them from all lines
         $matches = [];

tests/Type/UserErrorsType/InputObjectTest.php

Indentation +20 added lines, -20 removed lines

@@ -45,9 +45,9 @@ 
 
     public function testFieldsWithErrorCodesButNoValidate()
     {
-	    $this->expectExceptionMessage("If you specify errorCodes, you must also provide a validate callback");
+        $this->expectExceptionMessage("If you specify errorCodes, you must also provide a validate callback");
 
-	    $type  = new UserErrorsType([
+        $type  = new UserErrorsType([
             'type' => new InputObjectType([
                 'name' => 'bookInput',
                 'fields' => [
@@ -61,25 +61,25 @@ 
         ], ['updateBook']);
     }
 
-	public function testFieldsWithValidate()
-	{
-		$type  = new UserErrorsType([
-			'type' => new InputObjectType([
-				'name' => 'bookInput',
-				'fields' => [
-					'authorId' => [
-						'errorCodes' => ['unknownAuthor'],
-						'validate' => static function(int $authorId) {
+    public function testFieldsWithValidate()
+    {
+        $type  = new UserErrorsType([
+            'type' => new InputObjectType([
+                'name' => 'bookInput',
+                'fields' => [
+                    'authorId' => [
+                        'errorCodes' => ['unknownAuthor'],
+                        'validate' => static function(int $authorId) {
 
-						},
-						'type' => Type::id(),
-						'description' => 'An author Id',
-					],
-				],
-			])
-		], ['updateBook']);
+                        },
+                        'type' => Type::id(),
+                        'description' => 'An author Id',
+                    ],
+                ],
+            ])
+        ], ['updateBook']);
 
-		self::assertEquals(Utils::nowdoc('
+        self::assertEquals(Utils::nowdoc('
 			schema {
 			  query: UpdateBookError
 			}
@@ -111,5 +111,5 @@ 
 			}
 
 		'), SchemaPrinter::doPrint(new Schema(['query' => $type])));
-	}
+    }
 }

Spacing +3 added lines, -3 removed lines

@@ -17,7 +17,7 @@ 
 {
     public function testFieldsWithNoErrorCodes()
     {
-        $type  = new UserErrorsType([
+        $type = new UserErrorsType([
             'type' => new InputObjectType([
                 'name' => 'bookInput',
                 'fields' => [
@@ -47,7 +47,7 @@ 
     {
 	    $this->expectExceptionMessage("If you specify errorCodes, you must also provide a validate callback");
 
-	    $type  = new UserErrorsType([
+	    $type = new UserErrorsType([
             'type' => new InputObjectType([
                 'name' => 'bookInput',
                 'fields' => [
@@ -63,7 +63,7 @@ 
 
 	public function testFieldsWithValidate()
 	{
-		$type  = new UserErrorsType([
+		$type = new UserErrorsType([
 			'type' => new InputObjectType([
 				'name' => 'bookInput',
 				'fields' => [