Stop stock update if MSI stock is null

src/Actions/UpdateMsiStock.php

@@ -23,6 +23,10 @@ public function update(MagentoStock $model): void
 
         $availableSources = $this->getAvailableSources();
 
+        if ($model->msi_stock === null) {
+            return;
+        }
+
         foreach ($model->msi_stock as $location => $quantity) {
             if (! in_array($location, $availableSources)) {
                 continue;

src/Contracts/RetrievesStockSkus.php

@@ -11,5 +11,5 @@ interface RetrievesStockSkus
     public function retrieveAll(): Enumerable;
 
     /** @return Enumerable<int, string> */
-    public function retrieveUpdated(Carbon $from = null): Enumerable;
+    public function retrieveUpdated(?Carbon $from = null): Enumerable;
 }

src/Exceptions/UpdateException.php

@@ -7,7 +7,7 @@
 
 class UpdateException extends Exception
 {
-    public function __construct(string $sku, string $message, public array $payload = [], Throwable $previous = null)
+    public function __construct(string $sku, string $message, public array $payload = [], ?Throwable $previous = null)
     {
         parent::__construct("Failed to update $sku: $message", 0, $previous);
     }

src/Models/MagentoStock.php

@@ -18,8 +18,8 @@
  * @property bool $backorders
  * @property bool $magento_backorders_enabled
  * @property int $quantity
- * @property array $msi_stock
- * @property array $msi_status
+ * @property ?array $msi_stock
+ * @property ?array $msi_status
  * @property bool $retrieve
  * @property bool $update
  * @property ?Carbon $last_retrieved

src/Retriever/DummySkuRetriever.php

@@ -13,7 +13,7 @@ public function retrieveAll(): Enumerable
         return collect(['::sku_1::', '::sku_2::']);
     }
 
-    public function retrieveUpdated(Carbon $from = null): Enumerable
+    public function retrieveUpdated(?Carbon $from = null): Enumerable
     {
         return collect(['::sku_1::']);
     }

tests/Actions/UpdateMsiStockTest.php

@@ -135,6 +135,27 @@ function (Request $request) use ($expectedPayload) {
         Event::assertDispatched(StockUpdatedEvent::class);
     }
 
+    public function test_it_stops_when_stockdata_is_empty(): void
+    {
+        Http::fake([
+            '*/rest/all/V1/inventory/source-items*' => Http::response(),
+        ]);
+
+        /** @var UpdateMsiStock $action */
+        $action = app(UpdateMsiStock::class);
+
+        /** @var MagentoStock $stock */
+        $stock = MagentoStock::query()->first();
+
+        $stock->msi_stock = null;
+
+        $stock->msi_status = null;
+
+        $action->update($stock);
+
+        Event::assertNotDispatched(StockUpdatedEvent::class);
+    }
+
     public function test_it_logs_error(): void
     {
         Http::fake([