fix(database): support virtual properties

Closes #1474

Author: innocenzi

packages/database/src/Builder/QueryBuilders/InsertQueryBuilder.php

@@ -12,6 +12,7 @@
 use Tempest\Database\PrimaryKey;
 use Tempest\Database\Query;
 use Tempest\Database\QueryStatements\InsertStatement;
+use Tempest\Database\Virtual;
 use Tempest\Intl;
 use Tempest\Mapper\SerializerFactory;
 use Tempest\Reflection\ClassReflector;
@@ -390,6 +391,11 @@ private function resolveObjectData(object $model): array
             }
 
             $propertyName = $property->getName();
+
+            if ($property->hasAttribute(Virtual::class)) {
+                continue;
+            }
+
             $value = $property->getValue($model);
 
             if ($definition->getHasMany($propertyName)) {

packages/database/src/Builder/QueryBuilders/UpdateQueryBuilder.php

@@ -14,6 +14,7 @@
 use Tempest\Database\QueryStatements\HasWhereStatements;
 use Tempest\Database\QueryStatements\UpdateStatement;
 use Tempest\Database\QueryStatements\WhereStatement;
+use Tempest\Database\Virtual;
 use Tempest\Intl;
 use Tempest\Mapper\SerializerFactory;
 use Tempest\Reflection\ClassReflector;
@@ -473,6 +474,10 @@ private function convertObjectToArray(object $object): array
         $reflection = new ClassReflector($object);
 
         foreach ($reflection->getPublicProperties() as $property) {
+            if ($property->hasAttribute(Virtual::class)) {
+                continue;
+            }
+
             if ($property->isInitialized($object)) {
                 $result[$property->getName()] = $property->getValue($object);
             }

packages/database/src/IsDatabaseModel.php

@@ -9,6 +9,7 @@
 use Tempest\Database\Builder\QueryBuilders\SelectQueryBuilder;
 use Tempest\Database\Exceptions\RelationWasMissing;
 use Tempest\Database\Exceptions\ValueWasMissing;
+use Tempest\Database\Virtual;
 use Tempest\Reflection\ClassReflector;
 use Tempest\Reflection\PropertyReflector;
 
@@ -175,6 +176,10 @@ public function refresh(): self
         $refreshed = self::find(id: $primaryKeyValue)->first();
 
         foreach (new ClassReflector($refreshed)->getPublicProperties() as $property) {
+            if ($property->hasAttribute(Virtual::class)) {
+                continue;
+            }
+
             $property->setValue($this, $property->getValue($refreshed));
         }
 
@@ -198,6 +203,10 @@ public function load(string ...$relations): self
         $new = self::get($primaryKeyValue, $relations);
 
         foreach (new ClassReflector($new)->getPublicProperties() as $property) {
+            if ($property->hasAttribute(Virtual::class)) {
+                continue;
+            }
+
             $property->setValue($this, $property->getValue($new));
         }
 

packages/database/src/Virtual.php

@@ -6,6 +6,9 @@
 
 use Attribute;
 
+/**
+ * Virtual properties are ignored by the database mapper.
+ */
 #[Attribute(Attribute::TARGET_PROPERTY)]
 final readonly class Virtual
 {

tests/Integration/Database/Builder/IsDatabaseModelTest.php

@@ -433,7 +433,25 @@ public function test_no_result(): void
         $this->assertNull(A::select()->first());
     }
 
-    public function test_virtual_property(): void
+    public function test_create_with_virtual_property(): void
+    {
+        $this->migrate(
+            CreateMigrationsTable::class,
+            CreateATable::class,
+            CreateBTable::class,
+            CreateCTable::class,
+        );
+
+        $a = AWithVirtual::create(
+            b: new B(
+                c: new C(name: 'test'),
+            ),
+        );
+
+        $this->assertSame(-$a->id->value, $a->fake);
+    }
+
+    public function test_select_virtual_property(): void
     {
         $this->migrate(
             CreateMigrationsTable::class,
@@ -453,6 +471,36 @@ public function test_virtual_property(): void
         $this->assertSame(-$a->id->value, $a->fake);
     }
 
+    public function test_update_with_virtual_property(): void
+    {
+        $this->migrate(
+            CreateMigrationsTable::class,
+            CreateATable::class,
+            CreateBTable::class,
+            CreateCTable::class,
+        );
+
+        $a = AWithVirtual::create(
+            b: new B(
+                c: new C(name: 'test'),
+            ),
+        );
+
+        $a->update(
+            b: new B(
+                c: new C(name: 'updated'),
+            ),
+        );
+
+        $updatedA = AWithVirtual::select()
+            ->with('b.c')
+            ->where('id', $a->id)
+            ->first();
+
+        $this->assertSame(-$updatedA->id->value, $updatedA->fake);
+        $this->assertSame('updated', $updatedA->b->c->name);
+    }
+
     public function test_update_or_create(): void
     {
         $this->migrate(