Merge lp:~aisrael/charms/trusty/juju-gui/fix-automated-testing into lp:~juju-gui/charms/trusty/juju-gui/trunk

Proposed by Richard Harding
Status: Merged
Approved by: Francesco Banconi
Approved revision: 209
Merged at revision: 209
Proposed branch: lp:~aisrael/charms/trusty/juju-gui/fix-automated-testing
Merge into: lp:~juju-gui/charms/trusty/juju-gui/trunk
Diff against target: 49 lines (+16/-0)
2 files modified
config.yaml (+4/-0)
tests/tests.yaml (+12/-0)
To merge this branch: bzr merge lp:~aisrael/charms/trusty/juju-gui/fix-automated-testing
Reviewer Review Type Date Requested Status
Francesco Banconi Pending
Review via email: mp+238103@code.launchpad.net

This proposal supersedes a proposal from 2014-10-08.

Description of the change

Add test.yaml for automated tests.

To post a comment you must log in.
Revision history for this message
Francesco Banconi (frankban) wrote : Posted in a previous version of this proposal

Hi Adam,
thanks a lot for this branch, it looks nice, with just a minor request below.

Also note that lp:charms/juju-gui is the precise release branch of the GUI charm, while the development branch is lp:~juju-gui/charms/trusty/juju-gui/trunk, which is then pushed to both precise and trusty releases. Could you please repropose this against lp:~juju-gui/charms/trusty/juju-gui/trunk? I will take care of making charm releases later, so that we can test the GUI charm again with the bundletester.

review: Approve
Revision history for this message
Richard Harding (rharding) wrote :

I've retarged per Francesco's request using the resubmit proposal tool on the upper right. Francesco, can you double check all looks well and help walk it through the rest of the way please?

Revision history for this message
Francesco Banconi (frankban) wrote :

*** Submitted:

Add test.yaml for automated tests.

Revision history for this message
Francesco Banconi (frankban) wrote :

Thanks again Adam and Rick!
I merged the branch using "lbox submit -adopt".
I'll make a new charm release asap this week.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'config.yaml'
2--- config.yaml 2014-04-15 14:17:12 +0000
3+++ config.yaml 2014-10-13 06:18:03 +0000
4@@ -66,17 +66,20 @@
5 the GUI. Both ssl-cert-contents and ssl-key-contents must be provided.
6 If not, cetificates will be automatically generated.
7 type: string
8+ default:
9 ssl-key-contents:
10 description: |
11 The contents of the private key file to be used in SSL connections to
12 the GUI. Both ssl-cert-contents and ssl-key-contents must be provided.
13 If not, cetificates will be automatically generated.
14 type: string
15+ default:
16 login-help:
17 description: |
18 The help text shown to the user on the login screen. If not provided, a
19 default message is used, suggesting how to find the login credentials.
20 type: string
21+ default:
22 read-only:
23 description: |
24 Whether or not the GUI is in read-only mode. Note that read-only mode is
25@@ -103,6 +106,7 @@
26 If given, the password to use for the environment to immediately
27 connect. Do not set unless you understand and accept the risks.
28 type: string
29+ default:
30 sandbox:
31 description: |
32 Run using an in-memory sandbox rather than a real Juju backend. Sandbox
33
34=== added file 'tests/tests.yaml'
35--- tests/tests.yaml 1970-01-01 00:00:00 +0000
36+++ tests/tests.yaml 2014-10-13 06:18:03 +0000
37@@ -0,0 +1,12 @@
38+tests: "[0-9]*"
39+reset: false
40+packages:
41+ - build-essential
42+ - bzr
43+ - charm-tools
44+ - firefox
45+ - libapt-pkg-dev
46+ - libpython-dev
47+ - python-virtualenv
48+ - rsync
49+ - xvfb

Subscribers

People subscribed via source and target branches

to all changes: