💣 BaseAction

lib/vagrant-lxc/action.rb

@@ -1,7 +1,3 @@
-# TODO: Remove base action
-require 'vagrant-lxc/action/base_action'
-
-# TODO: Use require_relative
 require 'vagrant-lxc/action/boot'
 require 'vagrant-lxc/action/check_created'
 require 'vagrant-lxc/action/check_running'

lib/vagrant-lxc/action/base_action.rb

@@ -1,11 +0,0 @@
-module Vagrant
-  module LXC
-    module Action
-      class BaseAction
-        def initialize(app, env)
-          @app = app
-        end
-      end
-    end
-  end
-end

lib/vagrant-lxc/action/boot.rb

@@ -1,7 +1,11 @@
 module Vagrant
   module LXC
     module Action
-      class Boot < BaseAction
+      class Boot
+        def initialize(app, env)
+          @app = app
+        end
+
         def call(env)
           @env = env
 

lib/vagrant-lxc/action/check_created.rb

@@ -1,7 +1,11 @@
 module Vagrant
   module LXC
     module Action
-      class CheckCreated < BaseAction
+      class CheckCreated
+        def initialize(app, env)
+          @app = app
+        end
+
         def call(env)
           unless env[:machine].state.created?
             raise Vagrant::Errors::VMNotCreatedError

lib/vagrant-lxc/action/check_running.rb

@@ -1,7 +1,11 @@
 module Vagrant
   module LXC
     module Action
-      class CheckRunning < BaseAction
+      class CheckRunning
+        def initialize(app, env)
+          @app = app
+        end
+
         def call(env)
           unless env[:machine].state.running?
             raise Vagrant::Errors::VMNotRunningError

lib/vagrant-lxc/action/create.rb

@@ -1,7 +1,11 @@
 module Vagrant
   module LXC
     module Action
-      class Create < BaseAction
+      class Create
+        def initialize(app, env)
+          @app = app
+        end
+
         def call(env)
           base_name = env[:root_path].basename.to_s
           base_name.gsub!(/[^-a-z0-9_]/i, "")

lib/vagrant-lxc/action/created.rb

@@ -1,7 +1,11 @@
 module Vagrant
   module LXC
     module Action
-      class Created < BaseAction
+      class Created
+        def initialize(app, env)
+          @app = app
+        end
+
         def call(env)
           # Set the result to be true if the machine is created.
           env[:result] = env[:machine].state.created?

lib/vagrant-lxc/action/destroy.rb

@@ -1,7 +1,11 @@
 module Vagrant
   module LXC
     module Action
-      class Destroy < BaseAction
+      class Destroy
+        def initialize(app, env)
+          @app = app
+        end
+
         def call(env)
           env[:ui].info I18n.t("vagrant.actions.vm.destroy.destroying")
           env[:machine].provider.container.destroy

lib/vagrant-lxc/action/disconnect.rb

@@ -1,7 +1,11 @@
 module Vagrant
   module LXC
     module Action
-      class Disconnect < BaseAction
+      class Disconnect
+        def initialize(app, env)
+          @app = app
+        end
+
         def call(env)
           @app.call env
           # FIXME: Vagrant >= 1.1.3 should not need this

lib/vagrant-lxc/action/handle_box_metadata.rb

@@ -2,12 +2,12 @@ module Vagrant
   module LXC
     module Action
       # Prepare arguments to be used for lxc-create
-      class HandleBoxMetadata < BaseAction
+      class HandleBoxMetadata
         LXC_TEMPLATES_PATH = Pathname.new("/usr/share/lxc/templates")
         TEMP_PREFIX        = "vagrant-lxc-rootfs-temp-"
 
         def initialize(app, env)
-          super
+          @app    = app
           @logger = Log4r::Logger.new("vagrant::lxc::action::handle_box_metadata")
         end
 

lib/vagrant-lxc/action/is_running.rb

@@ -1,7 +1,11 @@
 module Vagrant
   module LXC
     module Action
-      class IsRunning < BaseAction
+      class IsRunning
+        def initialize(app, env)
+          @app = app
+        end
+
         def call(env)
           env[:result] = env[:machine].state.running?
 

lib/vagrant-lxc/action/network.rb

@@ -1,9 +1,12 @@
 module Vagrant
   module LXC
     module Action
-      class Network < BaseAction
+      class Network
+        def initialize(app, env)
+          @app = app
+        end
+
         def call(env)
-          # TODO: Validate network configuration prior to anything below
           @env = env
 
           env[:machine].config.vm.networks.each do |type, options|

lib/vagrant-lxc/action/share_folders.rb

@@ -1,7 +1,11 @@
 module Vagrant
   module LXC
     module Action
-      class ShareFolders < BaseAction
+      class ShareFolders
+        def initialize(app, env)
+          @app = app
+        end
+
         def call(env)
           @env = env
           prepare_folders

spec/unit/action/handle_box_metadata_spec.rb

@@ -1,6 +1,5 @@
 require 'unit_helper'
 
-require 'vagrant-lxc/action/base_action'
 require 'vagrant-lxc/action/handle_box_metadata'
 
 describe Vagrant::LXC::Action::HandleBoxMetadata do