- moved the parenthesis before map to the end so that the map works on

  the array and then the set is created from that, instead of map
  coercing the set to an array, which breaks things that call Set#add
- switching back to the previous more concise expression form

lib/puppet/provider/rabbitmq_user/rabbitmqctl.rb

@@ -111,12 +111,6 @@ def get_user_tags
     match = rabbitmqctl('-q', 'list_users').split(/\n/).collect do |line|
       line.match(/^#{Regexp.escape(resource[:name])}\s+\[(.*?)\]/)
     end.compact.first
-    if match
-      # RabbitMQ 3.3.5 and up separate tags with ', '
-      # 3.3.4 just separates with space
-      # this splits by space and then strips word-final commas
-      # (if you're ending a valid tag name with a comma, why)
-      Set.new(match[1].split(' ')).map{|x| x.gsub(/,$/, '')}
-    end
+    Set.new(match[1].split(' ').map{|x| x.gsub(/,$/, '')}) if match
   end
 end