From 154fab395268173eb10fe54993f2f8a1c7925c60 Mon Sep 17 00:00:00 2001
From: Le Philousophe <lephilousophe@users.noreply.github.com>
Date: Tue, 29 Aug 2023 12:16:04 +0200
Subject: [PATCH] TEST: Silence logs by default

Else tests on INI parser spam with expected warnings.
---
 backends/platform/null/null.cpp | 18 ++++++++++++------
 test/null_osystem.cpp           | 10 +++++++++-
 2 files changed, 21 insertions(+), 7 deletions(-)

diff --git a/backends/platform/null/null.cpp b/backends/platform/null/null.cpp
index 4be329cd6c4..4a6c430a7b1 100644
--- a/backends/platform/null/null.cpp
+++ b/backends/platform/null/null.cpp
@@ -77,7 +77,7 @@ typedef void (*sighandler_t)(int);
 
 class OSystem_NULL : public ModularMixerBackend, public ModularGraphicsBackend, Common::EventSource {
 public:
-	OSystem_NULL();
+	OSystem_NULL(bool silenceLogs);
 	virtual ~OSystem_NULL();
 
 	virtual void initBackend();
@@ -101,9 +101,11 @@ private:
 #elif defined(WIN32)
 	DWORD _startTime;
 #endif
+	bool _silenceLogs;
 };
 
-OSystem_NULL::OSystem_NULL() {
+OSystem_NULL::OSystem_NULL(bool silenceLogs) :
+	_silenceLogs(silenceLogs) {
 	#if defined(__amigaos4__)
 		_fsFactory = new AmigaOSFilesystemFactory();
 	#elif defined(__MORPHOS__)
@@ -228,6 +230,9 @@ void OSystem_NULL::quit() {
 }
 
 void OSystem_NULL::logMessage(LogMessageType::Type type, const char *message) {
+	if (_silenceLogs)
+		return;
+
 	FILE *output = 0;
 
 	if (type == LogMessageType::kInfo || type == LogMessageType::kDebug)
@@ -244,13 +249,13 @@ void OSystem_NULL::addSysArchivesToSearchSet(Common::SearchSet &s, int priority)
 	s.add("gui/themes", new Common::FSDirectory("gui/themes", 4), priority);
 }
 
-OSystem *OSystem_NULL_create() {
-	return new OSystem_NULL();
+OSystem *OSystem_NULL_create(bool silenceLogs) {
+	return new OSystem_NULL(silenceLogs);
 }
 
 #ifndef NULL_DRIVER_USE_FOR_TEST
 int main(int argc, char *argv[]) {
-	g_system = OSystem_NULL_create();
+	g_system = OSystem_NULL_create(false);
 	assert(g_system);
 
 	// Invoke the actual ScummVM main entry point:
@@ -262,7 +267,8 @@ int main(int argc, char *argv[]) {
 
 #else /* USE_NULL_DRIVER */
 
-OSystem *OSystem_NULL_create() {
+OSystem *OSystem_NULL_create(bool silenceLogs) {
+	(void)silenceLogs;
 	return NULL;
 }
 
diff --git a/test/null_osystem.cpp b/test/null_osystem.cpp
index 6f1d0540109..5c2bc808333 100644
--- a/test/null_osystem.cpp
+++ b/test/null_osystem.cpp
@@ -3,8 +3,16 @@
 #include "null_osystem.h"
 #include "../backends/platform/null/null.cpp"
 
+//#define DISPLAY_ERROR_MESSAGES
+
 void Common::install_null_g_system() {
-	g_system = OSystem_NULL_create();
+#ifdef DISPLAY_ERROR_MESSAGES
+	const bool silenceLogs = false;
+#else
+	const bool silenceLogs = true;
+#endif
+
+	g_system = OSystem_NULL_create(silenceLogs);
 }
 
 bool BaseBackend::setScaler(const char *name, int factor) {