[test suite] allow valgrind coverage test to run after --test-limit
authorMichael Rash <mbr@cipherdyne.org>
Sat, 11 May 2013 17:28:55 +0000 (13:28 -0400)
committerMichael Rash <mbr@cipherdyne.org>
Sat, 11 May 2013 17:28:55 +0000 (13:28 -0400)
test/test-fwknop.pl

index 634803d..0aad5b9 100755 (executable)
@@ -545,36 +545,6 @@ if ($enable_valgrind) {
     }
 }
 
-if ($enable_profile_coverage_check) {
-    push @tests,
-        {
-            'category' => 'profile coverage',
-            'detail'   => 'gcov profile coverage',
-            'function' => \&profile_coverage,
-            'fatal'    => $NO
-        };
-}
-
-if ($enable_valgrind) {
-    push @tests,
-        {
-            'category' => 'valgrind',
-            'subcategory' => 'fko-wrapper',
-            'detail'   => 'multiple libfko calls',
-            'function' => \&compile_execute_fko_wrapper,
-            'fatal'    => $NO
-        };
-
-    push @tests,
-        {
-            'category' => 'valgrind output',
-            'subcategory' => 'flagged functions',
-            'detail'   => '',
-            'function' => \&parse_valgrind_flagged_functions,
-            'fatal'    => $NO
-        };
-}
-
 ### print a summary of how many test buckets will be run
 my $test_buckets = 0;
 for my $test_hr (@tests) {
@@ -594,6 +564,34 @@ for my $test_hr (@tests) {
     }
 }
 
+if ($enable_profile_coverage_check) {
+    &run_test({
+        'category' => 'profile coverage',
+        'detail'   => 'gcov profile coverage',
+        'function' => \&profile_coverage,
+        'fatal'    => $NO}
+    );
+}
+
+if ($enable_valgrind) {
+    &run_test({
+        'category' => 'valgrind',
+        'subcategory' => 'fko-wrapper',
+        'detail'   => 'multiple libfko calls',
+        'function' => \&compile_execute_fko_wrapper,
+        'fatal'    => $NO}
+    );
+
+    &run_test({
+        'category' => 'valgrind output',
+        'subcategory' => 'flagged functions',
+        'detail'   => '',
+        'function' => \&parse_valgrind_flagged_functions,
+        'fatal'    => $NO}
+    );
+}
+
+
 &logr("\n");
 
 &remove_permissions_warnings() unless $include_permissions_warnings;