Merge pull request #157 from tidymodels/expect_snapshot-linting

lint expect_snapshot()
tidymodels · Nov 9, 2024 · 95e578b · 95e578b
2 parents 575663c + a913d6b
commit 95e578b
Show file tree

Hide file tree

Showing 12 changed files with 118 additions and 61 deletions.
diff --git a/tests/testthat/test-adasyn.R b/tests/testthat/test-adasyn.R
@@ -8,7 +8,8 @@ test_that("errors if there isn't enough data", {
   credit_data0$Status[1] <- "dummy"
   credit_data0$Status <- as.factor(credit_data0$Status)
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     recipe(Status ~ Age, data = credit_data0) %>%
       step_adasyn(Status) %>%
       prep()
@@ -32,13 +33,15 @@ test_that("basic usage", {
 test_that("bad data", {
   rec <- recipe(~., data = circle_example)
   # numeric check
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     rec %>%
       step_adasyn(x) %>%
       prep()
   )
   # Multiple variable check
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     rec %>%
       step_adasyn(class, id) %>%
       prep()
@@ -52,7 +55,8 @@ test_that("errors if character are present", {
     stringsAsFactors = FALSE
   )
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     recipe(~., data = df_char) %>%
       step_adasyn(x) %>%
       prep()
@@ -64,7 +68,8 @@ test_that("NA in response", {
 
   data("credit_data", package = "modeldata")
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     recipe(Job ~ Age, data = credit_data) %>%
       step_adasyn(Job) %>%
       prep()

diff --git a/tests/testthat/test-adasyn_impl.R b/tests/testthat/test-adasyn_impl.R
@@ -3,34 +3,41 @@ circle_example_num <- circle_example[, c("x", "y", "class")]
 test_that("adasyn() interfaces correctly", {
   expect_no_error(adasyn(circle_example_num, var = "class"))
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     adasyn(circle_example_num, var = "Class")
   )
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     adasyn(circle_example_num, var = c("class", "x"))
   )
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     adasyn(circle_example_num, var = "x")
   )
 
   circle_example0 <- circle_example_num
   circle_example0[1, 1] <- NA
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     adasyn(circle_example0, var = "class")
   )
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     adasyn(circle_example_num, var = "class", k = 0)
   )
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     adasyn(circle_example_num, var = "class", k = -1)
   )
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     adasyn(circle_example_num, var = "class", k = c(5, 10))
   )
 })

diff --git a/tests/testthat/test-bsmote.R b/tests/testthat/test-bsmote.R
@@ -44,13 +44,15 @@ test_that("basic usage", {
 test_that("bad data", {
   rec <- recipe(~., data = circle_example)
   # numeric check
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     rec %>%
       step_bsmote(x) %>%
       prep()
   )
   # Multiple variable check
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     rec %>%
       step_bsmote(class, id) %>%
       prep()
@@ -64,7 +66,8 @@ test_that("errors if character are present", {
     stringsAsFactors = FALSE
   )
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     recipe(~., data = df_char) %>%
       step_bsmote(x) %>%
       prep()
@@ -76,7 +79,8 @@ test_that("NA in response", {
 
   data("credit_data", package = "modeldata")
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     recipe(Job ~ Age, data = credit_data) %>%
       step_bsmote(Job) %>%
       prep()

diff --git a/tests/testthat/test-bsmote_impl.R b/tests/testthat/test-bsmote_impl.R
@@ -3,34 +3,41 @@ circle_example_num <- circle_example[, c("x", "y", "class")]
 test_that("bsmote() interfaces correctly", {
   expect_no_error(bsmote(circle_example_num, var = "class"))
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     bsmote(circle_example_num, var = "Class")
   )
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     bsmote(circle_example_num, var = c("class", "x"))
   )
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     bsmote(circle_example_num, var = "x")
   )
 
   circle_example0 <- circle_example_num
   circle_example0[1, 1] <- NA
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     bsmote(circle_example0, var = "class")
   )
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     bsmote(circle_example_num, var = "class", k = 0)
   )
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     bsmote(circle_example_num, var = "class", k = -1)
   )
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     bsmote(circle_example_num, var = "class", k = c(5, 10))
   )
 })

diff --git a/tests/testthat/test-downsample.R b/tests/testthat/test-downsample.R
@@ -1,5 +1,6 @@
 test_that("ratio deprecation", {
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     new_rec <- recipe(~., data = circle_example) %>%
       step_downsample(class, ratio = 2)
   )
@@ -22,13 +23,15 @@ test_that("basic usage", {
 test_that("bad data", {
   rec <- recipe(~., data = circle_example)
   # numeric check
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     rec %>%
       step_downsample(x) %>%
       prep()
   )
   # Multiple variable check
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     rec %>%
       step_downsample(class, id) %>%
       prep()

diff --git a/tests/testthat/test-nearmiss.R b/tests/testthat/test-nearmiss.R
@@ -15,13 +15,15 @@ test_that("basic usage", {
 test_that("bad data", {
   rec <- recipe(~., data = circle_example)
   # numeric check
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     rec %>%
       step_nearmiss(x) %>%
       prep()
   )
   # Multiple variable check
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     rec %>%
       step_nearmiss(class, id) %>%
       prep()
@@ -35,7 +37,8 @@ test_that("errors if character are present", {
     stringsAsFactors = FALSE
   )
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     recipe(~., data = df_char) %>%
       step_nearmiss(x) %>%
       prep()
@@ -47,7 +50,8 @@ test_that("NA in response", {
 
   data("credit_data", package = "modeldata")
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     recipe(Job ~ Age, data = credit_data) %>%
       step_nearmiss(Job) %>%
       prep()

diff --git a/tests/testthat/test-rose.R b/tests/testthat/test-rose.R
@@ -54,13 +54,15 @@ test_that("basic usage", {
 test_that("bad data", {
   rec <- recipe(~., data = circle_example)
   # numeric check
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     rec %>%
       step_rose(x) %>%
       prep()
   )
   # Multiple variable check
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     rec %>%
       step_rose(class, id) %>%
       prep()
@@ -74,7 +76,8 @@ test_that("NA in response", {
   credit_data0 <- credit_data
   credit_data0[1, 1] <- NA
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     recipe(Status ~ Age, data = credit_data0) %>%
       step_rose(Status) %>%
       prep()
@@ -124,7 +127,8 @@ test_that("only except 2 classes", {
     stringsAsFactors = FALSE
   )
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     recipe(~., data = df_char) %>%
       step_rose(x) %>%
       prep()

diff --git a/tests/testthat/test-smote.R b/tests/testthat/test-smote.R
@@ -8,7 +8,8 @@ test_that("errors if there isn't enough data", {
   credit_data0$Status[1] <- "dummy"
   credit_data0$Status <- as.factor(credit_data0$Status)
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     recipe(Status ~ Age, data = credit_data0) %>%
       step_smote(Status) %>%
       prep()
@@ -32,13 +33,15 @@ test_that("basic usage", {
 test_that("bad data", {
   rec <- recipe(~., data = circle_example)
   # numeric check
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     rec %>%
       step_smote(x) %>%
       prep()
   )
   # Multiple variable check
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     rec %>%
       step_smote(class, id) %>%
       prep()
@@ -52,7 +55,8 @@ test_that("errors if character are present", {
     stringsAsFactors = FALSE
   )
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     recipe(~., data = df_char) %>%
       step_smote(x) %>%
       prep()
@@ -64,7 +68,8 @@ test_that("NA in response", {
 
   data("credit_data", package = "modeldata")
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     recipe(Job ~ Age, data = credit_data) %>%
       step_smote(Job) %>%
       prep()

diff --git a/tests/testthat/test-smote_impl.R b/tests/testthat/test-smote_impl.R
@@ -24,34 +24,41 @@ test_that("smote() interfaces correctly", {
 
   expect_no_error(smote(circle_example_num, var = "class"))
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     smote(circle_example_num, var = "Class")
   )
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     smote(circle_example_num, var = c("class", "x"))
   )
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     smote(circle_example_num, var = "x")
   )
 
   circle_example0 <- circle_example_num
   circle_example0[1, 1] <- NA
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     smote(circle_example0, var = "class")
   )
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     smote(circle_example_num, var = "class", k = 0)
   )
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     smote(circle_example_num, var = "class", k = -1)
   )
 
-  expect_snapshot(error = TRUE,
+  expect_snapshot(
+    error = TRUE,
     smote(circle_example_num, var = "class", k = c(5, 10))
   )
 })