Merge pull request #1000 from cmyr/add-glyphdata-test-cases

[glyphdata] Add test to hit _split_glyph_name
googlefonts · Apr 19, 2024 · 5946df8 · 5946df8
2 parents e2ebf5b + a93588a
commit 5946df8
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/tests/glyphdata_test.py b/tests/glyphdata_test.py
@@ -141,6 +141,12 @@ def cat(n):
         self.assertEqual(cat("_aaa"), (None, None))
         self.assertEqual(cat("dal_alef-ar"), ("Letter", "Ligature"))
         self.assertEqual(cat("dal_lam-ar.dlig"), ("Letter", "Ligature"))
+        self.assertEqual(cat("po-khmer"), ("Letter", None))
+        self.assertEqual(cat("po-khmer.below"), ("Mark", "Nonspacing"))
+        # see https://github.com/googlefonts/glyphsLib/commit/68e4e9cf44c9919de
+        # this glyph is not in the data, and we want fallback to find po-khmer.below
+        # before po-khmer
+        self.assertEqual(cat("po-khmer.below.ro"), ("Mark", "Nonspacing"))
 
     def test_category_buy_unicode(self):
         def cat(n, u):