Fix E711 comparison to None

Related: https://pagure.io/freeipa/issue/8306
Signed-off-by: Christian Heimes <cheimes@redhat.com>
Reviewed-By: Florence Blanc-Renaud <frenaud@redhat.com>
This commit is contained in:
Christian Heimes 2020-04-30 14:55:15 +02:00
parent 86d76efcef
commit 9661807385
4 changed files with 10 additions and 9 deletions

View File

@ -801,7 +801,7 @@ class SimpleServiceInstance(Service):
def __enable(self): def __enable(self):
self.backup_state("enabled", self.is_enabled()) self.backup_state("enabled", self.is_enabled())
if self.gensvc_name == None: if self.gensvc_name is None:
self.enable() self.enable()
else: else:
self.ldap_configure(self.gensvc_name, self.fqdn, None, self.suffix) self.ldap_configure(self.gensvc_name, self.fqdn, None, self.suffix)

View File

@ -335,8 +335,9 @@ class selinuxusermap_add(LDAPCreate):
entry_attrs['ipaenabledflag'] = 'TRUE' entry_attrs['ipaenabledflag'] = 'TRUE'
validate_selinuxuser_inlist(ldap, entry_attrs['ipaselinuxuser']) validate_selinuxuser_inlist(ldap, entry_attrs['ipaselinuxuser'])
# hbacrule is not allowed when usercat or hostcat is set def is_to_be_set(x):
is_to_be_set = lambda x: x in entry_attrs and entry_attrs[x] != None """hbacrule is not allowed when usercat or hostcat is set"""
return x in entry_attrs and entry_attrs[x] is not None
are_local_members_to_be_set = any(is_to_be_set(attr) are_local_members_to_be_set = any(is_to_be_set(attr)
for attr in ('usercategory', for attr in ('usercategory',

View File

@ -136,7 +136,7 @@ class test_Fuzzy:
assert (False == self.klass()) is True assert (False == self.klass()) is True
assert (True == self.klass()) is True assert (True == self.klass()) is True
assert (None == self.klass()) is True assert (None == self.klass()) is True # noqa
def test_assert_deepequal(pytestconfig): def test_assert_deepequal(pytestconfig):

View File

@ -132,9 +132,9 @@ class test_hbactest(XMLRPC_test):
nodetail=True nodetail=True
) )
assert ret['value'] == True assert ret['value'] == True
assert ret['error'] == None assert ret['error'] is None
assert ret['matched'] == None assert ret['matched'] is None
assert ret['notmatched'] == None assert ret['notmatched'] is None
def test_c_hbactest_check_rules_enabled_detail(self): def test_c_hbactest_check_rules_enabled_detail(self):
""" """
@ -181,8 +181,8 @@ class test_hbactest(XMLRPC_test):
) )
assert ret['value'] == False assert ret['value'] == False
assert ret['matched'] == None assert ret['matched'] is None
assert ret['notmatched'] == None assert ret['notmatched'] is None
for rule in self.rule_names: for rule in self.rule_names:
assert u'%s_1x1' % (rule) in ret['error'] assert u'%s_1x1' % (rule) in ret['error']