erpnext-queue-client 2.9.2 → 2.9.4

package/dist/erpnext/doctypeSubmittableResourceRequest.js

@@ -5,6 +5,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ERPNextDoctypeSubmittableResourceRequest = void 0;
 const zod_1 = __importDefault(require("zod"));
+const logger_1 = require("../utils/logger");
 const doctypeResourceRequest_1 = require("./doctypeResourceRequest");
 const ERPNextDocTypeMeta_1 = require("./model/ERPNextDocTypeMeta");
 class ERPNextDoctypeSubmittableResourceRequest extends doctypeResourceRequest_1.ERPNextDoctypeResourceRequest {
@@ -15,15 +16,27 @@ class ERPNextDoctypeSubmittableResourceRequest extends doctypeResourceRequest_1.
         super(temporalClient, resourceName, mergedModel);
     }
     async cancel({ resourceId, }) {
-        return this.updateById({
-            resourceId,
-            inputValidationModel: zod_1.default
-                .object({ docstatus: ERPNextDocTypeMeta_1.Docstatus })
-                .describe("Cancel Input"),
-            body: {
-                docstatus: 2,
-            },
-        });
+        try {
+            return await this.updateById({
+                resourceId,
+                inputValidationModel: zod_1.default
+                    .object({ docstatus: ERPNextDocTypeMeta_1.Docstatus })
+                    .describe("Cancel Input"),
+                body: {
+                    docstatus: 2,
+                },
+            });
+        }
+        catch (err) {
+            const message = err instanceof Error ? err.message : String(err);
+            if (!message.includes("Cannot edit cancelled document"))
+                throw err;
+            const doc = await this.getById({ resourceId });
+            if (doc?.docstatus !== 2)
+                throw err;
+            logger_1.lg.info(`${this.resourceName} ${resourceId} is already cancelled`);
+            return doc;
+        }
     }
     async submit({ resourceId, }) {
         return this.updateById({

package/dist/erpnext/doctypeSubmittableResourceRequest.test.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/erpnext/doctypeSubmittableResourceRequest.test.js

@@ -0,0 +1,64 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const zod_1 = require("zod");
+const doctypeSubmittableResourceRequest_1 = require("./doctypeSubmittableResourceRequest");
+const TestModel = zod_1.z.object({ name: zod_1.z.string() });
+const cancelledDocError = new Error('Error 417: {"exception":"frappe.exceptions.ValidationError: Cannot edit cancelled document","exc_type":"ValidationError"}');
+function createRequest(executeERPNextRequestWorkflow) {
+    return new doctypeSubmittableResourceRequest_1.ERPNextDoctypeSubmittableResourceRequest({ executeERPNextRequestWorkflow }, "Purchase Receipt", TestModel);
+}
+describe("cancel", () => {
+    test("returns the updated document when the cancel request succeeds", async () => {
+        const doc = { name: "PR-1", docstatus: 2 };
+        const executeERPNextRequestWorkflow = vi
+            .fn()
+            .mockResolvedValue({ data: doc });
+        const request = createRequest(executeERPNextRequestWorkflow);
+        const result = await request.cancel({ resourceId: "PR-1" });
+        expect(result).toEqual(doc);
+        expect(executeERPNextRequestWorkflow).toHaveBeenCalledTimes(1);
+        const options = executeERPNextRequestWorkflow.mock.calls[0]?.[1];
+        expect(options.requestMethod).toBe("PUT");
+        expect(options.body).toEqual({ docstatus: 2 });
+    });
+    test("treats an already cancelled document as success", async () => {
+        const doc = { name: "PR-1", docstatus: 2 };
+        const executeERPNextRequestWorkflow = vi
+            .fn()
+            .mockRejectedValueOnce(cancelledDocError)
+            .mockResolvedValueOnce({ data: doc });
+        const request = createRequest(executeERPNextRequestWorkflow);
+        const result = await request.cancel({ resourceId: "PR-1" });
+        expect(result).toEqual(doc);
+        expect(executeERPNextRequestWorkflow).toHaveBeenCalledTimes(2);
+        const getOptions = executeERPNextRequestWorkflow.mock.calls[1]?.[1];
+        expect(getOptions.requestMethod).toBe("GET");
+        expect(getOptions.resourceId).toBe("PR-1");
+    });
+    test("rethrows when the document is not cancelled after the error", async () => {
+        const executeERPNextRequestWorkflow = vi
+            .fn()
+            .mockRejectedValueOnce(cancelledDocError)
+            .mockResolvedValueOnce({ data: { name: "PR-1", docstatus: 1 } });
+        const request = createRequest(executeERPNextRequestWorkflow);
+        await expect(request.cancel({ resourceId: "PR-1" })).rejects.toThrow("Cannot edit cancelled document");
+        expect(executeERPNextRequestWorkflow).toHaveBeenCalledTimes(2);
+    });
+    test("rethrows when the document no longer exists", async () => {
+        const executeERPNextRequestWorkflow = vi
+            .fn()
+            .mockRejectedValueOnce(cancelledDocError)
+            .mockRejectedValueOnce(new Error("Error 404: Not found"));
+        const request = createRequest(executeERPNextRequestWorkflow);
+        await expect(request.cancel({ resourceId: "PR-1" })).rejects.toThrow("Cannot edit cancelled document");
+        expect(executeERPNextRequestWorkflow).toHaveBeenCalledTimes(2);
+    });
+    test("rethrows unrelated errors without checking the document", async () => {
+        const executeERPNextRequestWorkflow = vi
+            .fn()
+            .mockRejectedValueOnce(new Error("Error 500: Internal Server Error"));
+        const request = createRequest(executeERPNextRequestWorkflow);
+        await expect(request.cancel({ resourceId: "PR-1" })).rejects.toThrow("Error 500");
+        expect(executeERPNextRequestWorkflow).toHaveBeenCalledTimes(1);
+    });
+});