X-Git-Url: https://osm.etsi.org/gitweb/?a=blobdiff_plain;f=osm_mon%2Ftests%2Funit%2Fcollector%2Fvnf_collectors%2Ftest_openstack.py;h=991d7f9fde001bfd5442b79e5f0dc13861dc059a;hb=94948de24a6ff7d03cce2168be5a83c0ed97196d;hp=42df03c71b2531d0dee023c4e920e7be30c5d252;hpb=4b9f79e2820a011f6ebbba7fb92cdc120b364650;p=osm%2FMON.git diff --git a/osm_mon/tests/unit/collector/vnf_collectors/test_openstack.py b/osm_mon/tests/unit/collector/vnf_collectors/test_openstack.py index 42df03c..991d7f9 100644 --- a/osm_mon/tests/unit/collector/vnf_collectors/test_openstack.py +++ b/osm_mon/tests/unit/collector/vnf_collectors/test_openstack.py @@ -20,6 +20,9 @@ # For those usages not covered by the Apache License, Version 2.0 please # contact: bdiaz@whitestack.com or glavado@whitestack.com ## + +# pylint: disable=E1101 + import datetime from unittest import TestCase, mock @@ -37,86 +40,161 @@ class CollectorTest(TestCase): def tearDown(self): super().tearDown() - @mock.patch.object(GnocchiBackend, '_build_neutron_client') - @mock.patch.object(GnocchiBackend, '_build_gnocchi_client') + @mock.patch.object(GnocchiBackend, "_build_neutron_client") + @mock.patch.object(GnocchiBackend, "_build_gnocchi_client") def test_collect_gnocchi_rate_instance(self, build_gnocchi_client, _): mock_gnocchi_client = mock.Mock() mock_gnocchi_client.metric = mock.Mock() - mock_gnocchi_client.metric.get_measures.return_value = [(datetime.datetime(2019, 4, 12, 15, 43, - tzinfo=datetime.timezone( - datetime.timedelta(0), - '+00:00')), 60.0, 0.0345442539), - (datetime.datetime(2019, 4, 12, 15, 44, - tzinfo=datetime.timezone( - datetime.timedelta(0), - '+00:00')), 60.0, 600000000)] + mock_vim_session = mock.Mock() + mock_gnocchi_client.metric.get_measures.return_value = [ + ( + datetime.datetime( + 2019, + 4, + 12, + 15, + 43, + tzinfo=datetime.timezone(datetime.timedelta(0), "+00:00"), + ), + 60.0, + 0.0345442539, + ), + ( + datetime.datetime( + 2019, + 4, + 12, + 15, + 44, + tzinfo=datetime.timezone(datetime.timedelta(0), "+00:00"), + ), + 60.0, + 600000000, + ), + ] build_gnocchi_client.return_value = mock_gnocchi_client - backend = GnocchiBackend({'_id': 'test_uuid'}) - value = backend._collect_instance_metric('cpu', 'test_resource_id') + backend = GnocchiBackend({"_id": "test_uuid"}, mock_vim_session) + value = backend._collect_instance_metric("cpu", "test_resource_id") self.assertEqual(value, 1.0) - mock_gnocchi_client.metric.get_measures.assert_called_once_with('cpu', - aggregation="rate:mean", - start=mock.ANY, - resource_id='test_resource_id') - - @mock.patch.object(GnocchiBackend, '_build_neutron_client') - @mock.patch.object(GnocchiBackend, '_build_gnocchi_client') + mock_gnocchi_client.metric.get_measures.assert_called_once_with( + "cpu", + aggregation="rate:mean", + start=mock.ANY, + resource_id="test_resource_id", + ) + + @mock.patch.object(GnocchiBackend, "_build_neutron_client") + @mock.patch.object(GnocchiBackend, "_build_gnocchi_client") def test_collect_gnocchi_non_rate_instance(self, build_gnocchi_client, _): mock_gnocchi_client = mock.Mock() - mock_gnocchi_client.metric.get_measures.return_value = [(datetime.datetime(2019, 4, 12, 15, 43, - tzinfo=datetime.timezone( - datetime.timedelta(0), - '+00:00')), 60.0, 0.0345442539), - (datetime.datetime(2019, 4, 12, 15, 44, - tzinfo=datetime.timezone( - datetime.timedelta(0), - '+00:00')), 60.0, 128)] + mock_vim_session = mock.Mock() + mock_gnocchi_client.metric.get_measures.return_value = [ + ( + datetime.datetime( + 2019, + 4, + 12, + 15, + 43, + tzinfo=datetime.timezone(datetime.timedelta(0), "+00:00"), + ), + 60.0, + 0.0345442539, + ), + ( + datetime.datetime( + 2019, + 4, + 12, + 15, + 44, + tzinfo=datetime.timezone(datetime.timedelta(0), "+00:00"), + ), + 60.0, + 128, + ), + ] build_gnocchi_client.return_value = mock_gnocchi_client - backend = GnocchiBackend({'_id': 'test_uuid'}) - value = backend._collect_instance_metric('memory.usage', 'test_resource_id') + backend = GnocchiBackend({"_id": "test_uuid"}, mock_vim_session) + value = backend._collect_instance_metric("memory.usage", "test_resource_id") self.assertEqual(value, 128) - mock_gnocchi_client.metric.get_measures.assert_called_once_with('memory.usage', - aggregation=None, - start=mock.ANY, - resource_id='test_resource_id') - - @mock.patch.object(GnocchiBackend, '_build_neutron_client') - @mock.patch.object(GnocchiBackend, '_build_gnocchi_client') + mock_gnocchi_client.metric.get_measures.assert_called_once_with( + "memory.usage", + aggregation=None, + start=mock.ANY, + resource_id="test_resource_id", + ) + + @mock.patch.object(GnocchiBackend, "_build_neutron_client") + @mock.patch.object(GnocchiBackend, "_build_gnocchi_client") def test_collect_gnocchi_no_metric(self, build_gnocchi_client, _): mock_gnocchi_client = mock.Mock() - mock_gnocchi_client.metric.get_measures.side_effect = gnocchiclient.exceptions.NotFound() + mock_vim_session = mock.Mock() + mock_gnocchi_client.metric.get_measures.side_effect = ( + gnocchiclient.exceptions.NotFound() + ) build_gnocchi_client.return_value = mock_gnocchi_client - backend = GnocchiBackend({'_id': 'test_uuid'}) - value = backend._collect_instance_metric('memory.usage', 'test_resource_id') + backend = GnocchiBackend({"_id": "test_uuid"}, mock_vim_session) + value = backend._collect_instance_metric("memory.usage", "test_resource_id") self.assertIsNone(value) - mock_gnocchi_client.metric.get_measures.assert_called_once_with('memory.usage', - aggregation=None, - start=mock.ANY, - resource_id='test_resource_id') - - @mock.patch.object(GnocchiBackend, '_build_neutron_client') - @mock.patch.object(GnocchiBackend, '_build_gnocchi_client') - def test_collect_interface_all_metric(self, build_gnocchi_client, build_neutron_client): + mock_gnocchi_client.metric.get_measures.assert_called_once_with( + "memory.usage", + aggregation=None, + start=mock.ANY, + resource_id="test_resource_id", + ) + + @mock.patch.object(GnocchiBackend, "_build_neutron_client") + @mock.patch.object(GnocchiBackend, "_build_gnocchi_client") + def test_collect_interface_all_metric( + self, build_gnocchi_client, build_neutron_client + ): mock_gnocchi_client = mock.Mock() - mock_gnocchi_client.resource.search.return_value = [{'id': 'test_id_1'}, {'id': 'test_id_2'}] - mock_gnocchi_client.metric.get_measures.return_value = [(datetime.datetime(2019, 4, 12, 15, 43, - tzinfo=datetime.timezone( - datetime.timedelta(0), - '+00:00')), 60.0, 0.0345442539), - (datetime.datetime(2019, 4, 12, 15, 44, - tzinfo=datetime.timezone( - datetime.timedelta(0), - '+00:00')), 60.0, 0.0333070363)] + mock_vim_session = mock.Mock() + mock_gnocchi_client.resource.search.return_value = [ + {"id": "test_id_1"}, + {"id": "test_id_2"}, + ] + mock_gnocchi_client.metric.get_measures.return_value = [ + ( + datetime.datetime( + 2019, + 4, + 12, + 15, + 43, + tzinfo=datetime.timezone(datetime.timedelta(0), "+00:00"), + ), + 60.0, + 0.0345442539, + ), + ( + datetime.datetime( + 2019, + 4, + 12, + 15, + 44, + tzinfo=datetime.timezone(datetime.timedelta(0), "+00:00"), + ), + 60.0, + 0.0333070363, + ), + ] build_gnocchi_client.return_value = mock_gnocchi_client - backend = GnocchiBackend({'_id': 'test_uuid'}) - value = backend._collect_interface_all_metric('packets_received', 'test_resource_id') + backend = GnocchiBackend({"_id": "test_uuid"}, mock_vim_session) + value = backend._collect_interface_all_metric( + "packets_received", "test_resource_id" + ) self.assertEqual(value, 0.0666140726) - mock_gnocchi_client.metric.get_measures.assert_any_call('packets_received', resource_id='test_id_1', - limit=1) - mock_gnocchi_client.metric.get_measures.assert_any_call('packets_received', resource_id='test_id_2', - limit=1) + mock_gnocchi_client.metric.get_measures.assert_any_call( + "packets_received", resource_id="test_id_1", limit=1 + ) + mock_gnocchi_client.metric.get_measures.assert_any_call( + "packets_received", resource_id="test_id_2", limit=1 + )