Complete deploy functionality and tests.

Complete deploy functionality and tests.

This combines three discrete improvements, each with their own commit.

revno: 423
  add machines for units

revno: 424
  add support for passing a YAML string for service configuration, as would be passed by file parsing

revno: 425
  add a nextChanges function and tests for it in isolation and with deploy and addUnit.

Note that rev 424 uses js-yaml 2.0.3, which seems to be a lot better than previous incarnations in my small tests.  I decided it was worth including.  As I said in the .min.js file, I committed this to the tree rather than using npm because the file we needed was not in npm.

rev 425, with the nextChanges, is the most "sketchy."  We'll have to see how that fares when we actually implement the wire protocol, but I think it should work.

https://code.launchpad.net/~gary/juju-gui/sandboxEnv-2/+merge/152805

(do not edit description out of merge proposal)

[gary.poster, 2013-03-12 00:40:15]

Please take a look.

[teknico, 2013-03-12 12:41:53]

LGTM

[bac, 2013-03-12 12:50:57]

LGTM modulo a couple of changes.

Before we'd not done any YAML parsing in JS due to Kapil's assertion that there
were no libraries that were acceptable.  Is the one you've chosen suitable?  Can
it be used in the parsing of config files when the user uploads one?  If so,
that would make passing YAML obsolete and would give a better user experience.

https://codereview.appspot.com/7741043/diff/1/app/store/env/fakebackend.js
File app/store/env/fakebackend.js (right):

https://codereview.appspot.com/7741043/diff/1/app/store/env/fakebackend.js#ne...
app/store/env/fakebackend.js:272: }
Originally improv accepted both and just gave precedence to configYAML.  The go
implementation does the same so it would be nice for this implementation to be
consistent.

https://codereview.appspot.com/7741043/diff/1/test/test_fakebackend.js
File test/test_fakebackend.js (right):

https://codereview.appspot.com/7741043/diff/1/test/test_fakebackend.js#newcod...
test/test_fakebackend.js:200: 
This test will need to be fixed if you make the previously suggested change.

[gary.poster, 2013-03-12 13:50:54]

*** Submitted:

Complete deploy functionality and tests.

This combines three discrete improvements, each with their own commit.

revno: 423
  add machines for units

revno: 424
  add support for passing a YAML string for service configuration, as would be
passed by file parsing

revno: 425
  add a nextChanges function and tests for it in isolation and with deploy and
addUnit.

Note that rev 424 uses js-yaml 2.0.3, which seems to be a lot better than
previous incarnations in my small tests.  I decided it was worth including.  As
I said in the .min.js file, I committed this to the tree rather than using npm
because the file we needed was not in npm.

rev 425, with the nextChanges, is the most "sketchy."  We'll have to see how
that fares when we actually implement the wire protocol, but I think it should
work.

R=teknico, bac
CC=
https://codereview.appspot.com/7741043