lp:~benji/juju-gui/bug-1110515-loading-message-2

Created by Benji York on 2013-02-13 and last modified on 2013-02-13
Get this branch:
bzr branch lp:~benji/juju-gui/bug-1110515-loading-message-2
Only Benji York can upload to this branch. If you are Benji York please log in for upload directions.

Branch merges

Related bugs

Related blueprints

Branch information

Owner:
Benji York
Project:
juju-gui
Status:
Development

Recent revisions

384. By Benji York on 2013-02-13

add missing file

383. By Benji York on 2013-02-13

make a new branch to try to fix lbox/Rietveld's mess

382. By Francesco Banconi on 2013-02-13

Test different browsers using the same env.

Re-deploying the charm to run tests in multiple browsers
is no longer required. Browsers can be specified as space
separated values in the environment variable
JUJU_GUI_TEST_BROWSERS. Chrome is the default.

Note that currently only chrome tests pass.

Other changes:

Implemented a clean up function that can be used to
restore a pristine Juju environment, and added a
test (deploy a charm) demonstrating how to use it.

bin/test-charm now exits with an error if one or
more tests fail.

Improved browser testing documentation.

QA:

JUJU_GUI_TEST_BROWSERS="chrome firefox ie" \
    bin/test-charm lp:juju-gui

You should see chrome tests passing, all the
others failing (or raising errors), and the charm
deployed only once.

R=gary.poster, matthew.scott
CC=
https://codereview.appspot.com/7307104

381. By Brad Crittenden on 2013-02-12

Add test for destroy service regression.

A recent change fixed a regression that occured when a service was destroyed.
The old service, which was removed from the database, was still in the
service_boxes object. Code was added to remove services from service_boxes if
it is removed from the database. This branch adds a unit test for that
functionality.

R=matthew.scott, gary.poster
CC=
https://codereview.appspot.com/7306095

380. By Jeff Pihach on 2013-02-12

Cleaned alert list styling and set max height

R=gary.poster, bac
CC=
https://codereview.appspot.com/7312084

379. By Madison Scott-Clary on 2013-02-12

Create ghost service while configuring.

When deploying from the charm panel, a ghost service is created in the environment while the user configures the charm. Once the charm is deployed, the service is made 'real'. If the deploy is canceled, the ghost is removed. If the panel is closed by a clickoutside event, the ghost remains, and clicking on it opens the panel back to the configuration pane (this allows panning, zooming, dragging, etc). If the service is dragged, it maintains its position after being deployed.

R=gary.poster, teknico
CC=
https://codereview.appspot.com/7308086

378. By Benji York on 2013-02-11

Add a warning about unsupported browsers.

The tests for the code are a bit unusual in that the code under test is
embedded in an HTML file, so it must be extracted and the test file built my
the Makefile so it can be tested. I don't think this approach is too heinous.

R=
CC=
https://codereview.appspot.com/7299071

377. By Benji York on 2013-02-11

Add a warning about unsupported browsers.

The tests for the code are a bit unusual in that the code under test is
embedded in an HTML file, so it must be extracted and the test file built my
the Makefile so it can be tested. I don't think this approach is too heinous.

R=bac, gary.poster
CC=
https://codereview.appspot.com/7299071

376. By Nicola Larosa on 2013-02-08

Surface more code docs.

Sorry for the big branch. A good chunk of it is the regenerated
"undocumented" file.

This branch goes over all 20 main source files, and gets most
comment blocks out of there and into the generated pages. It
rearranges the module and class directives so that all the
code parts are reachable from the pages. It also adds some
comment blocks to modules, classes and methods: the
undocumented count is now down to 173.

R=gary.poster, bac
CC=
https://codereview.appspot.com/7312067

375. By Francesco Banconi on 2013-02-08

Added GUI unit tests in the browser test suite.

Juju GUI unit tests are now included in the CI process.
Also added a test exercising the environment connection.

Updated the charm deployment process: "juju set" is no
longer used, and the charm configuration is now provided
by a temporary configuration file.

To run browser tests:

sudo apt-get install python-shelltoolbox python-selenium python-yaml juju
bin/test-charm lp:~frankban/juju-gui/bug-1117554-gui-tests-in-browser

R=gary.poster, teknico
CC=
https://codereview.appspot.com/7308068

Branch metadata

Branch format:
Branch format 7
Repository format:
Bazaar repository format 2a (needs bzr 1.16 or later)
Stacked on:
lp:juju-gui/experimental
This branch contains Public information 
Everyone can see this information.

Subscribers