fix: Carbon was being altered by google/cloud-spanner (#264)

Author: taka-oyama

src/Connection.php

@@ -24,6 +24,7 @@
 use Colopl\Spanner\Query\Processor as QueryProcessor;
 use Colopl\Spanner\Schema\Builder as SchemaBuilder;
 use Colopl\Spanner\Schema\Grammar as SchemaGrammar;
+use DateTimeImmutable;
 use DateTimeInterface;
 use Exception;
 use Generator;
@@ -425,7 +426,11 @@ protected function prepareBinding(BaseQueryGrammar $grammar, mixed $value): mixe
         // date string. Each query grammar maintains its own date string format
         // so we'll just ask the grammar for the format to get from the date.
         if ($value instanceof DateTimeInterface) {
-            return new Timestamp($value);
+            // Since Timestamp::__toString calls setTimezone() on the DateTime object,
+            // we need to clone the DateTime object to avoid changing the original object.
+            return ($value instanceof DateTimeImmutable)
+                ? new Timestamp($value)
+                : new Timestamp(DateTimeImmutable::createFromInterface($value));
         }
 
         if (is_array($value)) {

tests/ConnectionTest.php

@@ -601,4 +601,12 @@ public function test_getSchemaGrammar(): void
         $conn->useDefaultSchemaGrammar();
         $this->assertSame('test_', $conn->getSchemaGrammar()->getTablePrefix());
     }
+
+    public function test_binding_mutable_carbon_doesnt_change_timezone(): void
+    {
+        $conn = $this->getDefaultConnection();
+        $now = now()->setTimezone('Asia/Tokyo');
+        $conn->select('SELECT ?', [$now]);
+        $this->assertSame('Asia/Tokyo', $now->getTimezone()->getName());
+    }
 }