Raise errors by default

Disable by passing `catchErrors: true` to Robot

index.js

@@ -31,7 +31,7 @@ module.exports = (options = {}) => {
     debug: process.env.LOG_LEVEL === 'trace'
   });
   const server = createServer(webhook);
-  const robot = createRobot({app, webhook, cache, logger});
+  const robot = createRobot({app, webhook, cache, logger, catchErrors: true});
 
   // Forward webhooks to robot
   webhook.on('*', event => {

lib/robot.js

@@ -9,12 +9,12 @@ const Context = require('./context');
  * @property {logger} log - A logger
  */
 class Robot {
-  constructor({app, cache, logger, throwErrors} = {}) {
+  constructor({app, cache, logger, catchErrors} = {}) {
     this.events = new EventEmitter();
     this.app = app;
     this.cache = cache;
     this.log = wrapLogger(logger);
-    this.throwErrors = throwErrors;
+    this.catchErrors = catchErrors;
   }
 
   async receive(event) {
@@ -66,10 +66,9 @@ class Robot {
           const context = new Context(event, github);
           return callback(context, context /* DEPRECATED: for backward compat */);
         } catch (err) {
-          if (this.throwErrors) {
+          this.log.error(err);
+          if (!this.catchErrors) {
             throw err;
-          } else {
-            this.log.error(err);
           }
         }
       }

test/index.js

@@ -6,7 +6,7 @@ describe('Robot', () => {
   let event;
 
   beforeEach(() => {
-    robot = createRobot({throwErrors: true});
+    robot = createRobot();
     robot.auth = () => Promise.resolve({});
 
     event = {

test/robot.js

@@ -100,7 +100,11 @@ describe('Robot', function () {
         throw error;
       });
 
-      await robot.receive(event);
+      try {
+        await robot.receive(event);
+      } catch (err) {
+        // Expected
+      }
 
       expect(robot.log.error).toHaveBeenCalledWith(error);
     });