Merge "fixed several pep8 issues"
This commit is contained in:
commit
8de4ffab36
@ -570,7 +570,7 @@ class RoleApi(common_ldap.BaseLdap):
|
|||||||
try:
|
try:
|
||||||
roles = self._ldap_get_list(project_subtree, ldap.SCOPE_SUBTREE,
|
roles = self._ldap_get_list(project_subtree, ldap.SCOPE_SUBTREE,
|
||||||
query_params={
|
query_params={
|
||||||
self.member_attribute: user_dn})
|
self.member_attribute: user_dn})
|
||||||
except ldap.NO_SUCH_OBJECT:
|
except ldap.NO_SUCH_OBJECT:
|
||||||
roles = []
|
roles = []
|
||||||
res = []
|
res = []
|
||||||
|
@ -154,9 +154,8 @@ def notify_event_callbacks(service, resource_type, operation, payload):
|
|||||||
'operation': operation,
|
'operation': operation,
|
||||||
'payload': payload}
|
'payload': payload}
|
||||||
LOG.debug(_('Invoking callback %(cb_name)s for event '
|
LOG.debug(_('Invoking callback %(cb_name)s for event '
|
||||||
'%(service)s %(resource_type)s %(operation)s for'
|
'%(service)s %(resource_type)s %(operation)s for'
|
||||||
'%(payload)s'),
|
'%(payload)s'), subst_dict)
|
||||||
subst_dict)
|
|
||||||
cb(service, resource_type, operation, payload)
|
cb(service, resource_type, operation, payload)
|
||||||
|
|
||||||
|
|
||||||
|
@ -416,8 +416,8 @@ class TestEventCallbacks(test_v3.RestfulTestCase):
|
|||||||
|
|
||||||
def foo_callback(self, service, resource_type, operation,
|
def foo_callback(self, service, resource_type, operation,
|
||||||
payload):
|
payload):
|
||||||
callback_called.append(True) # uses callback_called
|
# uses callback_called from the closure
|
||||||
# from the closure
|
callback_called.append(True)
|
||||||
|
|
||||||
Foo()
|
Foo()
|
||||||
project_ref = self.new_project_ref(domain_id=self.domain_id)
|
project_ref = self.new_project_ref(domain_id=self.domain_id)
|
||||||
|
Loading…
Reference in New Issue
Block a user