Richard W.M. Jones
2010-Jun-03 13:55 UTC
[Libguestfs] [PATCH] generator: Allow individual tests to depend on daemon features.
-- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones virt-top is 'top' for virtual machines. Tiny program with many powerful monitoring features, net stats, disk stats, logging, etc. http://et.redhat.com/~rjones/virt-top -------------- next part -------------->From f9d08600c52dc0730e7dad8d9259b59e32aeece2 Mon Sep 17 00:00:00 2001From: Richard Jones <rjones at redhat.com> Date: Thu, 3 Jun 2010 14:31:58 +0100 Subject: [PATCH 2/4] generator: Allow individual tests to depend on daemon features. Using IfAvailable "featurename" we allow individual tests to only run if the feature is available in the daemon. This will allow us to extend testing to a lot more optional features such as NTFS. --- src/generator.ml | 14 ++++++++++++-- 1 files changed, 12 insertions(+), 2 deletions(-) diff --git a/src/generator.ml b/src/generator.ml index 939f1f6..4d74cb0 100755 --- a/src/generator.ml +++ b/src/generator.ml @@ -309,6 +309,9 @@ and test_prereq (* As for 'If' but the test runs _unless_ the code returns true. *) | Unless of string + (* Run the test only if 'string' is available in the daemon. *) + | IfAvailable of string + (* Some initial scenarios for testing. *) and test_init (* Do nothing, block devices could contain random stuff including @@ -6862,7 +6865,7 @@ is_available (const char *group) fun (_, _, _, _, tests, _, _) -> let tests = filter_map ( function - | (_, (Always|If _|Unless _), test) -> Some test + | (_, (Always|If _|Unless _|IfAvailable _), test) -> Some test | (_, Disabled, _) -> None ) tests in let seq = List.concat (List.map seq_of_test tests) in @@ -7068,7 +7071,7 @@ static int %s_skip (void) " test_name name (String.uppercase test_name) (String.uppercase name); (match prereq with - | Disabled | Always -> () + | Disabled | Always | IfAvailable _ -> () | If code | Unless code -> pr "static int %s_prereq (void)\n" test_name; pr "{\n"; @@ -7117,6 +7120,13 @@ static int %s (void) pr " }\n"; pr "\n"; generate_one_test_body name i test_name init test; + | IfAvailable group -> + pr " if (!is_available (\"%s\")) {\n" group; + pr " printf (\" %%s skipped (reason: %%s not available)\\n\", \"%s\", \"%s\");\n" test_name group; + pr " return 0;\n"; + pr " }\n"; + pr "\n"; + generate_one_test_body name i test_name init test; | Always -> generate_one_test_body name i test_name init test ); -- 1.6.6.1
Apparently Analagous Threads
- [PATCH] generator: Passing "" to StringList tests should turn into empty list.
- Re: [PATCH 06/27] daemon: Add unit tests of the ‘Utils’ module.
- [PATCH 0/6] Simple fixes for cross-compiling the daemon
- [PATCH] daemon/Win32: use gnulib modules connect, socket and symlinkat (for readlinkat).
- src/generator.ml is no more!