Merge lp:~ev/amulet/fix-no-relations into lp:~vila/amulet/ci

Proposed by Evan
Status: Merged
Merged at revision: 55
Proposed branch: lp:~ev/amulet/fix-no-relations
Merge into: lp:~vila/amulet/ci
Diff against target: 20 lines (+10/-0)
1 file modified
tests/test_deployer.py (+10/-0)
To merge this branch: bzr merge lp:~ev/amulet/fix-no-relations
Reviewer Review Type Date Requested Status
Vincent Ladeuil Pending
Review via email: mp+200976@code.launchpad.net

Description of the change

Added a test for the previous merged fix for handling an environment without relations set.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/test_deployer.py'
2--- tests/test_deployer.py 2013-08-08 01:56:09 +0000
3+++ tests/test_deployer.py 2014-01-09 10:11:20 +0000
4@@ -52,6 +52,16 @@
5 self.assertEqual(dmap['gojuju']['relations'], d.relations)
6 self.assertEqual(dmap['gojuju']['series'], d.series)
7
8+ # Try again without any relations.
9+ schema = '{"gojuju": {"series": "raring", "services": {"wordpress": \
10+ {"branch": "lp:charms/wordpress"}, "mysql": {"options": \
11+ {"tuning": "fastest"}, "branch": "lp:charms/mysql"}} \
12+ }}'
13+ dmap = json.loads(schema)
14+ d.load(dmap)
15+ self.assertEqual(dmap['gojuju']['services'], d.services)
16+ self.assertEqual(dmap['gojuju']['series'], d.series)
17+
18 def test_add(self):
19 d = Deployment(juju_env='gojuju')
20 d.add('charm')

Subscribers

People subscribed via source and target branches

to all changes: