Make base exception and add docstrings

Author: chillymosh

twitchio/ext/commands/bot.py

@@ -542,7 +542,7 @@ async def load_module(self, name: str, *, package: str | None = None) -> None:
             spec.loader.exec_module(module)  # type: ignore
         except Exception as e:
             del sys.modules[name]
-            raise ModuleLoadFailure(e) from e
+            raise ModuleLoadFailure(name, e) from e
 
         try:
             entry = getattr(module, "setup")
@@ -559,7 +559,7 @@ async def load_module(self, name: str, *, package: str | None = None) -> None:
         except Exception as e:
             del sys.modules[name]
             await self._remove_module_remnants(module.__name__)
-            raise ModuleLoadFailure(e) from e
+            raise ModuleLoadFailure(name, e) from e
 
         self.__modules[name] = module
 

twitchio/ext/commands/exceptions.py

@@ -126,21 +126,33 @@ def __init__(self, param: inspect.Parameter) -> None:
         super().__init__(f'"{param.name}" is a required argument which is missing.')
 
 
-class ModuleLoadFailure(TwitchioException):
-    def __init__(self, exc: Exception) -> None:
-        super().__init__(exc)
+class ModuleError(TwitchioException):
+    """Base exception for module related errors."""
 
 
-class NoEntryPointError(TwitchioException):
+class ModuleLoadFailure(ModuleError):
+    """An exception raised when a module failed to load during execution or `setup` entry point."""
+
+    def __init__(self, name: str, exc: Exception) -> None:
+        super().__init__(name, exc)
+
+
+class NoEntryPointError(ModuleError):
+    """Exception raised when the module does not have a `setup` entry point coroutine."""
+
     def __init__(self, msg: str) -> None:
         super().__init__(msg)
 
 
-class ModuleAlreadyLoadedError(TwitchioException):
+class ModuleAlreadyLoadedError(ModuleError):
+    """Exception raised when a module has already been loaded."""
+
     def __init__(self, msg: str) -> None:
         super().__init__(msg)
 
 
-class ModuleNotLoadedError(TwitchioException):
+class ModuleNotLoadedError(ModuleError):
+    """An exception raised when a module was not loaded."""
+
     def __init__(self, msg: str) -> None:
         super().__init__(msg)