Merge "Add meter rule to OSC"

3 files changed, 475 insertions, 0 deletions

diff --git a/openstackclient/tests/functional/network/v2/test_network_meter_rule.py b/openstackclient/tests/functional/network/v2/test_network_meter_rule.py
new file mode 100644
index 00000000..4f079e3c
--- /dev/null
+++ b/openstackclient/tests/functional/network/v2/test_network_meter_rule.py
@@ -0,0 +1,108 @@
+# Copyright (c) 2016, Intel Corporation.
+# All Rights Reserved.
+#
+#    Licensed under the Apache License, Version 2.0 (the "License"); you may
+#    not use this file except in compliance with the License. You may obtain
+#    a copy of the License at
+#
+#         http://www.apache.org/licenses/LICENSE-2.0
+#
+#    Unless required by applicable law or agreed to in writing, software
+#    distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+#    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
+#    License for the specific language governing permissions and limitations
+#    under the License.
+
+import re
+import uuid
+
+from openstackclient.tests.functional import base
+
+
+class TestMeterRule(base.TestCase):
+    """Functional tests for meter rule"""
+    METER_NAME = uuid.uuid4().hex
+    METER_ID = None
+    METER_RULE_ID = None
+
+    @classmethod
+    def setUpClass(cls):
+        # Set up some regex for matching below
+        cls.re_id = re.compile("id\s+\|\s+(\S+)")
+        cls.re_direction = re.compile("direction\s+\|\s+(\S+)")
+        cls.re_ip_prefix = re.compile(
+            "remote_ip_prefix\s+\|\s+([^|]+?)\s+\|"
+        )
+        cls.re_meter_id = re.compile("metering_label_id\s+\|\s+(\S+)")
+
+        raw_output = cls.openstack(
+            'network meter create ' + cls.METER_NAME
+        )
+
+        cls.METER_ID = re.search(cls.re_id, raw_output).group(1)
+
+    @classmethod
+    def tearDownClass(cls):
+        raw_output = cls.openstack('network meter delete ' + cls.METER_ID)
+        cls.assertOutput('', raw_output)
+
+    def test_meter_rule_delete(self):
+        """test create, delete"""
+
+        raw_output = self.openstack(
+            'network meter rule create ' +
+            '--remote-ip-prefix 10.0.0.0/8 ' +
+            self.METER_ID
+        )
+        rule_id = re.search(self.re_id, raw_output).group(1)
+        re_ip = re.search(self.re_ip_prefix, raw_output)
+
+        self.addCleanup(self.openstack,
+                        'network meter rule delete ' + rule_id)
+        self.assertIsNotNone(re_ip)
+        self.assertIsNotNone(rule_id)
+
+    def test_meter_rule_list(self):
+        """Test create, list, delete"""
+        raw_output = self.openstack(
+            'network meter rule create ' +
+            '--remote-ip-prefix 10.0.0.0/8 ' +
+            self.METER_ID
+        )
+        rule_id = re.search(self.re_id, raw_output).group(1)
+        self.addCleanup(self.openstack,
+                        'network meter rule delete ' + rule_id)
+        self.assertEqual(
+            '10.0.0.0/8',
+            re.search(self.re_ip_prefix, raw_output).group(1)
+        )
+
+        raw_output = self.openstack('network meter rule list')
+        self.assertIsNotNone(re.search(rule_id + "|\s+\|\s+\|\s+10.0.0.0/8",
+                                       raw_output))
+
+    def test_meter_rule_show(self):
+        """Test create, show, delete"""
+        raw_output = self.openstack(
+            'network meter rule create ' +
+            '--remote-ip-prefix 10.0.0.0/8 ' +
+            '--egress ' +
+            self.METER_ID
+        )
+        rule_id = re.search(self.re_id, raw_output).group(1)
+
+        self.assertEqual(
+            'egress',
+            re.search(self.re_direction, raw_output).group(1)
+        )
+
+        raw_output = self.openstack('network meter rule show ' + rule_id)
+
+        self.assertEqual(
+            '10.0.0.0/8',
+            re.search(self.re_ip_prefix, raw_output).group(1)
+        )
+        self.assertIsNotNone(rule_id)
+
+        self.addCleanup(self.openstack,
+                        'network meter rule delete ' + rule_id)
diff --git a/openstackclient/tests/unit/network/v2/fakes.py b/openstackclient/tests/unit/network/v2/fakes.py
index 524285ab..eb965339 100644
--- a/openstackclient/tests/unit/network/v2/fakes.py
+++ b/openstackclient/tests/unit/network/v2/fakes.py
@@ -1322,6 +1322,52 @@ class FakeNetworkMeter(object):
         return mock.Mock(side_effect=meter)
 
 
+class FakeNetworkMeterRule(object):
+    """Fake metering rule"""
+
+    @staticmethod
+    def create_one_rule(attrs=None):
+        """Create one meter rule"""
+        attrs = attrs or {}
+
+        meter_rule_attrs = {
+            'id': 'meter-label-rule-id-' + uuid.uuid4().hex,
+            'direction': 'ingress',
+            'excluded': False,
+            'metering_label_id': 'meter-label-id-' + uuid.uuid4().hex,
+            'remote_ip_prefix': '10.0.0.0/24',
+            'tenant_id': 'project-id-' + uuid.uuid4().hex,
+        }
+
+        meter_rule_attrs.update(attrs)
+
+        meter_rule = fakes.FakeResource(
+            info=copy.deepcopy(meter_rule_attrs),
+            loaded=True)
+
+        meter_rule.project_id = meter_rule_attrs['tenant_id']
+
+        return meter_rule
+
+    @staticmethod
+    def create_meter_rule(attrs=None, count=2):
+        """Create multiple meter rules"""
+
+        meter_rules = []
+        for i in range(0, count):
+            meter_rules.append(FakeNetworkMeterRule.
+                               create_one_rule(attrs))
+        return meter_rules
+
+    @staticmethod
+    def get_meter_rule(meter_rule=None, count=2):
+        """Get a list of meter rules"""
+        if meter_rule is None:
+            meter_rule = (FakeNetworkMeterRule.
+                          create_meter_rule(count))
+        return mock.Mock(side_effect=meter_rule)
+
+
 class FakeSubnetPool(object):
     """Fake one or more subnet pools."""
 
diff --git a/openstackclient/tests/unit/network/v2/test_network_meter_rule.py b/openstackclient/tests/unit/network/v2/test_network_meter_rule.py
new file mode 100644
index 00000000..af481793
--- /dev/null
+++ b/openstackclient/tests/unit/network/v2/test_network_meter_rule.py
@@ -0,0 +1,321 @@
+# Copyright (c) 2016, Intel Corporation.
+# All Rights Reserved.
+#
+#    Licensed under the Apache License, Version 2.0 (the "License"); you may
+#    not use this file except in compliance with the License. You may obtain
+#    a copy of the License at
+#
+#         http://www.apache.org/licenses/LICENSE-2.0
+#
+#    Unless required by applicable law or agreed to in writing, software
+#    distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+#    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
+#    License for the specific language governing permissions and limitations
+#    under the License.
+
+import mock
+from mock import call
+
+from osc_lib import exceptions
+
+from openstackclient.network.v2 import network_meter_rule
+from openstackclient.tests.unit.identity.v3 import fakes as identity_fakes_v3
+from openstackclient.tests.unit.network.v2 import fakes as network_fakes
+from openstackclient.tests.unit import utils as tests_utils
+
+
+class TestMeterRule(network_fakes.TestNetworkV2):
+    def setUp(self):
+        super(TestMeterRule, self).setUp()
+        self.network = self.app.client_manager.network
+        self.projects_mock = self.app.client_manager.identity.projects
+        self.domains_mock = self.app.client_manager.identity.domains
+
+
+class TestCreateMeterRule(TestMeterRule):
+    project = identity_fakes_v3.FakeProject.create_one_project()
+    domain = identity_fakes_v3.FakeDomain.create_one_domain()
+
+    new_rule = (
+        network_fakes.FakeNetworkMeterRule.
+        create_one_rule()
+    )
+
+    columns = (
+        'direction',
+        'excluded',
+        'id',
+        'metering_label_id',
+        'project_id',
+        'remote_ip_prefix',
+    )
+    data = (
+        new_rule.direction,
+        new_rule.excluded,
+        new_rule.id,
+        new_rule.metering_label_id,
+        new_rule.project_id,
+        new_rule.remote_ip_prefix,
+    )
+
+    def setUp(self):
+        super(TestCreateMeterRule, self).setUp()
+        fake_meter = network_fakes.FakeNetworkMeter.create_one_meter({
+            'id': self.new_rule.metering_label_id})
+
+        self.network.create_metering_label_rule = mock.Mock(
+            return_value=self.new_rule)
+        self.projects_mock.get.return_value = self.project
+        self.cmd = network_meter_rule.CreateMeterRule(self.app,
+                                                      self.namespace)
+        self.network.find_metering_label = mock.Mock(
+            return_value=fake_meter)
+
+    def test_create_no_options(self):
+        arglist = []
+        verifylist = []
+
+        self.assertRaises(tests_utils.ParserException, self.check_parser,
+                          self.cmd, arglist, verifylist)
+
+    def test_create_default_options(self):
+        arglist = [
+            self.new_rule.metering_label_id,
+            "--remote-ip-prefix", self.new_rule.remote_ip_prefix,
+        ]
+        verifylist = [
+            ('meter', self.new_rule.metering_label_id),
+            ('remote_ip_prefix', self.new_rule.remote_ip_prefix),
+        ]
+
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+        columns, data = (self.cmd.take_action(parsed_args))
+
+        self.network.create_metering_label_rule.assert_called_once_with(
+            **{'direction': 'ingress',
+               'metering_label_id': self.new_rule.metering_label_id,
+               'remote_ip_prefix': self.new_rule.remote_ip_prefix, }
+        )
+        self.assertEqual(self.columns, columns)
+        self.assertEqual(self.data, data)
+
+    def test_create_all_options(self):
+        arglist = [
+            "--ingress",
+            "--include",
+            self.new_rule.metering_label_id,
+            "--remote-ip-prefix", self.new_rule.remote_ip_prefix,
+        ]
+        verifylist = [
+            ('ingress', True),
+            ('include', True),
+            ('meter', self.new_rule.metering_label_id),
+            ('remote_ip_prefix', self.new_rule.remote_ip_prefix),
+        ]
+
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+        columns, data = (self.cmd.take_action(parsed_args))
+
+        self.network.create_metering_label_rule.assert_called_once_with(
+            **{'direction': self.new_rule.direction,
+               'excluded': self.new_rule.excluded,
+               'metering_label_id': self.new_rule.metering_label_id,
+               'remote_ip_prefix': self.new_rule.remote_ip_prefix, }
+        )
+        self.assertEqual(self.columns, columns)
+        self.assertEqual(self.data, data)
+
+
+class TestDeleteMeterRule(TestMeterRule):
+    def setUp(self):
+        super(TestDeleteMeterRule, self).setUp()
+        self.rule_list = \
+            network_fakes.FakeNetworkMeterRule.create_meter_rule(
+                count=2
+            )
+        self.network.delete_metering_label_rule = mock.Mock(return_value=None)
+
+        self.network.find_metering_label_rule = network_fakes \
+            .FakeNetworkMeterRule.get_meter_rule(
+                meter_rule=self.rule_list
+            )
+
+        self.cmd = network_meter_rule.DeleteMeterRule(self.app,
+                                                      self.namespace)
+
+    def test_delete_one_rule(self):
+        arglist = [
+            self.rule_list[0].id,
+        ]
+        verifylist = [
+            ('meter_rule_id', [self.rule_list[0].id]),
+        ]
+
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        result = self.cmd.take_action(parsed_args)
+
+        self.network.delete_metering_label_rule.assert_called_once_with(
+            self.rule_list[0]
+        )
+        self.assertIsNone(result)
+
+    def test_delete_multiple_rules(self):
+        arglist = []
+        for rule in self.rule_list:
+            arglist.append(rule.id)
+        verifylist = [
+            ('meter_rule_id', arglist),
+        ]
+
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        result = self.cmd.take_action(parsed_args)
+
+        calls = []
+        for rule in self.rule_list:
+            calls.append(call(rule))
+        self.network.delete_metering_label_rule.assert_has_calls(calls)
+        self.assertIsNone(result)
+
+    def test_delete_multiple_rules_exception(self):
+        arglist = [
+            self.rule_list[0].id,
+            'xxxx-yyyy-zzzz',
+            self.rule_list[1].id,
+        ]
+        verifylist = [
+            ('meter_rule_id', arglist),
+        ]
+
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        return_find = [
+            self.rule_list[0],
+            exceptions.NotFound('404'),
+            self.rule_list[1],
+        ]
+        self.network.find_metering_label_rule = mock.Mock(
+            side_effect=return_find
+        )
+
+        ret_delete = [
+            None,
+            exceptions.NotFound('404'),
+        ]
+        self.network.delete_metering_label_rule = mock.Mock(
+            side_effect=ret_delete
+        )
+
+        self.assertRaises(exceptions.CommandError, self.cmd.take_action,
+                          parsed_args)
+
+        calls = [
+            call(self.rule_list[0]),
+            call(self.rule_list[1]),
+        ]
+        self.network.delete_metering_label_rule.assert_has_calls(calls)
+
+
+class TestListMeterRule(TestMeterRule):
+    rule_list = \
+        network_fakes.FakeNetworkMeterRule.create_meter_rule(
+            count=2
+        )
+
+    columns = (
+        'ID',
+        'Excluded',
+        'Direction',
+        'Remote IP Prefix',
+    )
+
+    data = []
+
+    for rule in rule_list:
+        data.append((
+            rule.id,
+            rule.excluded,
+            rule.direction,
+            rule.remote_ip_prefix,
+        ))
+
+    def setUp(self):
+        super(TestListMeterRule, self).setUp()
+
+        self.network.metering_label_rules = mock.Mock(
+            return_value=self.rule_list
+        )
+
+        self.cmd = network_meter_rule.ListMeterRule(self.app,
+                                                    self.namespace)
+
+    def test_rule_list(self):
+        arglist = []
+        verifylist = []
+
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        columns, data = self.cmd.take_action(parsed_args)
+
+        self.network.metering_label_rules.assert_called_with()
+        self.assertEqual(self.columns, columns)
+        self.assertEqual(self.data, list(data))
+
+
+class TestShowMeterRule(TestMeterRule):
+    new_rule = (
+        network_fakes.FakeNetworkMeterRule.
+        create_one_rule()
+    )
+
+    columns = (
+        'direction',
+        'excluded',
+        'id',
+        'metering_label_id',
+        'project_id',
+        'remote_ip_prefix',
+    )
+
+    data = (
+        new_rule.direction,
+        new_rule.excluded,
+        new_rule.id,
+        new_rule.metering_label_id,
+        new_rule.project_id,
+        new_rule.remote_ip_prefix,
+    )
+
+    def setUp(self):
+        super(TestShowMeterRule, self).setUp()
+
+        self.cmd = network_meter_rule.ShowMeterRule(self.app,
+                                                    self.namespace)
+
+        self.network.find_metering_label_rule = \
+            mock.Mock(return_value=self.new_rule)
+
+    def test_show_no_options(self):
+        arglist = []
+        verifylist = []
+
+        self.assertRaises(tests_utils.ParserException, self.check_parser,
+                          self.cmd, arglist, verifylist)
+
+    def test_label_rule_show_option(self):
+        arglist = [
+            self.new_rule.id,
+        ]
+        verifylist = [
+            ('meter_rule_id', self.new_rule.id),
+        ]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        columns, data = self.cmd.take_action(parsed_args)
+
+        self.network.find_metering_label_rule.assert_called_with(
+            self.new_rule.id, ignore_missing=False
+        )
+        self.assertEqual(self.columns, columns)
+        self.assertEqual(self.data, data)