Merge ~barryprice/charm-canonical-livepatch/+git/canonical-livepatch-charm:multiseries-testing into ~livepatch-charmers/charm-canonical-livepatch:master

[Barry Price]

First attempt at multi-series testing

Unmerged commits

45c714c... by Barry Price on 2018-10-31

First attempt at multi-series testing

diff --git a/tests/99-autogen b/tests/multiseries
index d7bed72..41d46a1 100755
--- a/tests/99-autogen
+++ b/tests/multiseries
@@ -7,38 +7,41 @@ from yaml import safe_load
 
 
 class TestDeployment(unittest.TestCase):
+
+    series = 'bionic'  # latest LTS
+
     @classmethod
     def setUpClass(cls):
-        cls.deployment = amulet.Deployment(series='bionic')
+        cls.deployment = amulet.Deployment(series=series)
 
         # deploy mongodb as our parent
-        cls.deployment.add('mongodb')
+        cls.deployment.add('mongodb-{}'.format(series), 'mongodb')
 
         # deploy our own charm
-        cls.deployment.add('canonical-livepatch')
+        cls.deployment.add('livepatch-{}'.format(series), 'canonical-livepatch')
 
         # and deploy the nrpe subordinate to test nagios checks
-        cls.deployment.add('nrpe')
+        cls.deployment.add('nrpe-{}'.format(series), 'nrpe')
 
         # set nrpe to export its definitions
-        cls.deployment.configure('nrpe', {
+        cls.deployment.configure('nrpe-{}'.format(series), {
             'export_nagios_definitions': True,
         })
 
         # relate subordinates to parent charm
         cls.deployment.relate(
-            'mongodb:juju-info',
-            'canonical-livepatch:juju-info'
+            'mongodb-{}:juju-info'.format(series),
+            'livepatch-{}:juju-info'.format(series)
         )
         cls.deployment.relate(
-            'mongodb:nrpe-external-master',
-            'nrpe:nrpe-external-master'
+            'mongodb-{}:nrpe-external-master'.format(series),
+            'nrpe-{}:nrpe-external-master'.format(series)
         )
 
         # relate livepatch to nrpe for its own nagios checks
         cls.deployment.relate(
-            'canonical-livepatch:nrpe-external-master',
-            'nrpe:nrpe-external-master'
+            'livepatch-{}:nrpe-external-master'.format(series),
+            'nrpe-{}:nrpe-external-master'.format(series)
         )
 
         try:
@@ -51,7 +54,7 @@ class TestDeployment(unittest.TestCase):
             )
 
     def test_install(self):
-        livepatch = self.deployment.sentry['canonical-livepatch'][0]
+        livepatch = self.deployment.sentry['livepatch-{}'.format(series)][0]
 
         # verify the snap was installed
         output, exit_code = livepatch.run(
@@ -60,14 +63,14 @@ class TestDeployment(unittest.TestCase):
         self.assertEqual(exit_code, 0)
 
     def test_status(self):
-        livepatch = self.deployment.sentry['canonical-livepatch'][0]
+        livepatch = self.deployment.sentry['livepatch-{}'.format(series)][0]
 
         # run a status check - we expect this to return 1 due to no access key
         output, exit_code = livepatch.run('sudo canonical-livepatch status')
         self.assertEqual(exit_code, 1)
 
     def test_nagios_init(self):
-        livepatch = self.deployment.sentry['canonical-livepatch'][0]
+        livepatch = self.deployment.sentry['livepatch-{}'.format(series)][0]
 
         # check for nagios bits
         for path in [
@@ -79,12 +82,12 @@ class TestDeployment(unittest.TestCase):
             self.assertEqual(exit_code, 0)
 
     def test_nagios_context_change(self):
-        livepatch = self.deployment.sentry['canonical-livepatch'][0]
+        livepatch = self.deployment.sentry['livepatch-{}'.format(series)][0]
 
         test_context_name = 'amulet1'
 
         # set context name
-        self.deployment.configure('canonical-livepatch', {
+        self.deployment.configure('livepatch-{}'.format(series), {
             'nagios_context': test_context_name,
         })
 
@@ -99,7 +102,7 @@ class TestDeployment(unittest.TestCase):
         self.assertEqual(exit_code, 0)
 
     def test_channel_change(self):
-        livepatch = self.deployment.sentry['canonical-livepatch'][0]
+        livepatch = self.deployment.sentry['livepatch-{}'.format(series)][0]
 
         # verify the current channel
         output, exit_code = livepatch.run('snap info canonical-livepatch')
@@ -111,7 +114,7 @@ class TestDeployment(unittest.TestCase):
         self.assertEqual(channel, 'stable')
 
         # change channel to 'beta'
-        self.deployment.configure('canonical-livepatch', {
+        self.deployment.configure('livepatch-{}'.format(series), {
             'snap_channel': 'beta',
         })
 
@@ -128,12 +131,12 @@ class TestDeployment(unittest.TestCase):
         self.assertEqual(channel, 'beta')
 
     def test_nagios_servicegroup_change(self):
-        livepatch = self.deployment.sentry['canonical-livepatch'][0]
+        livepatch = self.deployment.sentry['livepatch-{}'.format(series)][0]
 
         test_servicegroup_name = 'amulet2'
 
         # set servicegroup name
-        self.deployment.configure('canonical-livepatch', {
+        self.deployment.configure('livepatch-{}'.format(series), {
             'nagios_servicegroups': test_servicegroup_name,
         })
 
@@ -150,4 +153,6 @@ class TestDeployment(unittest.TestCase):
 
 
 if __name__ == '__main__':
-    unittest.main()
+    for series in ['trusty', 'xenial', 'bionic']:
+        TestDeployment.series = series
+        unittest.main()