Add ftxvalidator CI run

diff --git a/.ci/ftxvalidator.py b/.ci/ftxvalidator.py
new file mode 100644 (file)
index 0000000..cf2d314
--- /dev/null
+++ b/.ci/ftxvalidator.py
@@ -0,0 +1,84 @@
+import argparse
+import glob
+import os
+import plistlib
+import sys
+import subprocess
+from collections import defaultdict
+
+from whatchanged import directory_check_types, CheckType
+
+severity_mapping = {
+    "kATSFontTestSeverityMinorError": "WARN",
+    "kATSFontTestSeverityFatalError": "FAIL",
+    "kATSFontTestSeverityInformation": "INFO",
+}
+
+default_levels = ["WARN", "FAIL"]
+
+
+def combine_severity(s1, s2):
+    if s1 == "FAIL" or s2 == "FAIL":
+        return "FAIL"
+    if s1 == "WARN" or s2 == "WARN":
+        return "WARN"
+    return None
+
+
+def parse_ftxvalidator_report(contents):
+    tree = plistlib.loads(contents)
+    worst_severity = None
+    for fonts in tree["kATSFontTestFontsTestedKey"]:
+        font = fonts["kATSFontTestFontPostScriptNameKey"]
+        results = fonts["kATSFontTestArrayKey"]
+        for result in results:
+            if not result["kATSFontTestResultKey"]:
+                continue
+            messages = result["kATSFontTestMessagesKey"]
+            this_messages = defaultdict(list)
+            for message in messages:
+                messagetext = message["kATSFontTestMessageTextKey"]
+                severity = severity_mapping.get(message["kATSFontTestResultKey"])
+                if severity not in default_levels:
+                    continue
+                this_messages[severity].append(messagetext)
+            if not this_messages:
+                continue
+            print(result["kATSFontTestIdentifierKey"])
+            for severity, messages in this_messages.items():
+                for message in messages:
+                    print(f"  {severity}: {message}")
+                    worst_severity = combine_severity(worst_severity, severity)
+            print()
+    return worst_severity
+
+
+def main():
+    parser = argparse.ArgumentParser()
+    parser.add_argument(
+        "--branch", default="origin/main", help="branch to compare current head against"
+    )
+    args = parser.parse_args()
+
+    worst_severity = None
+    for directory, check_type in directory_check_types(args.branch):
+        if check_type not in [CheckType.NEW_FAMILY, CheckType.MODIFIED_FAMILY]:
+            continue
+        fonts = glob.glob(os.path.join(directory, "*.ttf"))
+        for font in fonts:
+            print(f"Validating {font}")
+            result = subprocess.run(
+                ["/Library/Apple/usr/bin/ftxvalidator", "-r", font],
+                capture_output=True,
+                text=False,
+                check=True,
+            )
+            worst_severity = combine_severity(
+                worst_severity, parse_ftxvalidator_report(result.stdout)
+            )
+    if worst_severity == "FAIL":
+        sys.exit(1)
+
+
+if __name__ == "__main__":
+    main()

diff --git a/.github/workflows/test.yaml b/.github/workflows/test.yaml
index ee2d7f2a7aec5855a5c5d8c0a401cc1531c968a0..e11d4495b50c1ddc1c712b2a37a9189d6a62c9eb 100644 (file)
--- a/.github/workflows/test.yaml
+++ b/.github/workflows/test.yaml
@@ -114,3 +114,22 @@ jobs:
         with:
           name: qa
           path: out/
+
+  ftxvalidator:
+    name: Run ftxvalidator on new/changed fonts
+    runs-on: macos-latest
+    steps:
+      - uses: actions/checkout@v1
+      - name: Download and install
+        run: |
+          ${{secrets.OBTAIN_FONTTOOLS}}
+          hdiutil attach font_tools.dmg
+          sudo installer -pkg /Volumes/macOS\ Font\ Tools/macOS\ Font\ Tools.pkg -target /
+          hdiutil detach /Volumes/macOS\ Font\ Tools
+      - name: Set up Python 3.10
+        uses: actions/setup-python@v4.4.0
+        with:
+          python-version: '3.10'
+      - name: Test font with ftxvalidator
+        run: python3 .ci/ftxvalidator.py
+