https://codereview.appspot.com/7241062/diff/3001/juju/hooks/invoker.py#newcode255
juju/hooks/invoker.py:255:
list(set(idents).intersection(set(self._relation_contexts))))
It doesn't look like you test this method after its change to be a
generator. You previously returned the generator though it looks like so
it would have been yielded on anyway, I suspect this is still ok.
LGTM, subtle issue you're fixing there.
https:/ /codereview. appspot. com/7241062/ diff/3001/ juju/hooks/ invoker. py invoker. py (right):
File juju/hooks/
https:/ /codereview. appspot. com/7241062/ diff/3001/ juju/hooks/ invoker. py#newcode255 invoker. py:255: idents) .intersection( set(self. _relation_ contexts) )))
juju/hooks/
list(set(
It doesn't look like you test this method after its change to be a
generator. You previously returned the generator though it looks like so
it would have been yielded on anyway, I suspect this is still ok.
https:/ /codereview. appspot. com/7241062/