PB-302: Add icon size to metadata

app/icon.py

@@ -1,5 +1,7 @@
 import os
 
+from PIL import Image
+
 from flask import url_for
 
 from app.helpers.icons import get_icon_template_url
@@ -82,6 +84,16 @@ def get_icon_filepath(self):
             name_with_extension = f"{name_with_extension}.png"
         return os.path.abspath(os.path.join(IMAGE_FOLDER, self.icon_set.name, name_with_extension))
 
+    def get_size(self):
+        """
+        Lazily open image of icon to get the size of the icon from the metadata.
+
+        Returns:
+            A list with the size of the specified icon [x,y]
+        """
+        with Image.open(self.get_icon_filepath()) as img:
+            return img.size
+
     def serialize(self):
         """
         As we want to add "url" to the __dict__, we can't really use a json.dumps to generate our
@@ -97,5 +109,6 @@ def serialize(self):
             "anchor": self.anchor,
             "icon_set": self.icon_set.name,
             "url": self.get_icon_url(),
-            "template_url": get_icon_template_url(get_base_url())
+            "template_url": get_icon_template_url(get_base_url()),
+            "size": self.get_size()
         }

tests/unit_tests/test_all_icons.py

@@ -253,6 +253,19 @@ def test_all_icon_metadata_endpoint(self):
                         )
                         self.assertTrue(fraction > 0, msg='"anchor" fraction should be > 0')
 
+                    self.assertIn('size', json_response)
+                    self.assertIsInstance(
+                        json_response['size'],
+                        list,
+                        msg='"size" should be a list with x and y dimension'
+                    )
+                    self.assertEqual(
+                        len(json_response['size']), 2, msg='"size" should have two items; x and y'
+                    )
+                    for elem in json_response['size']:
+                        self.assertIsInstance(elem, (int), msg='"size" should be int')
+                        self.assertTrue(elem > 0, msg='"size" should be > 0')
+
     def test_all_icon_basic_image(self):
         """
         Checking URLs without scale or color