amassivus/clusterssh
1
0
Fork 0
mirror of https://github.com/duncs/clusterssh.git synced 2025-04-20 16:59:06 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Improve test coverage

Browse source
This commit is contained in:
Duncan Ferguson 2017-12-27 16:35:35 +00:00
parent c807b52129
commit f27c42c795
2 changed files with 23 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
lib/App/ClusterSSH.pm
View file

@ -261,7 +261,7 @@ sub resolve_names(@) {
if ( $self->config->{unique_servers} ) {
$self->debug( 3, 'removing duplicate server names' );
@servers = remove_repeated_servers(@servers);
@servers = $self->remove_repeated_servers(@servers);
}
$self->debug( 3, 'leaving with ', $_ ) foreach (@servers);
@ -270,7 +270,8 @@ sub resolve_names(@) {
}
sub remove_repeated_servers {
my %all = ();
my $self = shift;
my %all = ();
@all{@_} = 1;
return ( keys %all );
}

20
t/80clusterssh.t
View file

@ -29,4 +29,24 @@ $app = App::ClusterSSH->new();
isa_ok( $app, 'App::ClusterSSH' );
isa_ok( $app->config, 'App::ClusterSSH::Config' );
for my $submod (qw/ cluster helper options window /) {
trap {
$app->$submod;
};
$trap->quiet("$submod loaded okay");
}
trap {
$app->exit_prog;
};
$trap->quiet("No errors from exit_prog call");
my @provided = (qw/ one one one two two three four four four /);
my @expected = sort (qw/ one two three four /);
my @got;
trap {
@got = sort $app->remove_repeated_servers(@provided);
};
is_deeply( \@got, \@expected, "Repeated servers removed okay" );
done_testing();