diff options
Diffstat (limited to 'openstack_dashboard/dashboards/project/networks/ports/tests.py')
-rw-r--r-- | openstack_dashboard/dashboards/project/networks/ports/tests.py | 122 |
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')}) |