shadowfacts/fonttools
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[ttGlyphPen_test] ttProgram has __eq__ now, remove unnecessary assignments

Browse Source 
previously assertEqual would fail, despite the `program` attributes where both empty, as the object have different id(). The Program.__eq__ method now compares self.__dict__ == other.__dict__, hence the test pass.
This commit is contained in:
Cosimo Lupo 2016-03-11 15:10:27 +00:00
parent 35c46aa0e2
commit d4b9a2bec3
1 changed files with 0 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Lib/fontTools/pens/ttGlyphPen_test.py
View File

@ -68,7 +68,6 @@ class TTGlyphPenTest(unittest.TestCase):
pen.endPath() pen.endPath()
endPathGlyph = pen.glyph() endPathGlyph = pen.glyph()
endPathGlyph.program = closePathGlyph.program
self.assertEqual(closePathGlyph, endPathGlyph) self.assertEqual(closePathGlyph, endPathGlyph)
def test_glyph_errorOnUnendedContour(self): def test_glyph_errorOnUnendedContour(self):
@ -105,7 +104,6 @@ class TTGlyphPenTest(unittest.TestCase):
pen.closePath() pen.closePath()
plainGlyph = pen.glyph() plainGlyph = pen.glyph()
plainGlyph.program = compositeGlyph.program
self.assertEqual(plainGlyph, compositeGlyph) self.assertEqual(plainGlyph, compositeGlyph)