djkato/saleor-app-sdk-REDIS_APL
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Add req and res to onError callback for AsyncWebhook (#180)

Browse source
This commit is contained in:
Lukasz Ostrowski 2023-02-15 11:08:09 +01:00 committed by GitHub
parent dffd4f5318
commit 4f9c9c2816
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 6 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/handlers/next/saleor-async-webhook.test.ts
View file

@ -160,14 +160,12 @@ describe("SaleorAsyncWebhook", () => {
*/
expect(res.statusCode).toBe(401);
expect(res._getData()).toBe("My Body");
/**
* TODO This assertion fails, due to WebhookError constructor:
* [TypeError: Class constructor WebhookError cannot be invoked without 'new']
*/
expect(onErrorCallback).toHaveBeenCalledWith(
expect.objectContaining({
message: "Test error message",
})
}),
req,
res
);
/**

6
src/handlers/next/saleor-async-webhook.ts
View file

@ -20,7 +20,7 @@ interface WebhookManifestConfigurationBase {
asyncEvent: AsyncWebhookEventType;
isActive?: boolean;
apl: APL;
onError?(error: WebhookError | Error): void;
onError?(error: WebhookError | Error, req: NextApiRequest, res: NextApiResponse): void;
formatErrorResponse?(
error: WebhookError | Error,
req: NextApiRequest,
@ -163,7 +163,7 @@ export class SaleorAsyncWebhook<TPayload = unknown> {
debug(`Validation error: ${e.message}`);
if (this.onError) {
this.onError(e);
this.onError(e, req, res);
}
if (this.formatErrorResponse) {
@ -185,7 +185,7 @@ export class SaleorAsyncWebhook<TPayload = unknown> {
debug("Unexpected error: %O", e);
if (this.onError) {
this.onError(e);
this.onError(e, req, res);
}
if (this.formatErrorResponse) {