From da232cbeaf2a906fe9c8cf74537cb869b6543baa Mon Sep 17 00:00:00 2001
From: plum-lihui <huili@taosdata.com>
Date: Wed, 20 Apr 2022 20:02:10 +0800
Subject: [PATCH] [test: check error from valgrind log file]

---
 tests/script/sh/checkValgrind.sh          | 38 ++++++++++++
 tests/script/tsim/valgrind/checkError.sim | 76 +++++++++++++++++++++++
 tests/tsim/src/simExe.c                   |  6 ++
 3 files changed, 120 insertions(+)
 create mode 100755 tests/script/sh/checkValgrind.sh
 create mode 100644 tests/script/tsim/valgrind/checkError.sim

diff --git a/tests/script/sh/checkValgrind.sh b/tests/script/sh/checkValgrind.sh
new file mode 100755
index 0000000000..17d7ce1aff
--- /dev/null
+++ b/tests/script/sh/checkValgrind.sh
@@ -0,0 +1,38 @@
+#!/bin/bash
+
+set +e
+#set -x
+
+NODE_NAME=
+
+while getopts "n:" arg
+do
+  case $arg in
+    n)
+      NODE_NAME=$OPTARG
+      ;;
+    ?)
+      echo "unkown argument"
+      ;;
+  esac
+done
+
+SCRIPT_DIR=`dirname $0`
+cd $SCRIPT_DIR/../
+SCRIPT_DIR=`pwd`
+
+IN_TDINTERNAL="community"
+if [[ "$SCRIPT_DIR" == *"$IN_TDINTERNAL"* ]]; then
+  cd ../../..
+else
+  cd ../../
+fi
+
+TAOS_DIR=`pwd`
+LOG_DIR=$TAOS_DIR/sim/$NODE_NAME/log
+#CFG_DIR=$TAOS_DIR/sim/$NODE_NAME/cfg
+
+#echo ---- $LOG_DIR
+
+errors=`grep "ERROR SUMMARY:" ${LOG_DIR}/valgrind-taosd-*.log | cut -d ' ' -f 2,3,4,5 | tr -d "\n"`
+echo $errors
diff --git a/tests/script/tsim/valgrind/checkError.sim b/tests/script/tsim/valgrind/checkError.sim
new file mode 100644
index 0000000000..3b66cb4a8c
--- /dev/null
+++ b/tests/script/tsim/valgrind/checkError.sim
@@ -0,0 +1,76 @@
+system sh/stop_dnodes.sh
+system sh/deploy.sh -n dnode1 -i 1
+#system sh/deploy.sh -n dnode2 -i 2
+#system sh/deploy.sh -n dnode3 -i 3
+#system sh/deploy.sh -n dnode4 -i 4
+#system sh/cfg.sh -n dnode1 -c supportVnodes -v 0
+system sh/exec.sh -n dnode1 -s start
+#system sh/exec.sh -n dnode2 -s start
+#system sh/exec.sh -n dnode3 -s start
+#system sh/exec.sh -n dnode4 -s start
+
+sleep 2000
+
+#$loop_cnt = 0
+#check_dnode_ready:
+#	$loop_cnt = $loop_cnt + 1
+#	sleep 200
+#	if $loop_cnt == 10 then
+#	  print ====> dnode not ready!
+#		return -1
+#	endi
+#sql show dnodes
+#print ===> $rows $data[0][0] $data[0][1] $data[0][2] $data[0][3] $data[0][4] $data[0][5] $data[0][6]
+#print ===> $rows $data[1][0] $data[1][1] $data[1][2] $data[1][3] $data[1][4] $data[1][5] $data[1][6]
+#print ===> $rows $data[2][0] $data[2][1] $data[2][2] $data[2][3] $data[2][4] $data[2][5] $data[2][6]
+#print ===> $rows $data[3][0] $data[3][1] $data[3][2] $data[3][3] $data[3][4] $data[3][5] $data[3][6]
+#if $data[0][0] != 1 then
+#  return -1
+#endi
+#if $data[0][4] != ready then
+#  goto check_dnode_ready
+#endi
+#
+##sql connect
+#sql create dnode $hostname port 7200
+#sql create dnode $hostname port 7300
+#sql create dnode $hostname port 7400
+#
+#$loop_cnt = 0
+#check_dnode_ready_1:
+#$loop_cnt = $loop_cnt + 1
+#sleep 200
+#if $loop_cnt == 10 then
+#  print ====> dnodes not ready!
+#	return -1
+#endi
+#sql show dnodes
+#print ===> $rows $data[0][0] $data[0][1] $data[0][2] $data[0][3] $data[0][4] $data[0][5] $data[0][6]
+#print ===> $rows $data[1][0] $data[1][1] $data[1][2] $data[1][3] $data[1][4] $data[1][5] $data[1][6]
+#print ===> $rows $data[2][0] $data[2][1] $data[2][2] $data[2][3] $data[2][4] $data[2][5] $data[2][6]
+#print ===> $rows $data[3][0] $data[3][1] $data[3][2] $data[3][3] $data[3][4] $data[3][5] $data[3][6]
+#if $data[0][4] != ready then
+#  goto check_dnode_ready_1
+#endi
+#if $data[1][4] != ready then
+#  goto check_dnode_ready_1
+#endi
+#if $data[2][4] != ready then
+#  goto check_dnode_ready_1
+#endi
+#if $data[3][4] != ready then
+#  goto check_dnode_ready_1
+#endi
+
+#=========== please add any actions above =================
+
+print ====> stop all dondes to output valgrind log file
+system sh/exec.sh -n dnode1 -s stop -x SIGINT
+
+print ====> start to check if there are ERRORS in vagrind log file for each dnode
+# -n : dnode[x] be check
+system_content sh/checkValgrind.sh -n dnode1 
+print cmd return result----> [ $system_content ]
+if $system_content != @ERROR SUMMARY: 0 errors@ then
+  return -1
+endi 
diff --git a/tests/tsim/src/simExe.c b/tests/tsim/src/simExe.c
index 8b3edb6035..dbc3c2c460 100644
--- a/tests/tsim/src/simExe.c
+++ b/tests/tsim/src/simExe.c
@@ -441,6 +441,12 @@ void simStoreSystemContentResult(SScript *script, char *filename) {
   // if ((fd = fopen(filename, "r")) != NULL) {
   if ((pFile = taosOpenFile(filename, TD_FILE_READ)) != NULL) {
     taosReadFile(pFile, script->system_ret_content, MAX_SYSTEM_RESULT_LEN - 1);
+    int32_t len = strlen(script->system_ret_content);
+    for (int32_t i = 0; i < len; ++i) {
+      if (script->system_ret_content[i] == '\n' || script->system_ret_content[i] == '\r') {
+        script->system_ret_content[i] = 0;
+      }
+    }
     taosCloseFile(&pFile);
     char rmCmd[MAX_FILE_NAME_LEN] = {0};
     sprintf(rmCmd, "rm -f %s", filename);
-- 
GitLab