From 4771e7cd6c20b56c29900a763f1f2b162b66748b Mon Sep 17 00:00:00 2001
From: Cosimo Lupo <cosimo@anthrotype.com>
Date: Thu, 1 Mar 2018 19:39:24 +0000
Subject: [PATCH] [ttGlyphPen_test] add test for OverflowError

---
 Tests/pens/ttGlyphPen_test.py | 20 ++++++++++++++++++--
 1 file changed, 18 insertions(+), 2 deletions(-)

diff --git a/Tests/pens/ttGlyphPen_test.py b/Tests/pens/ttGlyphPen_test.py
index db88b4f8a..250dbe192 100644
--- a/Tests/pens/ttGlyphPen_test.py
+++ b/Tests/pens/ttGlyphPen_test.py
@@ -5,10 +5,11 @@ import os
 import unittest
 
 from fontTools import ttLib
+from fontTools.misc.testTools import TestCase
 from fontTools.pens.ttGlyphPen import TTGlyphPen, MAX_F2DOT14
 
 
-class TTGlyphPenTest(unittest.TestCase):
+class TTGlyphPenTest(TestCase):
 
     def runEndToEnd(self, filename):
         font = ttLib.TTFont()
@@ -184,7 +185,7 @@ class TTGlyphPenTest(unittest.TestCase):
 
         self.assertEqual(expectedGlyph, compositeGlyph)
 
-    def test_out_of_range_component_transform(self):
+    def test_out_of_range_transform_decomposed(self):
         componentName = 'a'
         glyphSet = {}
         pen = TTGlyphPen(glyphSet)
@@ -216,6 +217,21 @@ class TTGlyphPenTest(unittest.TestCase):
 
         self.assertEqual(expectedGlyph, compositeGlyph)
 
+    def test_transform_overflow_error(self):
+        componentName = 'a'
+        glyphSet = {}
+        pen = TTGlyphPen(glyphSet, decomposeOverflowingTransform=False)
+
+        pen.moveTo((0, 0))
+        pen.lineTo((0, 1))
+        pen.lineTo((1, 0))
+        pen.closePath()
+        glyphSet[componentName] = _TestGlyph(pen.glyph())
+
+        pen.addComponent(componentName, (2.00001, 0, 0, 1, 0, 0))
+        with self.assertRaisesRegex(OverflowError, "too large"):
+            pen.glyph()
+
 
 class _TestGlyph(object):
     def __init__(self, glyph):