Skip to content

Commit 6dd3f45

Browse files
Jenkinsopenstack-gerrit
authored andcommitted
Merge "Show "target_project_id" attribute properly for network rbac object"
2 parents 79459ca + 7f93d8c commit 6dd3f45

4 files changed

Lines changed: 5 additions & 12 deletions

File tree

functional/tests/network/v2/test_network_rbac.py

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,6 @@
1212

1313
import uuid
1414

15-
import testtools
16-
1715
from functional.common import test
1816

1917

@@ -57,18 +55,13 @@ def test_network_rbac_show(self):
5755
raw_output = self.openstack('network rbac show ' + self.ID + opts)
5856
self.assertEqual(self.ID + "\n", raw_output)
5957

60-
# TODO(Huanxuan Ao): This test can pass after bug
61-
# https://bugs.launchpad.net/python-openstackclient/+bug/1608903 fixed.
62-
@testtools.skip(
63-
'Skip because of the bug '
64-
'https://bugs.launchpad.net/python-openstackclient/+bug/1608903')
6558
def test_network_rbac_set(self):
6659
opts = self.get_opts(self.FIELDS)
6760
project_id = self.openstack(
6861
'project create ' + self.PROJECT_NAME + opts)
6962
self.openstack('network rbac set ' + self.ID +
7063
' --target-project ' + self.PROJECT_NAME)
71-
opts = self.get_opts(['target_project'])
64+
opts = self.get_opts(['target_project_id'])
7265
raw_output_rbac = self.openstack('network rbac show ' + self.ID + opts)
7366
raw_output_project = self.openstack(
7467
'project delete ' + self.PROJECT_NAME)

openstackclient/network/v2/network_rbac.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ def _get_columns(item):
3333
columns.append('project_id')
3434
if 'target_tenant' in columns:
3535
columns.remove('target_tenant')
36-
columns.append('target_project')
36+
columns.append('target_project_id')
3737
return tuple(sorted(columns))
3838

3939

openstackclient/tests/network/v2/fakes.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -520,7 +520,7 @@ def create_one_network_rbac(attrs=None):
520520
loaded=True)
521521
# Set attributes with special mapping in OpenStack SDK.
522522
rbac.project_id = rbac_attrs['tenant_id']
523-
rbac.target_project = rbac_attrs['target_tenant']
523+
rbac.target_project_id = rbac_attrs['target_tenant']
524524
return rbac
525525

526526
@staticmethod

openstackclient/tests/network/v2/test_network_rbac.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ class TestCreateNetworkRBAC(TestNetworkRBAC):
4949
'object_id',
5050
'object_type',
5151
'project_id',
52-
'target_project',
52+
'target_project_id',
5353
)
5454

5555
data = [
@@ -383,7 +383,7 @@ class TestShowNetworkRBAC(TestNetworkRBAC):
383383
'object_id',
384384
'object_type',
385385
'project_id',
386-
'target_project',
386+
'target_project_id',
387387
)
388388

389389
data = [

0 commit comments

Comments
 (0)