vagrant-proxyconf 0.5.3 → 0.6.0.rc1

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    MGJkNWQ0YjdiMjA4ZDQ0YmQ3MzM3ZDFjODM3MmEwODk2YmI2M2ZhMg==
+    Y2NiYWE0Y2MxNWIxYTY0ZjQ1MDIyNjQ1MTQ3NDkxYWM0MDYwZjk1OA==
   data.tar.gz: !binary |-
-    MGE0Y2VkOWUzOWNjZDg0NDQxYTU0MmZmMDdhY2Y2ZTZkZmRiMzY3OQ==
+    MTNjNmNmYjZhMDQ1MGNlYmU4YmY0MTc5NjAyZWM3ZjY1YzMyNjY0Zg==
 SHA512:
   metadata.gz: !binary |-
-    YzA5OWYxNTZmYWZmYzdiNzM0Y2YwMDVmZDMyYzM3YTc3YWY0MTUyMmZkNDYx
-    ZTc3YTJlNjc1YTE1Njk2ZTk1ZmEzNGU1Mjc3ZTA1MDAxNGQyZGVkNDNkZDU1
-    Mjc5YzQ4NzY2ZjAwZmQxMjYzZjVkYTgxZGUwNWU0YmMyMzE3N2U=
+    MDllNDZmOGNlNzU0ZDJlZmJjYjdhMWYwYjgzOTM0OTg4NWUzODExYTM5ZDkz
+    NmI5YTkzYzU3MzAxNGU5MDFjY2QwMzUwNDYxNDVkYjA2ODZkMWRhODUyMWQz
+    ZDUxZTk3YWFlMzg3MDJhODIzOTZmYTJiZjMzYzQ1ZDAxNjBlM2Y=
   data.tar.gz: !binary |-
-    YjRkN2YyYzI3ZDM0NDg5NTQ3ZDBkMTZkOTQzYWU2MDY3NTA4YjQzODYzMWEz
-    YTA2ZjEzZGZiNDk3MmJhNDA0OWE3MDAxNzc5YTc5YTVkN2VlYWIyM2U5ZjEx
-    OTAwN2IwYjMyN2U1YTM1YjQ2OTI0YjQyNmU3ZTU2ZjE4MjViNzM=
+    ODUwZjNlZTQ1MzZhNmFlMmE1NTYzMTAwN2M2NzQxNzRiYzU2ZGQzYjRiY2Ji
+    MjU1ZjllMGJhYWQ3MDdiODg4YWQwOWIzNWE3ODY3MjBjOTQ2OWQyNTBmMTE4
+    YTQ2MmFiN2ExMWM1MTU1ZTdhMjVhNjdlMjlhNWYwYjY5Nzg1OWU=

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+# 0.6.0.rc1 / 2013-10-10
+
+- Add support for the [vagrant-vbguest](https://github.com/dotless-de/vagrant-vbguest) plugin ([GH-30][])
+
 # 0.5.3 / 2013-09-30
 
 - Compatibility with vagrant-aws v0.4.0 ([GH-28][])
@@ -77,3 +81,4 @@
 [GH-27]: https://github.com/tmatilai/vagrant-proxyconf/issues/27 "Issue 27"
 [GH-28]: https://github.com/tmatilai/vagrant-proxyconf/issues/28 "Issue 28"
 [GH-29]: https://github.com/tmatilai/vagrant-proxyconf/issues/29 "Issue 29"
+[GH-30]: https://github.com/tmatilai/vagrant-proxyconf/issues/30 "Issue 30"

data/Gemfile

@@ -7,7 +7,7 @@ gem 'rake'
 gem 'rspec', '~> 2.11'
 gem 'simplecov', '~> 0.7', :require => false
 gem 'tailor', '~> 1.2'
-gem 'vagrant', github: 'mitchellh/vagrant', ref: 'v1.3.3'
+gem 'vagrant', github: 'mitchellh/vagrant', ref: 'v1.3.4'
 
 group :development do
   gem 'guard-rspec'

data/README.md

@@ -43,8 +43,7 @@ end
 
 This plugin requires Vagrant 1.2 or newer ([downloads](http://downloads.vagrantup.com/)).
 
-The plugin is supposed to be compatible with all Vagrant providers. Please file an [issue](https://github.com/tmatilai/vagrant-proxyconf/issues) if this is not the case.
-The following providers are confirmed to work:
+The plugin is supposed to be compatible with all Vagrant providers and other plugins. Please file an [issue](https://github.com/tmatilai/vagrant-proxyconf/issues) if this is not the case. The following providers are confirmed to work:
 [AWS](https://github.com/mitchellh/vagrant-aws),
 [Digital Ocean](https://github.com/smdahlen/vagrant-digitalocean),
 [VirtualBox](http://docs.vagrantup.com/v2/virtualbox),

data/development/Vagrantfile.example

@@ -15,6 +15,6 @@ Vagrant.configure('2') do |config|
 
   config.vm.box = BOX
   if BOX == 'ubuntu-12.04'
-    ubuntu.vm.box_url = 'https://opscode-vm-bento.s3.amazonaws.com/vagrant/opscode_ubuntu-12.04_provisionerless.box'
+    config.vm.box_url = 'https://opscode-vm-bento.s3.amazonaws.com/vagrant/opscode_ubuntu-12.04_provisionerless.box'
   end
 end

data/lib/vagrant-proxyconf/action.rb

@@ -0,0 +1,31 @@
+require_relative 'action/configure_apt_proxy'
+require_relative 'action/configure_chef_proxy'
+require_relative 'action/configure_env_proxy'
+require_relative 'action/only_once'
+
+module VagrantPlugins
+  module ProxyConf
+    # Middleware stack builders
+    class Action
+      # Returns an action middleware stack that configures the VM
+      #
+      # @param opts [Hash] the options to be passed to {OnlyOnce}
+      # @option (see OnlyOnce#initialize)
+      def self.configure(opts = {})
+        Vagrant::Action::Builder.build(OnlyOnce, opts, &config_actions)
+      end
+
+      private
+
+      # @return [Proc] the block that adds config actions to the specified
+      #   middleware builder
+      def self.config_actions
+        @actions ||= Proc.new do |builder|
+          builder.use ConfigureAptProxy
+          builder.use ConfigureChefProxy
+          builder.use ConfigureEnvProxy
+        end
+      end
+    end
+  end
+end

data/lib/vagrant-proxyconf/action/base.rb

@@ -1,33 +1,30 @@
-require 'log4r'
 require 'tempfile'
 require 'vagrant'
+require_relative '../logger'
 
 module VagrantPlugins
   module ProxyConf
     class Action
       # Base class for proxy configuration Actions
       class Base
-        attr_reader :logger
-
         def initialize(app, env)
-          @app    = app
-          @logger = Log4r::Logger.new('vagrant::proxyconf')
+          @app = app
         end
 
         def call(env)
-          @app.call env
-
           machine = env[:machine]
           config  = config(machine)
 
           if !config.enabled?
-            logger.debug I18n.t("vagrant_proxyconf.#{config_name}.not_enabled")
+            logger.info I18n.t("vagrant_proxyconf.#{config_name}.not_enabled")
           elsif !supported?(machine)
             env[:ui].info I18n.t("vagrant_proxyconf.#{config_name}.not_supported")
           else
             env[:ui].info I18n.t("vagrant_proxyconf.#{config_name}.configuring")
             configure_machine(machine, config)
           end
+
+          @app.call env
         end
 
         # @return [String] the name of the configuration section
@@ -37,6 +34,11 @@ module VagrantPlugins
 
         private
 
+        # @return [Log4r::Logger]
+        def logger
+          ProxyConf.logger
+        end
+
         # @return [Vagrant::Plugin::V2::Config] the configuration
         def config(machine)
           config = machine.config.public_send(config_name.to_sym)

data/lib/vagrant-proxyconf/action/configure_chef_proxy.rb

@@ -1,33 +1,37 @@
+require_relative '../logger'
+
 module VagrantPlugins
   module ProxyConf
     class Action
       # Action for configuring Chef provisioners
       class ConfigureChefProxy
-        attr_reader :logger
-
         def initialize(app, env)
-          @app    = app
-          @logger = Log4r::Logger.new('vagrant::proxyconf')
+          @app = app
         end
 
         def call(env)
-          @app.call env
-
           machine = env[:machine]
           config  = config(machine)
 
           if chef_provisioners(machine).empty?
-            logger.debug I18n.t("vagrant_proxyconf.chef_proxy.no_provisioners")
+            logger.info I18n.t("vagrant_proxyconf.chef_proxy.no_provisioners")
           elsif !config.enabled?
-            logger.debug I18n.t("vagrant_proxyconf.chef_proxy.not_enabled")
+            logger.info I18n.t("vagrant_proxyconf.chef_proxy.not_enabled")
           else
             env[:ui].info I18n.t("vagrant_proxyconf.chef_proxy.configuring")
             configure_chef_provisioners(machine, config)
           end
+
+          @app.call env
         end
 
         private
 
+        # @return [Log4r::Logger]
+        def logger
+          ProxyConf.logger
+        end
+
         # @return [Config::Proxy] the `config.proxy` configuration
         def config(machine)
           config = machine.config.proxy

data/lib/vagrant-proxyconf/action/only_once.rb

@@ -0,0 +1,61 @@
+require_relative '../logger'
+
+module VagrantPlugins
+  module ProxyConf
+    class Action
+      # A middleware class that builds and runs the stack based on the
+      # specified block, but only once.
+      class OnlyOnce
+        # @param opts [Hash] the options
+        # @option opts [Boolean] :before (false) should the block be called
+        #   before (instead of after) passing control to the next middleware
+        def initialize(app, env, opts = {}, &block)
+          raise ArgumentError, "A block must be given to OnlyOnce" if !block
+
+          @app    = app
+          @before = opts[:before]
+          @block  = block
+        end
+
+        def call(env)
+          @app.call(env) if !@before
+
+          if env[@block]
+            logger.debug "Skipping repeated '#{@block}' stack"
+          else
+            logger.debug "'#{@block}' stack invoked first time"
+            env[@block] = true
+
+            new_env = build_and_run_block(env)
+            env.merge!(new_env)
+          end
+
+          @app.call(env) if @before
+        end
+
+        def recover(env)
+          # Call back into our compiled application and recover it.
+          @child_app.recover(env) if @child_app
+        end
+
+        private
+
+        # @return [Log4r::Logger]
+        def logger
+          ProxyConf.logger
+        end
+
+        # Creates and runs a Builder based on the block given in initializer.
+        #
+        # @param env [Hash] the current environment
+        # @return [Hash] the new environment
+        def build_and_run_block(env)
+          builder = Vagrant::Action::Builder.new
+          @block.call(builder)
+          @child_app = builder.to_app(env)
+          Vagrant::Action::Runner.new.run(@child_app, env)
+        end
+      end
+    end
+  end
+end

data/lib/vagrant-proxyconf/logger.rb

@@ -0,0 +1,11 @@
+require 'log4r'
+
+module VagrantPlugins
+  # Base module for Vagrant Proxyconf plugin
+  module ProxyConf
+    # @return [Log4r::Logger] the logger instance for this plugin
+    def self.logger
+      @logger ||= Log4r::Logger.new('vagrant::proxyconf')
+    end
+  end
+end

data/lib/vagrant-proxyconf/plugin.rb

@@ -9,6 +9,9 @@ module VagrantPlugins
       # The minimum compatible Vagrant version
       MIN_VAGRANT_VERSION = '1.2.0'
 
+      # A list of plugins whose action classes we hook to if installed
+      OPTIONAL_PLUGIN_DEPENDENCIES = %w[vagrant-aws vagrant-omnibus vagrant-vbguest]
+
       # Verifies that the Vagrant version fulfills the requirements
       #
       # @raise [VagrantPlugins::ProxyConf::VagrantVersionError] if this plugin
@@ -27,8 +30,25 @@ module VagrantPlugins
         I18n.reload!
       end
 
+      # Ensures a dependent plugin is loaded before us if it is installed.
+      # Ignores {Vagrant::Errors::PluginLoadError} but passes other exceptions.
+      #
+      # @param plugin [String] the plugin name
+      def self.load_optional_dependency(plugin)
+        begin
+          Vagrant.require_plugin plugin
+        rescue Vagrant::Errors::PluginLoadError; end
+      end
+
+      # Loads the plugins to ensure their action hooks are registered before us.
+      # Uses alphabetical order to not change the default behaviour otherwise.
+      def self.load_optional_dependencies
+        OPTIONAL_PLUGIN_DEPENDENCIES.sort.each { |plugin| load_optional_dependency plugin }
+      end
+
       setup_i18n
       check_vagrant_version!
+      load_optional_dependencies
 
       name 'vagrant-proxyconf'
 
@@ -58,29 +78,25 @@ module VagrantPlugins
       end
 
       action_hook 'proxyconf_configure' do |hook|
-        if defined? VagrantPlugins::Omnibus::Action::InstallChef
-          # configure the proxies before vagrant-omnibus
-          register_hooks(hook, VagrantPlugins::Omnibus::Action::InstallChef)
-        else
-          register_hooks(hook, Vagrant::Action::Builtin::Provision)
-
-          # vagrant-aws < 0.4.0 uses a non-standard provision action
-          if defined? VagrantPlugins::AWS::Action::TimedProvision
-            register_hooks(hook, VagrantPlugins::AWS::Action::TimedProvision)
-          end
-        end
-      end
+        require_relative 'action'
+
+        # the standard provision action
+        hook.after Vagrant::Action::Builtin::Provision, Action.configure
 
-      private
+        # vagrant-aws < 0.4.0 uses a non-standard provision action
+        if defined?(VagrantPlugins::AWS::Action::TimedProvision)
+          hook.after VagrantPlugins::AWS::Action::TimedProvision, Action.configure
+        end
 
-      def self.register_hooks(hook, provision_action)
-        require_relative 'action/configure_apt_proxy'
-        require_relative 'action/configure_chef_proxy'
-        require_relative 'action/configure_env_proxy'
+        # configure the proxies before vagrant-omnibus
+        if defined?(VagrantPlugins::Omnibus::Action::InstallChef)
+          hook.after VagrantPlugins::Omnibus::Action::InstallChef, Action.configure
+        end
 
-        hook.after provision_action, Action::ConfigureAptProxy
-        hook.after provision_action, Action::ConfigureChefProxy
-        hook.after provision_action, Action::ConfigureEnvProxy
+        # configure the proxies before vagrant-vbguest
+        if defined?(VagrantVbguest::Middleware)
+          hook.before VagrantVbguest::Middleware, Action.configure(before: true)
+        end
       end
     end
   end

data/lib/vagrant-proxyconf/version.rb

@@ -1,5 +1,5 @@
 module VagrantPlugins
   module ProxyConf
-    VERSION = '0.5.3'
+    VERSION = '0.6.0.rc1'
   end
 end

data/locales/en.yml

@@ -10,7 +10,7 @@ en:
 
     chef_proxy:
       no_provisioners: |-
-        No Chef provisioners configured.
+        No Chef provisioners configured
       not_enabled: |-
         Default proxy not enabled or configured. Skipping Chef provisioner configuration.
       configuring: |-

data/spec/unit/vagrant-proxyconf/action/only_once_spec.rb

@@ -0,0 +1,136 @@
+require 'spec_helper'
+require 'vagrant-proxyconf/action/only_once'
+
+describe VagrantPlugins::ProxyConf::Action::OnlyOnce do
+  let(:app) { lambda { |env| } }
+  let(:env) { {} }
+
+  it "runs the stack first time" do
+    received  = nil
+    next_step = lambda { |env| received = "value" }
+
+    described_class.new(app, env) do |builder|
+      builder.use next_step
+    end.call({})
+
+    expect(received).to eq "value"
+  end
+
+  it "passes environment to the stack" do
+    received  = nil
+    next_step = lambda { |env| received = env[:foo] }
+
+    described_class.new(app, env) do |builder|
+      builder.use next_step
+    end.call({ foo: "value" })
+
+    expect(received).to eq "value"
+  end
+
+  it "updates the original environment" do
+    next_step = lambda { |env| env[:foo] = "value" }
+
+    described_class.new(app, env) do |builder|
+      builder.use next_step
+    end.call(env)
+
+    expect(env[:foo]).to eq "value"
+  end
+
+  it "runs the same stack only once" do
+    count     = 0
+    next_step = lambda { |env| count += 1 }
+    stack     = lambda { |builder| builder.use next_step }
+
+    described_class.new(app, env, &stack).tap do |instance|
+      instance.call(env)
+      instance.call(env)
+    end
+    described_class.new(app, env, &stack).call(env)
+
+    expect(count).to eq 1
+  end
+
+  it "runs different stacks" do
+    count     = 0
+    next_step = lambda { |env| count += 1 }
+    stack1    = lambda { |builder| builder.use next_step }
+    stack2    = lambda { |builder| builder.use next_step }
+
+    described_class.new(app, env, &stack1).call(env)
+    described_class.new(app, env, &stack2).call(env)
+
+    expect(count).to eq 2
+  end
+
+  it "calls the next app defore the block by default" do
+    received  = nil
+    next_app  = lambda { |env| env[:foo] = "value" }
+    next_step = lambda { |env| received = env[:foo] }
+
+    described_class.new(next_app, env) do |builder|
+      builder.use next_step
+    end.call({})
+
+    expect(received).to eq "value"
+  end
+
+  it "calls the next app after the block if specified" do
+    received  = nil
+    next_step = lambda { |env| env[:foo] = "value" }
+    next_app  = lambda { |env| received = env[:foo] }
+
+    described_class.new(next_app, env, before: true) do |builder|
+      builder.use next_step
+    end.call({})
+
+    expect(received).to eq "value"
+  end
+
+  it "calls the recover method for the sequence in an error" do
+    # Build the steps for the test
+    basic_step = Class.new do
+      def initialize(app, env)
+        @app = app
+        @env = env
+      end
+
+      def call(env)
+        @app.call(env)
+      end
+    end
+
+    step_a = Class.new(basic_step) do
+      def call(env)
+        env[:steps] << :call_A
+        super
+      end
+
+      def recover(env)
+        env[:steps] << :recover_A
+      end
+    end
+
+    step_b = Class.new(basic_step) do
+      def call(env)
+        env[:steps] << :call_B
+        super
+      end
+
+      def recover(env)
+        env[:steps] << :recover_B
+      end
+    end
+
+    instance = described_class.new(app, env) do |builder|
+      builder.use step_a
+      builder.use step_b
+    end
+
+    env[:steps] = []
+    instance.call(env)
+    instance.recover(env)
+
+    expect(env[:steps]).to eq [:call_A, :call_B, :recover_B, :recover_A]
+  end
+end

data/spec/unit/vagrant-proxyconf/action_spec.rb

@@ -0,0 +1,5 @@
+require 'spec_helper'
+require 'vagrant-proxyconf/action'
+
+describe VagrantPlugins::ProxyConf::Action do
+end

data/spec/unit/vagrant-proxyconf/logger_spec.rb

@@ -0,0 +1,16 @@
+require 'spec_helper'
+require 'vagrant-proxyconf/logger'
+
+describe VagrantPlugins::ProxyConf do
+
+  describe '.logger' do
+    subject { described_class.logger }
+
+    it { should be_a Log4r::Logger }
+
+    it "always returns the same instance" do
+      expect(subject).to be described_class.logger
+    end
+  end
+
+end

data/spec/unit/vagrant-proxyconf/plugin_spec.rb

@@ -3,7 +3,8 @@ require 'vagrant-proxyconf/plugin'
 
 describe VagrantPlugins::ProxyConf::Plugin do
 
-  describe ".check_vagrant_version!" do
+  describe '.check_vagrant_version!' do
+    subject { described_class.check_vagrant_version! }
     let(:min_vagrant_verision) { '1.2.3' }
     let(:err_msg) { /requires Vagrant #{min_vagrant_verision}/ }
 
@@ -16,27 +17,58 @@ describe VagrantPlugins::ProxyConf::Plugin do
     context "on too old Vagrant version" do
       let(:vagrant_version) { '1.1.5' }
       it "raises" do
-        expect { described_class.check_vagrant_version! }.to raise_error(err_msg)
+        expect { subject }.to raise_error(err_msg)
       end
       it "warns" do
         $stderr.should_receive(:puts).with(err_msg)
-        described_class.check_vagrant_version! rescue nil
+        subject rescue nil
       end
     end
 
     context "on exact required Vagrant version" do
       let(:vagrant_version) { min_vagrant_verision }
       it "does not raise" do
-        expect { described_class.check_vagrant_version! }.not_to raise_error
+        expect { subject }.not_to raise_error
       end
     end
 
     context "on newer Vagrant version" do
       let(:vagrant_version) { '1.3.5' }
       it "does not raise" do
-        expect { described_class.check_vagrant_version! }.not_to raise_error
+        expect { subject }.not_to raise_error
       end
     end
   end
 
+  describe '.load_optional_dependency' do
+    subject { described_class.load_optional_dependency(plugin_name) }
+    let(:plugin_name) { 'vagrant-foo' }
+
+    it "loads the specified plugin" do
+      expect(Vagrant).to receive(:require_plugin).with(plugin_name)
+      subject
+    end
+
+    it "ignores PluginLoadError" do
+      expect(Vagrant).to receive(:require_plugin).and_raise(Vagrant::Errors::PluginLoadError, plugin: plugin_name)
+      expect { subject }.not_to raise_error
+    end
+
+    it "won't ignore other error" do
+      expect(Vagrant).to receive(:require_plugin).and_raise(Vagrant::Errors::PluginLoadFailed, plugin: plugin_name)
+      expect { subject }.to raise_error(Vagrant::Errors::PluginLoadFailed)
+    end
+  end
+
+  describe '.load_optional_dependencies' do
+    let(:plugins) { %w[vagrant-foo vagrant-bar vagrant-baz] }
+    let(:loaded_plugins) { [] }
+
+    it "loads the plugins in alphabetical order" do
+      stub_const('VagrantPlugins::ProxyConf::Plugin::OPTIONAL_PLUGIN_DEPENDENCIES', plugins)
+      allow(described_class).to receive(:load_optional_dependency) { |plugin| loaded_plugins << plugin }
+      described_class.load_optional_dependencies
+      expect(loaded_plugins).to eq %w[vagrant-bar vagrant-baz vagrant-foo]
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: vagrant-proxyconf
 version: !ruby/object:Gem::Version
-  version: 0.5.3
+  version: 0.6.0.rc1
 platform: ruby
 authors:
 - Teemu Matilainen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-09-30 00:00:00.000000000 Z
+date: 2013-10-10 00:00:00.000000000 Z
 dependencies: []
 description: A Vagrant plugin that configures the virtual machine to use proxy servers
 email:
@@ -30,10 +30,12 @@ files:
 - development/README.md
 - development/Vagrantfile.example
 - lib/vagrant-proxyconf.rb
+- lib/vagrant-proxyconf/action.rb
 - lib/vagrant-proxyconf/action/base.rb
 - lib/vagrant-proxyconf/action/configure_apt_proxy.rb
 - lib/vagrant-proxyconf/action/configure_chef_proxy.rb
 - lib/vagrant-proxyconf/action/configure_env_proxy.rb
+- lib/vagrant-proxyconf/action/only_once.rb
 - lib/vagrant-proxyconf/cap/debian/apt_proxy_conf.rb
 - lib/vagrant-proxyconf/cap/linux/env_proxy_conf.rb
 - lib/vagrant-proxyconf/config/apt_proxy.rb
@@ -41,6 +43,7 @@ files:
 - lib/vagrant-proxyconf/config/key.rb
 - lib/vagrant-proxyconf/config/key_mixin.rb
 - lib/vagrant-proxyconf/config/proxy.rb
+- lib/vagrant-proxyconf/logger.rb
 - lib/vagrant-proxyconf/plugin.rb
 - lib/vagrant-proxyconf/version.rb
 - locales/en.yml
@@ -49,6 +52,8 @@ files:
 - spec/unit/vagrant-proxyconf/action/configure_apt_proxy_spec.rb
 - spec/unit/vagrant-proxyconf/action/configure_chef_proxy_spec.rb
 - spec/unit/vagrant-proxyconf/action/configure_env_proxy_spec.rb
+- spec/unit/vagrant-proxyconf/action/only_once_spec.rb
+- spec/unit/vagrant-proxyconf/action_spec.rb
 - spec/unit/vagrant-proxyconf/cap/debian/apt_proxy_conf_spec.rb
 - spec/unit/vagrant-proxyconf/cap/linux/env_proxy_conf_spec.rb
 - spec/unit/vagrant-proxyconf/config/apt_proxy_spec.rb
@@ -56,6 +61,7 @@ files:
 - spec/unit/vagrant-proxyconf/config/key_mixin_spec.rb
 - spec/unit/vagrant-proxyconf/config/key_spec.rb
 - spec/unit/vagrant-proxyconf/config/proxy_spec.rb
+- spec/unit/vagrant-proxyconf/logger_spec.rb
 - spec/unit/vagrant-proxyconf/plugin_spec.rb
 - vagrant-proxyconf.gemspec
 homepage: http://tmatilai.github.io/vagrant-proxyconf/
@@ -73,9 +79,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ! '>='
+  - - ! '>'
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubyforge_project: 
 rubygems_version: 2.1.4
@@ -88,6 +94,8 @@ test_files:
 - spec/unit/vagrant-proxyconf/action/configure_apt_proxy_spec.rb
 - spec/unit/vagrant-proxyconf/action/configure_chef_proxy_spec.rb
 - spec/unit/vagrant-proxyconf/action/configure_env_proxy_spec.rb
+- spec/unit/vagrant-proxyconf/action/only_once_spec.rb
+- spec/unit/vagrant-proxyconf/action_spec.rb
 - spec/unit/vagrant-proxyconf/cap/debian/apt_proxy_conf_spec.rb
 - spec/unit/vagrant-proxyconf/cap/linux/env_proxy_conf_spec.rb
 - spec/unit/vagrant-proxyconf/config/apt_proxy_spec.rb
@@ -95,5 +103,6 @@ test_files:
 - spec/unit/vagrant-proxyconf/config/key_mixin_spec.rb
 - spec/unit/vagrant-proxyconf/config/key_spec.rb
 - spec/unit/vagrant-proxyconf/config/proxy_spec.rb
+- spec/unit/vagrant-proxyconf/logger_spec.rb
 - spec/unit/vagrant-proxyconf/plugin_spec.rb
 has_rdoc: