Change the message of "error.consumption.validation.inheritedFromItem" when using canCreate

packages/consumption/src/consumption/ConsumptionCoreErrors.ts

@@ -306,6 +306,10 @@ class Requests {
         return new CoreError("error.consumption.requests.missingRelationship", message);
     }
 
+    public inheritedFromItem(message: string) {
+        return new ApplicationError("error.consumption.requests.validation.inheritedFromItem", message);
+    }
+
     private static readonly _decideValidation = class {
         public invalidNumberOfItems(message: string) {
             return new ApplicationError("error.consumption.requests.decide.validation.invalidNumberOfItems", message);

packages/consumption/src/modules/common/ValidationResult.ts

@@ -1,4 +1,5 @@
 import { ApplicationError } from "@js-soft/ts-utils";
+import { ConsumptionCoreErrors } from "../../consumption/ConsumptionCoreErrors";
 
 export abstract class ValidationResult {
     protected constructor(public readonly items: ValidationResult[]) {}
@@ -21,13 +22,7 @@ export abstract class ValidationResult {
 
     public static fromItems(items: ValidationResult[]): ValidationResult {
         return items.some((r) => r.isError())
-            ? ValidationResult.error(
-                  new ApplicationError(
-                      "error.consumption.validation.inheritedFromItem",
-                      "Some child items have errors. If this error occurred during the specification of a Request, call 'canCreate' to get more information."
-                  ),
-                  items
-              )
+            ? ValidationResult.error(ConsumptionCoreErrors.requests.inheritedFromItem("Some child items have errors."), items)
             : ValidationResult.success(items);
     }
 }

packages/consumption/src/modules/requests/outgoing/OutgoingRequestsController.ts

@@ -115,7 +115,12 @@ export class OutgoingRequestsController extends ConsumptionBaseController {
     private async _create(id: CoreId, content: Request, peer: CoreAddress) {
         const canCreateResult = await this.canCreate({ content, peer });
 
-        if (canCreateResult.isError()) throw canCreateResult.error;
+        if (canCreateResult.isError()) {
+            const error = ConsumptionCoreErrors.requests.inheritedFromItem("Some child items have errors. Call 'canCreate' to get more information.");
+            if (canCreateResult.error.equals(error)) throw error;
+
+            throw canCreateResult.error;
+        }
 
         const request = LocalRequest.from({
             id: id,

packages/consumption/test/modules/requests/DecideRequestParametersValidator.test.ts

@@ -272,8 +272,8 @@ describe("DecideRequestParametersValidator", function () {
             },
             expectedError: {
                 indexPath: [0],
-                code: "error.consumption.validation.inheritedFromItem",
-                message: "Some child items have errors. If this error occurred during the specification of a Request, call 'canCreate' to get more information."
+                code: "error.consumption.requests.validation.inheritedFromItem",
+                message: "Some child items have errors."
             }
         }
     ];
@@ -317,7 +317,7 @@ describe("DecideRequestParametersValidator", function () {
         }
 
         expect(validationResult).errorValidationResult({
-            code: "error.consumption.validation.inheritedFromItem"
+            code: "error.consumption.requests.validation.inheritedFromItem"
         });
 
         let childResult = validationResult;

packages/consumption/test/modules/requests/IncomingRequestsController.test.ts

@@ -364,15 +364,15 @@ describe("IncomingRequestsController", function () {
             });
 
             expect(validationResult).errorValidationResult({
-                code: "error.consumption.validation.inheritedFromItem",
-                message: "Some child items have errors. If this error occurred during the specification of a Request, call 'canCreate' to get more information."
+                code: "error.consumption.requests.validation.inheritedFromItem",
+                message: "Some child items have errors."
             });
             expect(validationResult.items).toHaveLength(2);
 
             expect(validationResult.items[0].isError()).toBe(false);
 
             expect(validationResult.items[1].isError()).toBe(true);
-            expect(validationResult.items[1]).errorValidationResult({ code: "error.consumption.validation.inheritedFromItem" });
+            expect(validationResult.items[1]).errorValidationResult({ code: "error.consumption.requests.validation.inheritedFromItem" });
 
             expect(validationResult.items[1].items).toHaveLength(3);
             expect(validationResult.items[1].items[0].isError()).toBe(true);
@@ -558,15 +558,15 @@ describe("IncomingRequestsController", function () {
             const validationResult = await When.iCallCanRejectWith(rejectParams);
 
             expect(validationResult).errorValidationResult({
-                code: "error.consumption.validation.inheritedFromItem",
-                message: "Some child items have errors. If this error occurred during the specification of a Request, call 'canCreate' to get more information."
+                code: "error.consumption.requests.validation.inheritedFromItem",
+                message: "Some child items have errors."
             });
             expect(validationResult.items).toHaveLength(2);
 
             expect(validationResult.items[0].isError()).toBe(false);
 
             expect(validationResult.items[1].isError()).toBe(true);
-            expect(validationResult.items[1]).errorValidationResult({ code: "error.consumption.validation.inheritedFromItem" });
+            expect(validationResult.items[1]).errorValidationResult({ code: "error.consumption.requests.validation.inheritedFromItem" });
 
             expect(validationResult.items[1].items).toHaveLength(3);
             expect(validationResult.items[1].items[0].isError()).toBe(true);

packages/consumption/test/modules/requests/OutgoingRequestsController.test.ts

@@ -151,8 +151,8 @@ describe("OutgoingRequestsController", function () {
                 }
             });
             expect(validationResult).errorValidationResult({
-                code: "error.consumption.validation.inheritedFromItem",
-                message: "Some child items have errors. If this error occurred during the specification of a Request, call 'canCreate' to get more information."
+                code: "error.consumption.requests.validation.inheritedFromItem",
+                message: "Some child items have errors."
             });
             expect(validationResult.items).toHaveLength(2);
 
@@ -184,18 +184,16 @@ describe("OutgoingRequestsController", function () {
                 }
             });
             expect(validationResult).errorValidationResult({
-                code: "error.consumption.validation.inheritedFromItem",
-                message: "Some child items have errors. If this error occurred during the specification of a Request, call 'canCreate' to get more information."
+                code: "error.consumption.requests.validation.inheritedFromItem",
+                message: "Some child items have errors."
             });
             expect(validationResult.items).toHaveLength(2);
 
             expect(validationResult.items[0].isError()).toBe(false);
 
             expect(validationResult.items[1].isError()).toBe(true);
-            expect((validationResult.items[1] as ErrorValidationResult).error.code).toBe("error.consumption.validation.inheritedFromItem");
-            expect((validationResult.items[1] as ErrorValidationResult).error.message).toBe(
-                "Some child items have errors. If this error occurred during the specification of a Request, call 'canCreate' to get more information."
-            );
+            expect((validationResult.items[1] as ErrorValidationResult).error.code).toBe("error.consumption.requests.validation.inheritedFromItem");
+            expect((validationResult.items[1] as ErrorValidationResult).error.message).toBe("Some child items have errors.");
 
             expect(validationResult.items[1].items).toHaveLength(1);
             expect(validationResult.items[1].items[0].isError()).toBe(true);
@@ -227,6 +225,11 @@ describe("OutgoingRequestsController", function () {
             await Then.itThrowsAnErrorWithTheErrorMessage("*content*Value is not defined*");
         });
 
+        test("throws that it is necessary to call 'canCreate' when at least one RequestItem is invalid", async function () {
+            await When.iTryToCreateAnOutgoingRequestWithIncorrectRequestItem();
+            await Then.itThrowsAnErrorWithTheErrorMessage("Some child items have errors. Call 'canCreate' to get more information.");
+        });
+
         test("throws when canCreate returns an error", async function () {
             const oldCanCreate = context.outgoingRequestsController.canCreate;
             context.outgoingRequestsController.canCreate = (_: ICreateOutgoingRequestParameters) => {

packages/consumption/test/modules/requests/RequestsIntegrationTest.ts

@@ -936,6 +936,30 @@ export class RequestsWhen {
         return Promise.resolve();
     }
 
+    public iTryToCreateAnOutgoingRequestWithIncorrectRequestItem(): Promise<void> {
+        const params: ICreateOutgoingRequestParameters = {
+            content: {
+                items: [
+                    TestRequestItem.from({
+                        mustBeAccepted: false,
+                        shouldFailAtCanCreateOutgoingRequestItem: true
+                    }),
+                    TestRequestItem.from({
+                        mustBeAccepted: false,
+                        shouldFailAtCanCreateOutgoingRequestItem: true
+                    })
+                ]
+            },
+            peer: CoreAddress.from("did:e:a-domain:dids:anidentity")
+        };
+
+        this.context.actionToTry = async () => {
+            await this.context.outgoingRequestsController.create(params as any);
+        };
+
+        return Promise.resolve();
+    }
+
     public iTryToCreateAnOutgoingRequestFromRelationshipTemplateResponseWithoutResponseSource(): Promise<void> {
         const paramsWithoutResponseSource: Omit<ICreateAndCompleteOutgoingRequestFromRelationshipTemplateResponseParameters, "responseSource"> = {
             response: TestObjectFactory.createResponse(),