From db8c9f81876e2b920a7143a33bade7cd2ffb5036 Mon Sep 17 00:00:00 2001
From: "Richard W.M. Jones" <rjones@redhat.com>
Date: Tue, 29 Nov 2011 17:51:59 +0000
Subject: [PATCH] extra tests: Add comments about what is not tested here.
 (cherry picked from commit efef3e3c1205e5476044f41fb88385e3ba07468d)

---
 extratests/Makefile.am | 33 +++++++++++++++++++++++++--------
 1 file changed, 25 insertions(+), 8 deletions(-)

diff --git a/extratests/Makefile.am b/extratests/Makefile.am
index 9c34413..e330e5c 100644
--- a/extratests/Makefile.am
+++ b/extratests/Makefile.am
@@ -24,6 +24,31 @@
 #   - valgrind
 #
 # These tests may fail for reasons which aren't necessarily real problems.
+#
+# XXX Not tested:
+#
+# ../clone/virt-sysprep
+#   - hard to test because it's a shell script
+#
+# ../edit/virt-edit
+#
+# Perl bindings
+# ../edit/virt-edit -e
+# ../tools/virt-win-reg (Perl)
+# ../tools/virt-make-fs (Perl)
+#   - hard because Perl doesn't cooperate by freeing memory
+#
+# Python bindings
+#   - as for Perl
+#
+# Ruby bindings
+#   - these look feasible to test using valgrind
+#
+# Java bindings
+#
+# ../fuse/guestmount
+#   - hard to test because guestmount forks into the background, and
+#     if valgrind reports errors it doesn't stop the test shell script
 
 EXTRA_DIST = suppressions
 
@@ -112,14 +137,6 @@ test-resize:
 test-sparsify:
 	$(MAKE) -C ../sparsify VG="$(VG)" check
 
-# XXX Not tested:
-# ../clone/virt-sysprep
-# ../edit/virt-edit
-# ../edit/virt-edit -e
-# ../fuse/guestmount
-# ../tools/virt-win-reg (Perl)
-# ../tools/virt-make-fs (Perl)
-
 # Run capitests under valgrind.
 test-capitests:
 	$(MAKE) -C ../capitests VG="$(RUN_VG)" check
-- 
1.8.3.1