From 5dff46a700a78c95cf9c5ca274920c7fca68ec0c Mon Sep 17 00:00:00 2001
From: Julia Kreger <juliaashleykreger@gmail.com>
Date: Fri, 19 Feb 2021 17:00:32 -0800
Subject: [PATCH] Project scope driver vendor pass-through
The tests were not enabled earlier on, for endpoints that
are not going to be exposed to users, really. So this patch
just updates them so they are tested as expected.
Change-Id: If3c989d5bcf03de5704e30165747642134952f75
---
ironic/tests/unit/api/test_acl.py | 3 ++-
ironic/tests/unit/api/test_rbac_project_scoped.yaml | 9 +++------
2 files changed, 5 insertions(+), 7 deletions(-)
diff --git a/ironic/tests/unit/api/test_acl.py b/ironic/tests/unit/api/test_acl.py
index b670738021..023c47c8c0 100644
--- a/ironic/tests/unit/api/test_acl.py
+++ b/ironic/tests/unit/api/test_acl.py
@@ -446,7 +446,8 @@ class TestRBACProjectScoped(TestACLBase):
'owner_port_ident': owned_node_port['uuid'],
'other_port_ident': other_port['uuid'],
'owner_portgroup_ident': owner_pgroup['uuid'],
- 'other_portgroup_ident': other_pgroup['uuid']})
+ 'other_portgroup_ident': other_pgroup['uuid'],
+ 'driver_name': 'fake-driverz'})
@ddt.file_data('test_rbac_project_scoped.yaml')
@ddt.unpack
diff --git a/ironic/tests/unit/api/test_rbac_project_scoped.yaml b/ironic/tests/unit/api/test_rbac_project_scoped.yaml
index 5f96ea8ddf..18a723c1bc 100644
--- a/ironic/tests/unit/api/test_rbac_project_scoped.yaml
+++ b/ironic/tests/unit/api/test_rbac_project_scoped.yaml
@@ -2214,22 +2214,19 @@ owner_reader_cannot_get_drivers_vendor_passthru:
path: '/v1/drivers/{driver_name}/vendor_passthru/methods'
method: get
headers: *owner_reader_headers
- assert_status: 403
- skip_reason: policy not implemented
+ assert_status: 500
lessee_reader_cannot_get_drivers_vendor_passthru:
path: '/v1/drivers/{driver_name}/vendor_passthru/methods'
method: get
headers: *lessee_reader_headers
- assert_status: 403
- skip_reason: policy not implemented
+ assert_status: 500
third_party_admin_cannot_get_drivers_vendor_passthru:
path: '/v1/drivers/{driver_name}/vendor_passthru/methods'
method: get
headers: *third_party_admin_headers
- assert_status: 403
- skip_reason: policy not implemented
+ assert_status: 500
# Node Bios - https://docs.openstack.org/api-ref/baremetal/#node-bios-nodes