Descriptionstate: avoid panic in watchers
If the state is closed underneath a watcher, the
underlying watcher.Watcher will die with no error.
That should not itself trigger a panic.
I have not added test coverage for every single
entry point that creates a given kind of watcher.
I feel that adds unnecessarily to the test burden, and
with this particular kind of test, inspecting the
test coverage shows clearly which watchers are
tested in this way.
https://code.launchpad.net/~rogpeppe/juju-core/380-avoid-watcher-panic/+merge/182915
(do not edit description out of merge proposal)
Patch Set 1 #Patch Set 2 : state: avoid panic in watchers #
Total comments: 6
Patch Set 3 : state: avoid panic in watchers #Patch Set 4 : state: avoid panic in watchers #Patch Set 5 : state: avoid panic in watchers #
MessagesTotal messages: 7
|