On 2012/07/01 02:32:22, hazmat wrote:
> this test also seems superfluous. the only impl file being modified by
the
> branch is the invoker, yet we have tests in numerous state test files
verifying
> what has already been verified by other tests in those files
https:/ /codereview. appspot. com/6324045/ diff/1/ juju/hooks/ tests/test_ invoker. py tests/test_ invoker. py (right):
File juju/hooks/
https:/ /codereview. appspot. com/6324045/ diff/1/ juju/hooks/ tests/test_ invoker. py#newcode1519 tests/test_ invoker. py:1519: (yield get_open_ ports() ),
juju/hooks/
unit_state.
On 2012/07/01 02:32:22, hazmat wrote:
> This test never verifies the implementation, namely that the container
ports are
> open.
Right you are, thanks
https:/ /codereview. appspot. com/6324045/ diff/1/ juju/state/ tests/test_ firewall. py tests/test_ firewall. py (right):
File juju/state/
https:/ /codereview. appspot. com/6324045/ diff/1/ juju/state/ tests/test_ firewall. py#newcode192 tests/test_ firewall. py:192: """
juju/state/
fair enough, removed
On 2012/07/01 02:32:22, hazmat wrote:
> this test also seems superfluous. the only impl file being modified by
the
> branch is the invoker, yet we have tests in numerous state test files
verifying
> what has already been verified by other tests in those files
https:/ /codereview. appspot. com/6324045/ diff/1/ juju/state/ tests/test_ service. py tests/test_ service. py (right):
File juju/state/
https:/ /codereview. appspot. com/6324045/ diff/1/ juju/state/ tests/test_ service. py#newcode2810 tests/test_ service. py:2810: self.assertEqual(
juju/state/
On 2012/07/01 02:32:22, hazmat wrote:
> how is this entire test not entirely superfluous?
removed
https:/ /codereview. appspot. com/6324045/