make ConversationFilter.include() accept variadic arguments for better API consistency

Author: imryche

blockkit/core.py

@@ -920,7 +920,7 @@ def __init__(
         exclude_external_shared_channels: bool | None = None,
     ):
         super().__init__()
-        self.include(include)
+        self.include(*include or ())
         self.exclude_bot_users(exclude_bot_users)
         self.exclude_external_shared_channels(exclude_external_shared_channels)
         self._add_validator(
@@ -929,12 +929,10 @@ def __init__(
             )
         )
 
-    def include(
-        self, include: list[Literal["im", "mpim", "private", "public"]] | None
-    ) -> Self:
+    def include(self, *include: Literal["im", "mpim", "private", "public"]) -> Self:
         return self._add_field(
             "include",
-            include,
+            list(include),
             validators=[
                 Typed(str),
                 Strings(self.IM, self.MPIM, self.PRIVATE, self.PUBLIC),

tests/test_core.py

@@ -1,5 +1,3 @@
-from blockkit.core import ColumnSettings
-from blockkit.core import RawText
 from datetime import date, datetime, time
 from zoneinfo import ZoneInfo
 
@@ -10,6 +8,7 @@
     Button,
     ChannelsSelect,
     Checkboxes,
+    ColumnSettings,
     ComponentValidationError,
     Confirm,
     Context,
@@ -29,7 +28,6 @@
     Image,
     ImageEl,
     Input,
-    Table,
     InputParameter,
     Markdown,
     Message,
@@ -45,6 +43,7 @@
     Overflow,
     PlainTextInput,
     RadioButtons,
+    RawText,
     RichBroadcastEl,
     RichChannelEl,
     RichColorEl,
@@ -64,6 +63,7 @@
     Section,
     SlackFile,
     StaticSelect,
+    Table,
     Text,
     TimePicker,
     Trigger,
@@ -386,7 +386,7 @@ def test_builds(self):
 
         got = (
             ConversationFilter()
-            .include(["public", "mpim"])
+            .include("public", "mpim")
             .exclude_bot_users()
             .exclude_external_shared_channels()
             .build()