Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit d696029

Browse files
committed
Replace knownfailureif with pytest.mark.xfail.
It was only required with nose because it needed a Matplotlib-provided plugin for nose.
1 parent 0dfd271 commit d696029

11 files changed

Lines changed: 32 additions & 37 deletions

lib/matplotlib/tests/test_backend_pdf.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,15 @@
1717
from matplotlib import pyplot as plt
1818
from matplotlib.testing.determinism import (_determinism_source_date_epoch,
1919
_determinism_check)
20-
from matplotlib.testing.decorators import (image_comparison, knownfailureif,
21-
cleanup)
20+
from matplotlib.testing.decorators import image_comparison, cleanup
2221
from matplotlib import dviread
2322

2423

24+
needs_tex = pytest.mark.xfail(
25+
not checkdep_tex(),
26+
reason="This test needs a TeX installation")
27+
28+
2529
@image_comparison(baseline_images=['pdf_use14corefonts'],
2630
extensions=['pdf'])
2731
def test_use14corefonts():
@@ -43,10 +47,6 @@ def test_use14corefonts():
4347
fontsize=14)
4448
ax.axhline(0.5, linewidth=0.5)
4549

46-
needs_tex = knownfailureif(
47-
not checkdep_tex(),
48-
"This test needs a TeX installation")
49-
5050

5151
@cleanup
5252
def test_type42():

lib/matplotlib/tests/test_backend_ps.py

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,25 +5,27 @@
55

66
import io
77
import re
8+
89
import numpy as np
10+
import pytest
911
import six
1012

1113
import matplotlib
1214
import matplotlib.pyplot as plt
1315
from matplotlib import patheffects
1416
from matplotlib.testing.determinism import (_determinism_source_date_epoch,
1517
_determinism_check)
16-
from matplotlib.testing.decorators import cleanup, knownfailureif
18+
from matplotlib.testing.decorators import cleanup
1719

1820

19-
needs_ghostscript = knownfailureif(
21+
needs_ghostscript = pytest.mark.xfail(
2022
matplotlib.checkdep_ghostscript()[0] is None,
21-
"This test needs a ghostscript installation")
23+
reason="This test needs a ghostscript installation")
2224

2325

24-
needs_tex = knownfailureif(
26+
needs_tex = pytest.mark.xfail(
2527
not matplotlib.checkdep_tex(),
26-
"This test needs a TeX installation")
28+
reason="This test needs a TeX installation")
2729

2830

2931
def _test_savefig_to_stringio(format='ps', use_log=False):

lib/matplotlib/tests/test_backend_svg.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,14 +13,14 @@
1313

1414
import matplotlib.pyplot as plt
1515
from matplotlib.testing.decorators import cleanup
16-
from matplotlib.testing.decorators import image_comparison, knownfailureif
16+
from matplotlib.testing.decorators import image_comparison
1717
import matplotlib
1818
from matplotlib import dviread
1919

2020

21-
needs_tex = knownfailureif(
21+
needs_tex = pytest.mark.xfail(
2222
not matplotlib.checkdep_tex(),
23-
"This test needs a TeX installation")
23+
reason="This test needs a TeX installation")
2424

2525

2626
@cleanup

lib/matplotlib/tests/test_basic.py

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
import six
55
import sys
66

7-
from ..testing.decorators import knownfailureif, skipif
7+
from ..testing.decorators import skipif
88

99

1010
SKIPIF_CONDITION = []
@@ -18,12 +18,6 @@ def test_simple():
1818
assert 1 + 1 == 2
1919

2020

21-
@knownfailureif(True)
22-
def test_simple_knownfail():
23-
# Test the known fail mechanism.
24-
assert 1 + 1 == 3
25-
26-
2721
@skipif(True, reason="skipif decorator test with bool condition passed")
2822
def test_skipif_bool():
2923
assert False, "skipif decorator does not work with bool condition"

lib/matplotlib/tests/test_colors.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,7 @@
1818
import matplotlib.colorbar as mcolorbar
1919
import matplotlib.cbook as cbook
2020
import matplotlib.pyplot as plt
21-
from matplotlib.testing.decorators import (image_comparison,
22-
cleanup, knownfailureif)
21+
from matplotlib.testing.decorators import image_comparison, cleanup
2322

2423

2524
def test_resample():
@@ -448,8 +447,8 @@ def test_light_source_shading_default():
448447
assert_array_almost_equal(rgb, expect, decimal=2)
449448

450449

451-
@knownfailureif((V(np.__version__) <= V('1.9.0')
452-
and V(np.__version__) >= V('1.7.0')))
450+
@pytest.mark.xfail(V('1.7.0') <= V(np.__version__) <= V('1.9.0'),
451+
reason='NumPy version is not buggy')
453452
# Numpy 1.9.1 fixed a bug in masked arrays which resulted in
454453
# additional elements being masked when calculating the gradient thus
455454
# the output is different with earlier numpy versions.

lib/matplotlib/tests/test_image.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,7 @@
99
import numpy as np
1010
from numpy.testing import assert_array_equal
1111

12-
from matplotlib.testing.decorators import (image_comparison,
13-
knownfailureif, cleanup)
12+
from matplotlib.testing.decorators import image_comparison, cleanup
1413
from matplotlib.image import (AxesImage, BboxImage, FigureImage,
1514
NonUniformImage, PcolorImage)
1615
from matplotlib.transforms import Bbox, Affine2D, TransformedBbox
@@ -31,6 +30,7 @@
3130
HAS_PIL = True
3231
except ImportError:
3332
HAS_PIL = False
33+
needs_pillow = pytest.mark.xfail(not HAS_PIL, reason='Test requires Pillow')
3434

3535

3636
@image_comparison(baseline_images=['image_interps'])
@@ -101,7 +101,7 @@ def test_image_python_io():
101101
plt.imread(buffer)
102102

103103

104-
@knownfailureif(not HAS_PIL)
104+
@needs_pillow
105105
def test_imread_pil_uint16():
106106
img = plt.imread(os.path.join(os.path.dirname(__file__),
107107
'baseline_images', 'test_image', 'uint16.tif'))
@@ -480,7 +480,7 @@ def test_nonuniformimage_setnorm():
480480
im.set_norm(plt.Normalize())
481481

482482

483-
@knownfailureif(not HAS_PIL)
483+
@needs_pillow
484484
@cleanup
485485
def test_jpeg_alpha():
486486
plt.figure(figsize=(1, 1), dpi=300)

lib/matplotlib/tests/test_mathtext.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
import pytest
1111

1212
import matplotlib
13-
from matplotlib.testing.decorators import image_comparison, knownfailureif, cleanup
13+
from matplotlib.testing.decorators import image_comparison, cleanup
1414
import matplotlib.pyplot as plt
1515
from matplotlib import mathtext
1616

lib/matplotlib/tests/test_mlab.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414

1515
import matplotlib.mlab as mlab
1616
import matplotlib.cbook as cbook
17-
from matplotlib.testing.decorators import knownfailureif, CleanupTestCase
17+
from matplotlib.testing.decorators import CleanupTestCase
1818

1919

2020
try:
@@ -2837,7 +2837,7 @@ def get_z(x, y):
28372837
np.ma.getmask(correct_zi_masked))
28382838

28392839

2840-
@knownfailureif(not HAS_NATGRID)
2840+
@pytest.mark.xfail(not HAS_NATGRID, reason='natgrid not installed')
28412841
def test_griddata_nn():
28422842
# z is a linear function of x and y.
28432843
def get_z(x, y):

lib/matplotlib/tests/test_patheffects.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@
22
unicode_literals)
33

44
import numpy as np
5+
import pytest
56

6-
from matplotlib.testing.decorators import (image_comparison, cleanup,
7-
knownfailureif)
7+
from matplotlib.testing.decorators import image_comparison, cleanup
88
import matplotlib.pyplot as plt
99
import matplotlib.patheffects as path_effects
1010

@@ -76,7 +76,7 @@ def test_patheffect3():
7676

7777

7878
@cleanup
79-
@knownfailureif(True)
79+
@pytest.mark.xfail
8080
def test_PathEffect_points_to_pixels():
8181
fig = plt.figure(dpi=150)
8282
p1, = plt.plot(range(10))

lib/matplotlib/tests/test_rcparams.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
import matplotlib as mpl
1919
import matplotlib.pyplot as plt
2020
from matplotlib.tests import assert_str_equal
21-
from matplotlib.testing.decorators import cleanup, knownfailureif
21+
from matplotlib.testing.decorators import cleanup
2222
import matplotlib.colors as mcolors
2323
from itertools import chain
2424
import numpy as np

0 commit comments

Comments
 (0)