Merge lp:~lazypower/charms/precise/quassel-core/update-50-deploy into lp:~paulgear/charms/precise/quassel-core/trunk

Proposed by Charles Butler
Status: Merged
Merged at revision: 25
Proposed branch: lp:~lazypower/charms/precise/quassel-core/update-50-deploy
Merge into: lp:~paulgear/charms/precise/quassel-core/trunk
Diff against target: 25 lines (+7/-6)
1 file modified
tests/50-basic-deploy (+7/-6)
To merge this branch: bzr merge lp:~lazypower/charms/precise/quassel-core/update-50-deploy
Reviewer Review Type Date Requested Status
Paul Gear Pending
Review via email: mp+255363@code.launchpad.net

Description of the change

Updates the 50-deploy test to remove the side-effects of the init() method

To post a comment you must log in.
26. By Charles Butler

Corrects an oversight of a leftover self. reference in the setUpClass

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/50-basic-deploy'
2--- tests/50-basic-deploy 2015-03-30 12:11:17 +0000
3+++ tests/50-basic-deploy 2015-04-07 14:14:26 +0000
4@@ -6,14 +6,15 @@
5
6 class TestDeployment(unittest.TestCase):
7
8- def __init__(self):
9+ @classmethod
10+ def setUpClass(cls):
11 try:
12- self.d = amulet.Deployment()
13- self.d.add('quassel-core')
14- self.d.setup(timeout=1800)
15- self.d.sentry.wait()
16+ cls.d = amulet.Deployment()
17+ cls.d.add('quassel-core')
18+ cls.d.setup(timeout=1800)
19+ cls.d.sentry.wait()
20
21- self.u = self.d.sentry.unit['quassel-core/0']
22+ cls.u = cls.d.sentry.unit['quassel-core/0']
23 except amulet.helpers.TimeoutError:
24 amulet.raise_status(amulet.SKIP, msg="Environment wasn't stood up in time")
25 except:

Subscribers

People subscribed via source and target branches