Skip to content

Commit

Permalink
remove query tests that used private interfaces (#745)
Browse files Browse the repository at this point in the history
  • Loading branch information
Bruce Martin committed Jan 19, 2023
1 parent 30bcd1e commit 578936a
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 23 deletions.
4 changes: 4 additions & 0 deletions apis/python/src/tiledbsoma/__init__.py
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
from somacore import AxisQuery, ExperimentAxisQuery

from .collection import Collection
from .dataframe import DataFrame
from .dense_nd_array import DenseNDArray
Expand Down Expand Up @@ -25,6 +27,8 @@
"get_storage_engine",
"TileDBObject",
"TileDBArray",
"AxisQuery",
"ExperimentAxisQuery",
"Collection",
"DenseNDArray",
"DoesNotExistError",
Expand Down
34 changes: 11 additions & 23 deletions apis/python/tests/test_experiment_query.py
Original file line number Diff line number Diff line change
Expand Up @@ -108,31 +108,23 @@ def test_experiment_query_all(soma_experiment):
query.X("raw").tables()
)

# read as table
read_result = query._read(
"raw", column_names={"obs": None, "var": None}, X_layers=()
)
assert len(read_result.X_layers) == 0
assert isinstance(read_result.obs, pa.Table)
assert isinstance(read_result.var, pa.Table)
assert isinstance(read_result.X, pa.Table)
assert read_result.obs == query.obs().concat()
assert read_result.var == query.var().concat()
assert (
read_result.X
== soma_experiment.ms["RNA"]
.X["raw"]
.read((slice(None), slice(None)))
.tables()
.concat()
)

# read as anndata
ad = query.to_anndata("raw")
assert ad.n_obs == query.n_obs and ad.n_vars == query.n_vars

assert set(ad.obs.keys().to_list()) == set(["soma_joinid", "label"])
assert set(ad.var.keys().to_list()) == set(["soma_joinid", "label"])

obs = soma_experiment.obs.read().concat().to_pandas()
obs.index = obs.index.map(str)
assert (obs == ad.obs).all().all()

var = soma_experiment.ms["RNA"].var.read().concat().to_pandas()
var.index = var.index.map(str)
assert (var == ad.var).all().all()

assert len(ad.layers) == 0

raw_X = (
soma_experiment.ms["RNA"]
.X["raw"]
Expand Down Expand Up @@ -284,10 +276,6 @@ def test_X_layers(soma_experiment):
)

with soma_experiment.axis_query("RNA") as query:
read_result = query._read("A", X_layers=["B"], column_names={})
assert read_result.X == A
assert read_result.X_layers["B"] == B

ad = query.to_anndata("B", X_layers=["A"])
ad_X_coo = ad.X.tocoo()
assert np.array_equal(B["soma_dim_0"], ad_X_coo.row)
Expand Down

0 comments on commit 578936a

Please sign in to comment.