backup: Fix tests/virsh-snapshot

[libvirt] [PATCH] backup: Fix tests/virsh-snapshot

Eric Blake posted 1 patch 4 years, 9 months ago

Download mbox

Test syntax-check passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20190726203424.14657-1-eblake@redhat.com

tests/virsh-snapshot | 2 ++
1 file changed, 2 insertions(+)

Expand all Fold all

[libvirt] [PATCH] backup: Fix tests/virsh-snapshot

Posted by Eric Blake 4 years, 9 months ago

Creating an 'exp' output file, but never comparing it against the
actual output, does no one any good. :)

Fixes: 280a2b41e
Signed-off-by: Eric Blake <eblake@redhat.com>
---

Pushing under the trivial rule.

 tests/virsh-snapshot | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/tests/virsh-snapshot b/tests/virsh-snapshot
index b23d949f62..1034140cde 100755
--- a/tests/virsh-snapshot
+++ b/tests/virsh-snapshot
@@ -207,6 +207,8 @@ Descendants:    0
 Metadata:       yes

 EOF
+sed '1,/^virsh #/d; /virsh #/d' < out > out.cooked || fail=1
+compare exp out.cooked || fail=1

 cat <<EOF > exp || fail=1
 error: invalid argument: parent s3 for moment s2 not found
-- 
2.20.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list