From 11f6c600e55ca5677f93624efe44d2605cdd908d Mon Sep 17 00:00:00 2001
From: lichao <lichao@aiotlink.com>
Date: 星期五, 21 五月 2021 20:18:38 +0800
Subject: [PATCH] reserve #,@ prefix for internal proc id and topic.

---
 utest/api_test.cpp |   61 +++++++++++++++++++++---------
 1 files changed, 43 insertions(+), 18 deletions(-)

diff --git a/utest/api_test.cpp b/utest/api_test.cpp
index e597533..3d842bf 100644
--- a/utest/api_test.cpp
+++ b/utest/api_test.cpp
@@ -27,6 +27,15 @@
 {
 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 {
 
@@ -60,7 +69,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,
@@ -120,10 +129,11 @@
 
 	// BHCleanup();
 	// return;
+	const std::string proc_id = "demo_client";
 	bool reg = false;
 	for (int i = 0; i < 3 && !reg; ++i) {
 		ProcInfo proc;
-		proc.set_proc_id("demo_client");
+		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;
@@ -137,24 +147,24 @@
 			// if (!reg) {
 			// 	int ec = 0;
 			// 	std::string msg;
-			// 	GetLastError(ec, msg);
+			// 	GetApiError(ec, msg);
 			// 	printf("reg error: %s\n", msg.c_str());
 			// }
 		} else {
 			int ec = 0;
 			std::string msg;
-			GetLastError(ec, msg);
+			GetApiError(ec, msg);
 			printf("register failed, %d, %s\n", ec, msg.c_str());
 		}
 
 		BHFree(reply, reply_len);
-		Sleep(1s);
+		// Sleep(1s);
 	}
 	if (!reg) {
 		return;
 	}
 
-	const std::string topic_ = "topic_";
+	const std::string topic_ = proc_id + "_topic_";
 
 	{ // Server Register Topics
 		MsgTopicList topics;
@@ -166,8 +176,21 @@
 		int reply_len = 0;
 		bool r = BHRegisterTopics(s.data(), s.size(), &reply, &reply_len, 1000);
 		DEFER1(BHFree(reply, reply_len));
-		// printf("register topic : %s\n", r ? "ok" : "failed");
-		// Sleep(1s);
+	}
+	{ // Server Register Topics
+		MsgTopicList topics;
+		topics.add_topic_list("@should_fail");
+		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));
+		if (!r) {
+			int ec = 0;
+			std::string msg;
+			GetApiError(ec, msg);
+			printf("register rpc failed, %d, %s\n", ec, msg.c_str());
+		}
 	}
 	auto PrintProcs = [](MsgQueryProcReply const &result) {
 		printf("query proc result: %d\n", result.proc_list().size());
@@ -204,7 +227,8 @@
 	{
 		// query procs with normal topic request
 		MsgRequestTopic req;
-		req.set_topic("@center_query_procs");
+		req.set_topic("#center_query_procs");
+		// req.set_data("{\"proc_id\":\"#center.node\"}");
 		std::string s(req.SerializeAsString());
 		// Sleep(10ms, false);
 		std::string dest(BHAddress().SerializeAsString());
@@ -214,11 +238,11 @@
 		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);
+		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;
-			GetLastError(ec, msg);
+			GetApiError(ec, msg);
 			printf("topic query proc error: %s\n", msg.c_str());
 		} else {
 			MsgRequestTopicReply ret;
@@ -232,10 +256,10 @@
 			// }
 		}
 	}
-	// return;
 
 	{ // Subscribe
 		MsgTopicList topics;
+		topics.add_topic_list("#center.node");
 		for (int i = 0; i < 10; ++i) {
 			topics.add_topic_list(topic_ + std::to_string(i * 2));
 		}
@@ -285,11 +309,11 @@
 		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);
+		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;
-			GetLastError(ec, msg);
+			GetApiError(ec, msg);
 			printf("request error: %s\n", msg.c_str());
 		} else {
 			MsgRequestTopicReply ret;
@@ -330,7 +354,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());
 				}
 			}
@@ -361,7 +385,7 @@
 
 	ThreadManager threads;
 
-#if 0
+#if 1
 	BHStartWorker(&ServerProc, &SubRecvProc, &ClientProc);
 #else
 	BHStartWorker(FServerCallback(), &SubRecvProc, &ClientProc);
@@ -372,7 +396,7 @@
 	threads.Launch(hb, &run);
 	threads.Launch(showStatus, &run);
 	int ncli = 10;
-	const int64_t nreq = 1; //000 * 100;
+	const int64_t nreq = 1000 * 100;
 
 	for (int i = 0; i < 10; ++i) {
 		SyncRequest(i);
@@ -396,10 +420,11 @@
 		}
 	}
 
+	Sleep(1s);
+
 	run = false;
 	threads.WaitAll();
 	auto &st = Status();
-	Sleep(1s);
 	printf("nreq: %8ld, nsrv: %8ld, nreply: %8ld\n", st.nrequest_.load(), st.nserved_.load(), st.nreply_.load());
 	BHCleanup();
 	printf("after cleanup\n");

--
Gitblit v1.8.0