Make sure that the test binding actually makes any output at all!

mlpack · Apr 24, 2024 · ed77e91 · ed77e91
1 parent 7d54cfb
commit ed77e91
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/src/mlpack/bindings/R/mlpack/tests/testthat/test-R_binding.R b/src/mlpack/bindings/R/mlpack/tests/testthat/test-R_binding.R
@@ -319,19 +319,19 @@ test_that("TestSerialization", {
 # Make sure that the verbose argument does anything at all.
 test_that("TestVerbose", {
   expect_output(test_r_binding(4.0, 12, "hello",
-                                          build_model=TRUE,
-                                          verbose=TRUE))
+                               build_model=TRUE,
+                               verbose=TRUE))
 })
 
 # Test that we get no output when verbose output is disabled.
 test_that("TestNotVerbose", {
   expect_silent(test_r_binding(4.0, 12, "hello",
-                                          build_model=TRUE))
+                               build_model=TRUE))
 })
 
 # Test that we get no output when verbose output is explicitly disabled.
 test_that("TestReallyNotVerbose", {
   expect_silent(test_r_binding(4.0, 12, "hello",
-                                          build_model=TRUE,
-                                          verbose=FALSE))
+                               build_model=TRUE,
+                               verbose=FALSE))
 })
diff --git a/src/mlpack/bindings/R/tests/test_r_binding_main.cpp b/src/mlpack/bindings/R/tests/test_r_binding_main.cpp
@@ -237,4 +237,7 @@ void BINDING_FUNCTION(util::Params& params, util::Timers& /* timers */)
     params.Get<double>("model_bw_out") =
         params.Get<GaussianKernel*>("model_in")->Bandwidth() * 2.0;
   }
+
+  // Provide some output if the user asked for it.
+  Log::Info << "Here is some verbose output!" << std::endl;
 }