Clean up tests and update tooling

Author: spawnia

src/DateScalar.php

@@ -44,7 +44,7 @@ public function parseLiteral($valueNode, ?array $variables = null): \DateTimeInt
      */
     protected function tryParsingDate(mixed $value, string $exceptionClass): \DateTimeInterface
     {
-        if (\is_string($value)) {
+        if (is_string($value)) {
             if (preg_match(static::regex(), $value, $matches) !== 1) {
                 $regex = static::regex();
                 throw new $exceptionClass("Value \"{$value}\" does not match \"{$regex}\". Make sure it's ISO 8601 compliant ");

tests/MixedScalarTest.php

@@ -14,7 +14,7 @@ final class MixedScalarTest extends TestCase
 {
     private Schema $schema;
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 
@@ -138,7 +138,7 @@ public static function literalToPhpMap(): iterable
         ];
     }
 
-    protected function executeQueryWithLiteral(string $literal): ExecutionResult
+    private function executeQueryWithLiteral(string $literal): ExecutionResult
     {
         $query = /** @lang GraphQL */ "
         {
@@ -152,10 +152,10 @@ protected function executeQueryWithLiteral(string $literal): ExecutionResult
         );
     }
 
-    protected function executeQueryWithJsonVariable(string $jsonLiteral): ExecutionResult
+    private function executeQueryWithJsonVariable(string $jsonLiteral): ExecutionResult
     {
         $query = /** @lang GraphQL */ '
-        query Foo($var: Mixed) {
+        query ($var: Mixed) {
             foo(bar: $var)
         }
         ';

tests/NullScalarTest.php

@@ -14,10 +14,9 @@ final class NullScalarTest extends TestCase
 {
     private Schema $schema;
 
-    /** @var mixed will be returned by field mixed */
-    private $return;
+    private mixed $return;
 
-    public function setUp(): void
+    protected function setUp(): void
     {
         parent::setUp();
 
@@ -74,7 +73,7 @@ public function testForbidsNonNullReturn(): void
         self::assertSame(['mixed' => null], $graphqlResult->data);
     }
 
-    protected function executeQueryWithLiteral(string $literal): ExecutionResult
+    private function executeQueryWithLiteral(string $literal): ExecutionResult
     {
         $query = /** @lang GraphQL */ "
         {
@@ -88,10 +87,10 @@ protected function executeQueryWithLiteral(string $literal): ExecutionResult
         );
     }
 
-    protected function executeQueryWithJsonVariable(string $jsonLiteral): ExecutionResult
+    private function executeQueryWithJsonVariable(string $jsonLiteral): ExecutionResult
     {
         $query = /** @lang GraphQL */ '
-        query Foo($var: Null) {
+        query ($var: Null) {
             foo(bar: $var)
         }
         ';