//
// SocketReactorTest.cpp
//
// Copyright (c) 2005-2006, Applied Informatics Software Engineering GmbH.
// and Contributors.
//
// SPDX-License-Identifier:	BSL-1.0
//


#include "SocketReactorTest.h"
#include "CppUnit/TestCaller.h"
#include "CppUnit/TestSuite.h"
#include "Poco/Net/SocketReactor.h"
#include "Poco/Net/SocketNotification.h"
#include "Poco/Net/SocketConnector.h"
#include "Poco/Net/SocketAcceptor.h"
#include "Poco/Net/ParallelSocketAcceptor.h"
#include "Poco/Net/StreamSocket.h"
#include "Poco/Net/ServerSocket.h"
#include "Poco/Net/SocketAddress.h"
#include "Poco/NObserver.h"
#include "Poco/Stopwatch.h"
#include "Poco/Exception.h"
#include "Poco/Thread.h"
#include <sstream>
#include <chrono>


using Poco::Net::SocketReactor;
using Poco::Net::ScopedSocketReactor;
using Poco::Net::SocketConnector;
using Poco::Net::SocketAcceptor;
using Poco::Net::ParallelSocketAcceptor;
using Poco::Net::StreamSocket;
using Poco::Net::ServerSocket;
using Poco::Net::SocketAddress;
using Poco::Net::SocketNotification;
using Poco::Net::ReadableNotification;
using Poco::Net::WritableNotification;
using Poco::Net::TimeoutNotification;
using Poco::Net::ErrorNotification;
using Poco::Net::ShutdownNotification;
using Poco::NObserver;
using Poco::Stopwatch;
using Poco::IllegalStateException;
using Poco::Thread;


namespace
{
	int DATA_SIZE = 1024;
	int REACTORS_COUNT = 8;
	int MAX_DATA_SIZE = DATA_SIZE * REACTORS_COUNT;

	class EchoServiceHandler
	{
	public:
		EchoServiceHandler(const StreamSocket& socket, SocketReactor& reactor):
			_socket(socket),
			_reactor(reactor)
		{
			_reactor.addEventHandler(_socket, NObserver<EchoServiceHandler, ReadableNotification>(*this, &EchoServiceHandler::onReadable));
		}

		~EchoServiceHandler() = default;

		void onReadable(const AutoPtr<ReadableNotification>& pNf)
		{
			char buffer[8];
			int n = _socket.receiveBytes(buffer, sizeof(buffer));
			if (n > 0)
			{
				_socket.sendBytes(buffer, n);
			}
			else
			{
				_reactor.removeEventHandler(_socket, NObserver<EchoServiceHandler, ReadableNotification>(*this, &EchoServiceHandler::onReadable));
				delete this;
			}
		}

	private:
		StreamSocket   _socket;
		SocketReactor& _reactor;
	};

	class ClientServiceHandler
	{
	public:
		ClientServiceHandler(const StreamSocket& socket, SocketReactor& reactor):
			_socket(socket),
			_reactor(reactor),
			_or(*this, &ClientServiceHandler::onReadable),
			_ow(*this, &ClientServiceHandler::onWritable),
			_ot(*this, &ClientServiceHandler::onTimeout),
			_os(*this, &ClientServiceHandler::onShutdown)
		{
			_timeout = false;
			_readableError = false;
			_writableError = false;
			_timeoutError = false;
			checkReadableObserverCount(0);
			_reactor.addEventHandler(_socket, _or);
			checkReadableObserverCount(1);
			checkWritableObserverCount(0);
			_reactor.addEventHandler(_socket, _ow);
			checkWritableObserverCount(1);
			checkTimeoutObserverCount(0);
			_reactor.addEventHandler(_socket, _ot);
			checkTimeoutObserverCount(1);
			_reactor.addEventHandler(_socket, _os);
		}

		~ClientServiceHandler()
		{
			_reactor.remove(_socket);

			// removing socket also removes all its observers;
			// these are here only to test that no bad things happen on multiple removeEventHandler calls
			// not recommended for production code, see https://github.com/pocoproject/poco/issues/5150
			_reactor.removeEventHandler(_socket, _or);
			_reactor.removeEventHandler(_socket, _ow);
			_reactor.removeEventHandler(_socket, _ot);
			_reactor.removeEventHandler(_socket, _os);
		}

		void onReadable(const AutoPtr<ReadableNotification>& pNf)
		{
			char buffer[32];
			int n = _socket.receiveBytes(buffer, sizeof(buffer));
			if (n > 0)
			{
				_str.write(buffer, n);
				_data += _str.str();
				_str.str("");
			}
			else
			{
				checkReadableObserverCount(1);
				_reactor.removeEventHandler(_socket, NObserver<ClientServiceHandler, ReadableNotification>(*this, &ClientServiceHandler::onReadable));
				checkReadableObserverCount(0);
				if (_once)
				{
					_reactor.stop();
					delete this;
					return;
				}
			}
			if (_data.size() == MAX_DATA_SIZE)
			{
				_reactor.stop();
				delete this;
			}
		}

		void onWritable(const AutoPtr<WritableNotification>& pNf)
		{
			checkWritableObserverCount(1);
			_reactor.removeEventHandler(_socket, NObserver<ClientServiceHandler, WritableNotification>(*this, &ClientServiceHandler::onWritable));
			checkWritableObserverCount(0);
			std::string data(DATA_SIZE, 'x');
			_socket.sendBytes(data.data(), (int) data.length());
			_socket.shutdownSend();
		}

		void onTimeout(const AutoPtr<TimeoutNotification>& pNf)
		{
			_timeout = true;
			if (_closeOnTimeout)
			{
				_reactor.stop();
				delete this;
			}
		}

		void onShutdown(const AutoPtr<ShutdownNotification>& pNf)
		{
			delete this;
		}

		static std::string data()
		{
			return _data;
		}

		static void resetData()
		{
			_data.clear();
		}

		static bool timeout()
		{
			return _timeout;
		}

		static bool getCloseOnTimeout()
		{
			return _closeOnTimeout;
		}

		static void setCloseOnTimeout(bool flag)
		{
			_closeOnTimeout = flag;
		}

		static bool readableError()
		{
			return _readableError;
		}

		static bool writableError()
		{
			return _writableError;
		}

		static bool timeoutError()
		{
			return _timeoutError;
		}

		static void setOnce(bool once = true)
		{
			_once = once;
		}

	private:
		void checkReadableObserverCount(std::size_t oro)
		{
			if (((oro == 0) && _reactor.hasEventHandler(_socket, _or)) ||
				((oro > 0) && !_reactor.hasEventHandler(_socket, _or)))
			{
				_readableError = true;
			}
		}

		void checkWritableObserverCount(std::size_t ow)
		{
			if (((ow == 0) && _reactor.hasEventHandler(_socket, _ow)) ||
				((ow > 0) && !_reactor.hasEventHandler(_socket, _ow)))
			{
				_writableError = true;
			}
		}

		void checkTimeoutObserverCount(std::size_t ot)
		{
			if (((ot == 0) && _reactor.hasEventHandler(_socket, _ot)) ||
				((ot > 0) && !_reactor.hasEventHandler(_socket, _ot)))
			{
				_timeoutError = true;
			}
		}

		StreamSocket                                         _socket;
		SocketReactor&                                       _reactor;
		NObserver<ClientServiceHandler, ReadableNotification> _or;
		NObserver<ClientServiceHandler, WritableNotification> _ow;
		NObserver<ClientServiceHandler, TimeoutNotification>  _ot;
		NObserver<ClientServiceHandler, ShutdownNotification> _os;
		std::stringstream                                    _str;
		static std::string                                   _data;
		static bool                                          _readableError;
		static bool                                          _writableError;
		static bool                                          _timeoutError;
		static bool                                          _timeout;
		static bool                                          _closeOnTimeout;
		static bool                                          _once;
	};

	std::string ClientServiceHandler::_data;
	bool ClientServiceHandler::_readableError = false;
	bool ClientServiceHandler::_writableError = false;
	bool ClientServiceHandler::_timeoutError = false;
	bool ClientServiceHandler::_timeout = false;
	bool ClientServiceHandler::_closeOnTimeout = false;
	bool ClientServiceHandler::_once = false;


	class FailConnector: public SocketConnector<ClientServiceHandler>
	{
	public:
		FailConnector(SocketAddress& address, SocketReactor& reactor):
			SocketConnector<ClientServiceHandler>(address, reactor),
			_failed(false),
			_shutdown(false)
		{
			reactor.addEventHandler(socket(), NObserver<FailConnector, TimeoutNotification>(*this, &FailConnector::onTimeout));
			reactor.addEventHandler(socket(), NObserver<FailConnector, ShutdownNotification>(*this, &FailConnector::onShutdown));
		}

		~FailConnector()
		{
			reactor()->remove(socket());

			// removing socket also removes all its observers;
			// these are here only to test that no bad things happen on multiple removeEventHandler calls
			// not recommended for production code, see https://github.com/pocoproject/poco/issues/5150
			reactor()->removeEventHandler(socket(), NObserver<FailConnector, TimeoutNotification>(*this, &FailConnector::onTimeout));
			reactor()->removeEventHandler(socket(), NObserver<FailConnector, ShutdownNotification>(*this, &FailConnector::onShutdown));
		}

		void onShutdown(const AutoPtr<ShutdownNotification>& pNf)
		{
			_shutdown = true;
		}

		void onTimeout(const AutoPtr<TimeoutNotification>& pNf)
		{
			_failed = true;
			reactor()->stop();
		}

		bool failed() const
		{
			return _failed;
		}

		bool shutdown() const
		{
			return _shutdown;
		}

	private:
		bool _failed;
		bool _shutdown;
	};

	class DataServiceHandler
	{
	public:
		using Data = std::vector<std::string>;

		DataServiceHandler(StreamSocket& socket, SocketReactor& reactor):
				_socket(socket),
				_reactor(reactor),
				_pos(0)
		{
			_data.resize(1);
			_reactor.addEventHandler(_socket, NObserver<DataServiceHandler, ReadableNotification>(*this, &DataServiceHandler::onReadable));
			_reactor.addEventHandler(_socket, NObserver<DataServiceHandler, ShutdownNotification>(*this, &DataServiceHandler::onShutdown));
			_socket.setBlocking(false);
		}

		~DataServiceHandler()
		{
			_reactor.remove(_socket);

			// removing socket also removes all its observers;
			// these are here only to test that no bad things happen on multiple removeEventHandler calls
			// not recommended for production code, see https://github.com/pocoproject/poco/issues/5150
			_reactor.removeEventHandler(_socket, NObserver<DataServiceHandler, ReadableNotification>(*this, &DataServiceHandler::onReadable));
			_reactor.removeEventHandler(_socket, NObserver<DataServiceHandler, ShutdownNotification>(*this, &DataServiceHandler::onShutdown));
		}

		void onReadable(const AutoPtr<ReadableNotification>& pNf)
		{
			char buffer[64];
			int n = 0;
			do
			{
				n = _socket.receiveBytes(&buffer[0], sizeof(buffer));
				if (n > 0)
				{
					_data[_pos].append(buffer, n);
					std::size_t pos;
					pos = _data[_pos].find('\n');
					if (pos != std::string::npos)
					{
						if (pos == _data[_pos].size() - 1)
						{
							_data[_pos].erase(pos, 1);
							_data.push_back(std::string());
						}
						else
						{
							_data.push_back(_data[_pos].substr(pos + 1));
							_data[_pos].erase(pos);
						}
						++_pos;
					}
				}
				else break;
			} while (true);
		}

		void onShutdown(const AutoPtr<ShutdownNotification>& pNf)
		{
			delete this;
		}

		static Data _data;

	private:
		StreamSocket   _socket;
		SocketReactor& _reactor;
		int            _pos;
	};

	DataServiceHandler::Data DataServiceHandler::_data;

	class SleepClientServiceHandler
	{
	public:
		SleepClientServiceHandler(Poco::Net::StreamSocket& socket, Poco::Net::SocketReactor& reactor)
		{
			Poco::Thread::sleep(500);
		}
	};

	class DummyServiceHandler
	{
	public:
		DummyServiceHandler(StreamSocket& socket, SocketReactor& reactor) : _socket(socket),
			_reactor(reactor)
		{
			_reactor.addEventHandler(_socket, NObserver<DummyServiceHandler, ReadableNotification>(*this, &DummyServiceHandler::onReadable));
			_reactor.addEventHandler(_socket, NObserver<DummyServiceHandler, ShutdownNotification>(*this, &DummyServiceHandler::onShutdown));
			_socket.setBlocking(false);
		}

		~DummyServiceHandler()
		{
			_reactor.removeEventHandler(_socket, NObserver<DummyServiceHandler, ReadableNotification>(*this, &DummyServiceHandler::onReadable));
			_reactor.removeEventHandler(_socket, NObserver<DummyServiceHandler, ShutdownNotification>(*this, &DummyServiceHandler::onShutdown));
		}

		void onReadable(const AutoPtr<ReadableNotification>& pNf)
		{
			std::vector<char> buffer;
			int n = 0;
			while ((n = _socket.available()))
			{
				if (n > buffer.size()) buffer.resize(n);
				n = _socket.receiveBytes(&buffer[0], buffer.size());
				if (0 == n) break;
			}
		}

		void onShutdown(const AutoPtr<ShutdownNotification>& pNf)
		{
			delete this;
		}

	private:
		StreamSocket   _socket;
		SocketReactor& _reactor;
	};


	// Handler that aggressively queries and modifies reactor from within notifications
	// Tests the NotificationCenter deadlock fix by calling removeEventHandler while
	// holding Observer mutex, concurrent with other threads adding/removing handlers
	class ConcurrentRemovalHandler
	{
	public:
		ConcurrentRemovalHandler(StreamSocket& sock, SocketReactor& r, std::atomic<int>& callCount):
			_socket(sock), _reactor(r), _callCount(callCount),
			_or(*this, &ConcurrentRemovalHandler::onReadable),
			_ow(*this, &ConcurrentRemovalHandler::onWritable),
			_removeCount(0)
		{
			_reactor.addEventHandler(_socket, _or);
			_reactor.addEventHandler(_socket, _ow);
		}

		~ConcurrentRemovalHandler()
		{
			// Use remove() for safe atomic cleanup - avoids race
			// with reactor dispatch that removeEventHandler() has
			try {
				_reactor.remove(_socket);
			} catch (...) {}
		}

		void onReadable(const AutoPtr<ReadableNotification>& pNf)
		{
			int count = ++_callCount;

			// Query reactor state while holding Observer mutex
			// This would deadlock if NC held lock while calling observer methods
			_reactor.hasEventHandler(_socket, _or);
			_reactor.hasEventHandler(_socket, _ow);
			_reactor.has(_socket);

			// Every 3rd call, remove a handler from within the notification
			// This is the exact scenario from the bug report
			if (count % 3 == 0 && _removeCount++ == 0)
			{
				_reactor.removeEventHandler(_socket, _ow);
			}

			// Read data to keep socket active
			char buffer[64];
			try {
				_socket.receiveBytes(buffer, sizeof(buffer));
			} catch (...) {}
		}

		void onWritable(const AutoPtr<WritableNotification>& pNf)
		{
			// Query reactor while in handler
			_reactor.hasEventHandler(_socket, _or);
		}

	private:
		StreamSocket _socket;
		SocketReactor& _reactor;
		std::atomic<int>& _callCount;
		NObserver<ConcurrentRemovalHandler, ReadableNotification> _or;
		NObserver<ConcurrentRemovalHandler, WritableNotification> _ow;
		int _removeCount;
	};
}


SocketReactorTest::SocketReactorTest(const std::string& name): CppUnit::TestCase(name)
{
}


SocketReactorTest::~SocketReactorTest() = default;


void SocketReactorTest::testSocketReactor()
{
	SocketAddress ssa;
	ServerSocket ss(ssa);
	SocketReactor reactor;
	SocketAcceptor<EchoServiceHandler> acceptor(ss, reactor);
	SocketAddress sa("127.0.0.1", ss.address().port());
	SocketConnector<ClientServiceHandler> connector(sa, reactor);
	ClientServiceHandler::setOnce(true);
	ClientServiceHandler::resetData();
	reactor.run();
	std::string data(ClientServiceHandler::data());
	assertTrue (data.size() == DATA_SIZE);
	assertTrue (!ClientServiceHandler::readableError());
	assertTrue (!ClientServiceHandler::writableError());
	assertTrue (!ClientServiceHandler::timeoutError());
}


void SocketReactorTest::testSetSocketReactor()
{
	SocketAddress ssa;
	ServerSocket ss(ssa);
	SocketReactor reactor;
	SocketAcceptor<EchoServiceHandler> acceptor(ss);
	acceptor.setReactor(reactor);
	SocketAddress sa("127.0.0.1", ss.address().port());
	SocketConnector<ClientServiceHandler> connector(sa, reactor);
	ClientServiceHandler::setOnce(true);
	ClientServiceHandler::resetData();
	reactor.run();
	std::string data(ClientServiceHandler::data());
	assertTrue (data.size() == DATA_SIZE);
	assertTrue (!ClientServiceHandler::readableError());
	assertTrue (!ClientServiceHandler::writableError());
	assertTrue (!ClientServiceHandler::timeoutError());
}


void SocketReactorTest::testParallelSocketReactor()
{
	SocketAddress ssa("127.0.0.1:22087");
	ServerSocket ss(ssa);
	SocketReactor reactor;
	ParallelSocketAcceptor<EchoServiceHandler, SocketReactor> acceptor(ss, reactor, REACTORS_COUNT);

	SocketAddress sa("127.0.0.1", ss.address().port());
	std::vector<SocketConnector<ClientServiceHandler>*> connectors;
	for (int i = 0; i < REACTORS_COUNT; ++i)
		connectors.push_back(new SocketConnector<ClientServiceHandler>(sa, reactor));

	ClientServiceHandler::setOnce(false);
	ClientServiceHandler::resetData();
	reactor.run();

	std::string data(ClientServiceHandler::data());
	assertTrue (data.size() == MAX_DATA_SIZE);
	assertTrue (!ClientServiceHandler::readableError());
	assertTrue (!ClientServiceHandler::writableError());
	assertTrue (!ClientServiceHandler::timeoutError());
	for (auto& c : connectors) delete c;
}


void SocketReactorTest::testSocketConnectorFail()
{
	SocketReactor reactor;
	reactor.setTimeout(Poco::Timespan(3, 0));
	SocketAddress sa("192.168.168.192", 12345);
	FailConnector connector(sa, reactor);
	assertTrue (!connector.failed());
	assertTrue (!connector.shutdown());
	reactor.run();
	assertTrue (connector.failed());
	assertTrue (connector.shutdown());
}


void SocketReactorTest::testSocketConnectorTimeout()
{
	ClientServiceHandler::setCloseOnTimeout(true);

	SocketAddress ssa;
	ServerSocket ss(ssa);
	SocketReactor reactor;
	SocketAddress sa("127.0.0.1", ss.address().port());
	SocketConnector<ClientServiceHandler> connector(sa, reactor);
	reactor.run();
	assertTrue (ClientServiceHandler::timeout());
}


void SocketReactorTest::testDataCollection()
{
	SocketAddress ssa;
	ServerSocket ss(ssa);
	SocketReactor reactor;
	SocketAcceptor<DataServiceHandler> acceptor(ss, reactor);
	Thread thread;
	thread.start(reactor);

	SocketAddress sa("127.0.0.1", ss.address().port());
	StreamSocket sock(sa);

	std::string data0("{"
					  "  \"src\":\"127.0.0.1\","
					  "  \"id\":\"test0\","
					  "  \"ts\":\"1524864651000001\","
					  "  \"data\":123"
					  "}\n");
	sock.sendBytes(data0.data(), static_cast<int>(data0.size()));
	std::string data1("{"
					  "  \"src\":\"127.0.0.1\","
					  "  \"id\":\"test1\","
					  "  \"ts\":\"1524864651123456\","
					  "  \"data\":"
					  "  ["
					  "   {"
					  "     \"tag1\":"
					  "     ["
					  "      {\"val1\":123},"
					  "      {\"val2\":\"abc\"}"
					  "     ]"
					  "   }"
					  "  ]"
					  "}\n");
	sock.sendBytes(data1.data(), static_cast<int>(data1.size()));
	std::string data2 = "{"
						"  \"src\":\"127.0.0.1\","
						"  \"id\":\"test2\","
						"  \"ts\":\"1524864652654321\","
						"  \"data\":"
						"  ["
						"    {"
						"     \"tag1\":"
						"     ["
						"      {"
						"       \"val1\":123,"
						"       \"val2\":\"abc\","
						"       \"val3\":42.123"
						"      },"
						"      {"
						"       \"val1\":987,"
						"       \"val2\":\"xyz\","
						"       \"val3\":24.321"
						"      }"
						"     ],"
						"     \"tag2\":"
						"     ["
						"      {"
						"       \"val1\":42.123,"
						"       \"val2\":123,"
						"       \"val3\":\"abc\""
						"      },"
						"      {"
						"       \"val1\":24.321,"
						"       \"val2\":987,"
						"       \"val3\":\"xyz\""
						"      }"
						"    ]"
						"   }"
						" ]"
						"}\n";
	sock.sendBytes(data2.data(), static_cast<int>(data2.size()));
	Thread::sleep(500);
	reactor.stop();
	thread.join();

	assertTrue (DataServiceHandler::_data.size() == 4);
	data0.erase(data0.size() - 1);
	assertTrue (DataServiceHandler::_data[0] == data0);
	data1.erase(data1.size() - 1);
	assertTrue (DataServiceHandler::_data[1] == data1);
	data2.erase(data2.size() - 1);
	assertTrue (DataServiceHandler::_data[2] == data2);
	assertTrue (DataServiceHandler::_data[3].empty());
}


void SocketReactorTest::testSocketConnectorDeadlock()
{
	{
		SocketAddress ssa;
		ServerSocket ss(ssa);
		SocketAddress sa("127.0.0.1", ss.address().port());
		SocketReactor reactor;
		Thread thread;
		int i = 0;
		while (++i < 10)
		{
			auto sc = new SocketConnector<SleepClientServiceHandler>(sa, reactor);
			thread.startFunc([&reactor]() { reactor.run(); });
			reactor.stop();
			thread.join();
			delete sc;
		}
	}

	int i = 0;
	while (++i < 10)
	{
		SocketAddress ssa;
		ServerSocket ss(ssa);
		SocketReactor reactor;
		SocketAcceptor<DummyServiceHandler> acceptor(ss, reactor);
		Thread thread;
		thread.start(reactor);

		SocketAddress sa("127.0.0.1", ss.address().port());
		StreamSocket sock(sa);

		std::string data("HELLO");
		sock.sendBytes(data.data(), static_cast<int>(data.size()));
		reactor.stop();
		thread.join();
	}
}


void SocketReactorTest::testSocketReactorWakeup()
{
	SocketReactor::Params params;
	params.pollTimeout = 1000000000;
	params.sleepLimit = 1000000000;
	SocketReactor reactor(params);
	Thread thread;
	Stopwatch sw;
	sw.start();
	thread.start(reactor);
	reactor.stop();
	thread.join();
	sw.stop();
	assertTrue (sw.elapsed() < 100000);
}


void SocketReactorTest::testSocketReactorRemove()
{
	SocketAddress ssa;
	ServerSocket ss(ssa);
	SocketReactor reactor;
	Thread thread;
	thread.start(reactor);

	SocketAddress sa("127.0.0.1", ss.address().port());
	StreamSocket sock(sa);
	StreamSocket accepted = ss.acceptConnection();

	// Add handlers
	NObserver<SocketReactorTest, ReadableNotification> obsRead(*this, &SocketReactorTest::onReadable);
	NObserver<SocketReactorTest, WritableNotification> obsWrite(*this, &SocketReactorTest::onWritable);
	reactor.addEventHandler(accepted, obsRead);
	reactor.addEventHandler(accepted, obsWrite);

	// Verify socket is registered
	assertTrue(reactor.has(accepted));
	assertTrue(reactor.hasEventHandler(accepted, obsRead));
	assertTrue(reactor.hasEventHandler(accepted, obsWrite));

	// Remove socket completely
	reactor.remove(accepted);

	// Verify socket is no longer registered
	assertTrue(!reactor.has(accepted));
	assertTrue(!reactor.hasEventHandler(accepted, obsRead));
	assertTrue(!reactor.hasEventHandler(accepted, obsWrite));

	reactor.stop();
	thread.join();
}


void SocketReactorTest::testConcurrentHandlerRemoval()
{
	// This test specifically exercises the NotificationCenter deadlock fix
	// by having handlers call removeEventHandler while holding Observer mutex,
	// while other threads are adding/removing handlers concurrently

	SocketAddress ssa;
	ServerSocket ss(ssa);
	ScopedSocketReactor reactor;

	SocketAddress sa("127.0.0.1", ss.address().port());
	std::vector<StreamSocket> sockets;
	std::vector<StreamSocket> accepted;

	// Create multiple socket connections
	const int numSockets = 4;
	for (int i = 0; i < numSockets; ++i)
	{
		sockets.push_back(StreamSocket(sa));
		accepted.push_back(ss.acceptConnection());
	}

	std::atomic<int> callCount(0);
	std::vector<ConcurrentRemovalHandler*> handlers;

	// Create handlers for all sockets
	for (auto& sock : accepted)
	{
		handlers.push_back(new ConcurrentRemovalHandler(sock, *reactor, callCount));
	}

	// Thread that continuously adds/removes handlers
	std::atomic<bool> stop(false);
	std::atomic<int> completedIterations(0);
	const int targetIterations = 20;

	Thread modifyThread;
	modifyThread.startFunc([&]() {
		for (int i = 0; i < targetIterations && !stop; ++i)
		{
			// Try to add/remove handlers concurrently with notifications
			for (auto& sock : accepted)
			{
				if (stop) break;
				NObserver<SocketReactorTest, ReadableNotification> obs(*this, &SocketReactorTest::onReadable);
				try {
					reactor->addEventHandler(sock, obs);
					Thread::sleep(1);
					reactor->removeEventHandler(sock, obs);
				} catch (...) {
					// Socket might be removed by handler, that's okay
				}
			}
			++completedIterations;
		}
	});

	// Send data to trigger notifications while polling for completion
	auto deadline = std::chrono::steady_clock::now() + std::chrono::seconds(10);
	int dataSent = 0;

	while (completedIterations < targetIterations)
	{
		if (std::chrono::steady_clock::now() >= deadline)
			break;

		// Send some data to trigger notifications
		if (dataSent < 5)
		{
			for (auto& sock : sockets)
			{
				std::string data = "test";
				try {
					sock.sendBytes(data.data(), (int)data.size());
				} catch (...) {}
			}
			++dataSent;
		}
		Thread::sleep(50);
	}

	bool timedOut = (completedIterations < targetIterations);

	// Always cleanup - must happen before fail() which throws
	stop = true;
	modifyThread.join();

	reactor->stop();

	for (auto* handler : handlers)
		delete handler;

	// ScopedSocketReactor automatically stops and joins on destruction

	if (timedOut)
		failmsg("Deadlock detected: concurrent handler removal test timed out");

	// Verify handlers were called
	assertTrue(callCount > 0);
}


void SocketReactorTest::onReadable(const Poco::AutoPtr<Poco::Net::ReadableNotification>& pNf)
{
}


void SocketReactorTest::onWritable(const Poco::AutoPtr<Poco::Net::WritableNotification>& pNf)
{
}


void SocketReactorTest::setUp()
{
	ClientServiceHandler::setCloseOnTimeout(false);
}


void SocketReactorTest::tearDown()
{
}


CppUnit::Test* SocketReactorTest::suite()
{
	CppUnit::TestSuite* pSuite = new CppUnit::TestSuite("SocketReactorTest");

	CppUnit_addTest(pSuite, SocketReactorTest, testSocketReactor);
	CppUnit_addTest(pSuite, SocketReactorTest, testSetSocketReactor);
	CppUnit_addTest(pSuite, SocketReactorTest, testParallelSocketReactor);
	CppUnit_addTest(pSuite, SocketReactorTest, testSocketConnectorFail);
	CppUnit_addTest(pSuite, SocketReactorTest, testSocketConnectorTimeout);
	CppUnit_addTest(pSuite, SocketReactorTest, testDataCollection);
	CppUnit_addTest(pSuite, SocketReactorTest, testSocketConnectorDeadlock);
	CppUnit_addTest(pSuite, SocketReactorTest, testSocketReactorWakeup);
	CppUnit_addTest(pSuite, SocketReactorTest, testSocketReactorRemove);
	CppUnit_addTest(pSuite, SocketReactorTest, testConcurrentHandlerRemoval);

	return pSuite;
}
