@@ -79,18 +79,19 @@ def test_default_master_roundtrips(ufo_module):
79
79
80
80
81
81
def test_roundtrip_self_ref (datadir , ufo_module ):
82
- """This test is to solve the problem of roundtrips sometimes resulting in broken
83
- ufos when working with bracket layers.
84
-
85
- Starting with UFO: It's okay to have a bracket glyph that references its base
86
- glyph as a component. However, on the glyphs.app side it's not okay because the base and
87
- bracket glyphs are merged into a single glyph with the bracket glyph now being a layer.
88
- Glyphsapp (2.6.1) does not like the self referencing component, but that's the best we can
89
- do to preserve the information
90
-
91
- The goal here is to make sure that the valid ufo we start with, is still valid in ufo after
92
- roundtriping. We are not concerned with making glyphs.app work properly at the intemediary
93
- stage.
82
+ """This test is to solve the problem of roundtrips sometimes resulting in
83
+ broken ufos when working with bracket layers.
84
+
85
+ Starting with UFO: It's okay to have a bracket glyph that references its
86
+ base glyph as a component. However, on the glyphs.app side it's not okay
87
+ because the base and bracket glyphs are merged into a single glyph with
88
+ the bracket glyph now being a layer. Glyphsapp (2.6.1) does not like the
89
+ self referencing component, but that's the best we can do to preserve the
90
+ information.
91
+
92
+ The goal here is to make sure that the valid ufo we start with, is still
93
+ valid in ufo after roundtriping. We are not concerned with making
94
+ glyphs.app work properly at the intemediary stage.
94
95
"""
95
96
path = datadir / "BracketSelfReference/BracketSelfReference.designspace"
96
97
designspace = designspaceLib .DesignSpaceDocument .fromfile (path )
0 commit comments