Correct tiledb_query_get_range_var call

NAMESPACE

@@ -245,6 +245,7 @@ export(tiledb_query_get_fragment_uri)
 export(tiledb_query_get_layout)
 export(tiledb_query_get_range)
 export(tiledb_query_get_range_num)
+export(tiledb_query_get_range_var)
 export(tiledb_query_import_buffer)
 export(tiledb_query_result_buffer_elements)
 export(tiledb_query_result_buffer_elements_vec)

R/Query.R

@@ -526,12 +526,12 @@ tiledb_query_get_range <- function(query, dimidx, rngidx) {
 #' Retrieve the query range for a variable-sized query dimension and range index
 #'
 #' @param query A TileDB Query object
-#' @param dimidx An integer index selecting the dimension
+#' @param dimidx An integer index selecting the variable-sized dimension
 #' @param rngidx An integer index selection the given range for the dimension
 #' @return An string vector with elements start and end for the query
 #' range for the given dimension and range index
 #' @export
-tiledb_query_get_range <- function(query, dimidx, rngidx) {
+tiledb_query_get_range_var <- function(query, dimidx, rngidx) {
   stopifnot(`Argument 'query' must be a tiledb_query object` = is(query, "tiledb_query"),
             `Argument 'dimidx' must be numeric` = is.numeric(dimidx),
             `Argument 'rngidx' must be numeric` = is.numeric(rngidx))

inst/tinytest/test_query.R

@@ -292,3 +292,22 @@ expect_silent(tiledb_query_set_subarray(qry, c(1L,5L), "INT32"))
 expect_silent(tiledb_query_submit(qry))
 expect_silent(tiledb_query_finalize(qry))
 expect_equal(tiledb_query_status(qry), "COMPLETE")
+
+
+uri <- tempfile()
+pp <- palmerpenguins::penguins
+fromDataFrame(pp, uri, sparse = TRUE, col_index = c("species", "year"))
+## dim 1: species
+qry <- tiledb_query(tiledb_array(uri), "READ")
+expect_equal(tiledb_query_get_range_num(qry, 1), 1)
+qry <- tiledb_query(tiledb_array(uri), "READ")
+expect_equal(tiledb_query_get_range_var(qry, 1, 1), c("", ""))
+qry <- tiledb_query(tiledb_array(uri), "READ")
+expect_error(tiledb_query_get_range_var(qry, 1, 2))  # wrong range
+## dim 2: year
+qry <- tiledb_query(tiledb_array(uri), "READ")
+expect_equal(tiledb_query_get_range_num(qry, 2), 1)
+qry <- tiledb_query(tiledb_array(uri), "READ")
+expect_equal(tiledb_query_get_range(qry, 2, 1), c(2007, 2009, 0))
+qry <- tiledb_query(tiledb_array(uri), "READ")
+expect_error(tiledb_query_get_range(qry, 2, 2))  # wrong range