Return from function 'collect_one_test_case' as sson it's detected that testcase should be skipped

parent 40a655b1
...@@ -390,6 +390,7 @@ sub collect_one_test_case($$$$$$$) { ...@@ -390,6 +390,7 @@ sub collect_one_test_case($$$$$$$) {
{ {
$tinfo->{'skip'}= 1; $tinfo->{'skip'}= 1;
$tinfo->{'comment'}= "No tests with sh scripts on Windows"; $tinfo->{'comment'}= "No tests with sh scripts on Windows";
return;
} }
else else
{ {
...@@ -404,6 +405,7 @@ sub collect_one_test_case($$$$$$$) { ...@@ -404,6 +405,7 @@ sub collect_one_test_case($$$$$$$) {
{ {
$tinfo->{'skip'}= 1; $tinfo->{'skip'}= 1;
$tinfo->{'comment'}= "No tests with sh scripts on Windows"; $tinfo->{'comment'}= "No tests with sh scripts on Windows";
return;
} }
else else
{ {
...@@ -448,6 +450,7 @@ sub collect_one_test_case($$$$$$$) { ...@@ -448,6 +450,7 @@ sub collect_one_test_case($$$$$$$) {
{ {
$tinfo->{'skip'}= 1; $tinfo->{'skip'}= 1;
$tinfo->{'disable'}= 1; # Sub type of 'skip' $tinfo->{'disable'}= 1; # Sub type of 'skip'
return;
} }
} }
...@@ -457,16 +460,19 @@ sub collect_one_test_case($$$$$$$) { ...@@ -457,16 +460,19 @@ sub collect_one_test_case($$$$$$$) {
{ {
$tinfo->{'skip'}= 1; $tinfo->{'skip'}= 1;
$tinfo->{'comment'}= "No IM with embedded server"; $tinfo->{'comment'}= "No IM with embedded server";
return;
} }
elsif ( $::opt_ps_protocol ) elsif ( $::opt_ps_protocol )
{ {
$tinfo->{'skip'}= 1; $tinfo->{'skip'}= 1;
$tinfo->{'comment'}= "No IM with --ps-protocol"; $tinfo->{'comment'}= "No IM with --ps-protocol";
return;
} }
elsif ( $::opt_skip_im ) elsif ( $::opt_skip_im )
{ {
$tinfo->{'skip'}= 1; $tinfo->{'skip'}= 1;
$tinfo->{'comment'}= "No IM tests(--skip-im)"; $tinfo->{'comment'}= "No IM tests(--skip-im)";
return;
} }
} }
else else
...@@ -477,18 +483,21 @@ sub collect_one_test_case($$$$$$$) { ...@@ -477,18 +483,21 @@ sub collect_one_test_case($$$$$$$) {
{ {
$tinfo->{'skip'}= 1; $tinfo->{'skip'}= 1;
$tinfo->{'comment'}= "Test need 'big-test' option"; $tinfo->{'comment'}= "Test need 'big-test' option";
return;
} }
if ( $tinfo->{'ndb_extra'} and ! $::opt_ndb_extra_test ) if ( $tinfo->{'ndb_extra'} and ! $::opt_ndb_extra_test )
{ {
$tinfo->{'skip'}= 1; $tinfo->{'skip'}= 1;
$tinfo->{'comment'}= "Test need 'ndb_extra' option"; $tinfo->{'comment'}= "Test need 'ndb_extra' option";
return;
} }
if ( $tinfo->{'require_manager'} ) if ( $tinfo->{'require_manager'} )
{ {
$tinfo->{'skip'}= 1; $tinfo->{'skip'}= 1;
$tinfo->{'comment'}= "Test need the _old_ manager(to be removed)"; $tinfo->{'comment'}= "Test need the _old_ manager(to be removed)";
return;
} }
if ( defined $tinfo->{'binlog_format'} and if ( defined $tinfo->{'binlog_format'} and
...@@ -496,12 +505,14 @@ sub collect_one_test_case($$$$$$$) { ...@@ -496,12 +505,14 @@ sub collect_one_test_case($$$$$$$) {
{ {
$tinfo->{'skip'}= 1; $tinfo->{'skip'}= 1;
$tinfo->{'comment'}= "Not running with binlog format '$tinfo->{'binlog_format'}'"; $tinfo->{'comment'}= "Not running with binlog format '$tinfo->{'binlog_format'}'";
return;
} }
if ( $tinfo->{'need_debug'} && ! $::debug_compiled_binaries ) if ( $tinfo->{'need_debug'} && ! $::debug_compiled_binaries )
{ {
$tinfo->{'skip'}= 1; $tinfo->{'skip'}= 1;
$tinfo->{'comment'}= "Test need debug binaries"; $tinfo->{'comment'}= "Test need debug binaries";
return;
} }
} }
...@@ -512,6 +523,7 @@ sub collect_one_test_case($$$$$$$) { ...@@ -512,6 +523,7 @@ sub collect_one_test_case($$$$$$$) {
{ {
$tinfo->{'skip'}= 1; $tinfo->{'skip'}= 1;
$tinfo->{'comment'}= "Can't restart a running server"; $tinfo->{'comment'}= "Can't restart a running server";
return;
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment