fix(skins): replace placeholder artwork

bongo_cat/__init__.py

@@ -6,7 +6,7 @@
 from .utils import resource_path, setup_logging
 from . import animations
 
-__version__ = "2.0.8"
+__version__ = "2.0.9"
 __author__ = "luinbytes"
 __description__ = "Interactive desktop pet that responds to keyboard, mouse, and controller inputs"
 

skins/neon/README.md

@@ -39,7 +39,3 @@ If you create a custom neon skin you'd like to share:
 1. Create a new folder in `skins/` with your theme name
 2. Add your images and a `skin.json` file
 3. Consider sharing it in the GitHub discussions!
-
----
-
-*Current images are placeholders - replace with your own neon-styled artwork!*

skins/retro/README.md

@@ -31,7 +31,3 @@ If you create a custom retro skin you'd like to share:
 1. Create a new folder in `skins/` with your theme name
 2. Add your images and a `skin.json` file
 3. Consider sharing it in the GitHub discussions!
-
----
-
-*Current images are placeholders - replace with your own retro-styled artwork!*

tests/test_skin_manager.py

@@ -1,12 +1,17 @@
 import json
 import os
 from pathlib import Path
+import struct
 import tempfile
 import unittest
 
 from bongo_cat.models.skin_manager import SkinManager
 
 
+BUILTIN_SKIN_IDS = ("default", "neon", "retro")
+POSE_FILENAMES = ("cat-rest.png", "cat-left.png", "cat-right.png")
+
+
 class TestSkinManagerPaths(unittest.TestCase):
     def test_explicit_relative_skin_directory_stays_authoritative(self):
         with tempfile.TemporaryDirectory() as working_dir:
@@ -40,5 +45,36 @@ def test_explicit_relative_skin_directory_stays_authoritative(self):
         self.assertEqual(["fixture"], manager.get_skin_ids())
 
 
+class TestBuiltInSkinAssets(unittest.TestCase):
+    def test_builtin_skin_assets_are_distinct_and_keep_dimensions(self):
+        repo_root = Path(__file__).resolve().parents[1]
+        skin_assets = {
+            skin_id: {
+                pose: repo_root / "skins" / skin_id / pose
+                for pose in POSE_FILENAMES
+            }
+            for skin_id in BUILTIN_SKIN_IDS
+        }
+
+        for skin_id in BUILTIN_SKIN_IDS[1:]:
+            for pose, default_path in skin_assets["default"].items():
+                with self.subTest(skin=skin_id, pose=pose):
+                    self.assertNotEqual(
+                        default_path.read_bytes(),
+                        skin_assets[skin_id][pose].read_bytes(),
+                        f"{skin_id}/{pose} must differ from default",
+                    )
+
+        for skin_id, poses in skin_assets.items():
+            for pose, path in poses.items():
+                with self.subTest(skin=skin_id, pose=pose):
+                    header = path.read_bytes()[:29]
+                    self.assertEqual(b"\x89PNG\r\n\x1a\n", header[:8])
+                    self.assertEqual(13, struct.unpack(">I", header[8:12])[0])
+                    self.assertEqual(b"IHDR", header[12:16])
+                    width, height = struct.unpack(">II", header[16:24])
+                    self.assertEqual((200, 126), (width, height))
+
+
 if __name__ == "__main__":
     unittest.main()