Skip to content

Commit

Permalink
Skip COO matrix tests for newer anndata (#3442)
Browse files Browse the repository at this point in the history
  • Loading branch information
flying-sheep authored Jan 21, 2025
1 parent 7dfd339 commit 6c89e1d
Showing 1 changed file with 15 additions and 2 deletions.
17 changes: 15 additions & 2 deletions tests/test_qc_metrics.py
Original file line number Diff line number Diff line change
@@ -1,10 +1,13 @@
from __future__ import annotations

from importlib.metadata import version

import numpy as np
import pandas as pd
import pytest
from anndata import AnnData
from anndata.tests.helpers import assert_equal
from packaging.version import Version
from scipy import sparse

import scanpy as sc
Expand All @@ -18,7 +21,7 @@
)
from testing.scanpy._helpers import as_sparse_dask_array, maybe_dask_process_context
from testing.scanpy._pytest.marks import needs
from testing.scanpy._pytest.params import ARRAY_TYPES
from testing.scanpy._pytest.params import ARRAY_TYPES, ARRAY_TYPES_MEM


@pytest.fixture
Expand Down Expand Up @@ -198,8 +201,18 @@ def adata_mito():
return adata_dense, init_var


skip_if_adata_0_12 = pytest.mark.skipif(
Version(version("anndata")) >= Version("0.12.0.dev0"),
reason="Newer AnnData removes implicit support for COO matrices",
)


@pytest.mark.parametrize(
"cls", [np.asarray, sparse.csr_matrix, sparse.csc_matrix, sparse.coo_matrix]
"cls",
[
*ARRAY_TYPES_MEM,
pytest.param(sparse.coo_matrix, marks=[skip_if_adata_0_12], id="scipy_coo"),
],
)
def test_qc_metrics_format(cls):
adata_dense, init_var = adata_mito()
Expand Down

0 comments on commit 6c89e1d

Please sign in to comment.