migrate UpdateAll action to use Updater class and remove hosts_file.rb

Author: pbitty

lib/vagrant-hostmanager/action/update_all.rb

@@ -1,18 +1,18 @@
-require 'vagrant-hostmanager/hosts_file'
+require 'vagrant-hostmanager/hosts_file/updater'
 require 'vagrant-hostmanager/util'
 
 module VagrantPlugins
   module HostManager
     module Action
       class UpdateAll
-        include HostsFile
 
         def initialize(app, env)
           @app = app
           @machine = env[:machine]
           @global_env = @machine.env
           @provider = @machine.provider_name
           @config = Util.get_config(@global_env)
+          @updater = HostsFile::Updater.new(@global_env, @provider)
           @logger = Log4r::Logger.new('vagrant::hostmanager::update_all')
         end
 
@@ -31,14 +31,14 @@ def call(env)
           @global_env.active_machines.each do |name, p|
             if p == @provider
               machine = @global_env.machine(name, p)
-              update_guest(machine)
+              @updater.update_guest(machine)
             end
           end
 
           # update /etc/hosts files on host if enabled
           if @machine.config.hostmanager.manage_host?
             env[:ui].info I18n.t('vagrant_hostmanager.action.update_host')
-            update_host
+            @updater.update_host
           end
         end
       end

lib/vagrant-hostmanager/hosts_file.rb

@@ -92,9 +92,9 @@ def update_content(file_content, resolving_machine, include_id)
         def get_hosts_file_entry(machine, resolving_machine)
           ip = get_ip_address(machine, resolving_machine)
           host = machine.config.vm.hostname || machine.name
-          aliases = machine.config.hostmanager.aliases.join(' ').chomp
+          aliases = machine.config.hostmanager.aliases
           if ip != nil
-            "#{ip}\t#{host} #{aliases}\n"
+            "#{ip}\t#{host}\n" + aliases.map{|a| "#{ip}\t#{a}"}.join("\n") + "\n"
           end
         end