no need to explicitly close: varLib.build() will do that.

This commit is contained in:
justvanrossum 2020-01-29 12:01:05 +01:00
parent 49507a9de8
commit 69800b0db5

View File

@ -485,7 +485,6 @@ class BuildTest(unittest.TestCase):
ds = DesignSpaceDocument.fromfile(ds_path) ds = DesignSpaceDocument.fromfile(ds_path)
master_fonts = load_masters(ds) master_fonts = load_masters(ds)
try:
# Trigger MVAR generation so varLib is forced to create deltas with a # Trigger MVAR generation so varLib is forced to create deltas with a
# sparse master inbetween. # sparse master inbetween.
font_0_os2 = ds.sources[0].font["OS/2"] font_0_os2 = ds.sources[0].font["OS/2"]
@ -559,9 +558,6 @@ class BuildTest(unittest.TestCase):
varfont, _, _ = build(ds) varfont, _, _ = build(ds)
mvar_tags = [vr.ValueTag for vr in varfont["MVAR"].table.ValueRecord] mvar_tags = [vr.ValueTag for vr in varfont["MVAR"].table.ValueRecord]
assert all(tag in mvar_tags for tag in fontTools.varLib.mvar.MVAR_ENTRIES) assert all(tag in mvar_tags for tag in fontTools.varLib.mvar.MVAR_ENTRIES)
finally:
for master in master_fonts:
master.close()
def test_varlib_build_VVAR_CFF2(self): def test_varlib_build_VVAR_CFF2(self):
ds_path = self.get_test_input('TestVVAR.designspace') ds_path = self.get_test_input('TestVVAR.designspace')