Correct channel permissions calculation, cleanup channel tests

disco/types/channel.py

@@ -162,18 +162,27 @@ class Channel(SlottedModel, Permissible):
         member = self.guild.get_member(user)
         base = self.guild.get_permissions(member)
 
-        ow_everyone = self.overwrites.get(self.guild_id)
+        # First grab and apply the everyone overwrite
-        if ow_everyone:
+        everyone = self.overwrites.get(self.guild_id)
-            base += ow_everyone.compiled
+        if everyone:
-
+            base -= everyone.deny
+            base += everyone.allow
+
+        denies = 0
+        allows = 0
         for role_id in member.roles:
-            ow_role = self.overwrites.get(role_id)
+            overwrite = self.overwrites.get(role_id)
-            if ow_role:
+            if overwrite:
-                base += ow_role.compiled
+                denies |= overwrite.deny
+                allows |= overwrite.allow
+
+        base -= denies
+        base += allows
 
         ow_member = self.overwrites.get(member.user.id)
         if ow_member:
-            base += ow_member.compiled
+            base -= ow_member.deny
+            base += ow_member.allow
 
         return base
 
@@ -200,7 +209,7 @@ class Channel(SlottedModel, Permissible):
         """
         Whether this channel is an NSFW channel.
         """
-        return self.type == ChannelType.GUILD_TEXT and (self.nsfw or NSFW_RE.match(self.name))
+        return bool(self.type == ChannelType.GUILD_TEXT and (self.nsfw or NSFW_RE.match(self.name)))
 
     @property
     def is_voice(self):

disco/types/guild.py

@@ -357,6 +357,7 @@ class Guild(SlottedModel, Permissible):
         if self.owner_id == member.id:
             return PermissionValue(Permissions.ADMINISTRATOR)
 
+        # Our value starts with the guilds default (@everyone) role permissions
         value = PermissionValue(self.roles.get(self.id).permissions)
 
         # Iterate over all roles the user has (plus the @everyone role)

tests/types/channel.py

@@ -1,25 +1,23 @@
-from unittest import TestCase
-
 from disco.types.channel import Channel, ChannelType
 
 
-class TestChannel(TestCase):
+def test_deprecated_nsfw_channel():
-    def test_deprecated_nsfw_channel(self):
+    channel = Channel(
-        channel = Channel(
+        name='nsfw-testing',
-            name='nsfw-testing',
+        type=ChannelType.GUILD_TEXT)
-            type=ChannelType.GUILD_TEXT)
+    assert channel.is_nsfw
-        self.assertTrue(channel.is_nsfw)
+
+    channel = Channel(
+        name='nsfw-testing',
+        type=ChannelType.GUILD_VOICE)
+    assert not channel.is_nsfw
 
-        channel = Channel(
+    channel = Channel(
-            name='nsfw-testing',
+        name='nsfw_testing',
-            type=ChannelType.GUILD_VOICE)
+        type=ChannelType.GUILD_TEXT)
-        self.assertFalse(channel.is_nsfw)
+    assert not channel.is_nsfw
 
-        channel = Channel(
-            name='nsfw_testing',
-            type=ChannelType.GUILD_TEXT)
-        self.assertFalse(channel.is_nsfw)
 
-    def test_nsfw_channel(self):
+def test_nsfw_channel():
-        channel = Channel(name='test', nsfw=True, type=ChannelType.GUILD_TEXT)
+    channel = Channel(name='test', nsfw=True, type=ChannelType.GUILD_TEXT)
-        assert channel.is_nsfw
+    assert channel.is_nsfw