style: black

Author: jorwoods

tableauserverclient/models/permissions_item.py

@@ -59,7 +59,10 @@ def __and__(self, other: "PermissionsRule") -> "PermissionsRule":
         capabilities = set((*self.capabilities.keys(), *other.capabilities.keys()))
         new_capabilities = {}
         for capability in capabilities:
-            if (self.capabilities.get(capability), other.capabilities.get(capability)) == (Permission.Mode.Allow, Permission.Mode.Allow):
+            if (self.capabilities.get(capability), other.capabilities.get(capability)) == (
+                Permission.Mode.Allow,
+                Permission.Mode.Allow,
+            ):
                 new_capabilities[capability] = Permission.Mode.Allow
             elif Permission.Mode.Deny in (self.capabilities.get(capability), other.capabilities.get(capability)):
                 new_capabilities[capability] = Permission.Mode.Deny
@@ -74,7 +77,10 @@ def __or__(self, other: "PermissionsRule") -> "PermissionsRule":
         for capability in capabilities:
             if Permission.Mode.Allow in (self.capabilities.get(capability), other.capabilities.get(capability)):
                 new_capabilities[capability] = Permission.Mode.Allow
-            elif (self.capabilities.get(capability), other.capabilities.get(capability)) == (Permission.Mode.Deny, Permission.Mode.Deny):
+            elif (self.capabilities.get(capability), other.capabilities.get(capability)) == (
+                Permission.Mode.Deny,
+                Permission.Mode.Deny,
+            ):
                 new_capabilities[capability] = Permission.Mode.Deny
 
         return PermissionsRule(self.grantee, new_capabilities)

test/test_permissionsrule.py

@@ -3,20 +3,27 @@
 import tableauserverclient as TSC
 from tableauserverclient.models.reference_item import ResourceReference
 
+
 class TestPermissionsRules(unittest.TestCase):
     def test_and(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,
-        })
+        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,
+            },
+        )
 
         composite = rule1 & rule2
 
@@ -25,25 +32,29 @@ def test_and(self):
         self.assertEqual(composite.capabilities.get(TSC.Permission.Capability.ViewComments), None)
         self.assertEqual(composite.capabilities.get(TSC.Permission.Capability.ExportXml), TSC.Permission.Mode.Deny)
 
-
     def test_or(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,
-        })
+        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,
+            },
+        )
 
         composite = rule1 | rule2
 
         self.assertEqual(composite.capabilities.get(TSC.Permission.Capability.ExportData), TSC.Permission.Mode.Allow)
         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)
-