From 58d904a328c0d849769b483e901a0be9426b8209 Mon Sep 17 00:00:00 2001
From: liuxiaolong <liuxiaolong@aiotlink.com>
Date: 星期二, 20 七月 2021 20:20:44 +0800
Subject: [PATCH] 调整Request C.BHFree的位置

---
 utest/api_test.cpp |  380 ++++++++++++++++++++++++++++++++++++++++++-----------
 1 files changed, 300 insertions(+), 80 deletions(-)

diff --git a/utest/api_test.cpp b/utest/api_test.cpp
index 766c0f8..13a552d 100644
--- a/utest/api_test.cpp
+++ b/utest/api_test.cpp
@@ -16,14 +16,26 @@
  * =====================================================================================
  */
 #include "bh_api.h"
+#include "json.h"
+#include "robust.h"
 #include "util.h"
 #include <atomic>
+#include <boost/lockfree/queue.hpp>
 
 using namespace bhome_msg;
 
 namespace
 {
 typedef std::atomic<uint64_t> Number;
+
+void GetApiError(int &ec, std::string &msg)
+{
+	void *pmsg = 0;
+	int msg_len = 0;
+	ec = BHGetLastError(&pmsg, &msg_len);
+	msg.assign((char *) pmsg, msg_len);
+	BHFree(pmsg, msg_len);
+}
 
 void Assign(Number &a, const Number &b) { a.store(b.load()); }
 struct MsgStatus {
@@ -49,7 +61,6 @@
 	static MsgStatus st;
 	return st;
 }
-} // namespace
 
 void SubRecvProc(const void *proc_id,
                  const int proc_id_len,
@@ -59,7 +70,7 @@
 	std::string proc((const char *) proc_id, proc_id_len);
 	MsgPublish pub;
 	pub.ParseFromArray(data, data_len);
-	// printf("Sub data, %s : %s\n", pub.topic().c_str(), pub.data().c_str());
+	printf("****************************************** Sub data, %s : %s\n", pub.topic().c_str(), pub.data().c_str());
 }
 
 void ServerProc(const void *proc_id,
@@ -96,46 +107,113 @@
 	// printf("client Recv reply : %s\n", reply.data().c_str());
 }
 
-BOOST_AUTO_TEST_CASE(MutexTest)
+} // namespace
+#include <chrono>
+using namespace std::chrono;
+// using namespace std::chrono_literals;
+
+bool Register(const std::string &proc_id)
 {
-	const std::string shm_name("ShmMutex");
-	// ShmRemover auto_remove(shm_name);
-	SharedMemory shm(shm_name, 1024 * 1024 * 10);
-
-	const std::string mtx_name("test_mutex");
-	const std::string int_name("test_int");
-	auto mtx = shm.find_or_construct<Mutex>(mtx_name.c_str())();
-	auto pi = shm.find_or_construct<int>(int_name.c_str())(100);
-	if (pi) {
-		auto old = *pi;
-		printf("int : %d, add1: %d\n", old, ++*pi);
-	}
-
-	auto TryLock = [&]() {
-		if (mtx->try_lock()) {
-			printf("try_lock ok\n");
-			return true;
+	bool reg = false;
+	for (int i = 0; i < 3 && !reg; ++i) {
+		ProcInfo proc;
+		proc.set_proc_id(proc_id);
+		proc.set_public_info("public info of demo_client. etc...");
+		std::string proc_buf(proc.SerializeAsString());
+		void *reply = 0;
+		int reply_len = 0;
+		reg = BHRegister(proc_buf.data(), proc_buf.size(), &reply, &reply_len, 2000);
+		if (reg) {
+			printf("register ok\n");
+			// bool r = BHUnregister(proc_buf.data(), proc_buf.size(), &reply, &reply_len, 2000);
+			// printf("unregister %s\n", r ? "ok" : "failed");
+			// reg = BHRegister(proc_buf.data(), proc_buf.size(), &reply, &reply_len, 2000);
+			// if (!reg) {
+			// 	int ec = 0;
+			// 	std::string msg;
+			// 	GetApiError(ec, msg);
+			// 	printf("reg error: %s\n", msg.c_str());
+			// }
 		} else {
-			printf("try_lock failed\n");
-			return false;
+			int ec = 0;
+			std::string msg;
+			GetApiError(ec, msg);
+			printf("register failed, %d, %s\n", ec, msg.c_str());
 		}
-	};
-	auto Unlock = [&]() {
-		mtx->unlock();
-		printf("unlocked\n");
-	};
 
-	if (mtx) {
-		printf("mtx exists\n");
-		if (TryLock()) {
-			if (TryLock()) {
-				Unlock();
-			}
-			// Unlock();
-		}
-	} else {
-		printf("mtx not exists\n");
+		BHFree(reply, reply_len);
+		// Sleep(1s);
 	}
+	return reg;
+}
+
+bool SyncRequest(const std::string &topic, const std::string &data)
+{ // SyncRequest
+	MsgRequestTopic req;
+	req.set_topic(topic);
+	req.set_data(data);
+	std::string s(req.SerializeAsString());
+	// Sleep(10ms, false);
+	std::string dest(BHAddress().SerializeAsString());
+	void *proc_id = 0;
+	int proc_id_len = 0;
+	DEFER1(BHFree(proc_id, proc_id_len););
+	void *reply = 0;
+	int reply_len = 0;
+	DEFER1(BHFree(reply, reply_len));
+	bool r = BHRequest(dest.data(), dest.size(), s.data(), s.size(), &proc_id, &proc_id_len, &reply, &reply_len, 100);
+	if (!r) {
+		int ec = 0;
+		std::string msg;
+		GetApiError(ec, msg);
+		printf("request error: %d, %s\n", ec, msg.c_str());
+	} else {
+		MsgRequestTopicReply ret;
+		ret.ParseFromArray(reply, reply_len);
+		printf("request result: %s\n", ret.data().c_str());
+	}
+	return r;
+}
+
+BOOST_AUTO_TEST_CASE(RestartTest)
+{
+	const std::string proc_id = "demo_node";
+	for (int i = 0; i < 5; ++i) {
+		printf("loop: %d --------------------------\n", i);
+		if (!Register(proc_id)) { continue; }
+
+		const std::string topic_ = "topic_";
+
+		{ // Server Register Topics
+			MsgTopicList topics;
+			for (int i = 0; i < 10; ++i) {
+				topics.add_topic_list(topic_ + std::to_string(i));
+			}
+			std::string s = topics.SerializeAsString();
+			void *reply = 0;
+			int reply_len = 0;
+			bool r = BHRegisterTopics(s.data(), s.size(), &reply, &reply_len, 1000);
+			DEFER1(BHFree(reply, reply_len));
+		}
+
+		BHStartWorker(&ServerProc, &SubRecvProc, &ClientProc);
+
+		for (int i = 0; i < 60; ++i) {
+			bool r = SyncRequest(topic_ + std::to_string(0), "request_data_" + std::to_string(i));
+			if (!r) {
+				void *msg = 0;
+				int msg_len = 0;
+				DEFER1(BHFree(msg, msg_len));
+				int ec = BHGetLastError(&msg, &msg_len);
+				if (ec == eNotRegistered) {
+					printf("need re-register\n");
+					break;
+				}
+			}
+			Sleep(1s);
+		}
+	}
+	BHCleanup();
 }
 
 BOOST_AUTO_TEST_CASE(ApiTest)
@@ -154,27 +232,14 @@
 
 	printf("maxsec: %ld\n", CountSeconds(max_time));
 
-	bool reg = false;
-	for (int i = 0; i < 3 && !reg; ++i) {
-		ProcInfo proc;
-		proc.set_proc_id("demo_client");
-		proc.set_public_info("public info of demo_client. etc...");
-		std::string proc_buf(proc.SerializeAsString());
-		void *reply = 0;
-		int reply_len = 0;
-		reg = BHRegister(proc_buf.data(), proc_buf.size(), &reply, &reply_len, 2000);
-		printf("register %s\n", reg ? "ok" : "failed");
-
-		BHFree(reply, reply_len);
-		Sleep(1s);
-	}
-	if (!reg) {
+	const std::string proc_id = "demo_client";
+	if (!Register(proc_id)) {
 		return;
 	}
 
-	const std::string topic_ = "topic_";
+	const std::string topic_ = proc_id + "_topic_";
 
-	{
+	{ // Server Register Topics
 		MsgTopicList topics;
 		for (int i = 0; i < 10; ++i) {
 			topics.add_topic_list(topic_ + std::to_string(i));
@@ -183,34 +248,178 @@
 		void *reply = 0;
 		int reply_len = 0;
 		bool r = BHRegisterTopics(s.data(), s.size(), &reply, &reply_len, 1000);
-		BHFree(reply, reply_len);
-		// printf("register topic : %s\n", r ? "ok" : "failed");
-		Sleep(1s);
+		DEFER1(BHFree(reply, reply_len));
 	}
-
-	{
-		MsgTopicList topics;
-		for (int i = 0; i < 10; ++i) {
-			topics.add_topic_list(topic_ + std::to_string(i * 2));
+	auto PrintProcs = [](MsgQueryProcReply const &result) {
+		printf("query proc result: %d\n", result.proc_list().size());
+		for (int i = 0; i < result.proc_list().size(); ++i) {
+			auto &info = result.proc_list(i);
+			printf("proc [%d] %s, %s, %s\n\ttopics\n", i,
+			       (info.online() ? "online" : "offline"),
+			       info.proc().proc_id().c_str(), info.proc().name().c_str());
+			auto PrintTopics = [](std::string const &name, auto &topic_list) {
+				printf("%s:[", name.c_str());
+				for (auto &t : topic_list) {
+					printf("%s,", t.c_str());
+				}
+				printf("]\n");
+			};
+			PrintTopics("service", info.service().topic_list());
+			PrintTopics("local_sub", info.local_sub().topic_list());
+			PrintTopics("net_sub", info.net_sub().topic_list());
+			printf("\n");
 		}
-		std::string s = topics.SerializeAsString();
+		printf("\n");
+	};
+	if (1) {
+		// query procs
+		std::string dest(BHAddress().SerializeAsString());
+		MsgQueryProc query;
+		std::string s = query.SerializeAsString();
 		void *reply = 0;
 		int reply_len = 0;
-		bool r = BHSubscribeTopics(s.data(), s.size(), &reply, &reply_len, 1000);
-		BHFree(reply, reply_len);
-		printf("subscribe topic : %s\n", r ? "ok" : "failed");
+		bool r = BHQueryProcs(dest.data(), dest.size(), s.data(), s.size(), &reply, &reply_len, 1000);
+		DEFER1(BHFree(reply, reply_len));
+		MsgQueryProcReply result;
+		if (result.ParseFromArray(reply, reply_len) && IsSuccess(result.errmsg().errcode())) {
+			PrintProcs(result);
+		} else {
+			printf("query proc error\n");
+		}
+		// printf("register topic : %s\n", r ? "ok" : "failed");
+		// Sleep(1s);
+	}
+	for (int i = 0; i < 3; ++i) {
+		// query procs with normal topic request
+		MsgRequestTopic req;
+		req.set_topic("#center_query_procs");
+		// req.set_data("{\"proc_id\":\"#center.node\"}");
+		std::string s(req.SerializeAsString());
+		// Sleep(10ms, false);
+		BHAddress host;
+		printf("query with ip set\n");
+		host.set_ip("127.0.0.1");
+		host.set_port(kBHCenterPort);
+		// center topic node address.
+		// host.set_mq_id(201);
+		// host.set_abs_addr(10072);
+
+		std::string dest(host.SerializeAsString());
+		void *proc_id = 0;
+		int proc_id_len = 0;
+		DEFER1(BHFree(proc_id, proc_id_len););
+		void *reply = 0;
+		int reply_len = 0;
+		DEFER1(BHFree(reply, reply_len));
+		bool r = BHRequest(dest.data(), dest.size(), s.data(), s.size(), &proc_id, &proc_id_len, &reply, &reply_len, 1000);
+		if (!r) {
+			int ec = 0;
+			std::string msg;
+			GetApiError(ec, msg);
+			printf("topic query proc error: %s\n", msg.c_str());
+		} else {
+			MsgRequestTopicReply ret;
+			ret.ParseFromArray(reply, reply_len);
+			printf("\ntopic query proc : %s\n", ret.data().c_str());
+			// MsgQueryProcReply result;
+			// if (result.ParseFromArray(ret.data().data(), ret.data().size()) && IsSuccess(result.errmsg().errcode())) {
+			// 	PrintProcs(result);
+			// } else {
+			// 	printf("topic query proc error\n");
+			// }
+		}
 	}
 
+	{ // Subscribe
+		auto Subscribe = [&](std::string topic, bool net) {
+			MsgTopicList topics;
+			topics.add_topic_list(topic);
+			for (int i = 0; i < 10; ++i) {
+				topics.add_topic_list(topic_ + std::to_string(i * 2));
+			}
+			std::string s = topics.SerializeAsString();
+			void *reply = 0;
+			int reply_len = 0;
+			bool r = false;
+			if (net) {
+				r = BHSubscribeNetTopics(s.data(), s.size(), &reply, &reply_len, 1000);
+			} else {
+				r = BHSubscribeTopics(s.data(), s.size(), &reply, &reply_len, 1000);
+			}
+			BHFree(reply, reply_len);
+			printf("subscribe topic %s: %s\n", topic.c_str(), (r ? "ok" : "failed"));
+		};
+		Subscribe("#center.node", false);
+		Subscribe("local0", false);
+		Subscribe("net0", true);
+	}
+
+	auto ServerLoop = [&](std::atomic<bool> *run) {
+		while (*run) {
+			void *proc_id = 0;
+			int proc_id_len = 0;
+			DEFER1(BHFree(proc_id, proc_id_len););
+			void *input = 0;
+			int input_len = 0;
+			DEFER1(BHFree(input, input_len));
+			void *src = 0;
+			if (BHReadRequest(&proc_id, &proc_id_len, &input, &input_len, &src, 10) && src) {
+
+				MsgRequestTopic request;
+				if (request.ParseFromArray(input, input_len)) {
+					MsgRequestTopicReply reply;
+					reply.set_data(" reply: " + request.data());
+					std::string s(reply.SerializeAsString());
+					// printf("%s", reply.data().c_str());
+					BHSendReply(src, s.data(), s.size());
+					++Status().nserved_;
+				}
+				src = 0;
+			}
+		}
+	};
+
+	std::atomic<bool> run(true);
+	ThreadManager threads;
+#if 1
 	BHStartWorker(&ServerProc, &SubRecvProc, &ClientProc);
+#else
+	BHStartWorker(FServerCallback(), &SubRecvProc, &ClientProc);
+	threads.Launch(ServerLoop, &run);
+#endif
+
+	auto Publish = [&](const std::string &topic, const std::string &data) {
+		MsgPublish pub;
+		pub.set_topic(topic);
+		pub.set_data(data);
+		std::string s(pub.SerializeAsString());
+		BHPublish(s.data(), s.size(), 0);
+	};
 
 	{
+		// publish
+		Publish(topic_ + std::to_string(0), "pub_data_" + std::string(104 * 1, 'a'));
 		for (int i = 0; i < 1; ++i) {
-			MsgPublish pub;
-			pub.set_topic(topic_ + std::to_string(i));
-			pub.set_data("pub_data_" + std::string(1024 * 1, 'a'));
-			std::string s(pub.SerializeAsString());
-			BHPublish(s.data(), s.size(), 0);
-			// Sleep(1s);
+
+			ssjson::Json net = ssjson::Json::Array();
+			ssjson::Json host;
+			host.put("serverId", "test_host");
+			host.put("ip", "127.0.0.1");
+			ssjson::Json topics = ssjson::Json::Array();
+			topics.push_back("aaaaa");
+			topics.push_back("bbbbb");
+			host.put("pubTopics", topics);
+			topics.push_back("net0");
+			topics.push_back("net1");
+			host.put("netSubTopics", topics);
+			net.push_back(host);
+
+			Publish("pub-allRegisterInfo-to-center", net.dump());
+			Sleep(1s);
+			Publish("local0", "local-abcd0");
+			Publish("net0", "net-abcd0");
+			Publish("local0", "local-abcd1");
+			Sleep(1s);
 		}
 	}
 
@@ -222,9 +431,11 @@
 			std::string s(req.SerializeAsString());
 			void *msg_id = 0;
 			int len = 0;
-			// Sleep(10ms, false);
-			bool r = BHAsyncRequest(s.data(), s.size(), 0, 0);
 			DEFER1(BHFree(msg_id, len););
+			// Sleep(10ms, false);
+			std::string dest(BHAddress().SerializeAsString());
+
+			bool r = BHAsyncRequest(dest.data(), dest.size(), s.data(), s.size(), 0, 0);
 			if (r) {
 				++Status().nrequest_;
 			} else {
@@ -234,7 +445,7 @@
 				if (last.exchange(now) < now) {
 					int ec = 0;
 					std::string msg;
-					GetLastError(ec, msg);
+					GetApiError(ec, msg);
 					printf("request topic error --------- : %s\n", msg.c_str());
 				}
 			}
@@ -260,32 +471,41 @@
 			printf("heartbeat: %s\n", r ? "ok" : "failed");
 		}
 	};
-	std::atomic<bool> run(true);
-	ThreadManager threads;
+
 	boost::timer::auto_cpu_timer timer;
 	threads.Launch(hb, &run);
 	threads.Launch(showStatus, &run);
 	int ncli = 10;
-	const uint64_t nreq = 1000 * 100;
+	const int64_t nreq = 1000; //* 100;
+
+	for (int i = 0; i < 10; ++i) {
+		SyncRequest(topic_ + std::to_string(0), "request_data_" + std::to_string(i));
+	}
+
 	for (int i = 0; i < ncli; ++i) {
 		threads.Launch(asyncRequest, nreq);
 	}
 
 	int same = 0;
-	int64_t last = 0;
-	while (last < nreq * ncli && same < 2) {
+	uint64_t last = 0;
+	while (last < nreq * ncli && same < 3) {
 		Sleep(1s, false);
 		auto cur = Status().nreply_.load();
 		if (last == cur) {
 			++same;
+			printf("same %d\n", same);
 		} else {
 			last = cur;
 			same = 0;
 		}
 	}
 
+	Sleep(1s);
+
 	run = false;
 	threads.WaitAll();
 	auto &st = Status();
 	printf("nreq: %8ld, nsrv: %8ld, nreply: %8ld\n", st.nrequest_.load(), st.nserved_.load(), st.nreply_.load());
+	BHCleanup();
+	printf("after cleanup\n");
 }
\ No newline at end of file

--
Gitblit v1.8.0