From 7bf69ca593b8f67f5b61cde39445760f37a2a0db Mon Sep 17 00:00:00 2001
From: Colin Rofls <colin@cmyr.net>
Date: Wed, 8 May 2024 16:54:41 -0400
Subject: [PATCH] [glyphs] Add test for anchor propagation on digraph

This is just to sanity check that our behaviour is matching glyphs.app
in the case where we have a glyph with multiple base components but
which is not a ligature.
---
 glyphs-reader/src/propagate_anchors.rs | 39 ++++++++++++++++++++++++++
 1 file changed, 39 insertions(+)

diff --git a/glyphs-reader/src/propagate_anchors.rs b/glyphs-reader/src/propagate_anchors.rs
index a7f165d80..9c7561cc5 100644
--- a/glyphs-reader/src/propagate_anchors.rs
+++ b/glyphs-reader/src/propagate_anchors.rs
@@ -643,6 +643,7 @@ mod tests {
     #[test]
     fn propagate_ligature_anchors() {
         // derived from the observed behaviour of glyphs 3.2.2 (3259)
+        // this is based on the IJ glyph in Oswald (ExtraLight)
         let mut glyphs = GlyphSetBuilder::new()
             .add_glyph("I", |glyph| {
                 glyph
@@ -682,6 +683,44 @@ mod tests {
         )
     }
 
+    #[test]
+    fn digraphs_arent_ligatures() {
+        // derived from the observed behaviour of glyphs 3.2.2 (3259)
+        // this is based on the IJ glyph in Oswald (ExtraLight)
+        let mut glyphs = GlyphSetBuilder::new()
+            .add_glyph("I", |glyph| {
+                glyph
+                    .add_anchor("bottom", (103, 0))
+                    .add_anchor("ogonek", (103, 0))
+                    .add_anchor("top", (103, 810))
+                    .add_anchor("topleft", (20, 810));
+            })
+            .add_glyph("J", |glyph| {
+                glyph
+                    .add_anchor("bottom", (133, 0))
+                    .add_anchor("top", (163, 810));
+            })
+            .add_glyph("IJ", |glyph| {
+                glyph
+                    .add_component("I", (0, 0))
+                    .add_component("J", (206, 0));
+            })
+            .build();
+        propagate_all_anchors_impl(&mut glyphs);
+        let ij = glyphs.get("IJ").unwrap();
+        // these were derived by running the built in glyphs.app propagate anchors
+        // method from the macro panel
+        assert_eq!(
+            ij.layers[0].anchors,
+            [
+                ("bottom", (339., 0.)),
+                ("ogonek", (103., 0.)),
+                ("top", (369., 810.)),
+                ("topleft", (20., 810.)),
+            ]
+        )
+    }
+
     #[test]
     fn propagate_across_layers() {
         // derived from the observed behaviour of glyphs 3.2.2 (3259)