diff options
| author | Pranith Kumar K <pkarampu@redhat.com> | 2016-07-22 13:58:22 +0530 |
|---|---|---|
| committer | Jeff Darcy <jdarcy@redhat.com> | 2016-07-22 10:01:45 -0700 |
| commit | c5bf5d98594a4237a72cf0d3c72925d5a5aa0f69 (patch) | |
| tree | 1c667cbaa751376788afbcadc9145db3c990ea49 | |
| parent | 1c9b229a860908ffc0c0134450c9218bc8079c22 (diff) | |
tests: Fix pending-heal-count checks
EXPECT_WITHIN takes regular expression to match the count,
so even when there are say 10 entries to heal, it would
think that the heal is complete. Fixed checking
pending heal count with correct regex.
Thanks to Xavi for finding this problem.
Change-Id: Ic593d22468b2b586bfca864962ffa0eda96b1d1f
BUG: 1332054
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
Reviewed-on: http://review.gluster.org/14985
Smoke: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Xavier Hernandez <xhernandez@datalab.es>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
| -rw-r--r-- | tests/bugs/disperse/bug-1236065.t | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/tests/bugs/disperse/bug-1236065.t b/tests/bugs/disperse/bug-1236065.t index a07e37adc04..9038cb95948 100644 --- a/tests/bugs/disperse/bug-1236065.t +++ b/tests/bugs/disperse/bug-1236065.t @@ -53,7 +53,7 @@ EXPECT '7' online_brick_count # active heal EXPECT_WITHIN $PROCESS_UP_TIMEOUT "[0-9][0-9]*" get_shd_process_pid TEST $CLI volume heal $V0 full -EXPECT_WITHIN $HEAL_TIMEOUT "0" get_pending_heal_count $V0 +EXPECT_WITHIN $HEAL_TIMEOUT "^0$" get_pending_heal_count $V0 TEST rm -f *.o TEST ec_test_make @@ -75,7 +75,7 @@ EXPECT '7' online_brick_count # self-healing EXPECT_WITHIN $PROCESS_UP_TIMEOUT "[0-9][0-9]*" get_shd_process_pid TEST $CLI volume heal $V0 full -EXPECT_WITHIN $HEAL_TIMEOUT "0" get_pending_heal_count $V0 +EXPECT_WITHIN $HEAL_TIMEOUT "^0$" get_pending_heal_count $V0 TEST rm -f *.o TEST ec_test_make @@ -93,5 +93,3 @@ TEST $CLI volume delete $V0 TEST rm -rf $B0/* cleanup; - -#G_TESTDEF_TEST_STATUS_CENTOS6=BAD_TEST,BUG=1332054 |