From 9c910b7be223704b6f151c1eb6794b72fdcc6af9 Mon Sep 17 00:00:00 2001
From: Milan Broz <mbroz@redhat.com>
Date: Thu, 28 Aug 2008 10:59:10 +0000
Subject: [PATCH] add vgrename by uuid test (Jaroslav Stava) fix vgsplit
 rejects last mda copy test (Jaroslav Stava)

---
 test/t-vgrename-usage.sh | 13 ++++++++++++-
 test/t-vgsplit-usage.sh  | 22 +++++++++++++---------
 2 files changed, 25 insertions(+), 10 deletions(-)

diff --git a/test/t-vgrename-usage.sh b/test/t-vgrename-usage.sh
index 690244ae6..ade3a0cd2 100755
--- a/test/t-vgrename-usage.sh
+++ b/test/t-vgrename-usage.sh
@@ -31,7 +31,8 @@ test_expect_success \
    f4=$(pwd)/4 && d4=$(loop_setup_ "$f4") &&
    vg1=$(this_test_)-1-$$          &&
    vg2=$(this_test_)-2-$$          &&
-   pvcreate $d1 $d2 $d3 $d4'
+   pvcreate $d1 $d2                &&
+   pvcreate --metadatacopies 0 $d3 $d4'
 
 test_expect_success \
   'vgrename normal operation - rename vg1 to vg2' \
@@ -40,6 +41,16 @@ test_expect_success \
    check_vg_field_ $vg2 vg_name $vg2 &&
    vgremove $vg2'
 
+test_expect_success \
+  "vgrename by uuid (bz231187)" '
+   vgcreate $vg1 $d1 $d3 &&
+   UUID=$(vgs --noheading -o vg_uuid $vg1) &&
+   check_vg_field_ $vg1 vg_uuid $UUID &&
+   vgrename $UUID $vg2 &&
+   check_vg_field_ $vg2 vg_name $vg2 &&
+   vgremove $vg2
+'
+
 test_done
 # Local Variables:
 # indent-tabs-mode: nil
diff --git a/test/t-vgsplit-usage.sh b/test/t-vgsplit-usage.sh
index d58b5c0c8..5a674564a 100755
--- a/test/t-vgsplit-usage.sh
+++ b/test/t-vgsplit-usage.sh
@@ -40,12 +40,11 @@ test_expect_success \
    lv2=$(this_test_)-test-lv2-$$          &&
    lv3=$(this_test_)-test-lv3-$$'
 
-for mdatype in 2
+for mdatype in 1 2
 do
 test_expect_success \
   "(lvm$mdatype) setup PVs" \
-  'pvcreate -M$mdatype $d1 $d2 $d3 $d4 &&
-   pvcreate -M$mdatype --metadatacopies 0 $d5'
+  'pvcreate -M$mdatype $d1 $d2 $d3 $d4'
 
 test_expect_success \
   "(lvm$mdatype) vgsplit accepts new vg as destination of split" \
@@ -200,19 +199,24 @@ test_expect_success \
    status=$?; echo status=$status; test $status = 5 &&
    vgremove -f $vg2 &&
    vgremove -f $vg1'
+done
+
+test_expect_success \
+  "(lvm2) setup PVs" '
+   pvcreate -M$mdatype --metadatacopies 0 $d5'
 
 test_expect_success \
   "(lvm2) vgsplit rejects to give away pv with the last mda copy" '
-   vgcreate -M$mdatype $vg1 $d5 $d2  &&
+   vgcreate -M2 $vg1 $d5 $d2  &&
    lvcreate -l 10 -n $lv1  $vg1 &&
    lvchange -an $vg1/$lv1 &&
    vg_validate_pvlv_counts_ $vg1 2 1 0 &&
-   vgsplit  $vg1 $vg2 $d5;
-   status=$?; echo status=$status; test $status != 0 &&
-   vg_validate_pvlv_counts_ $vg1 2 1 0 &&
-   vgremove -ff $vg1
+   { vgsplit  $vg1 $vg2 $d5;
+     status=$?; echo status=$status; test $status != 0 &&
+     vg_validate_pvlv_counts_ $vg1 2 1 0 &&
+     vgremove -ff $vg1
+   }
 '
-done
 
 test_expect_success \
   '(lvm2) vgsplit rejects split because metadata types differ' \
-- 
2.43.5