python-fontbro 0.31.0__tar.gz → 0.32.0__tar.gz

{python_fontbro-0.31.0/python_fontbro.egg-info → python_fontbro-0.32.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-fontbro
-Version: 0.31.0
+Version: 0.32.0
 Summary: friendly font operations on top of fontTools.
 Author-email: Fabio Caccamo <fabio.caccamo@gmail.com>
 Maintainer-email: Fabio Caccamo <fabio.caccamo@gmail.com>
@@ -43,7 +43,6 @@ Classifier: License :: OSI Approved :: MIT License
 Classifier: Natural Language :: English
 Classifier: Operating System :: OS Independent
 Classifier: Programming Language :: Python :: 3
-Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Classifier: Programming Language :: Python :: 3.13
@@ -52,6 +51,7 @@ Classifier: Programming Language :: Python :: Implementation :: PyPy
 Classifier: Topic :: Text Processing :: Fonts
 Classifier: Topic :: Multimedia :: Graphics
 Classifier: Topic :: Multimedia :: Graphics :: Graphics Conversion
+Requires-Python: >=3.11
 Description-Content-Type: text/markdown
 License-File: LICENSE.txt
 Requires-Dist: fonttools[lxml,pathops,unicode,woff]<5.0,>=4.43.0
@@ -184,7 +184,7 @@ font_clone = font.clone()
 ```python
 """
 Close the wrapped TTFont instance.
-After closing, any operation on the font raises OperationError,
+After closing, any operation on the font raises ClosedFontError,
 closing an already closed font does nothing.
 """
 
@@ -631,7 +631,7 @@ Gets the wrapped TTFont instance.
 :returns: The TTFont instance.
 :rtype: TTFont
 
-:raises OperationError: If the font has been closed.
+:raises ClosedFontError: If the font has been closed.
 """
 
 ttfont = font.get_ttfont()
@@ -1011,7 +1011,7 @@ If family_name is not defined it will be auto-detected.
     (only if the default instance style name is renamed)
 :type update_style_flags: bool
 
-:raises OperationError: if the font is not a variable font.
+:raises VariableFontRequiredError: if the font is not a variable font.
 """
 
 font.rename_variable_instances(
@@ -1131,7 +1131,7 @@ Save all instances of a variable font to specified directory in one or more form
     includes 'instance' (containing instance metadata) and 'files' (a dictionary
     with file formats as keys and file-paths as values).
 
-:raises TypeError: If the font is not a variable font.
+:raises VariableFontRequiredError: If the font is not a variable font.
 """
 
 saved_fonts = font.save_variable_instances(
@@ -1167,7 +1167,7 @@ The fsType field is left untouched if any argument is invalid.
 :raises ArgumentError: If a value is not a bool or None, if more than one
     of the mutually exclusive usage permissions is set to True, or if
     installable is set to False while no other usage permission is in effect.
-:raises OperationError: If the OS/2 table is not available in the font.
+:raises MissingTableError: If the OS/2 table is not available in the font.
 """
 
 font.set_embedding_permissions(
@@ -1187,7 +1187,7 @@ based on provided class_id and subclass_id.
 
 :param class_id: Numeric identifier of the font family class.
 :param subclass_id: Optional numeric identifier of the font family subclass (default is 0).
-:raises OperationError: If the OS/2 table is not available in the font.
+:raises MissingTableError: If the OS/2 table is not available in the font.
 :raises ArgumentError: If class_id is invalid or subclass_id is specified but invalid.
 """
 
@@ -1405,7 +1405,7 @@ If an axis min and max values are equal, the axis will be pinned.
 :param options: The options for the fontTools.varLib.instancer
 :type options: dictionary
 
-:raises TypeError: If the font is not a variable font
+:raises VariableFontRequiredError: If the font is not a variable font
 :raises ValueError: If the coordinates are not defined (empty)
 :raises ValueError: If the coordinates axes are all pinned
 """
@@ -1438,7 +1438,7 @@ If coordinates are not specified each axis will be pinned at its default value.
 :param options: The options for the fontTools.varLib.instancer
 :type options: dictionary
 
-:raises TypeError: If the font is not a variable font
+:raises VariableFontRequiredError: If the font is not a variable font
 :raises ValueError: If the coordinates axes are not all pinned
 """
 

{python_fontbro-0.31.0 → python_fontbro-0.32.0}/README.md

@@ -120,7 +120,7 @@ font_clone = font.clone()
 ```python
 """
 Close the wrapped TTFont instance.
-After closing, any operation on the font raises OperationError,
+After closing, any operation on the font raises ClosedFontError,
 closing an already closed font does nothing.
 """
 
@@ -567,7 +567,7 @@ Gets the wrapped TTFont instance.
 :returns: The TTFont instance.
 :rtype: TTFont
 
-:raises OperationError: If the font has been closed.
+:raises ClosedFontError: If the font has been closed.
 """
 
 ttfont = font.get_ttfont()
@@ -947,7 +947,7 @@ If family_name is not defined it will be auto-detected.
     (only if the default instance style name is renamed)
 :type update_style_flags: bool
 
-:raises OperationError: if the font is not a variable font.
+:raises VariableFontRequiredError: if the font is not a variable font.
 """
 
 font.rename_variable_instances(
@@ -1067,7 +1067,7 @@ Save all instances of a variable font to specified directory in one or more form
     includes 'instance' (containing instance metadata) and 'files' (a dictionary
     with file formats as keys and file-paths as values).
 
-:raises TypeError: If the font is not a variable font.
+:raises VariableFontRequiredError: If the font is not a variable font.
 """
 
 saved_fonts = font.save_variable_instances(
@@ -1103,7 +1103,7 @@ The fsType field is left untouched if any argument is invalid.
 :raises ArgumentError: If a value is not a bool or None, if more than one
     of the mutually exclusive usage permissions is set to True, or if
     installable is set to False while no other usage permission is in effect.
-:raises OperationError: If the OS/2 table is not available in the font.
+:raises MissingTableError: If the OS/2 table is not available in the font.
 """
 
 font.set_embedding_permissions(
@@ -1123,7 +1123,7 @@ based on provided class_id and subclass_id.
 
 :param class_id: Numeric identifier of the font family class.
 :param subclass_id: Optional numeric identifier of the font family subclass (default is 0).
-:raises OperationError: If the OS/2 table is not available in the font.
+:raises MissingTableError: If the OS/2 table is not available in the font.
 :raises ArgumentError: If class_id is invalid or subclass_id is specified but invalid.
 """
 
@@ -1341,7 +1341,7 @@ If an axis min and max values are equal, the axis will be pinned.
 :param options: The options for the fontTools.varLib.instancer
 :type options: dictionary
 
-:raises TypeError: If the font is not a variable font
+:raises VariableFontRequiredError: If the font is not a variable font
 :raises ValueError: If the coordinates are not defined (empty)
 :raises ValueError: If the coordinates axes are all pinned
 """
@@ -1374,7 +1374,7 @@ If coordinates are not specified each axis will be pinned at its default value.
 :param options: The options for the fontTools.varLib.instancer
 :type options: dictionary
 
-:raises TypeError: If the font is not a variable font
+:raises VariableFontRequiredError: If the font is not a variable font
 :raises ValueError: If the coordinates axes are not all pinned
 """
 

{python_fontbro-0.31.0 → python_fontbro-0.32.0}/fontbro/embedding_permissions.py

@@ -2,8 +2,9 @@ from __future__ import annotations
 
 from fontTools.ttLib import TTFont
 
-from fontbro.exceptions import ArgumentError, OperationError
+from fontbro.exceptions import ArgumentError
 from fontbro.flags import get_flag, set_flag
+from fontbro.tables import get_table_or_raise
 
 # Embedding Permissions:
 # https://learn.microsoft.com/en-us/typography/opentype/spec/os2#fstype
@@ -95,9 +96,7 @@ def set_embedding_permissions(
     Sets the embedding permissions in the OS/2 fsType field of the given font,
     the fsType field is left untouched if any argument is invalid.
     """
-    os2 = ttfont.get("OS/2")
-    if not os2:
-        raise OperationError("Invalid OS/2 table (doesn't exist).")
+    os2 = get_table_or_raise(ttfont, "OS/2")
 
     usage_permissions = {
         "installable": installable,

python_fontbro-0.32.0/fontbro/exceptions.py

@@ -0,0 +1,53 @@
+from __future__ import annotations
+
+
+class ArgumentError(ValueError):
+    pass
+
+
+class DataError(ValueError):
+    pass
+
+
+class OperationError(TypeError):
+    pass
+
+
+class ClosedFontError(OperationError):
+    """
+    The operation was called on a closed font.
+    """
+
+
+class MissingTableError(OperationError, KeyError):
+    """
+    The operation requires a table the font doesn't have.
+    It's also a KeyError, raised before when accessing the missing table.
+    """
+
+    def __init__(
+        self,
+        table_tag: str,
+        message: str | None = None,
+    ) -> None:
+        self.table_tag = table_tag
+        super().__init__(message or f"Invalid {table_tag} table (doesn't exist).")
+
+    def __reduce__(self) -> tuple[type[MissingTableError], tuple[str, str]]:
+        # pickle and copy recreate the exception passing its args (the message
+        # only) to __init__, the table tag must be passed too
+        return (self.__class__, (self.table_tag, str(self)))
+
+    def __str__(self) -> str:
+        # KeyError wraps the message in quotes
+        return str(self.args[0])
+
+
+class VariableFontRequiredError(OperationError):
+    """
+    The operation requires a variable font.
+    """
+
+
+class SanitizationError(Exception):
+    pass

{python_fontbro-0.31.0 → python_fontbro-0.32.0}/fontbro/family_classification.py

@@ -4,7 +4,8 @@ from typing import Any
 
 from fontTools.ttLib import TTFont
 
-from fontbro.exceptions import ArgumentError, OperationError
+from fontbro.exceptions import ArgumentError
+from fontbro.tables import get_table_or_raise
 from fontbro.utils import concat_names, find_item, read_json
 
 # Family Classification:
@@ -152,9 +153,7 @@ def set_family_classification(
     """
     Sets the family classification of the given font in the OS/2.sFamilyClass field.
     """
-    os2 = ttfont.get("OS/2")
-    if not os2:
-        raise OperationError("Invalid OS/2 table (doesn't exist).")
+    os2 = get_table_or_raise(ttfont, "OS/2")
 
     # validate class key and subclass key
     class_item, subclass_item = _get_family_classification_items(

{python_fontbro-0.31.0 → python_fontbro-0.32.0}/fontbro/font.py

@@ -32,7 +32,8 @@ from fontbro import variable as _variable
 from fontbro import version as _version
 from fontbro.exceptions import (
     ArgumentError,
-    OperationError,
+    ClosedFontError,
+    VariableFontRequiredError,
 )
 
 
@@ -325,7 +326,7 @@ class Font:
     ) -> None:
         """
         Close the wrapped TTFont instance.
-        After closing, any operation on the font raises OperationError,
+        After closing, any operation on the font raises ClosedFontError,
         closing an already closed font does nothing.
         """
         # TTFont.close() is idempotent (it closes the reader and sets it to None)
@@ -847,13 +848,13 @@ class Font:
         :returns: The TTFont instance.
         :rtype: TTFont
 
-        :raises OperationError: If the font has been closed.
+        :raises ClosedFontError: If the font has been closed.
         """
         # all the font operations get the TTFont instance through this method,
         # the TTFont can also be closed directly (bypassing this instance):
         # when closed, fontTools sets its reader to None
         if self._ttfont is None or self._ttfont.reader is None:
-            raise OperationError("Invalid operation, the font is closed.")
+            raise ClosedFontError("Invalid operation, the font is closed.")
         return self._ttfont
 
     def get_unicode_block_by_name(
@@ -1277,7 +1278,7 @@ class Font:
             (only if the default instance style name is renamed)
         :type update_style_flags: bool
 
-        :raises OperationError: if the font is not a variable font.
+        :raises VariableFontRequiredError: if the font is not a variable font.
         :return: None
         """
         ttfont = self.get_ttfont()
@@ -1512,10 +1513,10 @@ class Font:
             includes 'instance' (containing instance metadata) and 'files' (a dictionary
             with file formats as keys and file-paths as values).
 
-        :raises TypeError: If the font is not a variable font.
+        :raises VariableFontRequiredError: If the font is not a variable font.
         """
         if not self.is_variable():
-            raise OperationError("Only a variable font can be instantiated.")
+            raise VariableFontRequiredError("Only a variable font can be instantiated.")
 
         fsutil.assert_not_file(dirpath)
         fsutil.make_dirs(dirpath)
@@ -1570,7 +1571,7 @@ class Font:
 
         :param class_id: Numeric identifier of the font family class.
         :param subclass_id: Optional numeric identifier of the font family subclass (default is 0).
-        :raises OperationError: If the OS/2 table is not available in the font.
+        :raises MissingTableError: If the OS/2 table is not available in the font.
         :raises ArgumentError: If class_id is invalid or subclass_id is specified but invalid.
         """
         ttfont = self.get_ttfont()
@@ -1723,7 +1724,7 @@ class Font:
         :raises ArgumentError: If a value is not a bool or None, if more than one
             of the mutually exclusive usage permissions is set to True, or if
             installable is set to False while no other usage permission is in effect.
-        :raises OperationError: If the OS/2 table is not available in the font.
+        :raises MissingTableError: If the OS/2 table is not available in the font.
         """
         ttfont = self.get_ttfont()
         _embedding_permissions.set_embedding_permissions(
@@ -1860,7 +1861,7 @@ class Font:
         :param options: The options for the fontTools.varLib.instancer
         :type options: dictionary
 
-        :raises TypeError: If the font is not a variable font
+        :raises VariableFontRequiredError: If the font is not a variable font
         :raises ValueError: If the coordinates are not defined (blank)
         :raises ValueError: If the coordinates axes are all pinned
         """
@@ -1900,7 +1901,7 @@ class Font:
         :param options: The options for the fontTools.varLib.instancer
         :type options: dictionary
 
-        :raises TypeError: If the font is not a variable font
+        :raises VariableFontRequiredError: If the font is not a variable font
         :raises ValueError: If the coordinates axes are not all pinned
         """
         ttfont = self.get_ttfont()

{python_fontbro-0.31.0 → python_fontbro-0.32.0}/fontbro/metadata.py

@@ -4,4 +4,4 @@ __description__ = "friendly font operations on top of fontTools."
 __email__ = "fabio.caccamo@gmail.com"
 __license__ = "MIT"
 __title__ = "python-fontbro"
-__version__ = "0.31.0"
+__version__ = "0.32.0"

{python_fontbro-0.31.0 → python_fontbro-0.32.0}/fontbro/monospace.py

@@ -6,6 +6,7 @@ from collections import Counter
 from fontTools import unicodedata
 from fontTools.ttLib import TTFont
 
+from fontbro.tables import get_table_or_raise
 from fontbro.unicode import get_best_cmap
 
 # same criteria used by Fontbakery (glyph_metrics_stats condition):
@@ -36,9 +37,10 @@ def _get_widths(
     """
     Gets the advance widths of the given glyphs, zero widths are ignored.
     """
+    hmtx = get_table_or_raise(ttfont, "hmtx")
     return [
         advance
-        for glyph_name, (advance, _) in ttfont["hmtx"].metrics.items()
+        for glyph_name, (advance, _) in hmtx.metrics.items()
         if glyph_name in glyphs_names and advance != 0
     ]
 

{python_fontbro-0.31.0 → python_fontbro-0.32.0}/fontbro/names.py

@@ -6,7 +6,8 @@ from typing import Any
 from fontTools.misc.encodingTools import getEncoding
 from fontTools.ttLib import TTFont
 
-from fontbro.exceptions import ArgumentError, OperationError
+from fontbro.exceptions import ArgumentError, VariableFontRequiredError
+from fontbro.tables import get_table_or_raise
 from fontbro.utils import concat_names, remove_spaces
 
 # Names:
@@ -193,7 +194,7 @@ def set_name(
     and unicode flags.
     """
     name_id = _get_name_id(key)
-    name_table = ttfont["name"]
+    name_table = get_table_or_raise(ttfont, "name")
     if windows:
         # https://github.com/fonttools/fonttools/blob/main/Lib/fontTools/ttLib/tables/_n_a_m_e.py#L568
         # the windows record is always written, it's the one read first by get_name
@@ -313,7 +314,9 @@ def rename_variable_instances(
     so renaming it renames the font style too.
     """
     if "fvar" not in ttfont:
-        raise OperationError("Only the instances of a variable font can be renamed.")
+        raise VariableFontRequiredError(
+            "Only the instances of a variable font can be renamed."
+        )
     # style names are matched case-insensitively
     style_names = {key.lower(): value for key, value in (style_names or {}).items()}
     default_style_name = _get_default_instance_style_name(ttfont).lower()

{python_fontbro-0.31.0 → python_fontbro-0.32.0}/fontbro/pixel.py

@@ -9,6 +9,7 @@ from fontTools import unicodedata
 from fontTools.pens.recordingPen import DecomposingRecordingPen
 from fontTools.ttLib import TTFont
 
+from fontbro.tables import get_table_or_raise
 from fontbro.unicode import get_best_cmap
 
 Segment = tuple[float, float]
@@ -219,7 +220,7 @@ def is_pixel(
 
     # 2nd check: segments aligned to a plausible pixel grid,
     # it excludes squared ("techno") fonts
-    units_per_em = ttfont["head"].unitsPerEm
+    units_per_em = get_table_or_raise(ttfont, "head").unitsPerEm
     pixel_width, pixel_height = (
         _get_pixel_dimension(
             orthogonal_glyphs_segments, axis, units_per_em, min_glyphs_count

{python_fontbro-0.31.0 → python_fontbro-0.32.0}/fontbro/render.py

@@ -9,6 +9,7 @@ from fontTools.pens.svgPathPen import SVGPathPen
 from fontTools.ttLib import TTFont
 from PIL import Image, ImageDraw, ImageFont
 
+from fontbro.tables import get_table_or_raise
 from fontbro.unicode import get_best_cmap_or_raise
 
 
@@ -51,9 +52,9 @@ def get_svg(
     Gets an SVG representation of the given font rendering some text.
     """
     # get font metrics
-    units_per_em = ttfont["head"].unitsPerEm
+    units_per_em = get_table_or_raise(ttfont, "head").unitsPerEm
     scale = size / units_per_em
-    hhea = ttfont["hhea"]
+    hhea = get_table_or_raise(ttfont, "hhea")
     ascent = hhea.ascent * scale
     descent = hhea.descent * scale
     width = 0

{python_fontbro-0.31.0 → python_fontbro-0.32.0}/fontbro/tables.py

@@ -4,6 +4,8 @@ from typing import Any
 
 from fontTools.ttLib import TTFont
 
+from fontbro.exceptions import MissingTableError
+
 # https://learn.microsoft.com/en-us/typography/opentype/spec/otff#font-tables
 _TABLE_NAMES_BY_TAG: dict[str, str] = {
     # required tables
@@ -95,6 +97,19 @@ _TABLE_NAMES_BY_TAG: dict[str, str] = {
 }
 
 
+def get_table_or_raise(
+    ttfont: TTFont,
+    tag: str,
+) -> Any:
+    """
+    Gets the table with the given tag of the given font,
+    MissingTableError is raised if the font has no table with the given tag.
+    """
+    if tag not in ttfont:
+        raise MissingTableError(tag)
+    return ttfont[tag]
+
+
 def get_tables_tags(
     ttfont: TTFont,
     *,

{python_fontbro-0.31.0 → python_fontbro-0.32.0}/fontbro/variable.py

@@ -9,8 +9,9 @@ from fontTools.varLib.instancer import OverlapMode
 
 from fontbro import names as _names
 from fontbro import style_flags as _style_flags
-from fontbro.exceptions import ArgumentError, OperationError
+from fontbro.exceptions import ArgumentError, VariableFontRequiredError
 from fontbro.math import get_euclidean_distance
+from fontbro.tables import get_table_or_raise
 from fontbro.utils import slugify
 
 # Variable Axes:
@@ -159,7 +160,7 @@ def get_variable_instances(
     """
     if not is_variable(ttfont):
         return None
-    name_table = ttfont["name"]
+    name_table = get_table_or_raise(ttfont, "name")
     return [
         {
             "coordinates": instance.coordinates,
@@ -320,7 +321,7 @@ def to_sliced_variable(
     the variable axes at the given coordinates.
     """
     if not is_variable(ttfont):
-        raise OperationError("Only a variable font can be sliced.")
+        raise VariableFontRequiredError("Only a variable font can be sliced.")
 
     coordinates = get_sliced_coordinates(ttfont, coordinates)
 
@@ -364,7 +365,7 @@ def to_static(
     the variable axes at the given coordinates.
     """
     if not is_variable(ttfont):
-        raise OperationError("Only a variable font can be made static.")
+        raise VariableFontRequiredError("Only a variable font can be made static.")
 
     coordinates = get_static_coordinates(
         ttfont,

{python_fontbro-0.31.0 → python_fontbro-0.32.0}/pyproject.toml

@@ -40,7 +40,6 @@ classifiers = [
     "Natural Language :: English",
     "Operating System :: OS Independent",
     "Programming Language :: Python :: 3",
-    "Programming Language :: Python :: 3.10",
     "Programming Language :: Python :: 3.11",
     "Programming Language :: Python :: 3.12",
     "Programming Language :: Python :: 3.13",
@@ -59,6 +58,7 @@ dependencies = [
     "python-fsutil >= 0.16.0, < 1.0.0",
 ]
 dynamic = ["version"]
+requires-python = ">=3.11"
 maintainers = [
     { name = "Fabio Caccamo", email = "fabio.caccamo@gmail.com" },
 ]

{python_fontbro-0.31.0 → python_fontbro-0.32.0/python_fontbro.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-fontbro
-Version: 0.31.0
+Version: 0.32.0
 Summary: friendly font operations on top of fontTools.
 Author-email: Fabio Caccamo <fabio.caccamo@gmail.com>
 Maintainer-email: Fabio Caccamo <fabio.caccamo@gmail.com>
@@ -43,7 +43,6 @@ Classifier: License :: OSI Approved :: MIT License
 Classifier: Natural Language :: English
 Classifier: Operating System :: OS Independent
 Classifier: Programming Language :: Python :: 3
-Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Classifier: Programming Language :: Python :: 3.13
@@ -52,6 +51,7 @@ Classifier: Programming Language :: Python :: Implementation :: PyPy
 Classifier: Topic :: Text Processing :: Fonts
 Classifier: Topic :: Multimedia :: Graphics
 Classifier: Topic :: Multimedia :: Graphics :: Graphics Conversion
+Requires-Python: >=3.11
 Description-Content-Type: text/markdown
 License-File: LICENSE.txt
 Requires-Dist: fonttools[lxml,pathops,unicode,woff]<5.0,>=4.43.0
@@ -184,7 +184,7 @@ font_clone = font.clone()
 ```python
 """
 Close the wrapped TTFont instance.
-After closing, any operation on the font raises OperationError,
+After closing, any operation on the font raises ClosedFontError,
 closing an already closed font does nothing.
 """
 
@@ -631,7 +631,7 @@ Gets the wrapped TTFont instance.
 :returns: The TTFont instance.
 :rtype: TTFont
 
-:raises OperationError: If the font has been closed.
+:raises ClosedFontError: If the font has been closed.
 """
 
 ttfont = font.get_ttfont()
@@ -1011,7 +1011,7 @@ If family_name is not defined it will be auto-detected.
     (only if the default instance style name is renamed)
 :type update_style_flags: bool
 
-:raises OperationError: if the font is not a variable font.
+:raises VariableFontRequiredError: if the font is not a variable font.
 """
 
 font.rename_variable_instances(
@@ -1131,7 +1131,7 @@ Save all instances of a variable font to specified directory in one or more form
     includes 'instance' (containing instance metadata) and 'files' (a dictionary
     with file formats as keys and file-paths as values).
 
-:raises TypeError: If the font is not a variable font.
+:raises VariableFontRequiredError: If the font is not a variable font.
 """
 
 saved_fonts = font.save_variable_instances(
@@ -1167,7 +1167,7 @@ The fsType field is left untouched if any argument is invalid.
 :raises ArgumentError: If a value is not a bool or None, if more than one
     of the mutually exclusive usage permissions is set to True, or if
     installable is set to False while no other usage permission is in effect.
-:raises OperationError: If the OS/2 table is not available in the font.
+:raises MissingTableError: If the OS/2 table is not available in the font.
 """
 
 font.set_embedding_permissions(
@@ -1187,7 +1187,7 @@ based on provided class_id and subclass_id.
 
 :param class_id: Numeric identifier of the font family class.
 :param subclass_id: Optional numeric identifier of the font family subclass (default is 0).
-:raises OperationError: If the OS/2 table is not available in the font.
+:raises MissingTableError: If the OS/2 table is not available in the font.
 :raises ArgumentError: If class_id is invalid or subclass_id is specified but invalid.
 """
 
@@ -1405,7 +1405,7 @@ If an axis min and max values are equal, the axis will be pinned.
 :param options: The options for the fontTools.varLib.instancer
 :type options: dictionary
 
-:raises TypeError: If the font is not a variable font
+:raises VariableFontRequiredError: If the font is not a variable font
 :raises ValueError: If the coordinates are not defined (empty)
 :raises ValueError: If the coordinates axes are all pinned
 """
@@ -1438,7 +1438,7 @@ If coordinates are not specified each axis will be pinned at its default value.
 :param options: The options for the fontTools.varLib.instancer
 :type options: dictionary
 
-:raises TypeError: If the font is not a variable font
+:raises VariableFontRequiredError: If the font is not a variable font
 :raises ValueError: If the coordinates axes are not all pinned
 """
 

{python_fontbro-0.31.0 → python_fontbro-0.32.0}/python_fontbro.egg-info/SOURCES.txt

@@ -49,6 +49,7 @@ tests/test_collection.py
 tests/test_color.py
 tests/test_context_manager.py
 tests/test_embedding_permissions.py
+tests/test_exceptions.py
 tests/test_family_classification.py
 tests/test_family_name.py
 tests/test_features.py

python_fontbro-0.32.0/tests/test_exceptions.py

@@ -0,0 +1,134 @@
+import copy
+import pickle
+
+from fontbro.exceptions import (
+    ClosedFontError,
+    MissingTableError,
+    OperationError,
+    VariableFontRequiredError,
+)
+from tests import AbstractTestCase
+
+
+class ExceptionsTestCase(AbstractTestCase):
+    """
+    Test case for the specific exceptions (subclasses of OperationError).
+    """
+
+    def _get_static_font(self):
+        return self._get_font("/Roboto_Mono/static/RobotoMono-Regular.ttf")
+
+    def _get_variable_font(self):
+        return self._get_font("/Tourney/Tourney-VariableFont_wdth,wght.ttf")
+
+    def _assert_missing_table_error(self, context, table_tag):
+        error = context.exception
+        self.assertIsInstance(error, OperationError)
+        self.assertIsInstance(error, KeyError)
+        self.assertEqual(error.table_tag, table_tag)
+        self.assertEqual(str(error), f"Invalid {table_tag} table (doesn't exist).")
+
+    def test_closed_font_error(self):
+        font = self._get_static_font()
+        font.close()
+        with self.assertRaises(ClosedFontError) as context:
+            font.get_names()
+        self.assertIsInstance(context.exception, OperationError)
+        self.assertEqual(
+            str(context.exception), "Invalid operation, the font is closed."
+        )
+
+    def test_missing_table_error_with_custom_message(self):
+        error = MissingTableError("OS/2", "Custom message.")
+        self.assertEqual(error.table_tag, "OS/2")
+        self.assertEqual(str(error), "Custom message.")
+
+    def test_missing_table_error_pickle_and_copy(self):
+        # regression: the exception was recreated passing the message as table tag
+        for message in (None, "Custom message."):
+            error = MissingTableError("OS/2", message)
+            for error_copy in (pickle.loads(pickle.dumps(error)), copy.copy(error)):
+                with self.subTest(message=message):
+                    self.assertEqual(error_copy.table_tag, "OS/2")
+                    self.assertEqual(str(error_copy), str(error))
+
+    def test_missing_table_error_set_embedding_permissions(self):
+        font = self._get_static_font()
+        del font.get_ttfont()["OS/2"]
+        with self.assertRaises(MissingTableError) as context:
+            font.set_embedding_permissions(installable=True)
+        self._assert_missing_table_error(context, "OS/2")
+
+    def test_missing_table_error_set_family_classification(self):
+        font = self._get_static_font()
+        del font.get_ttfont()["OS/2"]
+        with self.assertRaises(MissingTableError) as context:
+            font.set_family_classification(class_id=1)
+        self._assert_missing_table_error(context, "OS/2")
+
+    def test_missing_table_error_set_name(self):
+        font = self._get_static_font()
+        del font.get_ttfont()["name"]
+        with self.assertRaises(MissingTableError) as context:
+            font.set_name("family_name", "Roboto Mono New")
+        self._assert_missing_table_error(context, "name")
+
+    def test_missing_table_error_get_variable_instances(self):
+        font = self._get_variable_font()
+        del font.get_ttfont()["name"]
+        with self.assertRaises(MissingTableError) as context:
+            font.get_variable_instances()
+        self._assert_missing_table_error(context, "name")
+
+    def test_missing_table_error_is_monospace(self):
+        font = self._get_static_font()
+        del font.get_ttfont()["hmtx"]
+        with self.assertRaises(MissingTableError) as context:
+            font.is_monospace()
+        self._assert_missing_table_error(context, "hmtx")
+
+    def test_missing_table_error_is_pixel(self):
+        font = self._get_font("/Silkscreen/Silkscreen-Regular.ttf")
+        ttfont = font.get_ttfont()
+        # load the glyphs before removing the head table (needed to load them)
+        ttfont["glyf"].ensureDecompiled()
+        del ttfont["head"]
+        with self.assertRaises(MissingTableError) as context:
+            font.is_pixel()
+        self._assert_missing_table_error(context, "head")
+
+    def test_missing_table_error_get_svg(self):
+        for table_tag in ("head", "hhea"):
+            with self.subTest(table_tag=table_tag):
+                font = self._get_static_font()
+                del font.get_ttfont()[table_tag]
+                with self.assertRaises(MissingTableError) as context:
+                    font.get_svg(text="Hello", size=12)
+                self._assert_missing_table_error(context, table_tag)
+
+    def test_variable_font_required_error(self):
+        operations = [
+            (
+                lambda font: font.rename_variable_instances(family_name="New"),
+                "Only the instances of a variable font can be renamed.",
+            ),
+            (
+                lambda font: font.save_variable_instances(self._get_font_temp_path()),
+                "Only a variable font can be instantiated.",
+            ),
+            (
+                lambda font: font.to_sliced_variable(coordinates={"wght": (100, 400)}),
+                "Only a variable font can be sliced.",
+            ),
+            (
+                lambda font: font.to_static(),
+                "Only a variable font can be made static.",
+            ),
+        ]
+        for operation, message in operations:
+            with self.subTest(message=message):
+                font = self._get_static_font()
+                with self.assertRaises(VariableFontRequiredError) as context:
+                    operation(font)
+                self.assertIsInstance(context.exception, OperationError)
+                self.assertEqual(str(context.exception), message)

{python_fontbro-0.31.0 → python_fontbro-0.32.0}/tests/test_instantiation.py

@@ -76,6 +76,15 @@ class InstantiationTestCase(AbstractTestCase):
         name_ids = {record.nameID for record in font.get_ttfont()["name"].names}
         self.assertEqual({name_id for name_id in name_ids if name_id > 255}, set())
 
+    def test_to_static_without_stat_table(self):
+        font = self._get_variable_font()
+        ttfont = font.get_ttfont()
+        del ttfont["STAT"]
+        font.to_static(style_name="Black")
+        self.assertFalse(font.is_variable())
+        self.assertNotIn("STAT", ttfont)
+        self.assertEqual(font.get_name(Font.NAME_POSTSCRIPT_NAME), "Tourney-Black")
+
     def test_to_static_with_style_name_invalid(self):
         font = self._get_variable_font()
         with self.assertRaises(ValueError):

{python_fontbro-0.31.0 → python_fontbro-0.32.0}/tests/test_rename.py

@@ -448,6 +448,36 @@ class RenameTestCase(AbstractTestCase):
         }
         self.assertEqual(names["Hair Italic"], "TourneyItalic-Hair")
 
+    def test_rename_variable_font_without_default_instance(self):
+        font = self._get_font("/Tourney/Tourney-VariableFont_wdth,wght.ttf")
+        fvar = font.get_ttfont()["fvar"]
+        # remove the default instance ("Thin")
+        fvar.instances.pop(0)
+        font.rename(style_name="Hair Line")
+        self.assertEqual(font.get_name(Font.NAME_POSTSCRIPT_NAME), "Tourney-HairLine")
+        self.assertEqual(font.get_name(fvar.instances[0].subfamilyNameID), "ExtraLight")
+
+    def test_rename_variable_font_without_instances_postscript_names(self):
+        font = self._get_font("/Inter/Inter-VariableFont_slnt,wght.ttf")
+        fvar = font.get_ttfont()["fvar"]
+        font.rename(family_name="Inter New")
+        self.assertEqual(font.get_name(Font.NAME_POSTSCRIPT_NAME), "InterNew-Regular")
+        for instance in fvar.instances:
+            self.assertEqual(instance.postscriptNameID, 0xFFFF)
+
+    def test_rename_variable_instances_with_style_name_shared_with_font_style_name(
+        self,
+    ):
+        # the instance style name record is the font subfamily name (2),
+        # so a new name record is added for the instance
+        font = self._get_font("/Tourney/Tourney-VariableFont_wdth,wght.ttf")
+        instance = font.get_ttfont()["fvar"].instances[1]
+        instance.subfamilyNameID = 2
+        font.rename_variable_instances(style_names={"regular": "Normal"})
+        self.assertGreater(instance.subfamilyNameID, 255)
+        self.assertEqual(font.get_name(instance.subfamilyNameID), "Normal")
+        self.assertEqual(font.get_name(Font.NAME_SUBFAMILY_NAME), "Regular")
+
     def test_rename_variable_instances_with_static_font(self):
         font = self._get_font("/Roboto_Mono/static/RobotoMono-Regular.ttf")
         with self.assertRaises(TypeError):

{python_fontbro-0.31.0 → python_fontbro-0.32.0}/tests/test_save.py

@@ -12,18 +12,18 @@ class SaveTestCase(AbstractTestCase):
     """
 
     def test_save_with_font_src_path_as_filepath_with_overwrite(self):
-        font_filepath = self._get_font_path(
+        font_filepath = self._get_font_temp_copy_path(
             "/Roboto_Mono/static/RobotoMono-Regular.ttf"
         )
-        font = self._get_font("/Roboto_Mono/static/RobotoMono-Regular.ttf")
+        font = Font(font_filepath)
         font_saved_path = font.save(font_filepath, overwrite=True)
         self.assertEqual(font_saved_path, font_filepath)
 
     def test_save_without_filepath_with_overwrite(self):
-        font_filepath = self._get_font_path(
+        font_filepath = self._get_font_temp_copy_path(
             "/Roboto_Mono/static/RobotoMono-Regular.ttf"
         )
-        font = self._get_font("/Roboto_Mono/static/RobotoMono-Regular.ttf")
+        font = Font(font_filepath)
         font_saved_path = font.save(overwrite=True)
         self.assertEqual(font_saved_path, font_filepath)
 
@@ -41,7 +41,9 @@ class SaveTestCase(AbstractTestCase):
             font.save(font_filepath)
 
     def test_save_fileobject(self):
-        font_filepath = self._get_font_path("/Noto_Sans_TC/NotoSansTC-Regular.otf")
+        font_filepath = self._get_font_temp_copy_path(
+            "/Noto_Sans_TC/NotoSansTC-Regular.otf"
+        )
         with open(font_filepath, "rb") as fh:
             font = Font(fh)
             font_saved_path = font.save(font_filepath, overwrite=True)

python_fontbro-0.32.0/tests/test_update_unicode_data.py

@@ -0,0 +1,20 @@
+import fsutil
+
+from scripts.update_data import update_unicode_data
+from tests import AbstractTestCase
+
+
+class UpdateUnicodeDataTestCase(AbstractTestCase):
+    """
+    Test case for updating the pre-computed unicode data.
+    """
+
+    def test_update_unicode_data(self):
+        # the data files are written to the temp directory,
+        # the package data files are not modified
+        dirpath = self._get_font_temp_path()
+        update_unicode_data(dirpath=dirpath)
+        for filename in ("unicode-blocks.json", "unicode-scripts.json"):
+            data = fsutil.read_file_json(fsutil.join_path(dirpath, filename))
+            self.assertTrue(len(data) > 0)
+            self.assertTrue(all(item["characters_total"] > 0 for item in data))

python_fontbro-0.31.0/fontbro/exceptions.py

@@ -1,14 +0,0 @@
-class ArgumentError(ValueError):
-    pass
-
-
-class DataError(ValueError):
-    pass
-
-
-class OperationError(TypeError):
-    pass
-
-
-class SanitizationError(Exception):
-    pass

python_fontbro-0.31.0/tests/test_update_unicode_data.py

@@ -1,11 +0,0 @@
-from scripts.update_data import update_unicode_data
-from tests import AbstractTestCase
-
-
-class UpdateUnicodeDataTestCase(AbstractTestCase):
-    """
-    Test case for updating the pre-computed unicode data.
-    """
-
-    def test_update_unicode_data(self):
-        update_unicode_data()