Merge pull request #30 from justbetter/feature/fix-double-retrieve

Do not set update to false if it's currently true

Author: VincentBean

src/Actions/Retrieval/SaveStock.php

@@ -28,7 +28,7 @@ public function save(StockData $stock, bool $forceUpdate): void
         $model->retrieve = false;
         $model->last_retrieved = now();
 
-        $model->update = $forceUpdate || $model->checksum !== $stock->checksum();
+        $model->update = $forceUpdate || $model->checksum !== $stock->checksum() || $model->update;
         $model->checksum = $stock->checksum();
 
         $model->save();

tests/Actions/Retrieval/SaveStockTest.php

@@ -92,4 +92,29 @@ public function it_can_force_update(): void
 
         $this->assertTrue($model->refresh()->update);
     }
+
+    #[Test]
+    public function it_does_not_reset_update_boolean(): void
+    {
+        $stockData = StockData::of([
+            'sku' => '::sku::',
+            'in_stock' => true,
+            'quantity' => 10,
+        ]);
+
+        /** @var SaveStock $action */
+        $action = app(SaveStock::class);
+        $action->save($stockData, false);
+
+        /** @var Stock $model */
+        $model = Stock::query()->firstWhere('sku', '=', '::sku::');
+
+        $this->assertTrue($model->update);
+
+        $model->update(['update' => true]);
+
+        $action->save($stockData, false);
+
+        $this->assertTrue($model->refresh()->update);
+    }
 }