TST: skip test_dviread if kpsewhich is not available

jankatins · matthew-brett · commit dcbb41d7bf0d · 2016-08-24T20:38:08.000-07:00
Also fix the docstring to mention that it is part of miktex...
diff --git a/lib/matplotlib/dviread.py b/lib/matplotlib/dviread.py
@@ -844,8 +844,8 @@ def find_tex_file(filename, format=None):
     `--format` option.
 
     Apparently most existing TeX distributions on Unix-like systems
-    use kpathsea. I hear MikTeX (a popular distribution on Windows)
-    doesn't use kpathsea, so what do we do? (TODO)
+    use kpathsea. It's also available as part of MikTeX, a popular
+    distribution on Windows.
 
     .. seealso::
 
diff --git a/lib/matplotlib/testing/decorators.py b/lib/matplotlib/testing/decorators.py
@@ -431,3 +431,24 @@ def backend_switcher(*args, **kwargs):
 
         return nose.tools.make_decorator(func)(backend_switcher)
     return switch_backend_decorator
+
+
+def skip_if_command_unavailable(cmd):
+    """
+    skips a test if a command is unavailable.
+
+    Parameters
+    ----------
+    cmd : list of str
+        must be a complete command which should not
+        return a non zero exit code, something like
+        ["latex", "-version"]
+    """
+    from matplotlib.compat.subprocess import check_output
+    try:
+        check_output(cmd)
+    except:
+        from nose import SkipTest
+        raise SkipTest('missing command: %s' % cmd[0])
+
+    return lambda f: f
diff --git a/lib/matplotlib/tests/test_dviread.py b/lib/matplotlib/tests/test_dviread.py
@@ -2,6 +2,8 @@
                         unicode_literals)
 
 import six
+from matplotlib.testing.decorators import skip_if_command_unavailable
+
 
 from nose.tools import assert_equal
 import matplotlib.dviread as dr
@@ -52,3 +54,19 @@ def test_PsfontsMap():
     assert_equal(entry.encoding, None)
     entry = fontmap['TeXfont9']
     assert_equal(entry.filename, '/absolute/font9.pfb')
+
+
+@skip_if_command_unavailable(["kpsewhich", "-version"])
+def test_dviread():
+    dir = os.path.join(os.path.dirname(__file__), 'baseline_images', 'dviread')
+    with open(os.path.join(dir, 'test.json')) as f:
+        correct = json.load(f)
+    with dr.Dvi(os.path.join(dir, 'test.dvi'), None) as dvi:
+        data = [{'text': [[t.x, t.y,
+                           six.unichr(t.glyph),
+                           six.text_type(t.font.texname),
+                           round(t.font.size, 2)]
+                          for t in page.text],
+                 'boxes': [[b.x, b.y, b.height, b.width] for b in page.boxes]}
+                for page in dvi]
+    assert_equal(data, correct)
