Fix Go sandbox AllWatcher initialization

Fix Go sandbox AllWatcher initialization

The Go sandbox AllWatcher was trying to send deltas immediately, which does not match the juju core implementation.  This fixes the behavior, with a test.

It also fixes a user-visible bug.  To qa, turn off the simulator in the config file, and then deploy a service in the default go sandbox.  Before my branch, the unit would never appear, because the delta for this change was lost.  After my branch, the unit appears correctly.

https://code.launchpad.net/~gary/juju-gui/fixGoSandboxWatcher2/+merge/190685

(do not edit description out of merge proposal)

[gary.poster, 2013-10-11 14:26:19]

Please take a look.

[matthew.scott, 2013-10-11 14:29:25]

LGTM, thanks for catching this!

[gary.poster, 2013-10-11 14:44:46]

*** Submitted:

Fix Go sandbox AllWatcher initialization

The Go sandbox AllWatcher was trying to send deltas immediately, which does not
match the juju core implementation.  This fixes the behavior, with a test.

It also fixes a user-visible bug.  To qa, turn off the simulator in the config
file, and then deploy a service in the default go sandbox.  Before my branch,
the unit would never appear, because the delta for this change was lost.  After
my branch, the unit appears correctly.

R=matthew.scott
CC=
https://codereview.appspot.com/14548050

[gary.poster, 2013-10-11 15:23:53]

On 2013/10/11 14:29:25, matthew.scott wrote:
> LGTM, thanks for catching this!

Thanks, Matt!