Clean up mock object.

Author: HexDecimal

tcod/event.py

@@ -89,16 +89,16 @@ def _pixel_to_tile(x: float, y: float) -> Tuple[float, float]:
 Point = NamedTuple("Point", [("x", int), ("y", int)])
 
 # manually define names for SDL macros
-BUTTON_LEFT = lib.SDL_BUTTON_LEFT
-BUTTON_MIDDLE = lib.SDL_BUTTON_MIDDLE
-BUTTON_RIGHT = lib.SDL_BUTTON_RIGHT
-BUTTON_X1 = lib.SDL_BUTTON_X1
-BUTTON_X2 = lib.SDL_BUTTON_X2
-BUTTON_LMASK = lib.SDL_BUTTON_LMASK
-BUTTON_MMASK = lib.SDL_BUTTON_MMASK
-BUTTON_RMASK = lib.SDL_BUTTON_RMASK
-BUTTON_X1MASK = lib.SDL_BUTTON_X1MASK
-BUTTON_X2MASK = lib.SDL_BUTTON_X2MASK
+BUTTON_LEFT = 1
+BUTTON_MIDDLE = 2
+BUTTON_RIGHT = 3
+BUTTON_X1 = 4
+BUTTON_X2 = 5
+BUTTON_LMASK = 0x1
+BUTTON_MMASK = 0x2
+BUTTON_RMASK = 0x4
+BUTTON_X1MASK = 0x8
+BUTTON_X2MASK = 0x10
 
 # reverse tables are used to get the tcod.event name from the value.
 _REVERSE_BUTTON_TABLE = {

tcod/libtcodpy.py

@@ -4404,7 +4404,7 @@ def sys_clipboard_get() -> str:
 @atexit.register
 def _atexit_verify() -> None:
     """Warns if the libtcod root console is implicitly deleted."""
-    if lib.TCOD_ctx.root:
+    if lib and lib.TCOD_ctx.root:
         warnings.warn(
             "The libtcod root console was implicitly deleted.\n"
             "Make sure the 'with' statement is used with the root console to"

tcod/loader.py

@@ -66,24 +66,18 @@ def get_sdl_version() -> str:
 class _Mock(object):
     """Mock object needed for ReadTheDocs."""
 
-    CData = ()  # This gets passed to an isinstance call.
-
     @staticmethod
     def def_extern() -> Any:
         """Pass def_extern call silently."""
         return lambda func: func
 
-    def __getattr__(self, attr: Any) -> Any:
-        """This object pretends to have everything."""
-        return self
-
-    def __call__(self, *args: Any, **kargs: Any) -> Any:
-        """Suppress any other calls"""
-        return self
+    def __getattr__(self, attr: str) -> None:
+        """Return None on any attribute."""
+        return None
 
-    def __str__(self) -> Any:
-        """Just have ? in case anything leaks as a parameter default."""
-        return "?"
+    def __bool__(self) -> bool:
+        """Allow checking for this mock object at import time."""
+        return False
 
 
 lib = None  # type: Any

tcod/path.py

@@ -62,6 +62,8 @@ def _get_pathcost_func(
     name: str,
 ) -> Callable[[int, int, int, int, Any], float]:
     """Return a properly cast PathCostArray callback."""
+    if not ffi:
+        return lambda x1, y1, x2, y2, _: 0
     return ffi.cast(  # type: ignore
         "TCOD_path_func_t", ffi.addressof(lib, name)
     )
@@ -162,7 +164,7 @@ def get_tcod_path_ffi(self) -> Tuple[Any, Any, Tuple[int, int]]:
             "struct PathCostArray*",
             (ffi.cast("char*", self.ctypes.data), self.strides),
         )
-        return callback, userdata, self.shape
+        return callback, userdata, (self.shape[0], self.shape[1])
 
 
 class _PathFinder(object):