Skip to content

Commit

Permalink
Update tests to account for stable release not having embeddings (#1226)
Browse files Browse the repository at this point in the history
  • Loading branch information
ivirshup committed Jul 8, 2024
1 parent af88f3d commit 82a3f86
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 11 deletions.
2 changes: 1 addition & 1 deletion api/python/cellxgene_census/tests/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -67,4 +67,4 @@ def census() -> soma.Collection:
def lts_census() -> soma.Collection:
import cellxgene_census

return cellxgene_census.open_soma(census_version="stable")
return cellxgene_census.open_soma(census_version="2023-12-15")
Original file line number Diff line number Diff line change
Expand Up @@ -77,6 +77,10 @@ def test_get_embedding_metadata_by_name(requests_mock: rm.Mocker) -> None:
)


@pytest.mark.xfail(
strict=True,
reason="Current stable release doesn't have embeddings available. This xfail should be removed once that's resolved.",
)
def test_get_embedding_by_name_w_version_aliases() -> None:
"""https://github.com/chanzuckerberg/cellxgene-census/issues/1202"""
# Only testing "stable" as "latest" doesn't have embeddings
Expand Down
10 changes: 0 additions & 10 deletions api/python/cellxgene_census/tests/test_get_anndata.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,16 +8,6 @@
import cellxgene_census


@pytest.fixture(scope="session")
def census() -> soma.Collection:
return cellxgene_census.open_soma(census_version="latest")


@pytest.fixture(scope="session")
def lts_census() -> soma.Collection:
return cellxgene_census.open_soma(census_version="stable")


@pytest.mark.live_corpus
def test_get_anndata_value_filter(census: soma.Collection) -> None:
ad = cellxgene_census.get_anndata(
Expand Down

0 comments on commit 82a3f86

Please sign in to comment.