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

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/updater.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