lib/testlab/container/actions.rb in testlab-0.7.4 vs lib/testlab/container/actions.rb in testlab-0.7.5

@@ -10,10 +10,11 @@
       #
       # @return [Boolean] True if successful.
       def create
         @ui.logger.debug { "Container Create: #{self.id} " }
 
+        (self.node.state == :not_created) and return false
         (self.lxc.state != :not_created) and return false
 
         please_wait(:ui => @ui, :message => format_object_action(self, 'Create', :green)) do
           configure
 
@@ -29,10 +30,11 @@
       #
       # @return [Boolean] True if successful.
       def destroy
         @ui.logger.debug { "Container Destroy: #{self.id} " }
 
+        (self.node.state == :not_created) and return false
         (self.lxc.state == :not_created) and return false
 
         please_wait(:ui => @ui, :message => format_object_action(self, 'Destroy', :red)) do
           self.lxc.destroy(%(-f))
           self.lxc_clone.destroy(%(-f))
@@ -47,10 +49,11 @@
       #
       # @return [Boolean] True if successful.
       def up
         @ui.logger.debug { "Container Up: #{self.id} " }
 
+        (self.node.state != :running) and return false
         (self.lxc.state == :running) and return false
 
         please_wait(:ui => @ui, :message => format_object_action(self, 'Up', :green)) do
           configure
 
@@ -77,9 +80,10 @@
       #
       # @return [Boolean] True if successful.
       def down
         @ui.logger.debug { "Container Down: #{self.id} " }
 
+        (self.node.state != :running) and return false
         (self.lxc.state != :running) and return false
 
         please_wait(:ui => @ui, :message => format_object_action(self, 'Down', :red)) do
           self.lxc.stop
           self.lxc.wait(:stopped)