Skip to content

Commit b49b782

Browse files
committed
Re-enable tests xfailed in pydata#3808
1 parent 564a291 commit b49b782

File tree

2 files changed

+2
-19
lines changed

2 files changed

+2
-19
lines changed

xarray/tests/test_cftimeindex.py

+2-15
Original file line numberDiff line numberDiff line change
@@ -450,7 +450,6 @@ def test_sel_date_scalar(da, date_type, index):
450450
assert_identical(result, expected)
451451

452452

453-
@pytest.mark.xfail(reason="https://github.com/pydata/xarray/issues/3751")
454453
@requires_cftime
455454
def test_sel_date_distant_date(da, date_type, index):
456455
expected = xr.DataArray(4).assign_coords(time=index[3])
@@ -513,20 +512,14 @@ def test_sel_date_scalar_backfill(da, date_type, index, sel_kwargs):
513512
[
514513
{"method": "pad", "tolerance": timedelta(days=20)},
515514
{"method": "backfill", "tolerance": timedelta(days=20)},
516-
pytest.param(
517-
{"method": "nearest", "tolerance": timedelta(days=20)},
518-
marks=pytest.mark.xfail(
519-
reason="https://github.com/pydata/xarray/issues/3751"
520-
),
521-
),
515+
{"method": "nearest", "tolerance": timedelta(days=20)},
522516
],
523517
)
524518
def test_sel_date_scalar_tolerance_raises(da, date_type, sel_kwargs):
525519
with pytest.raises(KeyError):
526520
da.sel(time=date_type(1, 5, 1), **sel_kwargs)
527521

528522

529-
@pytest.mark.xfail(reason="https://github.com/pydata/xarray/issues/3751")
530523
@requires_cftime
531524
@pytest.mark.parametrize(
532525
"sel_kwargs",
@@ -574,12 +567,7 @@ def test_sel_date_list_backfill(da, date_type, index, sel_kwargs):
574567
[
575568
{"method": "pad", "tolerance": timedelta(days=20)},
576569
{"method": "backfill", "tolerance": timedelta(days=20)},
577-
pytest.param(
578-
{"method": "nearest", "tolerance": timedelta(days=20)},
579-
marks=pytest.mark.xfail(
580-
reason="https://github.com/pydata/xarray/issues/3751"
581-
),
582-
),
570+
{"method": "nearest", "tolerance": timedelta(days=20)},
583571
],
584572
)
585573
def test_sel_date_list_tolerance_raises(da, date_type, sel_kwargs):
@@ -614,7 +602,6 @@ def range_args(date_type):
614602
]
615603

616604

617-
@pytest.mark.xfail(reason="https://github.com/pydata/xarray/issues/3751")
618605
@requires_cftime
619606
def test_indexing_in_series_getitem(series, index, scalar_args, range_args):
620607
for arg in scalar_args:

xarray/tests/test_interp.py

-4
Original file line numberDiff line numberDiff line change
@@ -586,7 +586,6 @@ def test_datetime_single_string():
586586
assert_allclose(actual.drop_vars("time"), expected)
587587

588588

589-
@pytest.mark.xfail(reason="https://github.com/pydata/xarray/issues/3751")
590589
@requires_cftime
591590
@requires_scipy
592591
def test_cftime():
@@ -613,7 +612,6 @@ def test_cftime_type_error():
613612
da.interp(time=times_new)
614613

615614

616-
@pytest.mark.xfail(reason="https://github.com/pydata/xarray/issues/3751")
617615
@requires_cftime
618616
@requires_scipy
619617
def test_cftime_list_of_strings():
@@ -635,7 +633,6 @@ def test_cftime_list_of_strings():
635633
assert_allclose(actual, expected)
636634

637635

638-
@pytest.mark.xfail(reason="https://github.com/pydata/xarray/issues/3751")
639636
@requires_cftime
640637
@requires_scipy
641638
def test_cftime_single_string():
@@ -697,7 +694,6 @@ def test_datetime_interp_noerror():
697694
a.interp(x=xi, time=xi.time) # should not raise an error
698695

699696

700-
@pytest.mark.xfail(reason="https://github.com/pydata/xarray/issues/3751")
701697
@requires_cftime
702698
def test_3641():
703699
times = xr.cftime_range("0001", periods=3, freq="500Y")

0 commit comments

Comments
 (0)