diff --git a/horizon/tables/base.py b/horizon/tables/base.py index 4088e18262..51a3affa56 100644 --- a/horizon/tables/base.py +++ b/horizon/tables/base.py @@ -1685,9 +1685,7 @@ class DataTable(object): By default, this returns a ``name`` attribute from the given object, but this can be overridden to return other values. """ - if hasattr(datum, 'name'): - return datum.name - return None + return getattr(datum, 'name', None) def has_prev_data(self): """Returns a boolean value indicating whether there is previous data diff --git a/openstack_dashboard/test/test_data/keystone_data.py b/openstack_dashboard/test/test_data/keystone_data.py index a1e9db6770..c4a831ace1 100644 --- a/openstack_dashboard/test/test_data/keystone_data.py +++ b/openstack_dashboard/test/test_data/keystone_data.py @@ -422,6 +422,6 @@ def data(TEST): ] } idp_mapping = mappings.Mapping( - mappings.MappingManager, + mappings.MappingManager(None), idp_mapping_dict) TEST.idp_mappings.add(idp_mapping) diff --git a/tox.ini b/tox.ini index badcc63535..6b3d80c7f8 100644 --- a/tox.ini +++ b/tox.ini @@ -29,6 +29,7 @@ commands = openstack_dashboard.dashboards.admin.metering \ openstack_dashboard.dashboards.admin.routers \ openstack_dashboard.dashboards.admin.volumes.volumes.tests \ + openstack_dashboard.dashboards.identity.identity_providers \ openstack_dashboard.dashboards.identity.users \ openstack_dashboard.dashboards.project.access_and_security.api_access.tests \ openstack_dashboard.dashboards.project.containers \