Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
fix (or at least silence) failing auto::gc tests, set gms as configur…
…e-time default
  • Loading branch information
cotto committed Feb 17, 2011
1 parent 8fcbee3 commit a81c3fc
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 5 deletions.
4 changes: 2 additions & 2 deletions config/auto/gc.pm
Expand Up @@ -38,7 +38,7 @@ sub _init {
sub runstep {
my ( $self, $conf ) = @_;

my $gc = $conf->options->get('gc') || '';
my $gc = $conf->options->get('gc') || 'gms';
$conf->debug(" ($gc) ");

my @known_gcs = qw<gms ms ms2 inf>;
Expand All @@ -51,7 +51,7 @@ sub runstep {
$self->set_result($gc);
}
else {
$conf->data->set(gc_flag => '');
$conf->data->set(gc_flag => '-DPARROT_GC_DEFAULT_TYPE=GMS');
$self->set_result('gms');
}

Expand Down
5 changes: 2 additions & 3 deletions t/steps/auto/gc-01.t
Expand Up @@ -41,9 +41,8 @@ my $step = test_step_constructor_and_description($conf);
\$stdout,
);
ok( $ret, "runstep() returned true value" );
note $stdout;
like( $stdout, qr/\(ms2\)/, "Got expected verbose output");
is($conf->data->get('gc_flag'), q{-DPARROT_GC_DEFAULT_TYPE=MS2},
like($stdout, qr/\(gms\)/, "Got expected verbose output");
like($conf->data->get('gc_flag'), qr{-DPARROT_GC_DEFAULT_TYPE=},
"Got expected value for 'gc_flag'");
}

Expand Down

0 comments on commit a81c3fc

Please sign in to comment.