chore: fix test and propety validation

Author: ChiragAgg5k

src/Agents/Schema.php

@@ -91,7 +91,9 @@ public function toJson(): array
     {
         $json = [];
         foreach ($this->object->getProperties() as $property => $value) {
-            $json[$property] = $value['description'].' ('.$value['type'].')';
+            $description = $value['description'] ?? '';
+            $type = $value['type'] ?? '';
+            $json[$property] = $description.' ('.$type.')';
         }
 
         return $json;

src/Agents/Schema/SchemaObject.php

@@ -33,6 +33,13 @@ class SchemaObject
      */
     public function __construct(array $properties = [])
     {
+        foreach ($properties as $name => $property) {
+            if (! $this->validateProperty($name, $property)) {
+                throw new \InvalidArgumentException(
+                    'Invalid type '.var_export($property['type'], true)." for property '$name'. Must be one of: ".implode(', ', self::getValidTypes())
+                );
+            }
+        }
         $this->properties = $properties;
     }
 
@@ -64,7 +71,7 @@ public function getProperty(string $name): ?array
      */
     public function addProperty(string $name, array $property): self
     {
-        if (! isset($property['type']) || ! in_array($property['type'], self::getValidTypes(), true)) {
+        if (! $this->validateProperty($name, $property)) {
             throw new \InvalidArgumentException(
                 'Invalid type '.var_export($property['type'], true)." for property '$name'. Must be one of: ".implode(', ', self::getValidTypes())
             );
@@ -89,6 +96,23 @@ public function getNames(): array
         return array_keys($this->properties);
     }
 
+    /**
+     * Validate a property
+     *
+     * @param  string  $name - name of the property
+     * @param  array<string, mixed>  $property - property definition (must be defined in JSON Schema format)
+     *
+     * @return bool
+     */
+    public function validateProperty(string $name, array $property): bool
+    {
+        if (! isset($property['type']) || ! in_array($property['type'], self::getValidTypes(), true)) {
+            return false;
+        }
+
+        return true;
+    }
+
     /**
      * @return array<int, string>
      */

tests/Agents/SchemaTest.php

@@ -24,9 +24,18 @@ class SchemaTest extends TestCase
     protected function setUp(): void
     {
         $this->object = new SchemaObject([
-            'id' => ['type' => SchemaObject::TYPE_STRING],
-            'name' => ['type' => SchemaObject::TYPE_STRING],
-            'age' => ['type' => SchemaObject::TYPE_INTEGER],
+            'id' => [
+                'type' => SchemaObject::TYPE_STRING,
+                'description' => 'The ID of the user',
+            ],
+            'name' => [
+                'type' => SchemaObject::TYPE_STRING,
+                'description' => 'The name of the user',
+            ],
+            'age' => [
+                'type' => SchemaObject::TYPE_INTEGER,
+                'description' => 'The age of the user',
+            ],
         ]);
         $this->schema = new Schema(
             $this->name,