View GitHub Repository graphql-php-validation-toolkit

Schedule Inspection
Subscribe
Passed
Pull Request — master (#1)
by Max
07:34 queued 02:12
created
Status
Category
Toggle Patches
tests/Type/ValidatedFieldDefinition/NonNullScalarValidationTest.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch 
@@ -55,13 +55,13 @@  discard block
 
 block discarded – undo
55 55 
             'fields' => [
56 56 
                 'title' => [
57 57 
                     'type' => Type::string(),
58 
-                    'resolve' => static function ($book) {
58 
+                    'resolve' => static function($book) {
59 59 
                         return $book['title'];
60 60 
                     },
61 61 
                 ],
62 62 
                 'author' => [
63 63 
                     'type' => $this->personType,
64 
-                    'resolve' => static function ($book) {
64 
+                    'resolve' => static function($book) {
65 65 
                         return $book['author'];
66 66 
                     },
67 67 
                 ],
 
@@ -71,7 +71,7 @@  discard block
 
 block discarded – undo
71 71 
         $this->schema = new Schema([
72 72 
             'mutation' => new ObjectType([
73 73 
                 'name' => 'Mutation',
74 
-                'fields' => function () {
74 
+                'fields' => function() {
75 75 
                     return [
76 76 
                         'updateBook' => new ValidatedFieldDefinition([
77 77 
                             'name' => 'updateBook',
 
@@ -80,7 +80,7 @@  discard block
 
 block discarded – undo
80 80 
                                 'bookId' => [
81 81 
                                     'type' => Type::nonNull(Type::id()),
82 82 
                                     'errorCodes' => ['bookNotFound'],
83 
-                                    'validate' => function ($bookId) {
83 
+                                    'validate' => function($bookId) {
84 84 
                                         if (isset($this->data['books'][$bookId])) {
85 85 
                                             return 0;
86 86 
                                         }
 
@@ -89,7 +89,7 @@  discard block
 
 block discarded – undo
89 89 
                                     },
90 90 
                                 ],
91 91 
                             ],
92 
-                            'resolve' => function ($value, $args) : array {
92 
+                            'resolve' => function($value, $args) : array {
93 93 
                                 return $this->data['books'][$args['bookId']];
94 94 
                             },
95 95 
                         ]),
Please login to merge, or discard this patch.
tests/Type/ValidatedFieldDefinition/ScalarValidationTest.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch 
@@ -58,13 +58,13 @@  discard block
 
 block discarded – undo
58 58 
             'fields' => [
59 59 
                 'title' => [
60 60 
                     'type' => Type::string(),
61 
-                    'resolve' => static function ($book) {
61 
+                    'resolve' => static function($book) {
62 62 
                         return $book['title'];
63 63 
                     },
64 64 
                 ],
65 65 
                 'author' => [
66 66 
                     'type' => $this->personType,
67 
-                    'resolve' => static function ($book) {
67 
+                    'resolve' => static function($book) {
68 68 
                         return $book['author'];
69 69 
                     },
70 70 
                 ],
 
@@ -77,7 +77,7 @@  discard block
 
 block discarded – undo
77 77 
             'query' => $this->query,
78 78 
             'mutation' => new ObjectType([
79 79 
                 'name' => 'Mutation',
80 
-                'fields' => function () {
80 
+                'fields' => function() {
81 81 
                     return [
82 82 
                         'updateBook' => new ValidatedFieldDefinition([
83 83 
                             'name' => 'updateBook',
 
@@ -86,7 +86,7 @@  discard block
 
 block discarded – undo
86 86 
                                 'bookId' => [
87 87 
                                     'type' => Type::id(),
88 88 
                                     'errorCodes' => ['bookNotFound'],
89 
-                                    'validate' => function ($bookId) {
89 
+                                    'validate' => function($bookId) {
90 90 
                                         if (isset($this->data['books'][$bookId])) {
91 91 
                                             return 0;
92 92 
                                         }
 
@@ -95,7 +95,7 @@  discard block
 
 block discarded – undo
95 95 
                                     },
96 96 
                                 ],
97 97 
                             ],
98 
-                            'resolve' => static function ($value, $args) : bool {
98 
+                            'resolve' => static function($value, $args) : bool {
99 99 
                                 return true;
100 100 
                             },
101 101 
                         ]),
Please login to merge, or discard this patch.
tests/Type/ErrorCodeTypeGenerationTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -23,7 +23,7 @@  discard block
 
 block discarded – undo
23 23 
                 'unknownUser',
24 24 
                 'userIsMinor',
25 25 
             ],
26 
-            'typeSetter' => static function ($type) use (&$types) {
26 
+            'typeSetter' => static function($type) use (&$types) {
27 27 
                 $types[$type->name] = $type;
28 28 
             },
29 29 
             'type' => new IDType(['name' => 'User']),
 
@@ -56,7 +56,7 @@  discard block
 
 block discarded – undo
56 56 
                     ],
57 57 
                 ],
58 58 
             ]),
59 
-            'typeSetter' => static function ($type) use (&$types) {
59 
+            'typeSetter' => static function($type) use (&$types) {
60 60 
                 $types[$type->name] = $type;
61 61 
             },
62 62 
         ], ['updateBook']);
 
@@ -84,7 +84,7 @@  discard block
 
 block discarded – undo
84 84 
                     ],
85 85 
                 ],
86 86 
             ]),
87 
-            'typeSetter' => static function ($type) use (&$types) {
87 
+            'typeSetter' => static function($type) use (&$types) {
88 88 
                 $types[$type->name] = $type;
89 89 
             },
90 90 
         ], ['updateBook']);
Please login to merge, or discard this patch.
tests/Utils.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -22,7 +22,7 @@
 
 block discarded – undo
22 22 
         }
23 23
24 24 
         // Toss out the first and last lines.
25 
-        $lines = array_slice($lines, 1, count($lines) - 2);
25 
+        $lines = array_slice($lines, 1, count($lines)-2);
26 26
27 27 
         // take the tabs from the first line, and subtract them from all lines
28 28 
         $matches = [];
Please login to merge, or discard this patch.
tests/Type/UserErrorsType/ListOfTest.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch 
@@ -38,7 +38,7 @@  discard block
 
 block discarded – undo
38 38 
         $type = new UserErrorsType([
39 39 
             'type' => Type::listOf(Type::id()),
40 40 
             'errorCodes'=> ['atLeastOneRequired'],
41 
-            'validate' => static function ($value) {
41 
+            'validate' => static function($value) {
42 42 
                 return 0;
43 43 
             },
44 44 
         ], ['users']);
 
@@ -70,10 +70,10 @@  discard block
 
 block discarded – undo
70 70 
         $type = new UserErrorsType([
71 71 
             'type' => Type::listOf(Type::listOf(Type::listOf(Type::id()))),
72 72 
             'errorCodes'=> ['atLeastOneRequired'],
73 
-            'validate' => static function ($value) {
73 
+            'validate' => static function($value) {
74 74 
                 return 0;
75 75 
             },
76 
-            'validateItem' => static function ($value) {
76 
+            'validateItem' => static function($value) {
77 77 
                 return 0;
78 78 
             }
79 79 
         ], ['users']);
 
@@ -117,9 +117,9 @@  discard block
 
 block discarded – undo
117 117
118 118 
     public function testListOfInputObjectWithValidationOnSelf()
119 119 
     {
120 
-        $type  = new UserErrorsType([
120 
+        $type = new UserErrorsType([
121 121 
 	        'validate' => static function(array $authors) {
122 
-        	    if(count($authors) < 1) {
122 
+        	    if (count($authors) < 1) {
123 123 
         	    	return ['atLeastOneAuthorRequired', "You must submit at least one author"];
124 124 
 	            }
125 125 
 	        },
 
@@ -161,10 +161,10 @@  discard block
 
 block discarded – undo
161 161
162 162 
 	public function testListOfInputObjectWithValidationOnWrappedSelf()
163 163 
 	{
164 
-		$type  = new UserErrorsType([
164 
+		$type = new UserErrorsType([
165 165 
 			'suberrorCodes' => ['firstNameOrLastNameRequired'],
166 166 
 			'validateItem' => static function(array $author) {
167 
-				if(!isset($author['firstName']) && !isset($author['lastName'])) {
167 
+				if (!isset($author['firstName']) && !isset($author['lastName'])) {
168 168 
 					return ['atLeastOneAuthorRequired', "You must submit a first name or a last name"];
169 169 
 				}
170 170 
 			},
Please login to merge, or discard this patch.
tests/Type/ValidatedFieldDefinition/ListOfScalarValidationTest.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch 
@@ -31,7 +31,7 @@  discard block
 
 block discarded – undo
31 31 
             'query' => $this->query,
32 32 
             'mutation' => new ObjectType([
33 33 
                 'name' => 'Mutation',
34 
-                'fields' => static function () {
34 
+                'fields' => static function() {
35 35 
                     return [
36 36 
                         'setPhoneNumbers' => new ValidatedFieldDefinition([
37 37 
                             'name' => 'setPhoneNumbers',
 
@@ -40,20 +40,20 @@  discard block
 
 block discarded – undo
40 40 
                                 'phoneNumbers' => [
41 41 
                                     'type' => Type::listOf(Type::listOf(Type::string())),
42 42 
                                     'errorCodes' => ['atLeastOneList'],
43 
-                                    'validate' => static function (array $phoneNumberLists) {
43 
+                                    'validate' => static function(array $phoneNumberLists) {
44 44 
                                         if (count($phoneNumberLists) < 1) {
45 45 
                                             return ['atLeastOneList', 'You must submit at least one list of numbers'];
46 46 
                                         }
47 47 
                                         return 0;
48 48 
                                     },
49 49 
                                     'suberrorCodes' => ['invalidPhoneNumber'],
50 
-                                    'validateItem' => static function ($phoneNumber) {
50 
+                                    'validateItem' => static function($phoneNumber) {
51 51 
                                         $res = preg_match('/^[0-9\-]+$/', $phoneNumber) === 1;
52 
-                                        return ! $res ? ['invalidPhoneNumber', 'That does not seem to be a valid phone number'] : 0;
52 
+                                        return !$res ? ['invalidPhoneNumber', 'That does not seem to be a valid phone number'] : 0;
53 53 
                                     },
54 54 
                                 ],
55 55 
                             ],
56 
-                            'resolve' => static function (array $phoneNumbers, $args) : bool {
56 
+                            'resolve' => static function(array $phoneNumbers, $args) : bool {
57 57 
                                 // ...
58 58 
                                 // stash them somewhere
59 59 
                                 // ...
 
@@ -110,7 +110,7 @@  discard block
 
 block discarded – undo
110 110 
                             [
111 111 
                                 'suberrors' =>
112 112 
 	                                [
113 
-	                                    'path' => [0,1],
113 
+	                                    'path' => [0, 1],
114 114 
 	                                    'code' => 'invalidPhoneNumber',
115 115 
 	                                ]
116 116 
                             ],
 
@@ -157,12 +157,12 @@  discard block
 
 block discarded – undo
157 157 
 		);
158 158
159 159 
 		static::assertEquals(
160 
-			array (
160 
+			array(
161 161 
 				'valid' => false,
162 162 
 				'suberrors' =>
163 
-					array (
163 
+					array(
164 164 
 						'phoneNumbers' =>
165 
-							array (
165 
+							array(
166 166 
 								'code' => 'atLeastOneList',
167 167 
 								'msg' => 'You must submit at least one list of numbers',
168 168 
 								'suberrors' => NULL,
 
@@ -217,18 +217,18 @@  discard block
 
 block discarded – undo
217 217
218 218 
         static::assertEmpty($res->errors);
219 219 
         static::assertEquals(
220 
-	        array (
220 
+	        array(
221 221 
 		        'valid' => false,
222 222 
 		        'suberrors' =>
223 
-			        array (
223 
+			        array(
224 224 
 				        'phoneNumbers' =>
225 
-					        array (
225 
+					        array(
226 226 
 						        'code' => NULL,
227 227 
 						        'msg' => NULL,
228 228 
 						        'suberrors' =>
229 
-							        array (
229 
+							        array(
230 230 
 								        'path' =>
231 
-									        array (
231 
+									        array(
232 232 
 										        0 => 1,
233 233 
 										        1 => 1,
234 234 
 									        ),
Please login to merge, or discard this patch.
tests/Type/ValidatedFieldDefinition/InputObjectValidationTest.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch 
@@ -61,7 +61,7 @@  discard block
 
 block discarded – undo
61 61 
                 'title' => [
62 62 
                     'type' => Type::string(),
63 63 
                     'description' => 'Enter a book title, no more than 10 characters in length',
64 
-                    'validate' => static function (string $title) {
64 
+                    'validate' => static function(string $title) {
65 65 
                         if (strlen($title) > 10) {
66 66 
                             return [1, 'book title must be less than 10 chaacters'];
67 67 
                         }
 
@@ -75,8 +75,8 @@  discard block
 
 block discarded – undo
75 75 
                         'unknownAuthor',
76 76 
                         'authorDeceased',
77 77 
                     ],
78 
-                    'validate' => function (string $authorId) {
79 
-                        if (! isset($this->data['people'][$authorId])) {
78 
+                    'validate' => function(string $authorId) {
79 
+                        if (!isset($this->data['people'][$authorId])) {
80 80 
                             return ['unknownAuthor', 'We have no record of that author'];
81 81 
                         }
82 82 
                         return 0;
 
@@ -91,7 +91,7 @@  discard block
 
 block discarded – undo
91 91 
             'query' => $this->query,
92 92 
             'mutation' => new ObjectType([
93 93 
                 'name' => 'Mutation',
94 
-                'fields' => function () {
94 
+                'fields' => function() {
95 95 
                     return [
96 96 
                         'updateBook' => new ValidatedFieldDefinition([
97 97 
                             'name' => 'updateBook',
 
@@ -103,7 +103,7 @@  discard block
 
 block discarded – undo
103 103 
 										'titleOrIdRequired'
104 104 
 	                                ],
105 105 
 	                                'validate' => static function(?array $bookAttributes) {
106 
-                        	            if(is_null($bookAttributes)) {
106 
+                        	            if (is_null($bookAttributes)) {
107 107 
                         	                return 0;
108 108 
 	                                    }
109 109 
                         	            
@@ -113,7 +113,7 @@  discard block
 
 block discarded – undo
113 113 
 	                                }
114 114 
                                 ],
115 115 
                             ],
116 
-                            'resolve' => static function ($value, $args) : bool {
116 
+                            'resolve' => static function($value, $args) : bool {
117 117 
                                 // ...
118 118 
                                 // do update
119 119 
                                 // ...
 
@@ -240,12 +240,12 @@  discard block
 
 block discarded – undo
240 240 
 		);
241 241
242 242 
 		static::assertEquals(
243 
-			array (
243 
+			array(
244 244 
 				'valid' => false,
245 245 
 				'suberrors' =>
246 
-					array (
246 
+					array(
247 247 
 						'bookAttributes' =>
248 
-							array (
248 
+							array(
249 249 
 								'code' => 'titleOrIdRequired',
250 250 
 								'msg' => 'You must supply at least one of title or author',
251 251 
 								'suberrors' => NULL,
 
@@ -351,7 +351,7 @@  discard block
 
 block discarded – undo
351 351
352 352 
         static::assertEmpty($res->errors);
353 353 
         static::assertEquals(
354 
-	        array (
354 
+	        array(
355 355 
 		        'valid' => true,
356 356 
 		        'suberrors' => NULL,
357 357 
 		        'result' => true,
Please login to merge, or discard this patch.
tests/Type/UserErrorsType/BasicTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -35,7 +35,7 @@  discard block
 
 block discarded – undo
35 35 
     public function testNoType() {
36 36 
 	    $this->expectExceptionMessage("You must specify a type for your field");
37 37 
 	    UserErrorsType::create([
38 
-		    'validate' => static function ($value) {
38 
+		    'validate' => static function($value) {
39 39 
 		    }
40 40 
 	    ], ['upsertSku']);
41 41 
     }
 
@@ -43,7 +43,7 @@  discard block
 
 block discarded – undo
43 43 
     public function testValidationWithNoErrorCodes()
44 44 
     {
45 45 
         $type = UserErrorsType::create([
46 
-            'validate' => static function ($value) {
46 
+            'validate' => static function($value) {
47 47 
             },
48 48 
             'type' => Type::id(),
49 49 
         ], ['upsertSku']);
Please login to merge, or discard this patch.
src/Type/Definition/UserErrorsType.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch 
@@ -30,7 +30,7 @@  discard block
 
 block discarded – undo
30 30 
         GraphQL\Utils\Utils::invariant($config['type'] instanceof Type, 'Must provide type.');
31 31
32 32 
         if (isset($config['errorCodes'])) {
33 
-        	if(isset($config['validate'])) {
33 
+        	if (isset($config['validate'])) {
34 34 
 		        /** code property */
35 35 
 		        $finalFields['code'] = [
36 36 
 			        'type' => $this->_set(new EnumType([

@@ -39,7 +39,7 @@  discard block
 
 block discarded – undo
39 39 
 				        'values' => $config['errorCodes'],
40 40 
 			        ]), $config),
41 41 
 			        'description' => 'An error code',
42 
-			        'resolve' => static function ($value) {
42 
+			        'resolve' => static function($value) {
43 43 
 				        return $value['error'][0] ?? null;
44 44 
 			        },
45 45 
 		        ];

@@ -50,7 +50,7 @@  discard block
 
 block discarded – undo
50 50 
 		        $finalFields['msg'] = [
51 51 
 			        'type' => Type::string(),
52 52 
 			        'description' => 'A natural language description of the issue',
53 
-			        'resolve' => static function ($value) {
53 
+			        'resolve' => static function($value) {
54 54 
 				        return $value['error'][1] ?? null;
55 55 
 			        },
56 56 
 		        ];

@@ -64,18 +64,18 @@  discard block
 
 block discarded – undo
64 64 
             $fields = [];
65 65 
             foreach ($type->getFields() as $key => $field) {
66 66 
                 $newType = static::create(
67 
-                    $field->config + ['typeSetter' => $config['typeSetter'] ?? null],
67 
+                    $field->config+['typeSetter' => $config['typeSetter'] ?? null],
68 68 
                     array_merge($path, [$key])
69 69 
                 );
70 70
71 
-                if (! $newType) {
71 
+                if (!$newType) {
72 72 
                     continue;
73 73 
                 }
74 74
75 75 
                 $fields[$key] = [
76 76 
                     'description' => 'Error for ' . $key,
77 77 
                     'type' => $newType,
78 
-                    'resolve' => static function ($value) use ($key) {
78 
+                    'resolve' => static function($value) use ($key) {
79 79 
                         return $value[$key] ?? null;
80 80 
                     },
81 81 
                 ];

@@ -92,7 +92,7 @@  discard block
 
 block discarded – undo
92 92 
                         'fields' => $fields,
93 93 
                     ]), $config),
94 94 
                     'description' => 'Suberrors for ' . ucfirst($path[count($path)-1]),
95 
-                    'resolve' => static function (array $value) {
95 
+                    'resolve' => static function(array $value) {
96 96 
                         return $value['suberrors'] ?? null;
97 97 
                     },
98 98 
                 ];

@@ -116,7 +116,7 @@  discard block
 
 block discarded – undo
116 116 
                 $finalFields['suberrors'] = [
117 117 
                     'description' => 'Suberrors for the list of ' . $type->ofType . ' items',
118 118 
                     'type' => $newType,
119 
-                    'resolve' => static function ($value) {
119 
+                    'resolve' => static function($value) {
120 120 
                         return $value['suberrors'] ?? null;
121 121 
                     },
122 122 
                 ];

@@ -130,7 +130,7 @@  discard block
 
 block discarded – undo
130 130 
             $finalFields['path'] = [
131 131 
                 'type' => Type::listOf(Type::int()),
132 132 
                 'description' => 'A path describing this items\'s location in the nested array',
133 
-                'resolve' => static function ($value) {
133 
+                'resolve' => static function($value) {
134 134 
                     return $value['path'];
135 135 
                 },
136 136 
             ];

@@ -167,7 +167,7 @@  discard block
 
 block discarded – undo
167 167 
             $config['fields']['code'] = $config['fields']['code'] ?? [
168 168 
                 'type' => Type::int(),
169 169 
                 'description' => 'A numeric error code. 0 on success, non-zero on failure.',
170 
-                'resolve' => static function ($value) {
170 
+                'resolve' => static function($value) {
171 171 
                     $error = $value['error'] ?? null;
172 172 
                     switch (gettype($error)) {
173 173 
                         case 'integer':

@@ -180,7 +180,7 @@  discard block
 
 block discarded – undo
180 180 
             $config['fields']['msg'] = $config['fields']['msg'] ?? [
181 181 
                 'type' => Type::string(),
182 182 
                 'description' => 'An error message.',
183 
-                'resolve' => static function ($value) {
183 
+                'resolve' => static function($value) {
184 184 
                     $error = $value['error'] ?? null;
185 185 
                     switch (gettype($error)) {
186 186 
                         case 'integer':
Please login to merge, or discard this patch.