[FEATURE] addArguments() & addPrefix() to last item in selectionSet - helpers for mghoneimy/php-graphql-oqm

src/Client.php

@@ -47,10 +47,10 @@ public function __construct(string $endpointUrl, array $authorizationHeaders = [
      * @param bool                        $resultsAsArray
      * @param array                       $variables
      *
-     * @return Results|null
+     * @return Results
      * @throws QueryError
      */
-    public function runQuery($query, bool $resultsAsArray = false, array $variables = []): ?Results
+    public function runQuery($query, bool $resultsAsArray = false, array $variables = []): Results
     {
         if ($query instanceof QueryBuilderInterface) {
             $query = $query->getQuery();
@@ -68,10 +68,10 @@ public function runQuery($query, bool $resultsAsArray = false, array $variables
      * @param bool   $resultsAsArray
      * @param array  $variables
      *
-     * @return Results|null
+     * @return Results
      * @throws QueryError
      */
-    public function runRawQuery(string $queryString, $resultsAsArray = false, array $variables = []): ?Results
+    public function runRawQuery(string $queryString, $resultsAsArray = false, array $variables = []): Results
     {
         // Set request headers for authorization and content type
         if (!empty($this->authorizationHeaders)) {

src/Query.php

@@ -189,7 +189,7 @@ protected function constructArguments(): string
             }
 
             // Convert argument values to graphql string literal equivalent
-            if (is_scalar($value)) {
+            if (is_scalar($value) || $value === null) {
                 // Convert scalar value to its literal in graphql
                 $value = StringLiteralFormatter::formatValueForRHS($value);
             } elseif (is_array($value)) {

tests/QueryTest.php

@@ -353,6 +353,31 @@ public function testBooleanArgumentValue(Query $query)
         return $query;
     }
 
+    /**
+     * @depends clone testEmptyArguments
+     *
+     * @covers  \GraphQL\Query::setArguments
+     * @covers  \GraphQL\Query::constructArguments
+     *
+     * @param Query $query
+     *
+     * @return Query
+     */
+    public function testNullArgumentValue(Query $query)
+    {
+        $query->setArguments(['arg1' => null]);
+        $this->assertEquals(
+            "query {
+Object(arg1: null) {
+
+}
+}"
+            , (string) $query
+        );
+
+        return $query;
+    }
+
     /**
      * @depends clone testEmptyArguments
      *