Add perlcritic test and make code comply

[gruntmaster-daemon.git] / lib / Gruntmaster / Daemon.pm

diff --git a/lib/Gruntmaster/Daemon.pm b/lib/Gruntmaster/Daemon.pm
index 1d1ca01f9544c895168dd27e7ed356c3af2cebaf..64f82312df71b5edde5d6e4dde3c0003e76be960 100644 (file)
--- a/lib/Gruntmaster/Daemon.pm
+++ b/lib/Gruntmaster/Daemon.pm
@@ -40,8 +40,7 @@ my $db;
 sub safe_can_nodie {
        my ($type, $sub, $name) = @_;
 
 sub safe_can_nodie {
        my ($type, $sub, $name) = @_;
 

-       return unless $name =~ /^\w+$/;
-       no strict 'refs';
+       return unless $name =~ /^\w+$/s;

        my $pkg = __PACKAGE__ . "::${type}::${name}";
        eval "require $pkg" or get_logger->warn("Error while requiring $pkg: $@");
        $pkg->can($sub);
        my $pkg = __PACKAGE__ . "::${type}::${name}";
        eval "require $pkg" or get_logger->warn("Error while requiring $pkg: $@");
        $pkg->can($sub);


@@ -87,7 +86,7 @@ sub process{
                prepare_files $meta;
                chomp $errors;
 
                prepare_files $meta;
                chomp $errors;
 

-               my ($files, $generator, $runner, $judge, $testcnt) = map { $meta->{$_} or die "Required parameter missing: $_"} qw/files generator runner judge testcnt/;
+               my ($files, $generator, $runner, $judge, $testcnt) = map { $meta->{$_} or die "Required parameter missing: $_\n"} qw/files generator runner judge testcnt/;

 
                $generator = safe_can Generator => generate => $generator;
                $runner = safe_can Runner => run => $runner;
 
                $generator = safe_can Generator => generate => $generator;
                $runner = safe_can Runner => run => $runner;


@@ -122,14 +121,14 @@ sub process{
                $meta->{$_} = $results{$_} for keys %results;
                $meta->{results} = \@full_results
        } catch {
                $meta->{$_} = $results{$_} for keys %results;
                $meta->{results} = \@full_results
        } catch {

-               s,(.*) at .*,$1,;
+               s/(.*) at .*/$1/s;

                chomp;
                $meta->{result} = -1;
                $meta->{result_text} = $_;
        };
        stopvms;
 
                chomp;
                $meta->{result} = -1;
                $meta->{result_text} = $_;
        };
        stopvms;
 

-       get_logger->info("Job result: " . $meta->{result_text});
+       get_logger->info('Job result: ' . $meta->{result_text});

        return unless $jobr;
        $jobr->update({
                result => $meta->{result},
        return unless $jobr;
        $jobr->update({
                result => $meta->{result},