uber / tchannel-go / 1089

82%

dev: 88%

push

travis-ci

SubChannel.SetHandler: override SubChannel Handlers

Added `SubChannel.SetHandler` which makes it possible to set a catch-all
Handler for all requests to a SubChannel. Using `SetHandler()` makes
`Register()` useless and will cause a panic.

As part of this change:

- Removed redundant `handlerMap` from Channel. Always use SubChannel's handler
  instead.
- Changed `handlerMap` to be a map of method name to handler
  (`map[string]Handler`) rather than map of service name to map of method name
  to handler (`map[string]map[string]Handler`). SubChannel dispatch happens in
  a different part of the system.
- Made `readableFragment.done()` idempotent.
- Ensure that the read last fragment is released when
  `InboundCallResponse.SendSystemError()` is called.

57 of 57 new or added lines in 7 files covered. (100.0%)

3009 of 3648 relevant lines covered (82.48%)

3588.46 hits per line