Merge pull request #221 from StyxOfDynamite/issue/220

issue/220
laravel-notification-channels · Feb 26, 2025 · ccf883b · ccf883b
2 parents b421a79 + ce3f66d
commit ccf883b
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/src/FcmMessage.php b/src/FcmMessage.php
@@ -76,10 +76,14 @@ public function condition(?string $condition): self
     }
 
     /**
-     * Set the message data.
+     * Set the message data, or throw exception if data is not an array of strings.
      */
     public function data(?array $data): self
     {
+        if (! empty(array_filter($data, fn ($value) => ! is_string($value)))) {
+            throw new \InvalidArgumentException('Data values must be strings.');
+        }
+
         $this->data = $data;
 
         return $this;

diff --git a/tests/FcmMessageTest.php b/tests/FcmMessageTest.php
@@ -63,6 +63,13 @@ public function test_it_can_set_data()
         $this->assertEquals(['data' => ['a' => 'b']], $message->toArray());
     }
 
+    public function test_it_throws_exception_on_invalid_data()
+    {
+        $this->expectException(\InvalidArgumentException::class);
+
+        FcmMessage::create()->data(['a' => 1]);
+    }
+
     public function test_it_can_set_custom_attributes()
     {
         $message = FcmMessage::create()