Fixed FontProperties to/from strings

TD22057 · TD22057 · commit bb14c7fb7239 · 2019-11-04T08:28:57.000-08:00
diff --git a/lib/matplotlib/fontconfig_pattern.py b/lib/matplotlib/fontconfig_pattern.py
@@ -177,19 +177,34 @@ def _property(self, s, loc, tokens):
 parse_fontconfig_pattern = lru_cache()(FontconfigPatternParser().parse)
 
 
+def _escape_val(val, escape_func):
+    if type(val) == list:
+        val = [escape_func(r'\\\1', str(x)) for x in val
+               if x is not None]
+        if val != []:
+            val = ','.join(val)
+    else:
+        val = escape_func(r'\\\1', str(val))
+
+    return val
+
+
 def generate_fontconfig_pattern(d):
     """
     Given a dictionary of key/value pairs, generates a fontconfig
     pattern string.
     """
     props = []
-    for key in 'family style variant weight stretch file size'.split():
+
+    # Family is added first w/o a keyword
+    family = d.get_family()
+    if family is not None and family != []:
+        props.append("%s" % (_escape_val(family, family_escape)))
+
+    # The other keys are added as key=value
+    for key in ['style', 'variant', 'weight', 'stretch', 'file', 'size']:
         val = getattr(d, 'get_' + key)()
         if val is not None and val != []:
-            if type(val) == list:
-                val = [value_escape(r'\\\1', str(x)) for x in val
-                       if x is not None]
-                if val != []:
-                    val = ','.join(val)
-            props.append(":%s=%s" % (key, val))
+            props.append(":%s=%s" % (key, _escape_val(val, value_escape)))
+
     return ''.join(props)
diff --git a/lib/matplotlib/tests/test_fontconfig.py b/lib/matplotlib/tests/test_fontconfig.py
@@ -0,0 +1,70 @@
+from matplotlib.font_manager import FontProperties
+
+
+# Attributes on FontProperties object to check for consistency
+keys = [
+    "get_family",
+    "get_style",
+    "get_variant",
+    "get_weight",
+    "get_size",
+    ]
+
+
+def test_fontconfig_pattern():
+    "Test converting a FontProperties to string then back."
+
+    # Defaults
+    test = "defaults "
+    f1 = FontProperties()
+    s = str(f1)
+
+    f2 = FontProperties(s)
+    for k in keys:
+        assert getattr(f1, k)() == getattr(f2, k)(), test + k
+
+    # Basic inputs
+    test = "basic "
+    f1 = FontProperties(family="serif", size=20, style="italic")
+    s = str(f1)
+
+    f2 = FontProperties(s)
+    for k in keys:
+        assert getattr(f1, k)() == getattr(f2, k)(), test + k
+
+    # Full set of inputs.
+    test = "full "
+    f1 = FontProperties(family="sans-serif", size=24, weight="bold",
+                        style="oblique", variant="small-caps",
+                        stretch="expanded")
+    s = str(f1)
+
+    f2 = FontProperties(s)
+    for k in keys:
+        assert getattr(f1, k)() == getattr(f2, k)(), test + k
+
+
+def test_fontconfig_str():
+    "Test FontProperties string conversions for correctness"
+
+    # Known good strings taken from actual font config specs on a linux box
+    # and modified for MPL defaults.
+
+    # Default values found by inspection.
+    test = "defaults "
+    s = ("sans\\-serif:style=normal:variant=normal:weight=normal"
+         ":stretch=normal:size=12.0")
+    font = FontProperties(s)
+    right = FontProperties()
+    for k in keys:
+        assert getattr(font, k)() == getattr(right, k)(), test + k
+
+    test = "full "
+    s = ("serif:size=24:style=oblique:variant=small-caps:weight=bold"
+         ":stretch=expanded")
+    font = FontProperties(s)
+    right = FontProperties(family="serif", size=24, weight="bold",
+                           style="oblique", variant="small-caps",
+                           stretch="expanded")
+    for k in keys:
+        assert getattr(font, k)() == getattr(right, k)(), test + k
