From 1462e4f70e6bc8e42f6a5b5e959b7e34f2774a73 Mon Sep 17 00:00:00 2001
From: Mattes D <github@xoft.cz>
Date: Sun, 26 Jun 2016 15:51:12 +0200
Subject: Self tests (#3242)

* SelfTests: Added a cEvent stress-test.

* cNetwork: Fixed startup event hang.

The original code used to hang with a ~ 1:50000 chance, because on Linux the cEvent was destroyed before its "Set()" returned.
---
 tests/CMakeLists.txt            |  1 +
 tests/OSSupport/CMakeLists.txt  | 50 ++++++++++++++++++++++++++++++++++
 tests/OSSupport/StressEvent.cpp | 60 +++++++++++++++++++++++++++++++++++++++++
 3 files changed, 111 insertions(+)
 create mode 100644 tests/OSSupport/CMakeLists.txt
 create mode 100644 tests/OSSupport/StressEvent.cpp

(limited to 'tests')

diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt
index 052a31835..a076ffed9 100644
--- a/tests/CMakeLists.txt
+++ b/tests/CMakeLists.txt
@@ -15,4 +15,5 @@ add_subdirectory(CompositeChat)
 add_subdirectory(HTTP)
 add_subdirectory(LoadablePieces)
 add_subdirectory(Network)
+add_subdirectory(OSSupport)
 add_subdirectory(SchematicFileSerializer)
diff --git a/tests/OSSupport/CMakeLists.txt b/tests/OSSupport/CMakeLists.txt
new file mode 100644
index 000000000..8fff570a4
--- /dev/null
+++ b/tests/OSSupport/CMakeLists.txt
@@ -0,0 +1,50 @@
+cmake_minimum_required (VERSION 2.6)
+
+enable_testing()
+
+include_directories(${CMAKE_SOURCE_DIR}/src/)
+
+add_definitions(-DTEST_GLOBALS=1)
+
+# Create a single OSSupport library that contains all the OSSupport code used in the tests:
+# Only needed for Windows; Linux already defines the OSSupport lib
+if (WIN32)
+	set (OSSupport_SRCS
+		${CMAKE_SOURCE_DIR}/src/OSSupport/CriticalSection.cpp
+		${CMAKE_SOURCE_DIR}/src/OSSupport/Event.cpp
+		${CMAKE_SOURCE_DIR}/src/StringUtils.cpp
+	)
+	set (OSSupport_HDRS
+		${CMAKE_SOURCE_DIR}/src/OSSupport/CriticalSection.h
+		${CMAKE_SOURCE_DIR}/src/OSSupport/Event.h
+		${CMAKE_SOURCE_DIR}/src/StringUtils.h
+		${CMAKE_SOURCE_DIR}/src/Globals.h
+	)
+	add_library(OSSupport
+		${OSSupport_SRCS}
+		${OSSupport_HDRS}
+	)
+endif()
+
+# Define individual tests:
+
+# StressEvent: Stress-test the cEvent implementation:
+add_executable(StressEvent-exe StressEvent.cpp)
+target_link_libraries(StressEvent-exe OSSupport)
+add_test(NAME StressEvent-test COMMAND StressEvent-exe)
+
+
+
+# Put all the tests into a solution folder (MSVC):
+set_target_properties(
+	StressEvent-exe
+	PROPERTIES FOLDER Tests/OSSupport
+)
+set_target_properties(
+	OSSupport
+	PROPERTIES FOLDER Lib
+)
+
+
+
+
diff --git a/tests/OSSupport/StressEvent.cpp b/tests/OSSupport/StressEvent.cpp
new file mode 100644
index 000000000..05c47b599
--- /dev/null
+++ b/tests/OSSupport/StressEvent.cpp
@@ -0,0 +1,60 @@
+
+// StressEvent.cpp
+
+// Stress-tests the cEvent implementation
+
+#include "Globals.h"
+#include <thread>
+
+
+
+
+
+/** Number of repetitions of the thread loops. */
+const int NUM_REPETITIONS = 5000;
+
+
+
+
+// Forward declarations are needed for clang
+void runThread(cEvent * a_Event1, cEvent * a_Event2, const char * a_ThreadName);
+
+
+
+
+
+/** Function that runs in a separate thread, notifies event1 and waits for event2, in a loop, NUM_REPETITIONS times.
+This basically simulates a producer / consumer pattern with 2 events, one for "queue empty", the other for "queue full". */
+void runThread(cEvent * a_Event1, cEvent * a_Event2, const char * a_ThreadName)
+{
+	LOG("Thread %s started", a_ThreadName);
+	for (int i = 0; i < NUM_REPETITIONS; ++i)
+	{
+		// LOGD("%s: Waiting for event %p (%d)", a_ThreadName, a_Event2, i);
+		a_Event2->Wait();
+		// LOGD("%s: Setting event %p (%d)", a_ThreadName, a_Event1, i);
+		a_Event1->SetAll();
+	}
+	LOG("Thread %s finished", a_ThreadName);
+}
+
+
+
+
+
+int main()
+{
+	LOG("Test started");
+	cEvent event1, event2;
+	event1.Set();
+	std::thread thread1(&runThread, &event1, &event2, "A");
+	std::thread thread2(&runThread, &event2, &event1, "B");
+	thread1.join();
+	thread2.join();
+	LOG("Test finished");
+	return 0;
+}
+
+
+
+
-- 
cgit v1.2.3