skip checking lookup functionality tests on CRAN due to flakiness with linux devel environments

Author: thegargiulian

NEWS.md

@@ -1,3 +1,7 @@
+# verdata 1.0.2
+
+* Removing a test of `mse()` when `lookup_estimates = TRUE`. This was a timing based test that was too flaky to be a reliable indicator of function performance across all test environment. Increasing the time tolerance in update 1.0.1 did not meaningfully resolve this issue as hoped.
+
 # verdata 1.0.1
 
 * Timing test for `mse()` using `lookup_estimates` option uses more generous comparison to ensure that test doesn't fail unnecessarily when estimates are correctly looked up.

tests/testthat/test-estimate_mse.R

@@ -164,6 +164,7 @@ testthat::test_that("mse function returns errors when inputs are misspecified",
 
 testthat::test_that("lookup function correctly finds strata that have and have not been estimated", {
 
+    testthat::skip_on_cran()
 
     local_dir <- system.file("extdata", "right", package = "verdata")
     replicates <- read_replicates(local_dir, "reclutamiento", replicate_nums = 1,

tests/testthat/test-lookup_estimates.R

@@ -38,6 +38,8 @@ my_stratum <- tibble::tibble(in_A, in_B) %>%
 
 testthat::test_that("Lookup existing estimates", {
 
+    testthat::skip_on_cran()
+
     estrato <- replicates %>%
         dplyr::filter(sexo == "HOMBRE",
                       yy_hecho == 1999,
@@ -54,6 +56,8 @@ testthat::test_that("Lookup existing estimates", {
 
 testthat::test_that("Lookup non existing estimates", {
 
+    testthat::skip_on_cran()
+
     estrato <- replicates %>%
         dplyr::filter(sexo == "HOMBRE",
                       yy_hecho == 1998,