NGCP / communication-js / 75 / 1

97%

master: 97%

push

travis-ci-com

Add UpdateHanlder#clearHandlers and simplify logic

 ## Why is the change being made?

This change is made because UpdateHandler requires a clearHandlers
function to remove all handlers (used from a required function
Messenger#stopSendingMessages).

 ## What has changed to address the problem?

This change adds the clearHandlers function to UpdateHandlers and adds
unit testing for it. This change also simplifies a lot of logic
concerning deleting handlers explicitly, and just calls the
handler.remove function instead.

This change also adds a throw statement if a handler is deleted but the
event does not exist.

 ## How was this change tested?

This change was tested with `npm test`.

 ## Related documents, URLs, commits

84 of 84 branches covered (100.0%)

Branch coverage included in aggregate %.

47 of 47 relevant lines covered (100.0%)

8.85 hits per line