Move test inside if block to prevent premature exit

grep returns a non-zero error code if the pattern is not found.  This
would cause the topology-manager.sh test to exit prematurely instead
of capturing and logging the error and returning zero as intended.

Issue-ID: MULTICLOUD-1240
Signed-off-by: Todd Malsbary <todd.malsbary@intel.com>
Change-Id: I29c4d30630c0f803325c2ed024d4c0b9e8a0e911

diff --git a/kud/tests/topology-manager.sh b/kud/tests/topology-manager.sh
index bbffd4d..7d43438 100755 (executable)
--- a/kud/tests/topology-manager.sh
+++ b/kud/tests/topology-manager.sh
@@ -15,9 +15,8 @@ set -o pipefail
 source _common.sh
 source _functions.sh
 
-ethernet_adpator_version=$( lspci | grep "Ethernet Controller XL710" | head -n 1 | cut -d " " -f 8 )
-if [ -z "$ethernet_adpator_version" ]; then
-    echo " Ethernet adapator version is not set. Topology manager test case cannot run on this machine"
+if [ -z "$( lspci | grep "Ethernet Controller XL710" | head -n 1 | cut -d " " -f 8 )" ]; then
+    echo "Ethernet adaptor version is not set. Topology manager test case cannot run on this machine"
     exit 0
 else
     echo "NIC card specs match. Topology manager option avaiable for this version."