Skip to content

Commit

Permalink
Temporarily suppress warnings instead of checking for multiple simult…
Browse files Browse the repository at this point in the history
…aneously
  • Loading branch information
Nick-Eagles committed Jul 18, 2024
1 parent 0b49cb2 commit f860a84
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 6 deletions.
16 changes: 12 additions & 4 deletions tests/testthat/test-build_spe.R
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
test_that(
"build_spe",
{
bfc <- BiocFileCache()
gtf_cache <- bfcrpath(
bfc <- BiocFileCache::BiocFileCache()
gtf_cache <- BiocFileCache::bfcrpath(
bfc,
paste0(
"ftp://ftp.ebi.ac.uk/pub/databases/gencode/Gencode_human/",
Expand Down Expand Up @@ -37,8 +37,16 @@ test_that(
prep_imagej_coords(sample_info, out_dir = spe_input_dir)
prep_imagej_image(sample_info, out_dir = spe_input_dir)

spe = build_spe(
sample_info, coords_dir = spe_input_dir, reference_gtf = gtf_cache
# TODO: find a way to match against the two warnings that
# simultaneously occur due to using a different GTF; not good to
# suppress all warnings
suppressWarnings(
{
spe = build_spe(
sample_info, coords_dir = spe_input_dir,
reference_gtf = gtf_cache
)
},
)

########################################################################
Expand Down
2 changes: 0 additions & 2 deletions tests/testthat/test-spe_to_seurat.R
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
library(SpatialExperiment)

test_that(
"spe_to_seurat",
{
Expand Down

0 comments on commit f860a84

Please sign in to comment.