Update state (#29)

Author: ramonrietdijk

src/Actions/Update/Sync/UpdateStock.php

@@ -49,6 +49,8 @@ public function update(Stock $stock): void
         $stock->update([
             'update' => false,
             'last_updated' => now(),
+            'last_failed' => null,
+            'fail_count' => 0,
         ]);
 
         event(new StockUpdatedEvent($stock));

src/Listeners/BulkOperationStatusListener.php

@@ -18,7 +18,11 @@ public function execute(BulkOperation $operation): void
         $stock = $operation->subject;
 
         if ($operation->status === OperationStatus::Complete) {
-            $stock->update(['last_updated' => now()]);
+            $stock->update([
+                'last_updated' => now(),
+                'last_failed' => null,
+                'fail_count' => 0,
+            ]);
 
             event(new StockUpdatedEvent($stock));
 

tests/Actions/Update/Sync/UpdateStockTest.php

@@ -38,6 +38,7 @@ public function it_resets_update_when_product_does_not_exist(): void
         $model->refresh();
 
         $this->assertFalse($model->update);
+        $this->assertNull($model->last_updated);
     }
 
     #[Test]
@@ -62,6 +63,8 @@ public function it_updates_simple_stock_and_backorders(): void
         $model = Stock::query()->create([
             'sku' => '::sku::',
             'update' => true,
+            'last_failed' => now(),
+            'fail_count' => 1,
         ]);
 
         /** @var UpdateStock $action */
@@ -72,6 +75,8 @@ public function it_updates_simple_stock_and_backorders(): void
 
         $this->assertFalse($model->update);
         $this->assertNotNull($model->last_updated);
+        $this->assertNull($model->last_failed);
+        $this->assertEquals(0, $model->fail_count);
 
         Event::assertDispatched(StockUpdatedEvent::class);
     }
@@ -95,6 +100,8 @@ public function it_updates_msi_stock(): void
         $model = Stock::query()->create([
             'sku' => '::sku::',
             'update' => true,
+            'last_failed' => now(),
+            'fail_count' => 1,
         ]);
 
         /** @var UpdateStock $action */
@@ -105,6 +112,8 @@ public function it_updates_msi_stock(): void
 
         $this->assertFalse($model->update);
         $this->assertNotNull($model->last_updated);
+        $this->assertNull($model->last_failed);
+        $this->assertEquals(0, $model->fail_count);
 
         Event::assertDispatched(StockUpdatedEvent::class);
     }

tests/Listeners/BulkOperationStatusListenerTest.php

@@ -24,6 +24,8 @@ public function it_handles_complete_status(): void
             'sku' => 'sku',
             'in_stock' => true,
             'quantity' => 5,
+            'last_failed' => now(),
+            'fail_count' => 1,
         ]);
 
         /** @var BulkRequest $request */
@@ -47,11 +49,15 @@ public function it_handles_complete_status(): void
 
         /** @var BulkOperationStatusListener $listener */
         $listener = app(BulkOperationStatusListener::class);
-
         $listener->execute($operation);
 
+        $model->refresh();
+
+        $this->assertNotNull($model->last_updated);
+        $this->assertNull($model->last_failed);
+        $this->assertEquals(0, $model->fail_count);
+
         Event::assertDispatched(StockUpdatedEvent::class);
-        $this->assertNotNull($model->refresh()->last_updated);
     }
 
     #[Test]
@@ -87,10 +93,12 @@ public function it_handles_failed_status(): void
 
         /** @var BulkOperationStatusListener $listener */
         $listener = app(BulkOperationStatusListener::class);
-
         $listener->execute($operation);
 
+        $model->refresh();
+
+        $this->assertNull($model->last_updated);
+
         Event::assertNotDispatched(StockUpdatedEvent::class);
-        $this->assertNull($model->refresh()->last_updated);
     }
 }