fix: Produkt-Formular ueberschreibt keinen zwischenzeitlich gebuchten Bestand
Das Speichern schrieb den Formularwert immer fest in den Lagerbestand. Wurde zwischen Oeffnen und Speichern ein Glas verkauft, kam es dadurch wieder in den Bestand, ohne dass es je zurueckkam. - Das Formular schickt den Bestand beim Oeffnen mit (quantityLoaded) - Feld unveraendert: Bestand wird nicht angefasst - Feld geaendert: Schreiben nur, wenn der Bestand noch dem geladenen Stand entspricht (Bedingung atomar im UPDATE); sonst wird nichts gespeichert und der aktuelle Wert angezeigt - Fehlt quantityLoaded (alter Tab), muss neu geladen werden Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
This commit is contained in:
1 parent
5848eb6ad5
commit
d52a762e8a
4 files changed
+133
-26
No files matched your search
@@ -1,5 +1,32 @@
|
||||
import { describe, test, expect } from "vitest";
|
||||
import { needsRestockOnCancel } from "./inventory";
|
||||
import { needsRestockOnCancel, planInventoryEdit } from "./inventory";
|
||||
|
||||
describe("planInventoryEdit", () => {
|
||||
test("laesst den Bestand unberuehrt, wenn das Feld nicht geaendert wurde", () => {
|
||||
// Zwischen Oeffnen und Speichern verkaufte Glaeser duerfen nicht
|
||||
// mit dem alten Formularwert ueberschrieben werden.
|
||||
expect(planInventoryEdit({ loaded: "105", submitted: 105 })).toEqual({ kind: "keep" });
|
||||
});
|
||||
|
||||
test("setzt den Bestand nur unter der Bedingung, dass er noch dem geladenen Wert entspricht", () => {
|
||||
expect(planInventoryEdit({ loaded: "105", submitted: 120 })).toEqual({
|
||||
kind: "set",
|
||||
expected: 105,
|
||||
quantity: 120,
|
||||
});
|
||||
});
|
||||
|
||||
test("verlangt ein Neuladen, wenn der geladene Wert fehlt", () => {
|
||||
// Formular aus einem Tab, der vor dem Deployment geoeffnet wurde.
|
||||
expect(planInventoryEdit({ loaded: "", submitted: 105 })).toEqual({ kind: "reload" });
|
||||
});
|
||||
|
||||
test("verlangt ein Neuladen bei einem manipulierten geladenen Wert", () => {
|
||||
expect(planInventoryEdit({ loaded: "-1", submitted: 105 })).toEqual({ kind: "reload" });
|
||||
expect(planInventoryEdit({ loaded: "1.5", submitted: 105 })).toEqual({ kind: "reload" });
|
||||
expect(planInventoryEdit({ loaded: "abc", submitted: 105 })).toEqual({ kind: "reload" });
|
||||
});
|
||||
});
|
||||
|
||||
describe("needsRestockOnCancel", () => {
|
||||
test("bucht zurueck, wenn es keine Retoure gibt", () => {
|
||||
|
||||
Reference in new issue
Block a user