Make VS Test Explorer happy by absorbing the text output of legacy tests

2 files changed, 12 insertions, 3 deletions

diff --git a/unit_tests/Catch2.runsettings b/unit_tests/Catch2.runsettings
index 9360bfb..c8e32ee 100644
--- a/unit_tests/Catch2.runsettings
+++ b/unit_tests/Catch2.runsettings
@@ -22,9 +22,6 @@
                 <!-- Combine: A single test executable is started to run multiple test cases. (Single: instance per test case) -->
                 <ExecutionMode>Combine</ExecutionMode>
 
-                <!-- In Milliseconds -->
-                <TestCaseTimeout>10000</TestCaseTimeout>
-
                 <!-- Working directory -->
                 <WorkingDirectoryRoot>Solution</WorkingDirectoryRoot>
                 <WorkingDirectory>Lanes</WorkingDirectory>
diff --git a/unit_tests/shared.cpp b/unit_tests/shared.cpp
index d2b4ef7..c2b8bf0 100644
--- a/unit_tests/shared.cpp
+++ b/unit_tests/shared.cpp
@@ -389,6 +389,18 @@ TEST_CASE("LuaState.doString")
 FileRunner::FileRunner(std::string_view const& where_)
 : LuaState{ LuaState::WithBaseLibs{ true }, LuaState::WithFixture{ true } }
 {
+    // _G.print = _nullprint
+    // because the VS Test Explorer doesn't appreciate the text output of some scripts, so absorb them
+    if constexpr (1) {
+        auto const _nullprint = +[](lua_State* const L_) { return 0; };
+        luaG_pushglobaltable(L);
+        lua_pushcfunction(L, _nullprint);
+        luaG_setfield(L, StackIndex{ -2 }, std::string_view{ "print" });
+        lua_pop(L, 1);
+        stackCheck(0);
+    }
+    // TODO: do the same with io.stderr:write?
+
     [[maybe_unused]] std::filesystem::path const _current{ std::filesystem::current_path() };
     std::filesystem::path _path{ where_ };
     root = std::filesystem::canonical(_path).generic_string();