Ensure fields exist before accessing them

Author: jeromegamez

tests/Integration/Database/TransactionTest.php

@@ -73,9 +73,11 @@ public function aTransactionKeepsTrackOfMultipleReferences(): void
             $secondSnapshot = $transaction->snapshot($secondRef);
 
             $firstCurrentValue = $firstSnapshot->getValue() ?? 0;
+            assert(is_int($firstCurrentValue));
             $newFirstValue = ++$firstCurrentValue;
 
             $secondCurrentValue = $secondSnapshot->getValue() ?? 0;
+            assert(is_int($secondCurrentValue));
             $newSecondValue = ++$secondCurrentValue;
 
             // Set the value without a transaction

tests/Unit/Messaging/CloudMessageTest.php

@@ -139,10 +139,16 @@ public function messageDataCanBeSetWithAnObjectOrAnArray(): void
 
         $fromObject = CloudMessage::new()->withData(MessageData::fromArray($data));
         $serializedFromObject = Json::decode(Json::encode($fromObject), true);
+
+        $this->assertArrayHasKey('data', $serializedFromObject);
+        $this->assertArrayHasKey('key', $serializedFromObject['data']);
         $this->assertSame('value', $serializedFromObject['data']['key']);
 
         $fromArray = CloudMessage::new()->withData($data);
         $serializedFromArray = Json::decode(Json::encode($fromArray), true);
+
+        $this->assertArrayHasKey('data', $serializedFromArray);
+        $this->assertArrayHasKey('key', $serializedFromArray['data']);
         $this->assertSame('value', $serializedFromArray['data']['key']);
 
         $this->assertSame($serializedFromObject, $serializedFromArray);

tests/Unit/Messaging/Processor/SetApnsContentAvailableIfNeededTest.php

@@ -37,6 +37,9 @@ public function itSetsTheExpectedPushType(array $messageData): void
 
         $processed = Json::decode(Json::encode(($this->processor)($message)), true);
 
+        $this->assertArrayHasKey('apns', $processed);
+        $this->assertArrayHasKey('payload', $processed['apns']);
+        $this->assertArrayHasKey('aps', $processed['apns']['payload']);
         $this->assertArrayHasKey('content-available', $processed['apns']['payload']['aps']);
         $this->assertSame(1, $processed['apns']['payload']['aps']['content-available']);
     }

tests/Unit/Messaging/Processor/SetApnsPushTypeIfNeededTest.php

@@ -36,6 +36,8 @@ public function itSetsTheExpectedPushType(string $expected, array $messageData):
 
         $processed = Json::decode(Json::encode(($this->processor)($message)), true);
 
+        $this->assertArrayHasKey('apns', $processed);
+        $this->assertArrayHasKey('headers', $processed['apns']);
         $this->assertArrayHasKey('apns-push-type', $processed['apns']['headers']);
         $this->assertSame($expected, $processed['apns']['headers']['apns-push-type']);
     }