Merge pull request #607 from hugovk/master

Fix test coverage

.gitignore

@@ -1,10 +1,56 @@
-*.pyc
+# Byte-compiled / optimized / DLL files
-*.egg-info
+__pycache__/
-build
+*.py[cod]
-dist
+
-.tox
+# C extensions
 *.so
-docs/_build
+
+# Distribution / packaging
+.Python
+env/
+bin/
+build/
+develop-eggs/
+dist/
+eggs/
+lib/
+lib64/
+parts/
+sdist/
+var/
+*.egg-info/
+.installed.cfg
+*.egg
+
+# Installer logs
+pip-log.txt
+pip-delete-this-directory.txt
+
+# Unit test / coverage reports
+htmlcov/
+.tox/
+.coverage
+.cache
+nosetests.xml
+coverage.xml
+
+# Translations
+*.mo
+
+# Mr Developer
+.mr.developer.cfg
+.project
+.pydevproject
+
+# Rope
+.ropeproject
+
+# Django stuff:
+*.log
+*.pot
+
+# Sphinx documentation
+docs/_build/
 
 # Vim cruft
 .*.swp
@@ -13,3 +59,4 @@ docs/_build
 *~
 \#*#
 .#*
+

.travis.yml

@@ -30,7 +30,7 @@ script:
   - python setup.py clean
   - python setup.py build_ext --inplace
   - coverage run --append --include=PIL/* selftest.py
-  - coverage run --append --include=PIL/* Tests/run.py
+  - python Tests/run.py --coverage
 
 after_success:
   - coverage report

Tests/tester.py

@@ -11,10 +11,8 @@ except NameError:
     # we expect a NameError on py2.x, since it doesn't have ResourceWarnings.
     pass
 
-
-
 import sys
-py3 = (sys.version_info >= (3,0))
+py3 = (sys.version_info >= (3, 0))
 
 # some test helpers
 
@@ -22,6 +20,7 @@ _target = None
 _tempfiles = []
 _logfile = None
 
+
 def success():
     import sys
     success.count += 1
@@ -29,8 +28,10 @@ def success():
         print(sys.argv[0], success.count, failure.count, file=_logfile)
     return True
 
+
 def failure(msg=None, frame=None):
-    import sys, linecache
+    import sys
+    import linecache
     failure.count += 1
     if _target:
         if frame is None:
@@ -49,6 +50,7 @@ def failure(msg=None, frame=None):
 
 success.count = failure.count = 0
 
+
 # predicates
 
 def assert_true(v, msg=None):
@@ -57,35 +59,39 @@ def assert_true(v, msg=None):
     else:
         failure(msg or "got %r, expected true value" % v)
 
+
 def assert_false(v, msg=None):
     if v:
         failure(msg or "got %r, expected false value" % v)
     else:
         success()
 
+
 def assert_equal(a, b, msg=None):
     if a == b:
         success()
     else:
         failure(msg or "got %r, expected %r" % (a, b))
 
+
 def assert_almost_equal(a, b, msg=None, eps=1e-6):
     if abs(a-b) < eps:
         success()
     else:
         failure(msg or "got %r, expected %r" % (a, b))
 
+
 def assert_deep_equal(a, b, msg=None):
     try:
         if len(a) == len(b):
-            if all([x==y for x,y in zip(a,b)]):
+            if all([x == y for x, y in zip(a, b)]):
                 success()
             else:
-                failure(msg or "got %s, expected %s" % (a,b))
+                failure(msg or "got %s, expected %s" % (a, b))
         else:
             failure(msg or "got length %s, expected %s" % (len(a), len(b)))
     except:
-        assert_equal(a,b,msg)
+        assert_equal(a, b, msg)
 
 
 def assert_match(v, pattern, msg=None):
@@ -95,8 +101,10 @@ def assert_match(v, pattern, msg=None):
     else:
         failure(msg or "got %r, doesn't match pattern %r" % (v, pattern))
 
+
 def assert_exception(exc_class, func):
-    import sys, traceback
+    import sys
+    import traceback
     try:
         func()
     except exc_class:
@@ -108,8 +116,10 @@ def assert_exception(exc_class, func):
     else:
         failure("expected %r exception, got no exception" % exc_class.__name__)
 
+
 def assert_no_exception(func):
-    import sys, traceback
+    import sys
+    import traceback
     try:
         func()
     except:
@@ -118,11 +128,14 @@ def assert_no_exception(func):
     else:
         success()
 
+
 def assert_warning(warn_class, func):
     # note: this assert calls func three times!
     import warnings
+
     def warn_error(message, category=UserWarning, **options):
         raise category(message)
+
     def warn_ignore(message, category=UserWarning, **options):
         pass
     warn = warnings.warn
@@ -141,15 +154,18 @@ def assert_warning(warn_class, func):
 
 from io import BytesIO
 
+
 def fromstring(data):
     from PIL import Image
     return Image.open(BytesIO(data))
 
+
 def tostring(im, format, **options):
     out = BytesIO()
     im.save(out, format, **options)
     return out.getvalue()
 
+
 def lena(mode="RGB", cache={}):
     from PIL import Image
     im = cache.get(mode)
@@ -165,6 +181,7 @@ def lena(mode="RGB", cache={}):
     cache[mode] = im
     return im
 
+
 def assert_image(im, mode, size, msg=None):
     if mode is not None and im.mode != mode:
         failure(msg or "got mode %r, expected %r" % (im.mode, mode))
@@ -173,6 +190,7 @@ def assert_image(im, mode, size, msg=None):
     else:
         success()
 
+
 def assert_image_equal(a, b, msg=None):
     if a.mode != b.mode:
         failure(msg or "got mode %r, expected %r" % (a.mode, b.mode))
@@ -184,6 +202,7 @@ def assert_image_equal(a, b, msg=None):
     else:
         success()
 
+
 def assert_image_similar(a, b, epsilon, msg=None):
     epsilon = float(epsilon)
     if a.mode != b.mode:
@@ -193,19 +212,25 @@ def assert_image_similar(a, b, epsilon, msg=None):
     diff = 0
     try:
         ord(b'0')
-        for abyte,bbyte in zip(a.tobytes(),b.tobytes()):
+        for abyte, bbyte in zip(a.tobytes(), b.tobytes()):
             diff += abs(ord(abyte)-ord(bbyte))
     except:
-        for abyte,bbyte in zip(a.tobytes(),b.tobytes()):
+        for abyte, bbyte in zip(a.tobytes(), b.tobytes()):
             diff += abs(abyte-bbyte)
     ave_diff = float(diff)/(a.size[0]*a.size[1])
     if epsilon < ave_diff:
-        return failure(msg or "average pixel value difference %.4f > epsilon %.4f" %(ave_diff, epsilon))
+        return failure(
+            msg or "average pixel value difference %.4f > epsilon %.4f" % (
+                ave_diff, epsilon))
     else:
         return success()
 
+
 def tempfile(template, *extra):
-    import os, os.path, sys, tempfile
+    import os
+    import os.path
+    import sys
+    import tempfile
     files = []
     root = os.path.join(tempfile.gettempdir(), 'pillow-tests')
     try:
@@ -222,11 +247,13 @@ def tempfile(template, *extra):
     _tempfiles.extend(files)
     return files[0]
 
+
 # test runner
 
 def run():
     global _target, _tests, run
-    import sys, traceback
+    import sys
+    import traceback
     _target = sys.modules["__main__"]
     run = None  # no need to run twice
     tests = []
@@ -251,29 +278,42 @@ def run():
                     sys.argv[0], lineno, v))
                 failure.count += 1
 
+
 def yield_test(function, *args):
     # collect delayed/generated tests
     _tests.append((function, args))
 
+
 def skip(msg=None):
     import os
     print("skip")
     os._exit(0)  # don't run exit handlers
 
+
 def ignore(pattern):
     """Tells the driver to ignore messages matching the pattern, for the
     duration of the current test."""
     print('ignore: %s' % pattern)
 
+
 def _setup():
     global _logfile
+
+    import sys
+    if "--coverage" in sys.argv:
+        import coverage
+        cov = coverage.coverage(auto_data=True, include="PIL/*")
+        cov.start()
+
     def report():
         if run:
             run()
         if success.count and not failure.count:
             print("ok")
             # only clean out tempfiles if test passed
-            import os, os.path, tempfile
+            import os
+            import os.path
+            import tempfile
             for file in _tempfiles:
                 try:
                     os.remove(file)
@@ -285,18 +325,9 @@ def _setup():
             except OSError:
                 pass
 
-        if "--coverage" in sys.argv:
+    import atexit
-            import coverage
-            coverage.stop()
-            # The coverage module messes up when used from inside an
-            # atexit handler.  Do an explicit save to make sure that
-            # we actually flush the coverage cache.
-            coverage.the_coverage.save()
-    import atexit, sys
     atexit.register(report)
-    if "--coverage" in sys.argv:
+
-        import coverage
-        coverage.start()
     if "--log" in sys.argv:
         _logfile = open("test.log", "a")