Apply suggestions from code review

Co-authored-by: Simon P. Couch <[email protected]>

Author: topepo

R/condense_control.R

@@ -24,7 +24,8 @@
 #'
 #' ctrl <- condense_control(ctrl, control_parsnip())
 #' str(ctrl)
-condense_control <- function(x, ref, call = rlang::caller_env()) {
+condense_control <- function(x, ref, ..., call = rlang::caller_env()) {
+  check_dots_empty()
   mismatch <- setdiff(names(ref), names(x))
   if (length(mismatch)) {
     cli::cli_abort(

R/contr_one_hot.R

@@ -25,9 +25,8 @@ contr_one_hot <- function(n, contrasts = TRUE, sparse = FALSE) {
   }
 
   if (is.character(n)) {
-    check_character(n, empty = FALSE)
     if (length(n) < 1) {
-      cli::cli_abort("A character vector for {.arg n} cannot be empty.")
+      cli::cli_abort("{.arg n} cannot be empty.")
     }
     names <- n
     n <- length(names)

R/glmnet-engines.R

@@ -370,7 +370,8 @@ format_glmnet_multinom_class <- function(pred, penalty, lvl, n_obs) {
         {.pkg tune} package.",
         "i" = "To predict multiple penalties, use {.fn multi_predict}."
       ),
-      call = call)
+      call = call
+    )
   }
 }
 
@@ -395,7 +396,8 @@ format_glmnet_multinom_class <- function(pred, penalty, lvl, n_obs) {
           "{.arg penalty} should be a single numeric value.",
           "i" = "{.fn multi_predict} can be used to get multiple predictions per row of data."
         ),
-        call = call)
+        call = call
+      )
     }
   }
 
@@ -406,7 +408,8 @@ format_glmnet_multinom_class <- function(pred, penalty, lvl, n_obs) {
       {.arg object$fit$lambda}. Predicting with a value of {.arg penalty}
       will give incorrect results from `glmnet()`."
       ),
-      call = call)
+      call = call
+    )
   }
 
   penalty

tests/testthat/test-condense_control.R

@@ -25,5 +25,4 @@ test_that("condense_control works", {
     invisible(control)
   }
   expect_snapshot(error = TRUE, control_test(ctrl))
-
 })

tests/testthat/test-contr_one_hot.R

@@ -1,5 +1,4 @@
 test_that('one-hot encoding contrasts', {
-
   contr_mat <- contr_one_hot(12)
   expect_equal(colnames(contr_mat), paste(1:12))
   expect_equal(rownames(contr_mat), paste(1:12))
@@ -17,5 +16,4 @@ test_that('one-hot encoding contrasts', {
   expect_snapshot(contr_one_hot(list()), error = TRUE)
   expect_snapshot(contr_one_hot(2, contrast = FALSE))
   expect_snapshot(contr_one_hot(2, sparse = TRUE))
-
 })

tests/testthat/test-misc.R

@@ -7,7 +7,6 @@ test_that('parsnip objects', {
   expect_false(has_multi_predict(lm_idea))
   expect_snapshot(error = TRUE, predict(lm_idea, mtcars))
 
-
   lm_fit <- fit(lm_idea, mpg ~ ., data = mtcars)
   expect_false(has_multi_predict(lm_fit))
   expect_false(has_multi_predict(extract_fit_engine(lm_fit)))