From a525f24c3ee276323088708dee70ee9bf9ceddd0 Mon Sep 17 00:00:00 2001
From: Romain Forlot <romain.forlot@iot.bzh>
Date: Fri, 5 Oct 2018 17:16:14 +0200
Subject: Unify test launch between target and native

On a target, the binder process name is taken from the widget "id",
now the native launcher script also launches the binder using the widget "id".

As the controller searches its configuration file using the binder process middle name
we need to differentiate between both "id" otherwise the wrong configuration file is used.
The commit renames configuration file according to the new "id".

Change-Id: I4a596dd1d54201a671bb68bf9aca6904286322c6
Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
---
 afm-test.native.sh | 44 +++++++++++++++++++++-----------------------
 1 file changed, 21 insertions(+), 23 deletions(-)

(limited to 'afm-test.native.sh')

diff --git a/afm-test.native.sh b/afm-test.native.sh
index 15be083..367319e 100644
--- a/afm-test.native.sh
+++ b/afm-test.native.sh
@@ -47,7 +47,7 @@ then
 	SERVICEPACKAGEDIR="$(readlink -f $1)"
 	TESTPACKAGEDIR="$(readlink -f $2)"
 else
-	echo "Error: you doesn't specified either the binding folder location either test widget folder location."
+	echo "Error: you did not specify either the binding folder location or test widget folder location."
 	usage
 	cleanNexit 1
 fi
@@ -67,25 +67,24 @@ else
 fi
 
 TESTCFGFILE=$(find "${TESTPACKAGEDIR}" -name "aft-*.json" -print | head -n1)
-TESTPROCNAME=$(grep '\"api\"' "${TESTCFGFILE}" | cut -d'"' -f4)
-[ -z "${TESTPROCNAME}" ]  && \
-echo "Can't find 'api' key from your test configuration file: ${TESTCFGFILE}.\n Either configuration file doesn't exist either it is incorrect." && \
-cleanNexit 4
+TESTAPINAME=$(grep '\"api\"' "${TESTCFGFILE}" | cut -d'"' -f4)
+[ ! -f "${TESTPACKAGEDIR}/config.xml" ] && \
+	echo "Error: you don't have the config.xml file. Please call 'make widget'" && \
+	cleanNexit 5
+TESTPROCNAME="afbd-$(grep -Eo 'id=".*" ' ${TESTPACKAGEDIR}/config.xml | cut -d'=' -f2 | tr -d '" '| tr '[:upper:]' '[:lower:]')"
 
 API=$(grep "provided-api" "${SERVICEPACKAGEDIR}/config.xml" -A1 2> /dev/null |  sed -r -e '1d' -e 's:.*"(.*)" v.*:\1:' 2> /dev/null)
-if [ -z "$API" ] && [ "$MODE" = "SERVICE" ]
-then
-	echo "Error: you doesn't have the config.xml file. Please call 'make widget'"
+[ -z "$API" ] && [ "$MODE" = "SERVICE" ] && \
+	echo "Error: you doesn't have the config.xml file. Please call 'make widget'" && \
 	cleanNexit 2
-else
-	ENV_API=$(echo ${API} | sed 's:[^a-zA-Z0-9_]:_:g')
-	declare AFT_${ENV_API}_CONFIG_PATH="${SERVICEPACKAGEDIR}"
-	declare AFT_$(echo ${ENV_API} | sed 's:[^a-zA-Z0-9_]:_:g')_PLUGIN_PATH="${SERVICEPACKAGEDIR}"
-	export AFT_${ENV_API}_CONFIG_PATH
-	export AFT_${ENV_API}_PLUGIN_PATH
-	PROCNAME="afbd-${API}"
-	SOCKETSERVICE="/tmp/$API"
-fi
+
+ENV_API=$(echo ${API} | sed 's:[^a-zA-Z0-9_]:_:g')
+declare AFT_${ENV_API}_CONFIG_PATH="${SERVICEPACKAGEDIR}"
+declare AFT_$(echo ${ENV_API} | sed 's:[^a-zA-Z0-9_]:_:g')_PLUGIN_PATH="${SERVICEPACKAGEDIR}"
+export AFT_${ENV_API}_CONFIG_PATH
+export AFT_${ENV_API}_PLUGIN_PATH
+PROCNAME="afbd-$(grep -Eo 'id=".*" ' ${SERVICEPACKAGEDIR}/config.xml | cut -d'=' -f2 | tr -d '" '| tr '[:upper:]' '[:lower:]')"
+SOCKETSERVICE="/tmp/$API"
 
 export AFT_CONFIG_PATH="${TESTPACKAGEDIR}"
 export AFT_PLUGIN_PATH="${TESTPACKAGEDIR}"
@@ -106,15 +105,15 @@ then
 			--workdir="${TESTPACKAGEDIR}" \
 			--ldpaths=${SERVICEPACKAGEDIR} \
 			--binding="${AFBTEST}" \
-			--call="${TESTPROCNAME}/launch_all_tests:{}" \
-			--call="${TESTPROCNAME}/exit:{}" \
+			--call="${TESTAPINAME}/launch_all_tests:{}" \
+			--call="${TESTAPINAME}/exit:{}" \
 			-vvv &> ${LOGFILETEST}
 elif [ ${MODE} = "SERVICE" ]
 then
 	pkill "$TESTPROCNAME"
 	pkill "$PROCNAME"
 
-	timeout -s 9 ${TIMEOUT} ${BINDER} --name=${PROCNAME} \
+	timeout -s 9 ${TIMEOUT} ${BINDER} --name="${PROCNAME}" \
 				--workdir="${SERVICEPACKAGEDIR}" \
 				--port=${PORTSERVICE} \
 				--ldpaths=. \
@@ -131,10 +130,9 @@ then
 				--workdir="${TESTPACKAGEDIR}" \
 				--binding="${AFBTEST}" \
 				--ws-client=unix:${SOCKETSERVICE} \
-				--call="${TESTPROCNAME}/launch_all_tests:{}" \
-				--call="${TESTPROCNAME}/exit:{}" \
+				--call="${TESTAPINAME}/launch_all_tests:{}" \
+				--call="${TESTAPINAME}/exit:{}" \
 				-vvv &> ${LOGFILETEST}
-
 else
 	echo "Error: No mode selected. Choose between SOLO or SERVICE"
 	usage
-- 
cgit 1.2.3-korg