Skip to content

Commit

Permalink
Fix join request multi-status error messge (#567)
Browse files Browse the repository at this point in the history
* Fix join request multi-status error messge

* Update test
  • Loading branch information
lounjukk authored Feb 3, 2025
1 parent 49a0d69 commit 694b8a7
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
4 changes: 3 additions & 1 deletion src/BusinessService.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,9 @@ describe(".business.*", () => {
} catch (e: unknown) {
if (e instanceof MultiStatusException) {
expect(e).toBeInstanceOf(MultiStatusException);
expect(e.message).toEqual("Failed to update 2 out of 3 join requests");
expect(e.message).toEqual(
"Failed to update 2 out of 3 join requests. Please try again later.",
);
expect((e.rawException as any)?.body).toEqual({
error: "Some join requests could not be processed",
"multi-status": [
Expand Down
2 changes: 1 addition & 1 deletion src/BusinessService.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1583,7 +1583,7 @@ class BusinessService extends Base {
const multiStatus = response["multi-status"];
const failed = multiStatus.filter((e) => e.status !== "success");

message = `Failed to update ${failed.length} out of ${multiStatus.length} join requests`;
message = `Failed to update ${failed.length} out of ${multiStatus.length} join requests. Please try again later.`;
}

return new MultiStatusException(message, error);
Expand Down

0 comments on commit 694b8a7

Please sign in to comment.