fix(assets): omit phantom stock line for ASSIGNED assets

src/actions/import.actions.ts

@@ -324,7 +324,7 @@ export async function importItems(formData: ImportFormType) {
       assetId: newAsset?.id || undefined,
       quantity: stock || 1,
       type: assigned ? "ASSIGNMENT" : "IN",
-      itemId: newItem?.id || undefined,
+      itemId: assigned ? undefined : newItem?.id || undefined,
       personId: newPerson?.id || undefined,
     }
 

src/use-cases/asset.use-cases.ts

@@ -179,7 +179,7 @@ export async function createAssetUseCase(
 
       await MovementService.create(
         {
-          itemId,
+          ...(status === "ASSIGNED" ? {} : { itemId }),
           assetId: newAsset.id,
           quantity: 1,
           type: status === "ASSIGNED" ? "ASSIGNMENT" : "IN",

tests/integration/use-cases/asset.use-cases.test.ts

@@ -235,10 +235,8 @@ describe("asset use-cases", () => {
       assignmentId: assignment.id,
       performedById: actor.id,
     })
-    expect(movements[0].stockLines[0]).toMatchObject({
-      itemId: item.id,
-      stockDelta: -1,
-    })
+    expect(movements[0].stockLines).toEqual([])
+    expect(movements[0].assetLines).toHaveLength(1)
     expect(movements[0].assetLines[0]).toMatchObject({
       assetId: result.assetId,
     })