Simplify "all_vhosts" in rabbitmq_queue provider

This is more idiomatic ruby, avoids creating array only to throw it away, and consistent with a method below.
voxpupuli · May 23, 2017 · fd70605 · fd70605
1 parent 2000ce7
commit fd70605
Showing 1 changed file with 1 addition and 5 deletions.
diff --git a/lib/puppet/provider/rabbitmq_queue/rabbitmqadmin.rb b/lib/puppet/provider/rabbitmq_queue/rabbitmqadmin.rb
@@ -24,11 +24,7 @@ def should_vhost
   end
 
   def self.all_vhosts
-    vhosts = []
-    rabbitmqctl('list_vhosts', '-q').split(/\n/).collect do |vhost|
-        vhosts.push(vhost)
-    end
-    vhosts
+    rabbitmqctl('list_vhosts', '-q').split(/\n/)
   end
 
   def self.all_queues(vhost)