test: log warning instead of throwing error for unknown events

Author: gr2m

test/unit/event-handler-on-test.js

@@ -1,3 +1,4 @@
+const simple = require('simple-mock')
 const test = require('tap').test
 
 const receiverOn = require('../../event-handler/on')
@@ -9,8 +10,11 @@ const state = {
 }
 
 test('receiver.on with invalid event name', t => {
-  t.throws(() => {
-    receiverOn(state, 'foo', noop)
-  })
+  simple.mock(console, 'warn').callFn(function () {})
+  receiverOn(state, 'foo', noop)
+  t.equals(console.warn.callCount, 1)
+  t.equals(console.warn.lastCall.arg, '"foo" is not a known webhook name (https://developer.github.com/v3/activity/events/types/)')
+
+  simple.restore()
   t.end()
 })