Fix #207 (#208)

ropensci · Dec 19, 2024 · 55c8038 · 55c8038
1 parent 7068815
commit 55c8038
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 1 deletion.
diff --git a/.Rbuildignore b/.Rbuildignore
@@ -24,3 +24,4 @@ notes.R
 ^revdep$
 ^LICENSE.note$
 ^.editorconfig$
+^tests/testdata/issue81.ods$
diff --git a/NEWS.md b/NEWS.md
@@ -1,3 +1,7 @@
+# readODS 2.3.2
+
+* Fix #207, trim off large or unused test files
+
 # readODS 2.3.1
 
 * Undo #173, use `cpp11` like 2.2.0

diff --git a/tests/testdata/layout_test.xls b/tests/testdata/layout_test.xls
diff --git a/tests/testdata/layout_test.xlsx b/tests/testdata/layout_test.xlsx
diff --git a/tests/testthat/test_issue81.R b/tests/testthat/test_issue81.R
@@ -11,12 +11,14 @@ test_that("issue 81, correctness", {
 })
 
 test_that("issue 81 real test", {
+    skip_if(!file.exists("../testdata/issue81.ods"))
     file <- "../testdata/issue81.ods"
     res <- read_ods(file, sheet = 2, skip = 4, as_tibble = FALSE)
     expect_equal(sum(is.na(res[,1])), 0)
 })
 
 test_that("issue 84", {
+    skip_if(!file.exists("../testdata/issue81.ods"))
     file <- "../testdata/issue81.ods"
     expect_error(read_ods(file, sheet = "Leavers"), NA)
-})
+})