Add more tests

.gitignore

@@ -122,3 +122,4 @@ local/
 *.sqlite3
 *.db
 local_stubs/
+*.*-invalid

cogs/archive.py

@@ -25,8 +25,8 @@
 class ArchiveCommandCog(TeXBotBaseCog):
     """Cog class that defines the "/archive" command and its call-back method."""
 
-    @staticmethod
-    async def autocomplete_get_categories(ctx: TeXBotAutocompleteContext) -> set[discord.OptionChoice]:  # noqa: E501
+    @classmethod
+    async def autocomplete_get_categories(cls, ctx: TeXBotAutocompleteContext) -> set[discord.OptionChoice]:  # noqa: E501
         """
         Autocomplete callable that generates the set of available selectable categories.
 

cogs/delete_all.py

@@ -20,8 +20,8 @@ class DeleteAllCommandsCog(TeXBotBaseCog):
         "Delete all instances of the selected object type from the backend database",
     )
 
-    @staticmethod
-    async def _delete_all(ctx: TeXBotApplicationContext, delete_model: type[AsyncBaseModel]) -> None:  # noqa: E501
+    @classmethod
+    async def _delete_all(cls, ctx: TeXBotApplicationContext, delete_model: type[AsyncBaseModel]) -> None:  # noqa: E501
         """Perform the actual deletion process of all instances of the given model class."""
         # noinspection PyProtectedMember
         await delete_model._default_manager.all().adelete()

cogs/edit_message.py

@@ -22,8 +22,8 @@ class EditMessageCommandCog(TeXBotBaseCog):
     # noinspection SpellCheckingInspection
     """Cog class that defines the "/editmessage" command and its call-back method."""
 
-    @staticmethod
-    async def autocomplete_get_text_channels(ctx: TeXBotAutocompleteContext) -> set[discord.OptionChoice]:  # noqa: E501
+    @classmethod
+    async def autocomplete_get_text_channels(cls, ctx: TeXBotAutocompleteContext) -> set[discord.OptionChoice]:  # noqa: E501
         """
         Autocomplete callable that generates the set of available selectable channels.
 

cogs/induct.py

@@ -12,8 +12,10 @@
 
 
 import contextlib
+import logging
 import random
 import re
+from logging import Logger
 from typing import Literal
 
 import discord
@@ -36,6 +38,8 @@
 )
 from utils.error_capture_decorators import capture_guild_does_not_exist_error
 
+logger: Logger = logging.getLogger("texbot")
+
 
 class InductSendMessageCog(TeXBotBaseCog):
     """Cog class that defines the "/induct" command and its call-back method."""
@@ -143,6 +147,8 @@ async def get_random_welcome_message(self, induction_member: discord.User | disc
 
         if "<User>" in random_welcome_message:
             if not induction_member:
+                logger.warning("No user provided, when retrieving a random welcome message.")
+
                 return await self.get_random_welcome_message(induction_member)
 
             random_welcome_message = random_welcome_message.replace(
@@ -154,6 +160,10 @@ async def get_random_welcome_message(self, induction_member: discord.User | disc
             try:
                 committee_role_mention: str = (await self.bot.committee_role).mention
             except CommitteeRoleDoesNotExistError:
+                logger.warning(
+                    "Committee role does not exist, when retrieving a random welcome message.",
+                )
+
                 return await self.get_random_welcome_message(induction_member)
             else:
                 random_welcome_message = random_welcome_message.replace(
@@ -163,6 +173,11 @@ async def get_random_welcome_message(self, induction_member: discord.User | disc
 
         if "<Purchase_Membership_URL>" in random_welcome_message:
             if not settings["PURCHASE_MEMBERSHIP_URL"]:
+                logger.warning(
+                    "Env variable `PURCHASE_MEMBERSHIP_URL` is not set, "
+                    "when retrieving a random welcome message.",
+                )
+
                 return await self.get_random_welcome_message(induction_member)
 
             random_welcome_message = random_welcome_message.replace(
@@ -263,8 +278,8 @@ async def _perform_induction(self, ctx: TeXBotApplicationContext, induction_memb
 class InductCommandCog(BaseInductCog):
     """Cog class that defines the "/induct" command and its call-back method."""
 
-    @staticmethod
-    async def autocomplete_get_members(ctx: TeXBotAutocompleteContext) -> set[discord.OptionChoice]:  # noqa: E501
+    @classmethod
+    async def autocomplete_get_members(cls, ctx: TeXBotAutocompleteContext) -> set[discord.OptionChoice]:  # noqa: E501
         """
         Autocomplete callable that generates the set of available selectable members.
 

cogs/remind_me.py

@@ -33,8 +33,8 @@
 class RemindMeCommandCog(TeXBotBaseCog):
     """Cog class that defines the "/remindme" command and its call-back method."""
 
-    @staticmethod
-    async def autocomplete_get_delays(ctx: TeXBotAutocompleteContext) -> set[str]:  # noqa: C901, PLR0912, PLR0915
+    @classmethod
+    async def autocomplete_get_delays(cls, ctx: TeXBotAutocompleteContext) -> set[str]:  # noqa: C901, PLR0912, PLR0915
         """
         Autocomplete callable that generates the common delay input values.
 

cogs/stats.py

@@ -136,7 +136,7 @@ def plot_bar_chart(data: dict[str, int], x_label: str, y_label: str, title: str,
 class StatsCommandsCog(TeXBotBaseCog):
     """Cog class that defines the "/stats" command group and its command call-back methods."""
 
-    _DISCORD_SERVER_NAME: Final[str] = f"""{
+    _DISCORD_GUILD_NAME: Final[str] = f"""{
         "the " if (
             settings["_GROUP_SHORT_NAME"] is not None
             and (
@@ -161,7 +161,7 @@ class StatsCommandsCog(TeXBotBaseCog):
 
     stats: discord.SlashCommandGroup = discord.SlashCommandGroup(
         "stats",
-        f"Various statistics about {_DISCORD_SERVER_NAME}",
+        f"Various statistics about {_DISCORD_GUILD_NAME}",
     )
 
     # noinspection SpellCheckingInspection
@@ -188,15 +188,15 @@ async def channel_stats(self, ctx: TeXBotApplicationContext, str_channel_id: str
         """
         channel_id: int = ctx.channel_id
 
-        if str_channel_id:
-            if not re.match(r"\A\d{17,20}\Z", str_channel_id):
+        if str_channel_id.strip():
+            if not re.match(r"\A\d{17,20}\Z", str_channel_id.strip()):
                 await self.command_send_error(
                     ctx,
-                    message=f"{str_channel_id!r} is not a valid channel ID.",
+                    message=f"{str_channel_id.strip()!r} is not a valid channel ID.",
                 )
                 return
 
-            channel_id = int(str_channel_id)
+            channel_id = int(str_channel_id.strip())
 
         # NOTE: Shortcut accessors are placed at the top of the function, so that the exceptions they raise are displayed before any further errors may be sent
         guild: discord.Guild = self.bot.main_guild
@@ -289,7 +289,7 @@ async def channel_stats(self, ctx: TeXBotApplicationContext, str_channel_id: str
 
     @stats.command(
         name="server",
-        description=f"Displays the stats for the whole of {_DISCORD_SERVER_NAME}",
+        description=f"Displays the stats for the whole of {_DISCORD_GUILD_NAME}",
     )
     async def server_stats(self, ctx: TeXBotApplicationContext) -> None:
         """
@@ -509,7 +509,7 @@ async def user_stats(self, ctx: TeXBotApplicationContext) -> None:
     # noinspection SpellCheckingInspection
     @stats.command(
         name="leftmembers",
-        description=f"Displays the stats about members that have left {_DISCORD_SERVER_NAME}",
+        description=f"Displays the stats about members that have left {_DISCORD_GUILD_NAME}",
     )
     async def left_member_stats(self, ctx: TeXBotApplicationContext) -> None:
         """

cogs/strike.py

@@ -716,8 +716,8 @@ async def on_member_ban(self, guild: discord.Guild, user: discord.User | discord
 class StrikeCommandCog(BaseStrikeCog):
     """Cog class that defines the "/strike" command and its call-back method."""
 
-    @staticmethod
-    async def strike_autocomplete_get_members(ctx: TeXBotAutocompleteContext) -> set[discord.OptionChoice]:  # noqa: E501
+    @classmethod
+    async def strike_autocomplete_get_members(cls, ctx: TeXBotAutocompleteContext) -> set[discord.OptionChoice]:  # noqa: E501
         """
         Autocomplete callable that generates the set of available selectable members.