Support smart formatting for relation-ids command

Support smart formatting for relation-ids command

Adds format_smart method for RelationIdsCli, which outputs whitespace separated (to be precise, per line) relation idents.

https://code.launchpad.net/~jimbaker/juju/relation-ids-whitespace-separated/+merge/103387

(do not edit description out of merge proposal)

[jimbaker, 2012-04-24 22:22:36]

Please take a look.

[hazmat, 2012-04-24 23:51:25]

fix looks straightforwards, its just not clear its as intended

https://codereview.appspot.com/6118047/diff/1/juju/hooks/tests/test_invoker.py
File juju/hooks/tests/test_invoker.py (right):

https://codereview.appspot.com/6118047/diff/1/juju/hooks/tests/test_invoker.p...
juju/hooks/tests/test_invoker.py:1105: """Verify `relation-ids` command returns
white-space separated ids."""
this isn't space separated its newline separated. what's the spec say regarding?

[jimbaker, 2012-04-25 00:21:15]

Please take a look.

[jimbaker, 2012-04-25 00:25:37]

Fixed in latest version of the MP

https://codereview.appspot.com/6118047/diff/1/juju/hooks/tests/test_invoker.py
File juju/hooks/tests/test_invoker.py (right):

https://codereview.appspot.com/6118047/diff/1/juju/hooks/tests/test_invoker.p...
juju/hooks/tests/test_invoker.py:1105: """Verify `relation-ids` command returns
white-space separated ids."""
The spec says: 
"The output is a whitespace separated list of relation ids, if any."
Per our conversation on IRC, updated docstring and comments to reflect this is
newline separated; but since shell/Python/presumably others treat newline as
just whitespace, this follows the letter of the spec. It also follows the
convention for relation-list.

[hazmat, 2012-04-25 02:58:48]

lgtm, pls also update the spec as thats what most people will read for docs.