lib/testlab/network/actions.rb in testlab-0.6.5 vs lib/testlab/network/actions.rb in testlab-0.6.6

@@ -5,39 +5,55 @@
 
       # Create the network
       def create
         @ui.logger.debug { "Network Create: #{self.id} " }
 
+        (self.state != :not_created) and return false
+
         please_wait(:ui => @ui, :message => format_object_action(self, 'Create', :green)) do
           self.node.ssh.exec(%(sudo brctl addbr #{self.bridge}), :silence => true, :ignore_exit_status => true)
         end
+
+        true
       end
 
       # Destroy the network
       def destroy
         @ui.logger.debug { "Network Destroy: #{self.id} " }
 
+        (self.state == :not_created) and return false
+
         please_wait(:ui => @ui, :message => format_object_action(self, 'Destroy', :red)) do
           self.node.ssh.exec(%(sudo brctl delbr #{self.bridge}), :silence => true, :ignore_exit_status => true)
         end
+
+        true
       end
 
       # Start the network
       def up
         @ui.logger.debug { "Network Up: #{self.id} " }
 
+        (self.state == :running) and return false
+
         please_wait(:ui => @ui, :message => format_object_action(self, 'Up', :green)) do
           self.node.ssh.exec(%(sudo ifconfig #{self.bridge} #{self.ip} netmask #{self.netmask} up), :silence => true, :ignore_exit_status => true)
         end
+
+        true
       end
 
       # Stop the network
       def down
         @ui.logger.debug { "Network Down: #{self.id} " }
 
+        (self.state != :running) and return false
+
         please_wait(:ui => @ui, :message => format_object_action(self, 'Down', :red)) do
           self.node.ssh.exec(%(sudo ifconfig #{self.bridge} down), :silence => true, :ignore_exit_status => true)
         end
+
+        true
       end
 
     end
 
   end