~bcsaller/juju-gui/databinding-safeupdate

Viewing all changes in revision 952.

  • Committer: Rick Harding
  • Author(s): Richard Harding
  • Date: 2013-08-15 00:47:04 UTC
  • mfrom: (948.1.6 ghost-config-upload)
  • Revision ID: rick.harding@canonical.com-20130815004704-3h97i5j9332xwb98
Fixes support for loading config from file.

The config file handling code was copied to the inspector, but the tests
weren't. Most of this branch is adding the tests from the old to the new.

- The two main bugs were that the variable name was called fileContent vs
configFileContent.
- The other was the deploy call looking on this.configFileContent vs on the
viewletManager where the content is kept.
- Small change to the onFileLoad to make testing easier. The UI is updated now
without having to fake it or to mock up handleFileChange in odd ways.
- Also did some small tweaks to alphabetize the var names and such in the
tests to make it easier to see what vars there are and find out if something
is global (in the test scope) or not.

R=jeff.pihach, matthew.scott
CC=
https://codereview.appspot.com/12797046

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: