summaryrefslogtreecommitdiff
path: root/openstack_dashboard/dashboards/project/networks/ports/tests.py
diff options
context:
space:
mode:
Diffstat (limited to 'openstack_dashboard/dashboards/project/networks/ports/tests.py')
-rw-r--r--openstack_dashboard/dashboards/project/networks/ports/tests.py122
1 files changed, 122 insertions, 0 deletions
diff --git a/openstack_dashboard/dashboards/project/networks/ports/tests.py b/openstack_dashboard/dashboards/project/networks/ports/tests.py
index 544f18d07..b0348797d 100644
--- a/openstack_dashboard/dashboards/project/networks/ports/tests.py
+++ b/openstack_dashboard/dashboards/project/networks/ports/tests.py
@@ -16,10 +16,13 @@
import collections
import copy
+from django.test.utils import override_settings
from django.urls import reverse
import mock
+from openstack_auth import utils as auth_utils
+
from horizon.workflows import views
from openstack_dashboard import api
@@ -288,6 +291,66 @@ class NetworkPortTests(test.TestCase):
self.mock_network_get.assert_called_once_with(
test.IsHttpRequest(), network.id)
+ @override_settings(POLICY_CHECK_FUNCTION='openstack_auth.policy.check')
+ @test.create_mocks({api.neutron: ('port_get',
+ 'network_get',
+ 'is_extension_supported')})
+ def test_add_allowed_address_pair_button_shown_to_network_owner(self):
+ port = self.ports.first()
+
+ url = reverse('horizon:project:networks:ports:addallowedaddresspairs',
+ args=[port.id])
+ classes = 'btn data-table-action btn-default ajax-modal'
+ link_name = "Add Allowed Address Pair"
+
+ expected_string = \
+ '<a id="allowed_address_pairs__action_AddAllowedAddressPair" ' \
+ 'class="%s" href="%s" title="Add Allowed Address Pair">' \
+ '<span class="fa fa-plus"></span> %s</a>' \
+ % (classes, url, link_name)
+
+ res = self.client.get(reverse('horizon:project:networks:ports:detail',
+ args=[port.id]))
+
+ self.assertTemplateUsed(res, 'horizon/common/_detail_tab_group.html')
+ self.assertIn(expected_string, res.context_data['tab_group'].render())
+
+ @override_settings(POLICY_CHECK_FUNCTION='openstack_auth.policy.check')
+ @test.create_mocks({api.neutron: ('port_get',
+ 'network_get',
+ 'is_extension_supported')})
+ def test_add_allowed_address_pair_button_disabled_to_other_tenant(self):
+ # Current user tenant_id is 1 so select port whose tenant_id is
+ # other than 1 for checking "Add Allowed Address Pair" button is not
+ # displayed on the screen.
+ user = auth_utils.get_user(self.request)
+
+ # select port such that tenant_id is different from user's tenant_id.
+ port = [p for p in self.ports.list()
+ if p.tenant_id != user.tenant_id][0]
+
+ self._stub_is_extension_supported(
+ {'allowed-address-pairs': False, 'mac-learning': False})
+
+ with mock.patch('openstack_auth.utils.get_user', return_value=user):
+ url = reverse(
+ 'horizon:project:networks:ports:addallowedaddresspairs',
+ args=[port.id])
+ classes = 'btn data-table-action btn-default ajax-modal'
+ link_name = "Add Allowed Address Pair"
+
+ expected_string = \
+ '<a id="allowed_address_pairs__action_AddAllowedAddressPair" ' \
+ 'class="%s" href="%s" title="Add Allowed Address Pair">' \
+ '<span class="fa fa-plus"></span> %s</a>' \
+ % (classes, url, link_name)
+
+ res = self.client.get(reverse(
+ 'horizon:project:networks:ports:detail', args=[port.id]))
+
+ self.assertNotIn(
+ expected_string, res.context_data['tab_group'].render())
+
@test.create_mocks({api.neutron: ('port_get',
'port_update')})
def test_port_add_allowed_address_pair(self):
@@ -345,6 +408,65 @@ class NetworkPortTests(test.TestCase):
self.assertFormErrors(res, 1)
self.assertContains(res, "Incorrect format for IP address")
+ @override_settings(POLICY_CHECK_FUNCTION='openstack_auth.policy.check')
+ @test.create_mocks({api.neutron: ('port_get',
+ 'network_get',
+ 'port_update',
+ 'is_extension_supported')})
+ def test_delete_address_pair_button_shown_to_network_owner(self):
+ port = self.ports.first()
+ classes = 'data-table-action btn-danger btn'
+
+ expected_string = \
+ '<button data-batch-action="true" ' \
+ 'id="allowed_address_pairs__action_delete" ' \
+ 'class="%s" name="action" help_text="This action cannot be ' \
+ 'undone." type="submit" value="allowed_address_pairs__delete">' \
+ '<span class="fa fa-trash"></span>' \
+ ' Delete</button>' \
+ % (classes)
+
+ res = self.client.get(reverse(
+ 'horizon:project:networks:ports:detail', args=[port.id]))
+
+ self.assertTemplateUsed(res, 'horizon/common/_detail_tab_group.html')
+ self.assertIn(expected_string, res.context_data['tab_group'].render())
+
+ @override_settings(POLICY_CHECK_FUNCTION='openstack_auth.policy.check')
+ @test.create_mocks({api.neutron: ('port_get',
+ 'network_get',
+ 'is_extension_supported')})
+ def test_delete_address_pair_button_disabled_to_other_tenant(self):
+ # Current user tenant_id is 1 so select port whose tenant_id is
+ # other than 1 for checking "Delete Allowed Address Pair" button is
+ # not displayed on the screen.
+ user = auth_utils.get_user(self.request)
+
+ # select port such that tenant_id is different from user's tenant_id.
+ port = [p for p in self.ports.list()
+ if p.tenant_id != user.tenant_id][0]
+
+ self._stub_is_extension_supported(
+ {'allowed-address-pairs': False, 'mac-learning': False})
+
+ with mock.patch('openstack_auth.utils.get_user', return_value=user):
+ classes = 'data-table-action btn-danger btn'
+
+ expected_string = \
+ '<button data-batch-action="true" ' \
+ 'id="allowed_address_pairs__action_delete" ' \
+ 'class="%s" name="action" help_text="This action cannot be ' \
+ 'undone." type="submit" ' \
+ 'value="allowed_address_pairs__delete">' \
+ '<span class="fa fa-trash"></span>' \
+ ' Delete</button>' % (classes)
+
+ res = self.client.get(reverse(
+ 'horizon:project:networks:ports:detail', args=[port.id]))
+
+ self.assertNotIn(
+ expected_string, res.context_data['tab_group'].render())
+
@test.create_mocks({api.neutron: ('port_get',
'is_extension_supported',
'port_update')})