From 71fb49962a3f3599cc91250558eb0c424aa80b15 Mon Sep 17 00:00:00 2001
From: Behdad Esfahbod <behdad@behdad.org>
Date: Mon, 13 Jun 2016 18:47:03 -0400
Subject: [PATCH] Improve perimeterPen tests

The tolerance on the pen is set to 0.5%, so it does not make sense
to test for accuracy for anything more than integers given the
values.
---
 Lib/fontTools/pens/perimeterPen_test.py | 18 ++++++++----------
 1 file changed, 8 insertions(+), 10 deletions(-)

diff --git a/Lib/fontTools/pens/perimeterPen_test.py b/Lib/fontTools/pens/perimeterPen_test.py
index 8434cfdf2..84e734207 100644
--- a/Lib/fontTools/pens/perimeterPen_test.py
+++ b/Lib/fontTools/pens/perimeterPen_test.py
@@ -3,8 +3,6 @@ from fontTools.misc.py23 import *
 from fontTools.pens.perimeterPen import PerimeterPen
 import unittest
 
-precision = 8
-
 def draw1_(pen):
     pen.moveTo( (254, 360) )
     pen.lineTo( (771, 367) )
@@ -126,42 +124,42 @@ class PerimeterPenTest(unittest.TestCase):
     def test_PScontour_clockwise_line_first(self):
         pen = PerimeterPen(None)
         draw1_(pen)
-        self.assertEqual(1588.68068486, round(pen.value, precision))
+        self.assertEqual(1589, round(pen.value))
 
     def test_PScontour_counterclockwise_line_last(self):
         pen = PerimeterPen(None)
         draw2_(pen)
-        self.assertEqual(1588.68068486, round(pen.value, precision))
+        self.assertEqual(1589, round(pen.value))
 
     def test_PScontour_clockwise_line_last(self):
         pen = PerimeterPen(None)
         draw3_(pen)
-        self.assertEqual(1588.68068486, round(pen.value, precision))
+        self.assertEqual(1589, round(pen.value))
 
     def test_PScontour_counterclockwise_line_first(self):
         pen = PerimeterPen(None)
         draw4_(pen)
-        self.assertEqual(1588.68068486, round(pen.value, precision))
+        self.assertEqual(1589, round(pen.value))
 
     def test_TTcontour_clockwise_line_first(self):
         pen = PerimeterPen(None)
         draw5_(pen)
-        self.assertEqual(1588.87336417, round(pen.value, precision))
+        self.assertEqual(1589, round(pen.value))
 
     def test_TTcontour_counterclockwise_line_last(self):
         pen = PerimeterPen(None)
         draw6_(pen)
-        self.assertEqual(1588.87336417, round(pen.value, precision))
+        self.assertEqual(1589, round(pen.value))
 
     def test_TTcontour_clockwise_line_last(self):
         pen = PerimeterPen(None)
         draw7_(pen)
-        self.assertEqual(1588.87336417, round(pen.value, precision))
+        self.assertEqual(1589, round(pen.value))
 
     def test_TTcontour_counterclockwise_line_first(self):
         pen = PerimeterPen(None)
         draw8_(pen)
-        self.assertEqual(1588.87336417, round(pen.value, precision))
+        self.assertEqual(1589, round(pen.value))
 
 
 if __name__ == '__main__':