NEWS and test

metrumresearchgroup · Oct 31, 2024 · fa5c0b2 · fa5c0b2
1 parent c1e83fc
commit fa5c0b2
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/NEWS.md b/NEWS.md
@@ -2,7 +2,7 @@
 
 ## New features and changes
 
-- `logRemove` added to package to assist with removing files from the QC log.
+- `logRemove` added to package to assist with removing files from the QC log. (#105)
 
 - `diffFiles` can now display the entire file that is being diffed. (#115)
 

diff --git a/tests/testthat/test-logRemove.R b/tests/testthat/test-logRemove.R
@@ -1,11 +1,14 @@
 with_demoRepo({
 
-  logRemove("script/data-assembly.R")
-
   testthat::test_that("logRemove removes the file from the QClog", {
+
     qclog <- logRead()
+    expect_true(nrow(qclog[qclog$file == "script/data-assembly.R",]) > 0)
+
+    logRemove("script/data-assembly.R")
+    qclog2 <- logRead()
 
-    expect_true(nrow(qclog[qclog$file == "script/data-assembly.R",]) == 0)
+    expect_true(nrow(qclog2[qclog2$file == "script/data-assembly.R",]) == 0)
   }) 
 })