Explicitly check for missing endpoints

[rgov]

The ModbusFunction.execute implementations invoke the callable returned by route_map.match(). The code assumes that if no match is found, then when calling the resulting None object, a TypeError will be thrown, and so it catches the exception and throws a IllegalDataAddressError instead.

However, by handling all TypeError exceptions the code is disguising TypeErrors thrown in other conditions. This is too general an exception when the code is really trying to handle one specific case. For instance, it hides the fact that a route's endpoint function might be declared with the wrong number of arguments.

This patch changes those exception handlers to simply an if statement. As a bonus it reduces indentation and reduces the (cyclomatic) complexity of the code; there is no longer the potential for a sudden jump to an exception handler out of a loop.

[coveralls]

Coverage decreased (-0.04%) to 95.824% when pulling 7527a56 on rgov:rzg/fix-exceptions into dcf35a1 on AdvancedClimateSystems:master.

[OrangeTux]

Many thanks for the PR! Also I like your good exploitation.