Merge lp:~marcoceppi/charms/precise/sbuild/tests into lp:charms/sbuild

Proposed by Marco Ceppi
Status: Merged
Merged at revision: 8
Proposed branch: lp:~marcoceppi/charms/precise/sbuild/tests
Merge into: lp:charms/sbuild
Diff against target: 64 lines (+54/-0)
2 files modified
tests/00-setup (+5/-0)
tests/99-autogen (+49/-0)
To merge this branch: bzr merge lp:~marcoceppi/charms/precise/sbuild/tests
Reviewer Review Type Date Requested Status
José Antonio Rey (community) Approve
Review Queue (community) automated testing Needs Fixing
Review via email: mp+240966@code.launchpad.net

Commit message

Tests for charm

To post a comment you must log in.
Revision history for this message
Review Queue (review-queue) wrote :

This items has failed automated testing! Results available here http://reports.vapour.ws/charm-tests/charm-bundle-test-10625-results

review: Needs Fixing (automated testing)
Revision history for this message
José Antonio Rey (jose) wrote :

In this case, when using bundletester to run the tests, they fail because of a default configuration option. It's asking to build a quantal environment, while it should try to build an env for a release that hasn't hit EOL.

However, tests are good and another bug will be filed against this charm for having a broken default config option.

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== added directory 'tests'
=== added file 'tests/00-setup'
--- tests/00-setup 1970-01-01 00:00:00 +0000
+++ tests/00-setup 2014-11-06 20:11:18 +0000
@@ -0,0 +1,5 @@
1#!/bin/bash
2
3sudo add-apt-repository ppa:juju/stable -y
4sudo apt-get update
5sudo apt-get install amulet python3-requests -y
06
=== added file 'tests/99-autogen'
--- tests/99-autogen 1970-01-01 00:00:00 +0000
+++ tests/99-autogen 2014-11-06 20:11:18 +0000
@@ -0,0 +1,49 @@
1#!/usr/bin/env python3
2
3import amulet
4import requests
5import unittest
6
7
8class TestDeployment(unittest.TestCase):
9 @classmethod
10 def setUpClass(cls):
11 cls.deployment = amulet.Deployment(series='precise')
12
13 cls.deployment.add('sbuild')
14
15 try:
16 cls.deployment.setup(timeout=900)
17 cls.deployment.sentry.wait()
18 except amulet.helpers.TimeoutError:
19 amulet.raise_status(amulet.SKIP, msg="Environment wasn't stood up in time")
20 except:
21 raise
22
23 def test_case(self):
24 # Now you can use self.deployment.sentry.unit[UNIT] to address each of
25 # the units and perform more in-depth steps. You can also reference
26 # the first unit as self.unit.
27 # There are three test statuses that can be triggered with
28 # amulet.raise_status():
29 # - amulet.PASS
30 # - amulet.FAIL
31 # - amulet.SKIP
32 # Each unit has the following methods:
33 # - .info - An array of the information of that unit from Juju
34 # - .file(PATH) - Get the details of a file on that unit
35 # - .file_contents(PATH) - Get plain text output of PATH file from that unit
36 # - .directory(PATH) - Get details of directory
37 # - .directory_contents(PATH) - List files and folders in PATH on that unit
38 # - .relation(relation, service:rel) - Get relation data from return service
39 # add tests here to confirm service is up and working properly
40 # For example, to confirm that it has a functioning HTTP server:
41 # page = requests.get('http://{}'.format(self.unit.info['public-address']))
42 # page.raise_for_status()
43 # More information on writing Amulet tests can be found at:
44 # https://juju.ubuntu.com/docs/tools-amulet.html
45 pass
46
47
48if __name__ == '__main__':
49 unittest.main()

Subscribers

People subscribed via source and target branches

to all changes: