removed unrequired explicit 'list' data levels
Robert 'phaylon' Sedlacek [Mon, 18 Jun 2012 22:22:09 +0000 (22:22 +0000)]
lib/System/Introspector/Probe/Puppet.pm
lib/System/Introspector/Probe/Repositories/Git.pm
lib/System/Introspector/Probe/Users.pm
t/puppet.t
t/users.t

index 811dea0..4c3fde5 100644 (file)
@@ -29,11 +29,11 @@ sub _gather_resources {
     return transform_exceptions {
         my @lines = output_from_file $self->resources_file;
         chomp @lines;
-        return { list => [ map {
+        return [ map {
             m{^(\w+)\[(.*)\]$}
                 ? [$1, $2]
                 : [__error__ => $_];
-        } @lines ] };
+        } @lines ];
     };
 }
 
@@ -42,7 +42,7 @@ sub _gather_classes {
     return transform_exceptions {
         my @lines = output_from_file $self->classes_file;
         chomp @lines;
-        return { list => \@lines };
+        return \@lines;
     };
 }
 
index c416c07..610372d 100644 (file)
@@ -92,7 +92,7 @@ sub _find_changes {
     local $ENV{GIT_DIR} = $dir;
     my @lines = lines_from_command
         ['git', 'diff', '--name-only', $local, $remote];
-    return { list => \@lines };
+    return \@lines;
 }
 
 sub _gather_git_config {
index 6c1a82a..0b4047a 100644 (file)
@@ -78,7 +78,7 @@ sub _gather_ssh_keys {
 sub _gather_user_groups {
     my ($self, $user) = @_;
     my $groups = output_from_command [groups => $user];
-    return { list => [split m{\s+}, $groups] };
+    return [split m{\s+}, $groups];
 }
 
 sub _deparse_htpasswd_line {
index 946ae6d..1e49879 100644 (file)
@@ -10,10 +10,10 @@ my $probe = System::Introspector::Probe::Puppet->new(
 );
 
 my $data = $probe->gather;
-is_deeply $data->{classes}{list},
+is_deeply $data->{classes},
     [qw( user::foo settings user::foo user::bar )],
     'classes parsing';
-is_deeply $data->{resources}{list},
+is_deeply $data->{resources},
     [[user => 'foo'],
      [exec => 'ls -lha'],
      [file => '/home/foo/quux'],
index ccd6826..efaba05 100644 (file)
--- a/t/users.t
+++ b/t/users.t
@@ -44,9 +44,11 @@ do {
         },
     }, 'ssh key data';
 
-    is_deeply $user->{groups}, {
-        list => [qw( testfoo_group_A testfoo_group_B testfoo_group_C )],
-    }, 'groups list';
+    is_deeply $user->{groups}, [qw(
+        testfoo_group_A
+        testfoo_group_B
+        testfoo_group_C
+    )], 'groups list';
 
     my $tab = $user->{crontab};
     ok $tab, 'got crontab results';