feat: add onAny and onError methods from @octokit/webhooks (#1480)

Co-authored-by: Gregor Martynus <39992+gr2m@users.noreply.github.com>

README.md

@@ -21,6 +21,14 @@ module.exports = (app) => {
     });
     return context.octokit.issues.createComment(issueComment);
   });
+
+  app.onAny(async (context) => {
+    context.log.info({ event: context.name, action: context.payload.action });
+  });
+
+  app.onError(async (error) => {
+    context.log.error(error);
+  });
 };
 ```
 

docs/webhooks.md

@@ -40,11 +40,11 @@ module.exports = (app) => {
 };
 ```
 
-You can also use `app.webhooks.onAny()` to listen for any event that your app is subscribed to:
+You can also use `app.onAny()` to listen for any event that your app is subscribed to:
 
 ```js
 module.exports = (app) => {
-  app.webhooks.onAny(async (context) => {
+  app.onAny(async (context) => {
     context.log.info({ event: context.name, action: context.payload.action });
   });
 };

src/probot.ts

@@ -36,6 +36,8 @@ export class Probot {
   public log: DeprecatedLogger;
   public version: String;
   public on: ProbotWebhooks["on"];
+  public onAny: ProbotWebhooks["onAny"];
+  public onError: ProbotWebhooks["onError"];
   public auth: (
     installationId?: number,
     log?: Logger
@@ -100,6 +102,9 @@ export class Probot {
       return this.webhooks.on(eventNameOrNames, callback);
     };
 
+    this.onAny = this.webhooks.onAny;
+    this.onError = this.webhooks.onError;
+
     this.version = VERSION;
   }
 

test/probot.test.ts

@@ -330,6 +330,19 @@ describe("Probot", () => {
       expect(spy).toHaveBeenCalled();
     });
 
+    it("calls callback with onAny", async () => {
+      const probot = new Probot({
+        appId,
+        privateKey,
+      });
+
+      const spy = jest.fn();
+      probot.onAny(spy);
+
+      await probot.receive(event);
+      expect(spy).toHaveBeenCalled();
+    });
+
     it("calls callback x amount of times when an array of x actions is passed", async () => {
       const probot = new Probot({
         appId,