ignore Safe\unserialize - only in newer safe-php, breaking change to use it now

Author: spawnia

benchmarks/ASTUnserializationBench.php

@@ -5,8 +5,6 @@
 use GraphQL\Language\Parser;
 use Nuwave\Lighthouse\Schema\AST\DocumentAST;
 
-use function Safe\unserialize;
-
 /**
  * @BeforeMethods({"prepareSchema"})
  */
@@ -54,6 +52,7 @@ public function prepareSchema(): void
      */
     public function benchUnserializeDocumentNode(): void
     {
+        // @phpstan-ignore theCodingMachineSafe.function (Safe\unserialize is not available in thecodingmachine/safe ^1 and ^2)
         unserialize($this->documentNode);
     }
 
@@ -64,6 +63,7 @@ public function benchUnserializeDocumentNode(): void
      */
     public function benchUnserializeDocumentAST(): void
     {
+        // @phpstan-ignore theCodingMachineSafe.function (Safe\unserialize is not available in thecodingmachine/safe ^1 and ^2)
         unserialize($this->documentAST);
     }
 }

src/Execution/ContextSerializer.php

@@ -9,8 +9,6 @@
 use Nuwave\Lighthouse\Support\Contracts\GraphQLContext;
 use Nuwave\Lighthouse\Support\Contracts\SerializesContext;
 
-use function Safe\unserialize;
-
 class ContextSerializer implements SerializesContext
 {
     use SerializesAndRestoresModelIdentifiers;
@@ -44,6 +42,7 @@ public function unserialize(string $context): GraphQLContext
         [
             'request' => $rawRequest,
             'user' => $rawUser
+            // @phpstan-ignore theCodingMachineSafe.function (Safe\unserialize is not available in thecodingmachine/safe ^1 and ^2)
         ] = unserialize($context);
 
         if ($rawRequest) {

src/Subscriptions/Storage/RedisStorageManager.php

@@ -10,8 +10,6 @@
 use Nuwave\Lighthouse\Subscriptions\Contracts\StoresSubscriptions;
 use Nuwave\Lighthouse\Subscriptions\Subscriber;
 
-use function Safe\unserialize;
-
 /**
  * Stores subscribers and topics in redis.
  *
@@ -89,6 +87,7 @@ public function subscribersByTopic(string $topic): Collection
 
                 // Other entries may contain invalid values.
                 try {
+                    // @phpstan-ignore theCodingMachineSafe.function (Safe\unserialize is not available in thecodingmachine/safe ^1 and ^2)
                     $subscriber = unserialize($subscriber);
 
                     // This key exists so remove it from the list of missing keys.
@@ -166,6 +165,7 @@ protected function getSubscriber(string $channelKey): ?Subscriber
         $subscriber = $this->connection->command('get', [$channelKey]);
 
         return is_string($subscriber)
+            // @phpstan-ignore theCodingMachineSafe.function (Safe\unserialize is not available in thecodingmachine/safe ^1 and ^2)
             ? unserialize($subscriber)
             : null;
     }

src/Subscriptions/Subscriber.php

@@ -11,8 +11,6 @@
 use Nuwave\Lighthouse\Support\Contracts\GraphQLContext;
 use Nuwave\Lighthouse\Support\Contracts\SerializesContext;
 
-use function Safe\unserialize;
-
 class Subscriber
 {
     /**
@@ -105,6 +103,7 @@ public function __unserialize(array $data): void
         $this->topic = $data['topic'];
 
         $documentNode = AST::fromArray(
+            // @phpstan-ignore theCodingMachineSafe.function (Safe\unserialize is not available in thecodingmachine/safe ^1 and ^2)
             unserialize($data['query']),
         );
         assert($documentNode instanceof DocumentNode, 'We know the type since it is set during construction and serialized.');

tests/Integration/Async/AsyncDirectiveTest.php

@@ -10,8 +10,6 @@
 use Nuwave\Lighthouse\Support\Contracts\GraphQLContext;
 use Tests\DBTestCase;
 
-use function Safe\unserialize;
-
 final class AsyncDirectiveTest extends DBTestCase
 {
     public function testDispatchesMutation(): void
@@ -40,6 +38,7 @@ public function testDispatchesMutation(): void
         foreach ($jobs as $job) {
             $this->assertInstanceOf(AsyncMutation::class, $job);
 
+            // @phpstan-ignore theCodingMachineSafe.function (Safe\unserialize is not available in thecodingmachine/safe ^1 and ^2)
             $jobCycledThroughSerialization = unserialize(serialize($job));
             $this->assertInstanceOf(AsyncMutation::class, $jobCycledThroughSerialization);
             Container::getInstance()->call([$jobCycledThroughSerialization, 'handle']);
@@ -73,6 +72,7 @@ public function testDispatchesMutationOnCustomQueue(): void
             $this->assertInstanceOf(AsyncMutation::class, $job);
             $this->assertSame('custom', $job->queue);
 
+            // @phpstan-ignore theCodingMachineSafe.function (Safe\unserialize is not available in thecodingmachine/safe ^1 and ^2)
             $jobCycledThroughSerialization = unserialize(serialize($job));
             $this->assertInstanceOf(AsyncMutation::class, $jobCycledThroughSerialization);
             Container::getInstance()->call([$jobCycledThroughSerialization, 'handle']);

tests/Unit/Schema/AST/DocumentASTTest.php

@@ -11,8 +11,6 @@
 use Tests\TestCase;
 use Tests\Utils\Models\User;
 
-use function Safe\unserialize;
-
 final class DocumentASTTest extends TestCase
 {
     public function testParsesSimpleSchema(): void
@@ -100,6 +98,7 @@ public function testBeSerialized(): void
         directive @foo on FIELD
         GRAPHQL);
 
+        // @phpstan-ignore theCodingMachineSafe.function (Safe\unserialize is not available in thecodingmachine/safe ^1 and ^2)
         $reserialized = unserialize(
             serialize($documentAST),
         );

tests/Unit/Subscriptions/SubscriberTest.php

@@ -12,8 +12,6 @@
 use Tests\TestCase;
 use Tests\TestsSerialization;
 
-use function Safe\unserialize;
-
 final class SubscriberTest extends TestCase
 {
     use TestsSerialization;
@@ -47,6 +45,7 @@ public function testSerializable(): void
 
         $channel = $subscriber->channel;
 
+        // @phpstan-ignore theCodingMachineSafe.function (Safe\unserialize is not available in thecodingmachine/safe ^1 and ^2)
         $serialized = unserialize(serialize($subscriber));
 
         $this->assertInstanceOf(Subscriber::class, $serialized);