mirror of
https://salsa.debian.org/freeipa-team/freeipa.git
synced 2025-02-25 18:55:28 -06:00
ipalib.aci: Fix bugs in comparison
- regression inbe6edef6e4: The __ne__ special method was named incorrectly - regression in1ea6def129: The targetattr operator was never compared Include some new comparison tests. Reviewed-By: Martin Kosek <mkosek@redhat.com>
This commit is contained in:
committed by
Martin Kosek
parent
c123d95084
commit
a2aca68f63
@@ -238,8 +238,8 @@ class ACI:
|
||||
|
||||
if set(self.target.get('targetattr', {}).get('expression', ())) != set(b.target.get('targetattr',{}).get('expression', ())):
|
||||
return False
|
||||
if self.target.get('targetattr',{}).get('operator') != b.target.get('targetattr',{}).get('operator'):
|
||||
return False
|
||||
if self.target.get('targetattr',{}).get('operator') != b.target.get('targetattr',{}).get('operator'):
|
||||
return False
|
||||
|
||||
if self.target.get('target',{}).get('expression') != b.target.get('target',{}).get('expression'):
|
||||
return False
|
||||
@@ -255,5 +255,5 @@ class ACI:
|
||||
|
||||
__eq__ = isequal
|
||||
|
||||
def __neq__(self, b):
|
||||
def __ne__(self, b):
|
||||
return not self == b
|
||||
|
||||
Reference in New Issue
Block a user