1berry/MariaDB-server
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fixed test failure that ended with "There is no group named 'mysqld.10' that can be used to resolve 'port' for test"

Browse Source 
mysql-test/lib/My/ConfigFactory.pm:
  Added test name to error output
mysql-test/mysql-test-run.pl:
  Added testname as part of ConfigFactory for better error message
mysql-test/suite/rpl/t/rpl_test_framework.cnf:
  Added space after # to ensure that ConfigFactory treat comments as comments
This commit is contained in:
Michael Widenius 2011-02-23 02:00:30 +02:00
parent 988d263e25
commit e079f3c84a
3 changed files with 22 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
mysql-test/lib/My/ConfigFactory.pm
View File

@ -358,7 +358,7 @@ sub post_check_client_group {
if (! defined $option){
#print $config;
croak "Could not get value for '$name_from'";
croak "Could not get value for '$name_from' for test $self->{testname}";
}
$config->insert($client_group_name, $name_to, $option->value())
}
@ -446,7 +446,7 @@ sub resolve_at_variable {
my $from_group= $config->group($group_name)
or croak "There is no group named '$group_name' that ",
"can be used to resolve '$option_name'";
"can be used to resolve '$option_name' for test '$self->{testname}'";
my $from= $from_group->value($option_name);
$config->insert($group->name(), $option->name(), $from)
@ -623,6 +623,7 @@ sub new_config {
HOSTS => $hosts,
NEXT_HOST => 0,
SERVER_ID => 1,
testname => $args->{testname},
}, $class;
# add auto-options

1
mysql-test/mysql-test-run.pl
View File

@ -3826,6 +3826,7 @@ sub run_testcase ($$) {
# Generate new config file from template
$config= My::ConfigFactory->new_config
( {
testname => $tinfo->{name},
basedir => $basedir,
testdir => $glob_mysql_test_dir,
template_path => $tinfo->{template_path},