]> granicus.if.org Git - curl/commitdiff
runtests: ignore disabled even when ranges are given
authorDaniel Stenberg <daniel@haxx.se>
Tue, 2 Oct 2018 07:08:56 +0000 (09:08 +0200)
committerDaniel Stenberg <daniel@haxx.se>
Tue, 2 Oct 2018 11:07:06 +0000 (13:07 +0200)
runtests.pl support running a range of tests, like "44 to 127". Starting
now, the code makes sure that even such given ranges will ignore tests
that are marked as disabled.

Disabled tests can still be run by explictly specifying that test
number.

Closes #3075

tests/runtests.pl

index 7258fdf31818a4a8d7dd65f375f8ded782c1594e..b83dc4646f08b284ff9be06a9a616175bd0faf71 100755 (executable)
@@ -5215,6 +5215,12 @@ sub runtimestats {
     logmsg "\n";
 }
 
+# globally disabled tests
+disabledtests("$TESTDIR/DISABLED");
+
+# locally disabled tests, ignored by git etc
+disabledtests("$TESTDIR/DISABLED.local");
+
 #######################################################################
 # Check options to this test program
 #
@@ -5369,8 +5375,17 @@ EOHELP
     elsif($ARGV[0] =~ /^(\d+)/) {
         $number = $1;
         if($fromnum >= 0) {
-            for($fromnum .. $number) {
-                push @testthis, $_;
+            for my $n ($fromnum .. $number) {
+                if($disabled{$n}) {
+                    # skip disabled test cases
+                    my $why = "configured as DISABLED";
+                    $skipped++;
+                    $skipped{$why}++;
+                    $teststat[$n]=$why; # store reason for this test case
+                }
+                else {
+                    push @testthis, $n;
+                }
             }
             $fromnum = -1;
         }
@@ -5528,12 +5543,6 @@ sub disabledtests {
     }
 }
 
-# globally disabled tests
-disabledtests("$TESTDIR/DISABLED");
-
-# locally disabled tests, ignored by git etc
-disabledtests("$TESTDIR/DISABLED.local");
-
 #######################################################################
 # If 'all' tests are requested, find out all test numbers
 #