Merge ~raharper/cloud-init:fix-unittest-assert-called into cloud-init:master

Proposed by Ryan Harper
Status: Merged
Merge reported by: Scott Moser
Merged at revision: not available
Proposed branch: ~raharper/cloud-init:fix-unittest-assert-called
Merge into: cloud-init:master
Diff against target: 31 lines (+3/-3)
1 file modified
tests/unittests/test_datasource/test_digitalocean.py (+3/-3)
Reviewer Review Type Date Requested Status
cloud-init Commiters Pending
Review via email: mp+307583@code.launchpad.net

Commit message

unittests: fix use of mock 2.0 'assert_called' when running make check

Some of the new DigitalOcean unittests were written to use
'assert_called', which is only available in mock versions 2.0.
Because of this, the failure would only occur in releases less
than yakkety and not in 'tox'.

Description of the change

unittests: fix use of mock 2.0 'assert_called' when running make check

Some of the new DigitalOcean unittests were written to use
'assert_called', which is only available in mock versions 2.0.
Because of this, the failure would only occur in releases less
than yakkety and not in 'tox'.

To post a comment you must log in.
Revision history for this message
Scott Moser (smoser) wrote :

I added a tox environment to make sure this isnt busted in the future.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/tests/unittests/test_datasource/test_digitalocean.py b/tests/unittests/test_datasource/test_digitalocean.py
2index bdfe0ba..7bde082 100644
3--- a/tests/unittests/test_datasource/test_digitalocean.py
4+++ b/tests/unittests/test_datasource/test_digitalocean.py
5@@ -162,7 +162,7 @@ class TestDataSourceDigitalOcean(TestCase):
6 m_read_sysinfo.return_value = (False, None)
7 ds = self.get_ds(get_sysinfo=None)
8 self.assertEqual(False, ds.get_data())
9- m_read_sysinfo.assert_called()
10+ self.assertTrue(m_read_sysinfo.called)
11
12 @mock.patch('cloudinit.sources.helpers.digitalocean.read_metadata')
13 def test_metadata(self, mock_readmd):
14@@ -172,7 +172,7 @@ class TestDataSourceDigitalOcean(TestCase):
15 ret = ds.get_data()
16 self.assertTrue(ret)
17
18- mock_readmd.assert_called()
19+ self.assertTrue(mock_readmd.called)
20
21 self.assertEqual(DO_META.get('user_data'), ds.get_userdata_raw())
22 self.assertEqual(DO_META.get('vendor_data'), ds.get_vendordata_raw())
23@@ -196,7 +196,7 @@ class TestDataSourceDigitalOcean(TestCase):
24 ret = ds.get_data()
25 self.assertTrue(ret)
26
27- mock_readmd.assert_called()
28+ self.assertTrue(mock_readmd.called)
29
30 # Multiple keys
31 self.assertEqual(metadata['public_keys'], ds.get_public_ssh_keys())

Subscribers

People subscribed via source and target branches