Improved subset unicodes arg to support also list of int codepoints.

Author: fabiocaccamo

fontbro/subset.py

@@ -8,7 +8,12 @@
 def parse_unicodes(unicodes):
     unicodes = unicodes or ""
     if isinstance(unicodes, (list, set, tuple)):
+        # convert possible int codepoints to hex str
+        for index in range(len(unicodes)):
+            code = unicodes[index]
+            unicodes[index] = f"{code:04X}" if isinstance(code, int) else code
         unicodes = ",".join(list(set(unicodes)))
+    assert isinstance(unicodes, str)
     # replace possible — ‐ − (— ‐ −) with -
     unicodes = re.sub(r"[\—\‐\−]", "-", unicodes)
     # remove U+, \u, u if present

tests/test_subset.py

@@ -54,18 +54,32 @@ def test_subset_with_unicodes_str_and_unicode_escape_prefix(self):
         chars_count = font.get_characters_count()
         self.assertEqual(chars_count, 240)
 
-    def test_subset_with_unicodes_list(self):
+    def test_subset_with_unicodes_list_of_str(self):
         font = self._get_font("/Roboto_Mono/static/RobotoMono-Regular.ttf")
         chars_count = font.get_characters_count()
         self.assertEqual(chars_count, 875)
         font.subset(unicodes=["0000—007F", "0100—017F", "0180—024F"])
         chars_count = font.get_characters_count()
         self.assertEqual(chars_count, 240)
 
-    def test_subset_with_unicodes_list_with_u_prefix(self):
+    def test_subset_with_unicodes_list_of_str_with_u_prefix(self):
         font = self._get_font("/Roboto_Mono/static/RobotoMono-Regular.ttf")
         chars_count = font.get_characters_count()
         self.assertEqual(chars_count, 875)
         font.subset(unicodes=["u0000—u007F", "u0100—u017F", "u0180—u024F"])
         chars_count = font.get_characters_count()
         self.assertEqual(chars_count, 240)
+
+    def test_subset_with_unicodes_list_of_int(self):
+        font = self._get_font("/Roboto_Mono/static/RobotoMono-Regular.ttf")
+        chars_count = font.get_characters_count()
+        self.assertEqual(chars_count, 875)
+        unicodes = [*range(65, 91)]
+        font.subset(unicodes=unicodes)
+        chars_count = font.get_characters_count()
+        self.assertEqual(chars_count, len(unicodes))
+        self.assertEqual(chars_count, 26)
+        self.assertEqual(
+            "".join([char["character"] for char in font.get_characters()]),
+            "ABCDEFGHIJKLMNOPQRSTUVWXYZ",
+        )