Skip to content

Commit

Permalink
silence messages in testing output
Browse files Browse the repository at this point in the history
  • Loading branch information
PietrH committed Jul 14, 2023
1 parent a22bd7c commit 1d40a02
Show file tree
Hide file tree
Showing 7 changed files with 134 additions and 82 deletions.
4 changes: 3 additions & 1 deletion tests/testthat/test-get_cam_op.R
Original file line number Diff line number Diff line change
Expand Up @@ -140,6 +140,8 @@ test_that(
)

test_that("filtering predicates are allowed and work well", {
filtered_cam_op_matrix <- get_cam_op(mica, pred_lt("longitude", 4.0))
filtered_cam_op_matrix <- suppressMessages(
get_cam_op(mica, pred_lt("longitude", 4.0))
)
expect_equal(rownames(filtered_cam_op_matrix), "Mica Viane")
})
8 changes: 5 additions & 3 deletions tests/testthat/test-get_custom_effort.R
Original file line number Diff line number Diff line change
Expand Up @@ -163,9 +163,11 @@ test_that("right columns, cols types, right relative number of rows", {
test_that("check effort and unit values", {
tot_effort <- get_custom_effort(mica)
# filtering deployments reduces effort value
filter_deploys <- get_custom_effort(mica,
pred_gte("latitude", 51.18),
group_by = "year"
filter_deploys <- suppressMessages(
get_custom_effort(mica,
pred_gte("latitude", 51.18),
group_by = "year"
)
)
expect_lt(filter_deploys$effort, tot_effort$effort)

Expand Down
45 changes: 24 additions & 21 deletions tests/testthat/test-get_n_individuals.R
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
test_that("get_n_individuals returns the right structure of dataframe", {

# species arg specified
output_anas_platyrhyncos <- get_n_individuals(mica,
output_anas_platyrhyncos <- suppressMessages(get_n_individuals(mica,
species = "Anas platyrhynchos"
)
))

# type list
expect_type(output_anas_platyrhyncos, "list")
Expand Down Expand Up @@ -64,9 +63,9 @@ test_that(paste(
n_deployments <- length(deployments)

# calculate get_n_individuals for a species undetected in one deployment
output_martes_foina <- get_n_individuals(mica,
output_martes_foina <- suppressMessages(get_n_individuals(mica,
species = "Martes foina"
)
))

# number of rows should be equal to number of deployments
expect_equal(nrow(output_martes_foina), n_deployments)
Expand Down Expand Up @@ -104,34 +103,33 @@ test_that("species = 'all' returns the same of using a vector with all species",

test_that("species is case insensitive", {
expect_equal(
get_n_individuals(mica, species = "Anas platyrhynchos"),
get_n_individuals(mica, species = toupper("Anas platyrhynchos"))
suppressMessages(get_n_individuals(mica, species = "Anas platyrhynchos")),
suppressMessages(get_n_individuals(mica, species = toupper("Anas platyrhynchos")))
)
})

test_that(paste(
"species accepts use of common names and return",
"the same as using scientic name"
), {

# define scientific name
scn <- "Anas platyrhynchos"
# define correspondent vernacular name
vn <- "Mallard"

# get number of individuals for both cases
output_anas_platyrhyncos <- get_n_individuals(mica, species = scn)
output_mallard <- get_n_individuals(mica, species = vn)
output_anas_platyrhyncos <- suppressMessages(get_n_individuals(mica, species = scn))
output_mallard <- suppressMessages(get_n_individuals(mica, species = vn))

# same outputs
expect_equal(output_anas_platyrhyncos, output_mallard)
})

test_that("if subset of species is specified, less individuals are returned", {
output_all_species <- get_n_individuals(mica)
output_anas_platyrhyncos <- get_n_individuals(mica,
output_anas_platyrhyncos <- suppressMessages(get_n_individuals(mica,
species = "Anas platyrhynchos"
)
))

expect_true(sum(output_all_species$n) >= sum(output_anas_platyrhyncos$n))
})
Expand Down Expand Up @@ -166,10 +164,10 @@ test_that("number of individuals is equal to sum of counts", {
dplyr::filter(scientificName == species) %>%
dplyr::pull(count) %>%
sum()
n_individuals <- get_n_individuals(mica,
n_individuals <- suppressMessages(get_n_individuals(mica,
species = "Mallard",
pred("deploymentID", deploy_id)
)
))
expect_equal(n_individuals$n, n_individuals_via_count)
})

Expand All @@ -180,7 +178,8 @@ test_that("sex filters data correctly", {
dplyr::filter(sex == sex_value) %>%
dplyr::pull(count) %>%
sum()
n_individuals_females <- get_n_individuals(mica, species = NULL, sex = sex_value)
n_individuals_females <-
suppressMessages(get_n_individuals(mica, species = NULL, sex = sex_value))
tot_n_individuals_females <- sum(n_individuals_females$n)
expect_equal(tot_n_individuals_females, n_individuals_via_count)
expect_equal(nrow(n_individuals_females), nrow(mica$data$deployments))
Expand All @@ -193,10 +192,10 @@ test_that("multiple sex values allowed", {
dplyr::filter(sex %in% sex_value) %>%
dplyr::pull(count) %>%
sum()
n_individuals_females_undefined <- get_n_individuals(mica,
n_individuals_females_undefined <- suppressMessages(get_n_individuals(mica,
species = NULL,
sex = sex_value
)
))
tot_n_individuals_females_undefined <- sum(n_individuals_females_undefined$n)
expect_equal(
tot_n_individuals_females_undefined,
Expand All @@ -215,7 +214,9 @@ test_that("life stage filters data correctly", {
dplyr::filter(lifeStage == life_stage_value) %>%
dplyr::pull(count) %>%
sum()
n_individuals_juvenile <- get_n_individuals(mica, species = NULL, life_stage = life_stage_value)
n_individuals_juvenile <- suppressMessages(
get_n_individuals(mica, species = NULL, life_stage = life_stage_value)
)
tot_n_individuals_juvenile <- sum(n_individuals_juvenile$n)
expect_equal(tot_n_individuals_juvenile, n_individuals_juvenile_via_count)
expect_equal(nrow(n_individuals_juvenile), nrow(mica$data$deployments))
Expand All @@ -228,9 +229,11 @@ test_that("multiple age values allowed", {
dplyr::filter(lifeStage %in% life_stage_value) %>%
dplyr::pull(count) %>%
sum()
n_individuals_juvenile_adult <- get_n_individuals(mica,
species = NULL,
life_stage = life_stage_value
n_individuals_juvenile_adult <- suppressMessages(
get_n_individuals(mica,
species = NULL,
life_stage = life_stage_value
)
)
tot_n_individuals_juvenile_adult <- sum(n_individuals_juvenile_adult$n)
expect_equal(
Expand Down
66 changes: 42 additions & 24 deletions tests/testthat/test-get_n_obs.R
Original file line number Diff line number Diff line change
@@ -1,8 +1,9 @@
test_that("get_n_obs returns the right structure of dataframe", {

# species arg specified
output_anas_platyrhyncos <- get_n_obs(mica,
species = "Anas platyrhynchos"
output_anas_platyrhyncos <- suppressMessages(
get_n_obs(mica,
species = "Anas platyrhynchos"
)
)

# type list
Expand Down Expand Up @@ -64,8 +65,10 @@ test_that(paste(
n_deployments <- length(deployments)

# calculate get_n_obs for a species undetected in one deployment
output_ondatra_zibethicus <- get_n_obs(mica,
species = "Anas strepera"
output_ondatra_zibethicus <- suppressMessages(
get_n_obs(mica,
species = "Anas strepera"
)
)

# number of rows should be equal to number of deployments
Expand Down Expand Up @@ -104,33 +107,36 @@ test_that("species = 'all' returns the same of using a vector with all species",

test_that("species is case insensitive", {
expect_equal(
get_n_obs(mica, species = "Anas platyrhynchos"),
get_n_obs(mica, species = toupper("ANAS platYrhyncHOS"))
suppressMessages(
get_n_obs(mica, species = "Anas platyrhynchos")
),
suppressMessages(get_n_obs(mica, species = toupper("ANAS platYrhyncHOS")))
)
})

test_that(paste(
"species accepts use of common names and return",
"the same as using scientic name"
), {

# define scientific name
scn <- "Anas platyrhynchos"
# define correspondent vernacular name
vn <- "Mallard"

# get number of observations for both cases
output_anas_platyrhyncos <- get_n_obs(mica, species = scn)
output_mallard <- get_n_obs(mica, species = vn)
output_anas_platyrhyncos <- suppressMessages(get_n_obs(mica, species = scn))
output_mallard <- suppressMessages(get_n_obs(mica, species = vn))

# same outputs
expect_equal(output_anas_platyrhyncos, output_mallard)
})

test_that("if subset of species is specified, less observations are returned", {
output_all_species <- get_n_obs(mica)
output_anas_platyrhyncos <- get_n_obs(mica,
species = "Anas platyrhynchos"
output_anas_platyrhyncos <- suppressMessages(
get_n_obs(mica,
species = "Anas platyrhynchos"
)
)

expect_true(sum(output_all_species$n) >= sum(output_anas_platyrhyncos$n))
Expand Down Expand Up @@ -170,27 +176,29 @@ test_that(paste(
dplyr::pull(.data$sequenceID) %>%
dplyr::n_distinct()
# one sequenceID linked to two observations (different age, sex and count)
n_obs <- get_n_obs(mica,
n_obs <- suppressMessages(get_n_obs(mica,
species = "Mallard",
pred("deploymentID", deploy_id)
)
))
expect_equal(n_obs$n, n_obs_via_sequence_id)
})

test_that("sex filters data correctly", {
sex_value <- "female"
n_obs_females <- get_n_obs(mica, species = NULL, sex = sex_value)
n_obs_females <- suppressMessages(
get_n_obs(mica, species = NULL, sex = sex_value)
)
tot_n_obs_females <- sum(n_obs_females$n)
expect_equal(tot_n_obs_females, 1)
expect_equal(nrow(n_obs_females), nrow(mica$data$deployments))
})

test_that("multiple sex values allowed", {
sex_value <- c("female", "unknown")
n_obs_females_unknown <- get_n_obs(mica,
n_obs_females_unknown <- suppressMessages(get_n_obs(mica,
species = NULL,
sex = sex_value
)
))
tot_n_obs_females_unknown <- sum(n_obs_females_unknown$n)
expect_equal(
tot_n_obs_females_unknown,
Expand All @@ -209,15 +217,19 @@ test_that("life_stage filters data correctly", {
dplyr::filter(.data$lifeStage %in% life_stage_value) %>%
dplyr::distinct(.data$sequenceID) %>%
nrow()
n_obs_subadult <- get_n_obs(mica, species = NULL, life_stage = life_stage_value)
n_obs_subadult <- suppressMessages(
get_n_obs(mica, species = NULL, life_stage = life_stage_value)
)
tot_n_obs_subadult <- sum(n_obs_subadult$n)
expect_equal(tot_n_obs_subadult, n_obs_subadult_via_distinct)
expect_equal(nrow(n_obs_subadult), nrow(mica$data$deployments))
})

test_that("multiple age values allowed", {
life_stage_value <- c("subadult", "adult")
n_obs_subadult_adult <- get_n_obs(mica, species = NULL, life_stage = life_stage_value)
n_obs_subadult_adult <- suppressMessages(
get_n_obs(mica, species = NULL, life_stage = life_stage_value)
)
tot_n_obs_subadult_adult <- sum(n_obs_subadult_adult$n)
n_obs_subadult_adult_calculate <-
mica$data$observations %>%
Expand All @@ -238,7 +250,9 @@ test_that(paste(
), {
species_value <- "Anas platyrhynchos"
sex_value <- "female"
n_obs <- get_n_obs(mica, species = species_value, sex = sex_value)
n_obs <- suppressMessages(
get_n_obs(mica, species = species_value, sex = sex_value)
)
expect_true(all(n_obs$scientificName %in% species_value))
expect_true(all(species_value %in% n_obs$scientificName))
})
Expand All @@ -255,10 +269,14 @@ test_that("Filter by date of deployments via predicates works correctly", {
mica_with_obs_filtered_manually$data$observations <-
mica_with_obs_filtered_manually$data$observations %>%
dplyr::filter(.data$deploymentID %in% deploys_filtered)
obs_filtered_man <- get_n_obs(mica_with_obs_filtered_manually,
pred_lt(arg = "end", value = end_date)) %>%
obs_filtered_man <- suppressMessages(get_n_obs(
mica_with_obs_filtered_manually,
pred_lt(arg = "end", value = end_date)
)) %>%
dplyr::arrange(deploymentID, scientificName)
obs_filtered <- get_n_obs(mica, pred_lt(arg = "end", value = end_date)) %>%
obs_filtered <- suppressMessages(
get_n_obs(mica, pred_lt(arg = "end", value = end_date))
) %>%
dplyr::arrange(deploymentID, scientificName)
expect_equal(obs_filtered, obs_filtered_man)
})
})
26 changes: 18 additions & 8 deletions tests/testthat/test-get_rai.R
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,10 @@ test_that("get_rai returns error if no species is specified", {
})

test_that("get_rai returns the right dataframe", {
output_anas_platyrhyncos <- get_rai(mica,
species = "Anas platyrhynchos"
output_anas_platyrhyncos <- suppressMessages(
get_rai(mica,
species = "Anas platyrhynchos"
)
)

# type list
Expand Down Expand Up @@ -66,15 +68,19 @@ test_that("get_rai returns the same if 'all' is used instead of vector with all

test_that("species is case insensitive", {
expect_equal(
get_rai(mica, species = "Anas platyrhynchos"),
get_rai(mica, species = toupper("Anas platyrhynchos"))
suppressMessages(get_rai(mica, species = "Anas platyrhynchos")),
suppressMessages(get_rai(mica, species = toupper("Anas platyrhynchos")))
)
})

test_that("sex filters data correctly", {
sex_value <- "female"
n_obs_females <- get_n_obs(mica, species = "Mallard", sex = sex_value)
rai_females <- get_rai(mica, species = "Mallard", sex = sex_value)
n_obs_females <- suppressMessages(
get_n_obs(mica, species = "Mallard", sex = sex_value)
)
rai_females <- suppressMessages(
get_rai(mica, species = "Mallard", sex = sex_value)
)
# same first two cols as in get_n_obs
expect_equal(names(n_obs_females)[1:2], names(rai_females)[1:2])
expect_equal(nrow(n_obs_females), nrow(rai_females))
Expand All @@ -85,8 +91,12 @@ test_that("sex filters data correctly", {

test_that("life_stage filters data correctly", {
life_stage_value <- "subadult"
n_obs_subadult <- get_n_obs(mica, species = "Mallard", life_stage = life_stage_value)
rai_subadult <- get_rai(mica, species = "Mallard", life_stage = life_stage_value)
n_obs_subadult <- suppressMessages(
get_n_obs(mica, species = "Mallard", life_stage = life_stage_value)
)
rai_subadult <- suppressMessages(
get_rai(mica, species = "Mallard", life_stage = life_stage_value)
)
# same first two cols as in get_n_obs
expect_equal(names(n_obs_subadult)[1:2], names(rai_subadult)[1:2])
expect_equal(nrow(n_obs_subadult), nrow(rai_subadult))
Expand Down
Loading

0 comments on commit 1d40a02

Please sign in to comment.