feat: add eq comparison for PermissionsRule

Author: jorwoods

tableauserverclient/models/permissions_item.py

@@ -53,9 +53,16 @@ def __init__(self, grantee: ResourceReference, capabilities: Dict[str, str]) ->
     def __repr__(self):
         return "<PermissionsRule grantee={}, capabilities={}>".format(self.grantee, self.capabilities)
 
+    def __eq__(self, other: "PermissionsRule") -> bool:
+        return self.grantee == other.grantee and self.capabilities == other.capabilities
+
     def __and__(self, other: "PermissionsRule") -> "PermissionsRule":
         if self.grantee != other.grantee:
             raise ValueError("Cannot AND two permissions rules with different grantees")
+
+        if self.capabilities == other.capabilities:
+            return self
+
         capabilities = set((*self.capabilities.keys(), *other.capabilities.keys()))
         new_capabilities = {}
         for capability in capabilities:
@@ -72,6 +79,10 @@ def __and__(self, other: "PermissionsRule") -> "PermissionsRule":
     def __or__(self, other: "PermissionsRule") -> "PermissionsRule":
         if self.grantee != other.grantee:
             raise ValueError("Cannot OR two permissions rules with different grantees")
+
+        if self.capabilities == other.capabilities:
+            return self
+
         capabilities = set((*self.capabilities.keys(), *other.capabilities.keys()))
         new_capabilities = {}
         for capability in capabilities:

test/test_permissionsrule.py

@@ -58,3 +58,49 @@ def test_or(self):
         self.assertEqual(composite.capabilities.get(TSC.Permission.Capability.Delete), TSC.Permission.Mode.Allow)
         self.assertEqual(composite.capabilities.get(TSC.Permission.Capability.ViewComments), TSC.Permission.Mode.Allow)
         self.assertEqual(composite.capabilities.get(TSC.Permission.Capability.ExportXml), TSC.Permission.Mode.Deny)
+
+    def test_eq_false(self):
+        grantee = ResourceReference("a", "user")
+        rule1 = TSC.PermissionsRule(
+            grantee,
+            {
+                TSC.Permission.Capability.ExportData: TSC.Permission.Mode.Allow,
+                TSC.Permission.Capability.Delete: TSC.Permission.Mode.Deny,
+                TSC.Permission.Capability.ViewComments: TSC.Permission.Mode.Allow,
+                TSC.Permission.Capability.ExportXml: TSC.Permission.Mode.Deny,
+            },
+        )
+        rule2 = TSC.PermissionsRule(
+            grantee,
+            {
+                TSC.Permission.Capability.ExportData: TSC.Permission.Mode.Allow,
+                TSC.Permission.Capability.Delete: TSC.Permission.Mode.Allow,
+                TSC.Permission.Capability.ExportXml: TSC.Permission.Mode.Deny,
+            },
+        )
+
+        self.assertNotEqual(rule1, rule2)
+
+    def test_eq_true(self):
+        grantee = ResourceReference("a", "user")
+        rule1 = TSC.PermissionsRule(
+            grantee,
+            {
+                TSC.Permission.Capability.ExportData: TSC.Permission.Mode.Allow,
+                TSC.Permission.Capability.Delete: TSC.Permission.Mode.Deny,
+                TSC.Permission.Capability.ViewComments: TSC.Permission.Mode.Allow,
+                TSC.Permission.Capability.ExportXml: TSC.Permission.Mode.Deny,
+            },
+        )
+        rule2 = TSC.PermissionsRule(
+            grantee,
+            {
+                TSC.Permission.Capability.ExportData: TSC.Permission.Mode.Allow,
+                TSC.Permission.Capability.Delete: TSC.Permission.Mode.Deny,
+                TSC.Permission.Capability.ViewComments: TSC.Permission.Mode.Allow,
+                TSC.Permission.Capability.ExportXml: TSC.Permission.Mode.Deny,
+            },
+        )
+        self.assertEqual(rule1, rule2)
+
+