← Back to team overview

maria-developers team mailing list archive

[Branch ~maria-captains/maria/5.1] Rev 2741: MBug#443014: Too many 'skipped' messages in mysql-test-run

 

------------------------------------------------------------
revno: 2741
committer: knielsen@xxxxxxxxxxxxxxx
branch nick: work-5.1
timestamp: Mon 2009-10-05 14:26:57 +0200
message:
  MBug#443014: Too many 'skipped' messages in mysql-test-run
  
  Remove mysql-test-run.pl unnecessary and confusing 'skipped' messages.
  
   - In mysql-test-run.pl, we auto-generate combinations of replication
     tests. But this sometimes generates combinations that are meaningless,
     like running a test that requires row-based replication with
     statement-based. These superfluous combinationes should not be
     reported as skipped, they should just be deleted.
  
   - Remove ndb suites from default suites, as we do not support NDB
     in MariaDB.
  
  Keep skip messages resulting from running mysql-test-run.pl in special
  ways, eg. --mysqld=--binlog-format=statement.
modified:
  mysql-test/lib/mtr_cases.pm
  mysql-test/mysql-test-run.pl


--
lp:maria
https://code.launchpad.net/~maria-captains/maria/5.1

Your team Maria developers is subscribed to branch lp:maria.
To unsubscribe from this branch go to https://code.launchpad.net/~maria-captains/maria/5.1/+edit-subscription.
=== modified file 'mysql-test/lib/mtr_cases.pm'
--- mysql-test/lib/mtr_cases.pm	2009-09-07 20:50:10 +0000
+++ mysql-test/lib/mtr_cases.pm	2009-10-05 12:26:57 +0000
@@ -560,8 +560,12 @@
 sub optimize_cases {
   my ($cases)= @_;
 
+  my @new_cases= ();
+
   foreach my $tinfo ( @$cases )
   {
+    push @new_cases, $tinfo;
+
     # Skip processing if already marked as skipped
     next if $tinfo->{skip};
 
@@ -615,6 +619,13 @@
 	  $tinfo->{'skip'}= 1;
 	  $tinfo->{'comment'}=
 	    "Doesn't support --binlog-format='$test_binlog_format'";
+          # This test was added as a replication combination, but it is not
+          # actually ever possible to run it, as it is not made for this
+          # combination.
+          # So delete it from the list, rather than confuse the user with a
+          # message that this test is skipped (it is not really, just run
+          # with other combinations).
+          pop(@new_cases);
 	  next;
 	}
       }
@@ -683,6 +694,7 @@
       }
     }
   }
+  @$cases= @new_cases;
 }
 
 

=== modified file 'mysql-test/mysql-test-run.pl'
--- mysql-test/mysql-test-run.pl	2009-09-26 02:25:56 +0000
+++ mysql-test/mysql-test-run.pl	2009-10-05 12:26:57 +0000
@@ -126,13 +126,13 @@
 # executables will be used by the test suite.
 our $opt_vs_config = $ENV{'MTR_VS_CONFIG'};
 
-my $DEFAULT_SUITES= "binlog,federated,main,maria,ndb,rpl,rpl_ndb,innodb";
+my $DEFAULT_SUITES= "binlog,federated,main,maria,rpl,innodb";
 my $opt_suites;
 
 our $opt_usage;
 our $opt_list_options;
 our $opt_suites;
-our $opt_suites_default= "main,backup,backup_engines,binlog,rpl,rpl_ndb,ndb"; # Default suites to run
+our $opt_suites_default= "main,backup,backup_engines,binlog,rpl"; # Default suites to run
 our $opt_script_debug= 0;  # Script debugging, enable with --script-debug
 our $opt_verbose= 0;  # Verbose output, enable with --verbose
 our $exe_mysql;