webonyx / graphql-php

src/Validator/Rules/VariablesAreInputTypes.php

Doc Comments +4 added lines

@@ -35,6 +35,10 @@
         ];
     }
 
+    /**
+     * @param string $variableName
+     * @param string $typeName
+     */
     public static function nonInputTypeOnVarMessage($variableName, $typeName)
     {
         return sprintf('Variable "$%s" cannot be non-input type "%s".', $variableName, $typeName);

tests/Executor/DirectivesTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
     }
 
     /**
-     * @param Source|string $doc
+     * @param string $doc
      *
      * @return mixed[]
      */

tests/Executor/Promise/SyncPromiseAdapterTest.php

Doc Comments +12 added lines

@@ -82,6 +82,10 @@ 
         self::assertValidPromise($promise, null, 'A', SyncPromise::FULFILLED);
     }
 
+    /**
+     * @param Promise $promise
+     * @param null|string $expectedNextReason
+     */
     private static function assertValidPromise($promise, $expectedNextReason, $expectedNextValue, $expectedNextState)
     {
         self::assertInstanceOf('GraphQL\Executor\Promise\Promise', $promise);
@@ -93,10 +97,18 @@ 
         $onRejectedCalled  = false;
 
         $promise->then(
+
+            /**
+             * @param Promise $nextValue
+             */
             static function ($nextValue) use (&$actualNextValue, &$onFulfilledCalled) {
                 $onFulfilledCalled = true;
                 $actualNextValue   = $nextValue;
             },
+
+            /**
+             * @param Promise $reason
+             */
             static function (Throwable $reason) use (&$actualNextReason, &$onRejectedCalled) {
                 $onRejectedCalled = true;
                 $actualNextReason = $reason->getMessage();

tests/Executor/SyncTest.php

Doc Comments +16 added lines

@@ -82,11 +82,19 @@ 
         self::assertSync(['errors' => [['message' => 'Must provide an operation.']]], $result);
     }
 
+    /**
+     * @param Schema $schema
+     * @param \GraphQL\Language\AST\DocumentNode $doc
+     * @param string $rootValue
+     */
     private function execute($schema, $doc, $rootValue = null)
     {
         return Executor::promiseToExecute($this->promiseAdapter, $schema, $doc, $rootValue);
     }
 
+    /**
+     * @param Promise $actualResult
+     */
     private static function assertSync($expectedFinalArray, $actualResult) : void
     {
         $message = 'Failed assertion that execution was synchronous';
@@ -146,6 +154,9 @@ 
         $this->assertAsync(['data' => ['syncField' => 'rootValue', 'asyncField' => 'rootValue']], $result);
     }
 
+    /**
+     * @param Promise $actualResult
+     */
     private function assertAsync($expectedFinalArray, $actualResult)
     {
         $message = 'Failed assertion that execution was asynchronous';
@@ -180,6 +191,11 @@ 
         );
     }
 
+    /**
+     * @param Schema $schema
+     * @param string $doc
+     * @param string $rootValue
+     */
     private function graphqlSync($schema, $doc, $rootValue = null)
     {
         return GraphQL::promiseToExecute($this->promiseAdapter, $schema, $doc, $rootValue);

tests/Executor/VariablesTest.php

Doc Comments +3 added lines

@@ -109,6 +109,9 @@
         );
     }
 
+    /**
+     * @param string $query
+     */
     private function executeQuery($query, $variableValues = null)
     {
         $document = Parser::parse($query);

tests/Language/LexerTest.php

Doc Comments +4 added lines

@@ -52,6 +52,10 @@
         return $lexer->advance();
     }
 
+    /**
+     * @param integer $line
+     * @param integer $column
+     */
     private function loc($line, $column)
     {
         return new SourceLocation($line, $column);

tests/Language/ParserTest.php

Doc Comments +9 added lines

@@ -149,6 +149,11 @@ 
         Parser::parse('query Foo($x: Boolean = false @bar) { field }');
     }
 
+    /**
+     * @param string $text
+     * @param string $message
+     * @param SourceLocation $location
+     */
     private function expectSyntaxError($text, $message, $location)
     {
         $this->expectException(SyntaxError::class);
@@ -161,6 +166,10 @@ 
         }
     }
 
+    /**
+     * @param integer $line
+     * @param integer $column
+     */
     private function loc($line, $column)
     {
         return new SourceLocation($line, $column);

tests/Language/SchemaParserTest.php

Doc Comments +15 added lines

@@ -78,6 +78,9 @@ 
         ];
     }
 
+    /**
+     * @param string $name
+     */
     private function typeNode($name, $loc)
     {
         return [
@@ -294,6 +297,11 @@ 
         );
     }
 
+    /**
+     * @param string $text
+     * @param string $message
+     * @param SourceLocation $location
+     */
     private function expectSyntaxError($text, $message, $location)
     {
         $this->expectException(SyntaxError::class);
@@ -306,6 +314,10 @@ 
         }
     }
 
+    /**
+     * @param integer $line
+     * @param integer $column
+     */
     private function loc($line, $column)
     {
         return new SourceLocation($line, $column);
@@ -533,6 +545,9 @@ 
         self::assertEquals($expected, TestUtils::nodeToArray($doc));
     }
 
+    /**
+     * @param string $name
+     */
     private function enumValueNode($name, $loc)
     {
         return [

tests/Language/VisitorTest.php

Doc Comments +3 added lines

@@ -107,6 +107,9 @@
         ]);
     }
 
+    /**
+     * @param DocumentNode $ast
+     */
     private function checkVisitorFnArgs($ast, $args, $isEdited = false)
     {
         /** @var Node $node */