From 2592fa0409d5215473b84012c69ac34d0270ef08 Mon Sep 17 00:00:00 2001
From: hari gowtham <hgowtham@redhat.com>
Date: Fri, 28 Apr 2017 00:09:50 +0530
Subject: Tier/cli: detach status xml output

Problem: detach status xml output was broken because
of the wrong argument. The status_op sent to verify
whether it is a tier status command was as false.

Fix: the argument being passed was changed from false
to true.

Change-Id: I8cdd4dd972d6bfbb61c1182cbf4097767f83c7c5
BUG: 1446362
Signed-off-by: hari gowtham <hgowtham@redhat.com>
Reviewed-on: https://review.gluster.org/17131
Smoke: Gluster Build System <jenkins@build.gluster.org>
Tested-by: hari gowtham <hari.gowtham005@gmail.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
Reviewed-by: N Balachandran <nbalacha@redhat.com>
---
 tests/basic/tier/new-tier-cmds.t | 9 +++++++++
 1 file changed, 9 insertions(+)

(limited to 'tests/basic/tier')

diff --git a/tests/basic/tier/new-tier-cmds.t b/tests/basic/tier/new-tier-cmds.t
index 3c16154265d..d341e62dc51 100644
--- a/tests/basic/tier/new-tier-cmds.t
+++ b/tests/basic/tier/new-tier-cmds.t
@@ -27,6 +27,11 @@ function tier_daemon_status {
                 | sed -n '/.*<status>\([0-9]*\).*/s//\1/p'
 }
 
+function detach_xml_status {
+        $CLI_1 volume tier $V0 detach status --xml | sed -n \
+        '/.*<opErrstr>Detach tier status successful/p' | wc -l
+}
+
 cleanup;
 
 #setup cluster and test volume
@@ -52,10 +57,14 @@ TEST ! $CLI_1 volume rebalance $V0 tier status
 
 EXPECT "Tier command failed" $CLI_1 volume tier $V0 detach status
 
+EXPECT "0" detach_xml_status
+
 #after starting detach tier the detach tier status should display the status
 
 TEST $CLI_1 volume tier $V0 detach start
 
+EXPECT "1" detach_xml_status
+
 EXPECT_WITHIN $PROCESS_UP_TIMEOUT "1" tier_detach_status
 
 #kill a node
-- 
cgit