shmax / graphql-php-validation-toolkit

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/ListOfTest.php

Spacing +7 added lines, -7 removed lines

@@ -38,7 +38,7 @@ 
         $type = new UserErrorsType([
             'type' => Type::listOf(Type::id()),
             'errorCodes'=> ['atLeastOneRequired'],
-            'validate' => static function ($value) {
+            'validate' => static function($value) {
                 return 0;
             },
         ], ['users']);
@@ -70,10 +70,10 @@ 
         $type = new UserErrorsType([
             'type' => Type::listOf(Type::listOf(Type::listOf(Type::id()))),
             'errorCodes'=> ['atLeastOneRequired'],
-            'validate' => static function ($value) {
+            'validate' => static function($value) {
                 return 0;
             },
-            'validateItem' => static function ($value) {
+            'validateItem' => static function($value) {
                 return 0;
             }
         ], ['users']);
@@ -117,9 +117,9 @@ 
 
     public function testListOfInputObjectWithValidationOnSelf()
     {
-        $type  = new UserErrorsType([
+        $type = new UserErrorsType([
 	        'validate' => static function(array $authors) {
-        	    if(count($authors) < 1) {
+        	    if (count($authors) < 1) {
         	    	return ['atLeastOneAuthorRequired', "You must submit at least one author"];
 	            }
 	        },
@@ -161,10 +161,10 @@ 
 
 	public function testListOfInputObjectWithValidationOnWrappedSelf()
 	{
-		$type  = new UserErrorsType([
+		$type = new UserErrorsType([
 			'suberrorCodes' => ['firstNameOrLastNameRequired'],
 			'validateItem' => static function(array $author) {
-				if(!isset($author['firstName']) && !isset($author['lastName'])) {
+				if (!isset($author['firstName']) && !isset($author['lastName'])) {
 					return ['atLeastOneAuthorRequired', "You must submit a first name or a last name"];
 				}
 			},

tests/Type/ValidatedFieldDefinition/ListOfScalarValidationTest.php

Spacing +14 added lines, -14 removed lines

@@ -31,7 +31,7 @@ 
             'query' => $this->query,
             'mutation' => new ObjectType([
                 'name' => 'Mutation',
-                'fields' => static function () {
+                'fields' => static function() {
                     return [
                         'setPhoneNumbers' => new ValidatedFieldDefinition([
                             'name' => 'setPhoneNumbers',
@@ -40,20 +40,20 @@ 
                                 'phoneNumbers' => [
                                     'type' => Type::listOf(Type::listOf(Type::string())),
                                     'errorCodes' => ['atLeastOneList'],
-                                    'validate' => static function (array $phoneNumberLists) {
+                                    'validate' => static function(array $phoneNumberLists) {
                                         if (count($phoneNumberLists) < 1) {
                                             return ['atLeastOneList', 'You must submit at least one list of numbers'];
                                         }
                                         return 0;
                                     },
                                     'suberrorCodes' => ['invalidPhoneNumber'],
-                                    'validateItem' => static function ($phoneNumber) {
+                                    'validateItem' => static function($phoneNumber) {
                                         $res = preg_match('/^[0-9\-]+$/', $phoneNumber) === 1;
-                                        return ! $res ? ['invalidPhoneNumber', 'That does not seem to be a valid phone number'] : 0;
+                                        return !$res ? ['invalidPhoneNumber', 'That does not seem to be a valid phone number'] : 0;
                                     },
                                 ],
                             ],
-                            'resolve' => static function (array $phoneNumbers, $args) : bool {
+                            'resolve' => static function(array $phoneNumbers, $args) : bool {
                                 // ...
                                 // stash them somewhere
                                 // ...
@@ -110,7 +110,7 @@ 
                             [
                                 'suberrors' =>
 	                                [
-	                                    'path' => [0,1],
+	                                    'path' => [0, 1],
 	                                    'code' => 'invalidPhoneNumber',
 	                                ]
                             ],
@@ -157,12 +157,12 @@ 
 		);
 
 		static::assertEquals(
-			array (
+			array(
 				'valid' => false,
 				'suberrors' =>
-					array (
+					array(
 						'phoneNumbers' =>
-							array (
+							array(
 								'code' => 'atLeastOneList',
 								'msg' => 'You must submit at least one list of numbers',
 								'suberrors' => NULL,
@@ -217,18 +217,18 @@ 
 
         static::assertEmpty($res->errors);
         static::assertEquals(
-	        array (
+	        array(
 		        'valid' => false,
 		        'suberrors' =>
-			        array (
+			        array(
 				        'phoneNumbers' =>
-					        array (
+					        array(
 						        'code' => NULL,
 						        'msg' => NULL,
 						        'suberrors' =>
-							        array (
+							        array(
 								        'path' =>
-									        array (
+									        array(
 										        0 => 1,
 										        1 => 1,
 									        ),

tests/Type/ValidatedFieldDefinition/InputObjectValidationTest.php

Spacing +10 added lines, -10 removed lines

@@ -61,7 +61,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'];
                         }
@@ -75,8 +75,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;
@@ -91,7 +91,7 @@ 
             'query' => $this->query,
             'mutation' => new ObjectType([
                 'name' => 'Mutation',
-                'fields' => function () {
+                'fields' => function() {
                     return [
                         'updateBook' => new ValidatedFieldDefinition([
                             'name' => 'updateBook',
@@ -103,7 +103,7 @@ 
 										'titleOrIdRequired'
 	                                ],
 	                                'validate' => static function(?array $bookAttributes) {
-                        	            if(is_null($bookAttributes)) {
+                        	            if (is_null($bookAttributes)) {
                         	                return 0;
 	                                    }
                         	            
@@ -113,7 +113,7 @@ 
 	                                }
                                 ],
                             ],
-                            'resolve' => static function ($value, $args) : bool {
+                            'resolve' => static function($value, $args) : bool {
                                 // ...
                                 // do update
                                 // ...
@@ -240,12 +240,12 @@ 
 		);
 
 		static::assertEquals(
-			array (
+			array(
 				'valid' => false,
 				'suberrors' =>
-					array (
+					array(
 						'bookAttributes' =>
-							array (
+							array(
 								'code' => 'titleOrIdRequired',
 								'msg' => 'You must supply at least one of title or author',
 								'suberrors' => NULL,
@@ -351,7 +351,7 @@ 
 
         static::assertEmpty($res->errors);
         static::assertEquals(
-	        array (
+	        array(
 		        'valid' => true,
 		        'suberrors' => NULL,
 		        'result' => true,

tests/Type/UserErrorsType/BasicTest.php

Spacing +2 added lines, -2 removed lines

@@ -35,7 +35,7 @@ 
     public function testNoType() {
 	    $this->expectExceptionMessage("You must specify a type for your field");
 	    UserErrorsType::create([
-		    'validate' => static function ($value) {
+		    'validate' => static function($value) {
 		    }
 	    ], ['upsertSku']);
     }
@@ -43,7 +43,7 @@ 
     public function testValidationWithNoErrorCodes()
     {
         $type = UserErrorsType::create([
-            'validate' => static function ($value) {
+            'validate' => static function($value) {
             },
             'type' => Type::id(),
         ], ['upsertSku']);

src/Type/Definition/UserErrorsType.php

Spacing +11 added lines, -11 removed lines

@@ -30,7 +30,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([

@@ -39,7 +39,7 @@ 
 				        'values' => $config['errorCodes'],

 			        ]), $config),

 			        'description' => 'An error code',

-			        'resolve' => static function ($value) {

+			        'resolve' => static function($value) {

 				        return $value['error'][0] ?? null;

 			        },

 		        ];

@@ -50,7 +50,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;

 			        },

 		        ];

@@ -64,18 +64,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;

                     },

                 ];

@@ -92,7 +92,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;

                     },

                 ];

@@ -116,7 +116,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;

                     },

                 ];

@@ -130,7 +130,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'];

                 },

             ];

@@ -167,7 +167,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':

@@ -180,7 +180,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

Spacing +12 added lines, -12 removed lines

@@ -34,14 +34,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'];

                     },

                 ],

@@ -58,17 +58,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);

                 }

 

@@ -80,19 +80,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) {

@@ -120,11 +120,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

@@ -148,7 +148,7 @@ 
                 }

 

                 $fields = $type->getFields();

-                if(is_array($value)) {

+                if (is_array($value)) {

 	                foreach ($value as $key => $subValue) {

 		                $config = $fields[$key]->config;

 		                $res['suberrors'][$key] = $this->_validate($config, $subValue);

tests/Type/UserErrorsType/ScalarTest.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
     {
         $type = new UserErrorsType([
             'errorCodes' => ['invalidColor'],
-            'validate' => static function ($value) {
+            'validate' => static function($value) {
                 return $value ? 0 : 1;
             },
             'type' => new IDType(['name' => 'Color']),

tests/Type/UserErrorsType/InputObjectTest.php

Spacing +2 added lines, -2 removed lines

@@ -17,7 +17,7 @@ 
 {
     public function testFieldsWithNoErrorCodes()
     {
-        $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' => [