fix: add setter/getter for limit/offset to avoid refelction usage

Author: larskemper

src/Migration/MigrationContext.php

@@ -26,8 +26,8 @@ public function __construct(
         private ?GatewayInterface $gateway = null,
         private ?DataSet $dataSet = null,
         private readonly string $runUuid = '',
-        private readonly int $offset = 0,
-        private readonly int $limit = 0,
+        private int $offset = 0,
+        private int $limit = 0,
     ) {
     }
 
@@ -89,8 +89,18 @@ public function getOffset(): int
         return $this->offset;
     }
 
+    public function setOffset(int $offset): void
+    {
+        $this->offset = $offset;
+    }
+
     public function getLimit(): int
     {
         return $this->limit;
     }
+
+    public function setLimit(int $limit): void
+    {
+        $this->limit = $limit;
+    }
 }

src/Migration/MigrationContextInterface.php

@@ -34,5 +34,9 @@ public function getDataSet(): ?DataSet;
 
     public function getOffset(): int;
 
+    public function setOffset(int $offset): void;
+
     public function getLimit(): int;
+
+    public function setLimit(int $limit): void;
 }

tests/LocalConnectionTestCase.php

@@ -70,9 +70,8 @@ abstract protected function getDataSet(): DataSet;
 
     protected function setLimitAndOffset(int $limit, int $offset): void
     {
-        $reflectionClass = new \ReflectionClass($this->getMigrationContext());
-        (new \ReflectionProperty($reflectionClass->getName(), 'limit'))->setValue($this->getMigrationContext(), $limit);
-        (new \ReflectionProperty($reflectionClass->getName(), 'offset'))->setValue($this->getMigrationContext(), $offset);
+        $this->getMigrationContext()->setLimit($limit);
+        $this->getMigrationContext()->setOffset($offset);
     }
 
     protected function getExternalConnection(): Connection