fission 0.1.0 → 0.2.0

data/CHANGELOG.md

@@ -2,5 +2,12 @@
 
 ## head
 
+## 0.2.0 (07/13/2011)
+* add 'status' command
+* add 'start' command
+* add 'stop' command
+* add 'suspend' command
+* add support for cloning single and split disk VMs
+
 ## 0.1.0 (05/17/2011)
 Initial release

data/README.md

@@ -1,19 +1,7 @@
 # Fission
 
 ## Intro
-Fission is a simple command line tool for cloning of VMware Fusion VMs.
-
-
-## Config
-By default, fission will use the default VMware Fusion VM directory
-(~/Documents/Virtual Machines.localized/) when cloning.  If you want to use a
-different directory, you can set this in a config file.
-
-The config file needs to be in yaml format and live at '~/.fissionrc'
-
-    $cat ~/.fissionrc
-    ---
-    vm_dir: "/vm"
+Fission is a simple command line tool for managing VMware Fusion VMs.
 
 
 ## Install
@@ -22,7 +10,29 @@ The config file needs to be in yaml format and live at '~/.fissionrc'
 
 ## Usage
 ### Clone
-    fission clone existing_vm new_vm
+    fission clone existing_vm new_vm [--start]
+
+If you provide '--start', then the new VM will be powered on after cloning
+
+### Start
+    fission start my_vm
+
+Starts the VM
+
+### Status
+    fission status
+
+Displays the status (running or not) of all of the VMs found
+
+### Stop
+    fission stop my_vm
+
+Stops the VM
+
+### Suspend
+    fission suspend my_vm
+
+Suspends the VM
 
 ### Help
     fission -h
@@ -32,7 +42,23 @@ or just
     fission
 
 
+## Config
+By default, fission will use the default VMware Fusion VM directory
+(~/Documents/Virtual Machines.localized/) when cloning.  If you want to use a
+different directory, you can set this in a config file.
+
+The config file needs to be in yaml format and live at '~/.fissionrc'
+
+    $cat ~/.fissionrc
+    ---
+    vm_dir: "/vm"
+
+
 ## Other Notable Info
+The name of the VM used in the previous examples should be the directory name 
+of the VM minus the '.vmwarevm' extension.  Typically the VM name and the 
+directory name are the same.
+
 As of now, VMware Fusion doesn't provide an easy, out of
 the box, way to modify the personality (hostname, ip, etc.) of a VM.  Because of
 this, a clone created by fission is an _exact_ copy of the original (including
@@ -40,8 +66,8 @@ hostname, ip address, etc.).  Most likely, this isn't what you want.
 
 One approach is to create a VM which will act as a template.  Create the VM with
 the desired install method (ideally with easy install) and settings, but do not
-power on the VM.  You can create clones from this VM and when you power it on,
-it will start the OS install process (and assign a new ip, etc.)
+power on the VM.  You can then create clones from this VM 'template'.  When you 
+power on the clone, it will start the OS install process (and assign a new ip, etc.).
 
 
 ## Contribute
@@ -49,3 +75,7 @@ it will start the OS install process (and assign a new ip, etc.)
 * Make your feature addition or bug fix (with tests) in a topic branch
 * Bonus points for not mucking with the gemspec or version
 * Send a pull request and I'll get it integrated
+
+
+## License
+See LICENSE

data/lib/fission.rb

@@ -8,8 +8,13 @@ $:.unshift File.join(File.dirname(__FILE__), *%w[.. lib])
 require 'fission/cli'
 require 'fission/command'
 require 'fission/command/clone'
+require 'fission/command/start'
+require 'fission/command/status'
+require 'fission/command/stop'
+require 'fission/command/suspend'
 require 'fission/config'
 require 'fission/core_ext/class'
+require 'fission/core_ext/file'
 require 'fission/core_ext/object'
 require 'fission/ui'
 require 'fission/vm'

data/lib/fission/cli.rb

@@ -28,15 +28,22 @@ module Fission
         exit(1)
       end
 
-      case args.first
-      when 'clone'
-        @command = Fission::Command::Clone.new args.drop 1
-        @command.execute
+      if commands.include?(args.first)
+        @cmd = Fission::Command.const_get(args.first.capitalize).new args.drop 1
+        @cmd.execute
       else
         show_all_help(optparse)
-        exit(0)
+        exit(1)
+      end
+    end
+
+    def self.commands
+      cmds = []
+      Dir.entries(File.join(File.dirname(__FILE__), 'command')).select do |file|
+        cmds << File.basename(file, '.rb') unless File.directory? file
       end
 
+      cmds
     end
 
     private

data/lib/fission/command/start.rb

@@ -0,0 +1,42 @@
+module Fission
+  class Command
+    class Start < Command
+
+      def initialize(args=[])
+        super
+      end
+
+      def execute
+        unless args.count == 1
+          Fission.ui.output self.class.help
+          Fission.ui.output ""
+          Fission.ui.output_and_exit "Incorrect arguments for start command", 1
+        end
+
+        vm_name = args.first
+
+        unless Fission::VM.exists? vm_name
+          Fission.ui.output_and_exit "Unable to find the VM #{vm_name} (#{Fission::VM.path(vm_name)})", 1 
+        end
+
+        if VM.all_running.include?(vm_name)
+          Fission.ui.output ''
+          Fission.ui.output_and_exit "VM '#{vm_name}' is already running", 0
+        end
+
+        Fission.ui.output "Starting '#{vm_name}'"
+        @vm = Fission::VM.new vm_name
+        @vm.start
+      end
+
+      def option_parser
+        optparse = OptionParser.new do |opts|
+          opts.banner = "\nstart usage: fission start vm"
+        end
+
+        optparse
+      end
+
+    end
+  end
+end

data/lib/fission/command/status.rb

@@ -0,0 +1,31 @@
+module Fission
+  class Command
+    class Status < Command
+
+      def initialize(args=[])
+        super
+      end
+
+      def execute
+        all_vms = VM.all
+        all_running_vms = VM.all_running
+
+        longest_vm_name = all_vms.max { |a,b| a.length <=> b.length }
+
+        VM.all.each do |vm|
+          status = all_running_vms.include?(vm) ? '[running]' : '[not running]'
+          Fission.ui.output_printf "%-#{longest_vm_name.length}s   %s\n", vm, status
+        end
+      end
+
+      def option_parser
+        optparse = OptionParser.new do |opts|
+          opts.banner = "\nstatus usage: fission status"
+        end
+
+        optparse
+      end
+
+    end
+  end
+end

data/lib/fission/command/stop.rb

@@ -0,0 +1,42 @@
+module Fission
+  class Command
+    class Stop < Command
+
+      def initialize(args=[])
+        super
+      end
+
+      def execute
+        unless args.count == 1
+          Fission.ui.output self.class.help
+          Fission.ui.output ""
+          Fission.ui.output_and_exit "Incorrect arguments for stop command", 1
+        end
+
+        vm_name = args.first
+
+        unless Fission::VM.exists? vm_name
+          Fission.ui.output_and_exit "Unable to find the VM #{vm_name} (#{Fission::VM.path(vm_name)})", 1 
+        end
+
+        unless VM.all_running.include?(vm_name)
+          Fission.ui.output ''
+          Fission.ui.output_and_exit "VM '#{vm_name}' is not running", 0
+        end
+
+        Fission.ui.output "Stopping '#{vm_name}'"
+        @vm = Fission::VM.new vm_name
+        @vm.stop
+      end
+
+      def option_parser
+        optparse = OptionParser.new do |opts|
+          opts.banner = "\nstop usage: fission stop vm"
+        end
+
+        optparse
+      end
+
+    end
+  end
+end

data/lib/fission/command/suspend.rb

@@ -0,0 +1,42 @@
+module Fission
+  class Command
+    class Suspend < Command
+
+      def initialize(args=[])
+        super
+      end
+
+      def execute
+        unless args.count == 1
+          Fission.ui.output self.class.help
+          Fission.ui.output ""
+          Fission.ui.output_and_exit "Incorrect arguments for suspend command", 1
+        end
+
+        vm_name = args.first
+
+        unless Fission::VM.exists? vm_name
+          Fission.ui.output_and_exit "Unable to find the VM #{vm_name} (#{Fission::VM.path(vm_name)})", 1 
+        end
+
+        unless VM.all_running.include?(vm_name)
+          Fission.ui.output ''
+          Fission.ui.output_and_exit "VM '#{vm_name}' is not running", 0
+        end
+
+        Fission.ui.output "suspending '#{vm_name}'"
+        @vm = Fission::VM.new vm_name
+        @vm.suspend
+      end
+
+      def option_parser
+        optparse = OptionParser.new do |opts|
+          opts.banner = "\nsuspend usage: fission suspend vm"
+        end
+
+        optparse
+      end
+
+    end
+  end
+end

data/lib/fission/config.rb

@@ -13,6 +13,7 @@ module Fission
       end
 
       @attributes['vmrun_bin'] = '/Library/Application Support/VMware Fusion/vmrun'
+      @attributes['vmrun_cmd'] = "#{@attributes['vmrun_bin'].gsub(' ', '\ ')} -T fusion"
     end
 
     private

data/lib/fission/core_ext/file.rb

@@ -0,0 +1,7 @@
+class File
+  # from ptools
+  def self.binary?(file)
+    s = (File.read(file, File.stat(file).blksize) || "").split(//)
+    ((s.size - s.grep(" ".."~").size) / s.size.to_f) > 0.30
+  end
+end

data/lib/fission/ui.rb

@@ -10,6 +10,10 @@ module Fission
       @stdout.puts s
     end
 
+    def output_printf(string, key, value)
+      @stdout.send :printf, string, key, value
+    end
+
     def output_and_exit(s, exit_code)
       output s
       exit exit_code

data/lib/fission/version.rb

@@ -1,3 +1,3 @@
 module Fission
-  VERSION = "0.1.0"
+  VERSION = "0.2.0"
 end

data/lib/fission/vm.rb

@@ -7,7 +7,7 @@ module Fission
     end
 
     def start
-      command = "#{Fission.config.attributes['vmrun_bin'].gsub(' ', '\ ' )} -T fusion start #{conf_file.gsub ' ', '\ '} gui 2>&1"
+      command = "#{Fission.config.attributes['vmrun_cmd']} start #{conf_file.gsub ' ', '\ '} gui 2>&1"
       output = `#{command}`
 
       if $?.exitstatus == 0
@@ -17,10 +17,62 @@ module Fission
       end
     end
 
+    def stop
+      command = "#{Fission.config.attributes['vmrun_cmd']} stop #{conf_file.gsub ' ', '\ '} 2>&1"
+      output = `#{command}`
+
+      if $?.exitstatus == 0
+        Fission.ui.output "VM stopped"
+      else
+        Fission.ui.output "There was a problem stopping the VM.  The error was:\n#{output}"
+      end
+    end
+
+    def suspend
+      command = "#{Fission.config.attributes['vmrun_cmd']} suspend #{conf_file.gsub ' ', '\ '} 2>&1"
+      output = `#{command}`
+
+      if $?.exitstatus == 0
+        Fission.ui.output "VM suspended"
+      else
+        Fission.ui.output "There was a problem suspending the VM.  The error was:\n#{output}"
+      end
+    end
+
     def conf_file
       File.join self.class.path(@name), "#{@name}.vmx"
     end
 
+    def self.all
+      vm_dirs = Dir[File.join Fission.config.attributes['vm_dir'], '*.vmwarevm'].select do |d|
+        File.directory? d
+      end
+
+      vm_dirs.map { |d| File.basename d, '.vmwarevm' }
+    end
+
+    def self.all_running
+      command = "#{Fission.config.attributes['vmrun_cmd']} list"
+
+      output = `#{command}`
+
+      vms = []
+
+      if $?.exitstatus == 0
+        output.split("\n").each do |vm|
+          if vm.include?('.vmx')
+            if File.exists?(vm) && (File.extname(vm) == '.vmx')
+              vms << File.basename(vm, '.vmx')
+            end
+          end
+        end
+      else
+        Fission.ui.output_and_exit "Unable to determine the list of running VMs", 1
+      end
+
+      vms
+    end
+
     def self.exists?(vm_name)
       File.directory? path(vm_name)
     end
@@ -50,11 +102,13 @@ module Fission
     end
 
     def self.update_config(from, to)
-      ['.vmdk', '.vmx', '.vmxf'].each do |ext|
+      ['.vmx', '.vmxf', '.vmdk'].each do |ext|
         file = File.join path(to), "#{to}#{ext}"
-        text = File.read file
-        text.gsub! from, to
-        File.open(file, 'w'){ |f| f.print text }
+
+        unless File.binary?(file)
+          text = (File.read file).gsub from, to
+          File.open(file, 'w'){ |f| f.print text }
+        end
       end
     end
 

data/spec/fission/cli_spec.rb

@@ -6,6 +6,12 @@ describe Fission::CLI do
     Fission.stub!(:ui).and_return(Fission::UI.new(@string_io))
   end
 
+  describe 'self.commands' do
+    it 'should return the list of commands' do
+      Fission::CLI.commands.should == ['clone', 'start', 'status', 'stop', 'suspend']
+    end
+  end
+
   describe 'execute' do
 
     describe 'with no arguments' do
@@ -45,17 +51,50 @@ describe Fission::CLI do
 
     end
 
-    describe 'clone' do
+    describe 'with the sub command' do
       before :each do
-        @clone_mock = mock('clone_mock')
-        Fission::Command::Clone.stub!(:help).and_return('')
+        @cmd_mock = mock('cmd')
+        @cmd_mock.should_receive(:execute)
+      end
+
+      describe 'clone' do
+        it "should try to clone the vm" do
+          Fission::Command::Clone.stub!(:help).and_return('')
+          Fission::Command::Clone.should_receive(:new).with(['foo', 'bar']).and_return(@cmd_mock)
+          Fission::CLI.execute ['clone', 'foo', 'bar']
+        end
       end
 
-      it "should try to clone the vm" do
-        @clone_mock.should_receive(:execute)
-        Fission::Command::Clone.should_receive(:new).with(['foo', 'bar']).and_return(@clone_mock)
+      describe 'start' do
+        it "should try to start the vm" do
+          Fission::Command::Start.stub!(:help).and_return('')
+          Fission::Command::Start.should_receive(:new).with(['foo']).and_return(@cmd_mock)
+          Fission::CLI.execute ['start', 'foo']
+        end
+      end
 
-        Fission::CLI.execute ['clone', 'foo', 'bar']
+      describe 'status' do
+        it "should try to get the status for all VMs" do
+          Fission::Command::Status.stub!(:help).and_return('')
+          Fission::Command::Status.should_receive(:new).and_return(@cmd_mock)
+          Fission::CLI.execute ['status']
+        end
+      end
+
+      describe 'stop' do
+        it "should try to stop the vm" do
+          Fission::Command::Stop.stub!(:help).and_return('')
+          Fission::Command::Stop.should_receive(:new).with(['foo']).and_return(@cmd_mock)
+          Fission::CLI.execute ['stop', 'foo']
+        end
+      end
+
+      describe 'suspend' do
+        it "should try to suspend the vm" do
+          Fission::Command::Suspend.stub!(:help).and_return('')
+          Fission::Command::Suspend.should_receive(:new).with(['foo']).and_return(@cmd_mock)
+          Fission::CLI.execute ['suspend', 'foo']
+        end
       end
     end
 

data/spec/fission/command/start_spec.rb

@@ -0,0 +1,71 @@
+require File.expand_path('../../../spec_helper.rb', __FILE__)
+
+describe Fission::Command::Start do
+  before :all do
+    @vm_info = ['foo']
+  end
+
+  before :each do
+    @string_io = StringIO.new
+    Fission.stub!(:ui).and_return(Fission::UI.new(@string_io))
+  end
+
+  describe 'execute' do
+    it "should output an error and the help when no VM argument is passed in" do
+      Fission::Command::Start.should_receive(:help)
+
+      lambda {
+        command = Fission::Command::Start.new
+        command.execute
+      }.should raise_error SystemExit
+
+      @string_io.string.should match /Incorrect arguments for start command/
+    end
+
+    it "should output an error and exit if it can't find the vm" do
+      Fission::VM.should_receive(:exists?).with(@vm_info.first).and_return(false)
+
+      lambda {
+        command = Fission::Command::Start.new @vm_info
+        command.execute
+      }.should raise_error SystemExit
+
+      @string_io.string.should match /Unable to find the VM #{@vm_info.first}/
+    end
+
+
+    it "should output and exit if the vm is already running" do
+      Fission::VM.should_receive(:exists?).with(@vm_info.first).and_return(true)
+      Fission::VM.should_receive(:all_running).and_return(@vm_info)
+
+      lambda {
+        command = Fission::Command::Start.new @vm_info
+        command.execute
+      }.should raise_error SystemExit
+
+      @string_io.string.should match /VM '#{@vm_info.first}' is already running/
+    end
+
+    it 'should try to start the vm if it is not running' do
+      @vm_mock = mock('vm_mock')
+      Fission::VM.should_receive(:exists?).with(@vm_info.first).and_return(true)
+      Fission::VM.should_receive(:all_running).and_return([])
+      Fission::VM.should_receive(:new).with(@vm_info.first).and_return(@vm_mock)
+      @vm_mock.should_receive(:start)
+
+      command = Fission::Command::Start.new @vm_info
+      command.execute
+
+      @string_io.string.should match /Starting '#{@vm_info.first}'/
+    end
+
+  end
+
+  describe 'help' do
+    it 'should output info for this command' do
+      output = Fission::Command::Start.help
+
+      output.should match /start vm/
+    end
+  end
+end

data/spec/fission/command/status_spec.rb

@@ -0,0 +1,38 @@
+require File.expand_path('../../../spec_helper.rb', __FILE__)
+
+describe Fission::Command::Status do
+  before :each do
+    @string_io = StringIO.new
+    Fission.stub!(:ui).and_return(Fission::UI.new(@string_io))
+  end
+
+  describe 'execute' do
+    before :each do
+      Fission::VM.stub!(:all).and_return(['foo', 'bar', 'baz'])
+      Fission::VM.stub!(:all_running).and_return(['foo', 'baz'])
+    end
+
+    it 'should output the VMs which are running' do
+      command = Fission::Command::Status.new
+      command.execute
+
+      @string_io.string.should match /foo.+[running]/
+      @string_io.string.should match /baz.+[running]/
+    end
+
+    it 'should output the VMs which are not running' do
+      command = Fission::Command::Status.new
+      command.execute
+
+      @string_io.string.should match /bar.+[not running]/
+    end
+  end
+
+  describe 'help' do
+    it 'should output info for this command' do
+      output = Fission::Command::Status.help
+
+      output.should match /status/
+    end
+  end
+end

data/spec/fission/command/stop_spec.rb

@@ -0,0 +1,71 @@
+require File.expand_path('../../../spec_helper.rb', __FILE__)
+
+describe Fission::Command::Stop do
+  before :all do
+    @vm_info = ['foo']
+  end
+
+  before :each do
+    @string_io = StringIO.new
+    Fission.stub!(:ui).and_return(Fission::UI.new(@string_io))
+  end
+
+  describe 'execute' do
+    it "should output an error and the help when no VM argument is passed in" do
+      Fission::Command::Stop.should_receive(:help)
+
+      lambda {
+        command = Fission::Command::Stop.new
+        command.execute
+      }.should raise_error SystemExit
+
+      @string_io.string.should match /Incorrect arguments for stop command/
+    end
+
+    it "should output an error and exit if it can't find the vm" do
+      Fission::VM.should_receive(:exists?).with(@vm_info.first).and_return(false)
+
+      lambda {
+        command = Fission::Command::Stop.new @vm_info
+        command.execute
+      }.should raise_error SystemExit
+
+      @string_io.string.should match /Unable to find the VM #{@vm_info.first}/
+    end
+
+
+    it "should output and exit if the vm is not running" do
+      Fission::VM.should_receive(:exists?).with(@vm_info.first).and_return(true)
+      Fission::VM.should_receive(:all_running).and_return([])
+
+      lambda {
+        command = Fission::Command::Stop.new @vm_info
+        command.execute
+      }.should raise_error SystemExit
+
+      @string_io.string.should match /VM '#{@vm_info.first}' is not running/
+    end
+
+    it 'should try to stop the vm if it is not running' do
+      @vm_mock = mock('vm_mock')
+      Fission::VM.should_receive(:exists?).with(@vm_info.first).and_return(true)
+      Fission::VM.should_receive(:all_running).and_return(@vm_info)
+      Fission::VM.should_receive(:new).with(@vm_info.first).and_return(@vm_mock)
+      @vm_mock.should_receive(:stop)
+
+      command = Fission::Command::Stop.new @vm_info
+      command.execute
+
+      @string_io.string.should match /Stopping '#{@vm_info.first}'/
+    end
+
+  end
+
+  describe 'help' do
+    it 'should output info for this command' do
+      output = Fission::Command::Stop.help
+
+      output.should match /stop vm/
+    end
+  end
+end

data/spec/fission/command/suspend_spec.rb

@@ -0,0 +1,71 @@
+require File.expand_path('../../../spec_helper.rb', __FILE__)
+
+describe Fission::Command::Suspend do
+  before :all do
+    @vm_info = ['foo']
+  end
+
+  before :each do
+    @string_io = StringIO.new
+    Fission.stub!(:ui).and_return(Fission::UI.new(@string_io))
+  end
+
+  describe 'execute' do
+    it "should output an error and the help when no VM argument is passed in" do
+      Fission::Command::Suspend.should_receive(:help)
+
+      lambda {
+        command = Fission::Command::Suspend.new
+        command.execute
+      }.should raise_error SystemExit
+
+      @string_io.string.should match /Incorrect arguments for suspend command/
+    end
+
+    it "should output an error and exit if it can't find the vm" do
+      Fission::VM.should_receive(:exists?).with(@vm_info.first).and_return(false)
+
+      lambda {
+        command = Fission::Command::Suspend.new @vm_info
+        command.execute
+      }.should raise_error SystemExit
+
+      @string_io.string.should match /Unable to find the VM #{@vm_info.first}/
+    end
+
+
+    it "should output and exit if the vm is not running" do
+      Fission::VM.should_receive(:exists?).with(@vm_info.first).and_return(true)
+      Fission::VM.should_receive(:all_running).and_return([])
+
+      lambda {
+        command = Fission::Command::Suspend.new @vm_info
+        command.execute
+      }.should raise_error SystemExit
+
+      @string_io.string.should match /VM '#{@vm_info.first}' is not running/
+    end
+
+    it 'should try to suspend the vm if it is running' do
+      @vm_mock = mock('vm_mock')
+      Fission::VM.should_receive(:exists?).with(@vm_info.first).and_return(true)
+      Fission::VM.should_receive(:all_running).and_return([@vm_info.first])
+      Fission::VM.should_receive(:new).with(@vm_info.first).and_return(@vm_mock)
+      @vm_mock.should_receive(:suspend)
+
+      command = Fission::Command::Suspend.new @vm_info
+      command.execute
+
+      @string_io.string.should match /suspending '#{@vm_info.first}'/
+    end
+
+  end
+
+  describe 'help' do
+    it 'should output info for this command' do
+      output = Fission::Command::Suspend.help
+
+      output.should match /suspend vm/
+    end
+  end
+end

data/spec/fission/config_spec.rb

@@ -25,6 +25,13 @@ describe Fission::Config do
       end
     end
 
+    it 'should set vmrun_cmd' do
+      FakeFS do
+        @config = Fission::Config.new
+        @config.attributes['vmrun_cmd'].should == '/Library/Application\ Support/VMware\ Fusion/vmrun -T fusion'
+      end
+    end
+
   end
 
 end

data/spec/fission/ui_spec.rb

@@ -11,6 +11,14 @@ describe Fission::UI do
     end
   end
 
+  describe 'output_printf' do
+    it 'should pass the arguments to printf' do
+      @output = mock('output')
+      @output.should_receive(:printf).with('foo', 'bar', 'baz')
+      Fission::UI.new(@output).output_printf('foo', 'bar', 'baz')
+    end
+  end
+
   describe 'output_and_exit' do
     it 'should show the desired text and exit with the desired exit code' do
       Fission::UI.any_instance.should_receive(:exit).and_return(1)

data/spec/fission/vm_spec.rb

@@ -13,9 +13,12 @@ describe Fission::VM do
 
   describe 'start' do
     it 'should output that it was successful' do
+      $?.should_receive(:exitstatus).and_return(0)
       Fission.stub!(:ui).and_return(Fission::UI.new(@string_io))
       @vm = Fission::VM.new('foo')
-      @vm.should_receive(:`).with("#{Fission.config.attributes['vmrun_bin'].gsub ' ', '\ '} -T fusion start #{(File.join(Fission::VM.path('foo'), 'foo.vmx')).gsub ' ', '\ '} gui 2>&1").and_return("it's all good")
+      @vm.should_receive(:`).
+          with("#{Fission.config.attributes['vmrun_cmd']} start #{@vm.conf_file.gsub(' ', '\ ')} gui 2>&1").
+          and_return("it's all good")
       @vm.start
 
       @string_io.string.should match /VM started/
@@ -25,19 +28,126 @@ describe Fission::VM do
       $?.should_receive(:exitstatus).and_return(1)
       Fission.stub!(:ui).and_return(Fission::UI.new(@string_io))
       @vm = Fission::VM.new('foo')
-      @vm.should_receive(:`).with("#{Fission.config.attributes['vmrun_bin'].gsub ' ', '\ '} -T fusion start #{(File.join(Fission::VM.path('foo'), 'foo.vmx')).gsub ' ', '\ '} gui 2>&1").and_return("it blew up")
+      @vm.should_receive(:`).
+          with("#{Fission.config.attributes['vmrun_cmd']} start #{@vm.conf_file.gsub(' ', '\ ')} gui 2>&1").
+          and_return("it blew up")
       @vm.start
 
       @string_io.string.should match /There was a problem starting the VM.+it blew up.+/m
     end
   end
 
+  describe 'stop' do
+    it 'should output that it was successful' do
+      $?.should_receive(:exitstatus).and_return(0)
+      Fission.stub!(:ui).and_return(Fission::UI.new(@string_io))
+      @vm = Fission::VM.new('foo')
+      @vm.should_receive(:`).
+          with("#{Fission.config.attributes['vmrun_cmd']} stop #{@vm.conf_file.gsub ' ', '\ '} 2>&1").
+          and_return("it's all good")
+      @vm.stop
+
+      @string_io.string.should match /VM stopped/
+    end
+
+    it 'it should output that it was unsuccessful' do
+      $?.should_receive(:exitstatus).and_return(1)
+      Fission.stub!(:ui).and_return(Fission::UI.new(@string_io))
+      @vm = Fission::VM.new('foo')
+      @vm.should_receive(:`).
+          with("#{Fission.config.attributes['vmrun_cmd']} stop #{@vm.conf_file.gsub ' ', '\ '} 2>&1").
+          and_return("it blew up")
+      @vm.stop
+
+      @string_io.string.should match /There was a problem stopping the VM.+it blew up.+/m
+    end
+  end
+
+  describe 'suspend' do
+    it 'should output that it was successful' do
+      $?.should_receive(:exitstatus).and_return(0)
+      Fission.stub!(:ui).and_return(Fission::UI.new(@string_io))
+      @vm = Fission::VM.new('foo')
+      @vm.should_receive(:`).
+          with("#{Fission.config.attributes['vmrun_cmd']} suspend #{@vm.conf_file.gsub ' ', '\ '} 2>&1").
+          and_return("it's all good")
+      @vm.suspend
+
+      @string_io.string.should match /VM suspended/
+    end
+
+    it 'it should output that it was unsuccessful' do
+      $?.should_receive(:exitstatus).and_return(1)
+      Fission.stub!(:ui).and_return(Fission::UI.new(@string_io))
+      @vm = Fission::VM.new('foo')
+      @vm.should_receive(:`).
+          with("#{Fission.config.attributes['vmrun_cmd']} suspend #{@vm.conf_file.gsub ' ', '\ '} 2>&1").
+          and_return("it blew up")
+      @vm.suspend
+
+      @string_io.string.should match /There was a problem suspending the VM.+it blew up.+/m
+    end
+  end
+
   describe 'conf_file' do
     it 'should return the path to the conf file' do
       Fission::VM.new('foo').conf_file.should == File.join(Fission.config.attributes['vm_dir'], 'foo.vmwarevm', 'foo.vmx')
     end
   end
 
+  describe "self.all" do
+    it "should return the list of VMs" do
+      vm_root = Fission.config.attributes['vm_dir']
+      Dir.should_receive(:[]).and_return(["#{File.join vm_root, 'foo.vmwarevm' }", "#{File.join vm_root, 'bar.vmwarevm' }"])
+
+      vm_root = Fission.config.attributes['vm_dir']
+      File.should_receive(:directory?).with("#{File.join vm_root, 'foo.vmwarevm'}").and_return(true)
+      File.should_receive(:directory?).with("#{File.join vm_root, 'bar.vmwarevm'}").and_return(true)
+      Fission::VM.all.should == ['foo', 'bar']
+    end
+
+    it "should not return an item in the list if it isn't a directory" do
+      vm_root = Fission.config.attributes['vm_dir']
+      Dir.should_receive(:[]).and_return((['foo', 'bar', 'baz'].map { |i| File.join vm_root, "#{i}.vmwarevm"}))
+      File.should_receive(:directory?).with("#{File.join vm_root, 'foo.vmwarevm'}").and_return(true)
+      File.should_receive(:directory?).with("#{File.join vm_root, 'bar.vmwarevm'}").and_return(true)
+      File.should_receive(:directory?).with("#{File.join vm_root, 'baz.vmwarevm'}").and_return(false)
+      Fission::VM.all.should == ['foo', 'bar']
+    end
+
+    it "should only query for items with an extension of .vmwarevm" do
+      dir_arg = File.join Fission.config.attributes['vm_dir'], '*.vmwarevm'
+      Dir.should_receive(:[]).with(dir_arg).and_return(['foo.vmwarevm', 'bar.vmwarevm'])
+      Fission::VM.all
+    end
+  end
+
+  describe 'self.all_running' do
+    it 'should list the running vms' do
+      list_output = "Total running VMs: 2\n/vm/foo.vmwarevm/foo.vmx\n/vm/bar.vmwarevm/bar.vmx\n/vm/baz.vmwarevm/baz.vmx\n"
+
+      $?.should_receive(:exitstatus).and_return(0)
+      Fission::VM.should_receive(:`).with("#{Fission.config.attributes['vmrun_cmd']} list").and_return(list_output)
+      [ 'foo', 'bar', 'baz'].each do |vm|
+        File.should_receive(:exists?).with("/vm/#{vm}.vmwarevm/#{vm}.vmx").and_return(true)
+      end
+
+      Fission::VM.all_running.should == ['foo', 'bar', 'baz']
+    end
+
+    it 'should output an error and exit if unable to get the list of running vms' do
+      $?.should_receive(:exitstatus).and_return(1)
+      Fission::VM.should_receive(:`).with("#{Fission.config.attributes['vmrun_cmd']} list").and_return("it blew up")
+      Fission.stub!(:ui).and_return(Fission::UI.new(@string_io))
+
+      lambda {
+        Fission::VM.all_running
+      }.should raise_error SystemExit
+
+      @string_io.string.should match /Unable to determine the list of running VMs/
+    end
+  end
+
   describe "self.path" do
     it "should return the path of the vm" do
       vm_path = File.join(Fission.config.attributes['vm_dir'], 'foo.vmwarevm').gsub '\\', ''
@@ -61,45 +171,75 @@ describe Fission::VM do
     end
   end
 
-
   describe "self.clone" do
     before :each do
       Fission.stub!(:ui).and_return(Fission::UI.new(@string_io))
-    end
+      @source_vm = 'foo'
+      @target_vm = 'bar'
+      @vm_files = ['.vmx', '.vmxf', '.vmdk', '-s001.vmdk', '-s002.vmdk', '.vmsd']
 
-    it 'should clone the vm to the target' do
-      source_vm = 'foo'
-      target_vm = 'bar'
-      vm_files = [ '.vmx',
-                   '.vmxf',
-                   '.vmdk',
-                   '-s001.vmdk',
-                   '-s002.vmdk',
-                   '.vmsd' ]
+      FakeFS.activate!
 
-      FakeFS do
-        FileUtils.mkdir_p Fission::VM.path('foo')
+      FileUtils.mkdir_p Fission::VM.path('foo')
+
+      @vm_files.each do |file|
+        FileUtils.touch File.join(Fission::VM.path('foo'), "#{@source_vm}#{file}")
+      end
 
-        vm_files.each do |file|
-          FileUtils.touch File.join(Fission::VM.path('foo'), "#{source_vm}#{file}")
-        end
+      ['.vmx', '.vmxf', '.vmdk'].each do |ext|
+        File.open(File.join(Fission::VM.path('foo'), "foo#{ext}"), 'w') { |f| f.write 'foo.vmdk'}
+      end
 
-        File.open(File.join(Fission::VM.path('foo'), 'foo.vmx'), 'w') { |f| f.write 'foo.vmdk'}
+      ['.vmx', '.vmxf'].each do |ext|
+        File.should_receive(:binary?).with(File.join(Fission::VM.path('bar'), "bar#{ext}")).and_return(false)
+      end
+    end
 
-        Fission::VM.clone source_vm, target_vm
+    after :each do
+      FakeFS.deactivate!
+    end
 
-        File.directory?(Fission::VM.path('bar')).should == true
+    it 'should copy the vm files to the target' do
+      File.should_receive(:binary?).with(File.join(Fission::VM.path('bar'), "bar.vmdk")).and_return(true)
+      Fission::VM.clone @source_vm, @target_vm
+
+      File.directory?(Fission::VM.path('bar')).should == true
+
+      @vm_files.each do |file|
+        File.file?(File.join(Fission::VM.path('bar'), "#{@target_vm}#{file}")).should == true
+      end
+    end
 
-        vm_files.each do |file|
-          File.file?(File.join(Fission::VM.path('bar'), "#{target_vm}#{file}")).should == true
-        end
+    it 'should update the target vm config files' do
+      File.should_receive(:binary?).with(File.join(Fission::VM.path('bar'), "bar.vmdk")).and_return(true)
+      Fission::VM.clone @source_vm, @target_vm
 
-        conf_file = File.read File.join(Fission::VM.path('bar'), 'bar.vmx')
-        conf_file.should == 'bar.vmdk'
+      ['.vmx', '.vmxf'].each do |ext|
+        File.read(File.join(Fission::VM.path('bar'), "bar#{ext}")).should_not match /foo/
+        File.read(File.join(Fission::VM.path('bar'), "bar#{ext}")).should match /bar/
       end
+    end
+
+    it "should not try to update the vmdk file if it's not a sparse disk" do
+      File.should_receive(:binary?).with(File.join(Fission::VM.path('bar'), "bar.vmdk")).and_return(true)
+      Fission::VM.clone @source_vm, @target_vm
+
+      File.read(File.join(Fission::VM.path('bar'), 'bar.vmdk')).should match /foo/
+    end
+
+    it "should update the vmdk when a sparse disk is found" do
+      File.should_receive(:binary?).with(File.join(Fission::VM.path('bar'), "bar.vmdk")).and_return(false)
+      Fission::VM.clone @source_vm, @target_vm
+
+      File.read(File.join(Fission::VM.path('bar'), 'bar.vmdk')).should match /bar/
+    end
+
+    it 'should output that the clone was successful' do
+      File.should_receive(:binary?).with(File.join(Fission::VM.path('bar'), "bar.vmdk")).and_return(true)
+      Fission::VM.clone @source_vm, @target_vm
 
-      @string_io.string.should match /Cloning #{source_vm} to #{target_vm}/
-      @string_io.string.should match /Configuring #{target_vm}/
+      @string_io.string.should match /Cloning #{@source_vm} to #{@target_vm}/
+        @string_io.string.should match /Configuring #{@target_vm}/
     end
 
   end

metadata

@@ -2,7 +2,7 @@
 name: fission
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors: 
 - Tommy Bishop
@@ -10,7 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-05-18 00:00:00 Z
+date: 2011-07-14 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: rspec
@@ -58,14 +58,23 @@ files:
 - lib/fission/cli.rb
 - lib/fission/command.rb
 - lib/fission/command/clone.rb
+- lib/fission/command/start.rb
+- lib/fission/command/status.rb
+- lib/fission/command/stop.rb
+- lib/fission/command/suspend.rb
 - lib/fission/config.rb
 - lib/fission/core_ext/class.rb
+- lib/fission/core_ext/file.rb
 - lib/fission/core_ext/object.rb
 - lib/fission/ui.rb
 - lib/fission/version.rb
 - lib/fission/vm.rb
 - spec/fission/cli_spec.rb
 - spec/fission/command/clone_spec.rb
+- spec/fission/command/start_spec.rb
+- spec/fission/command/status_spec.rb
+- spec/fission/command/stop_spec.rb
+- spec/fission/command/suspend_spec.rb
 - spec/fission/command_spec.rb
 - spec/fission/config_spec.rb
 - spec/fission/ui_spec.rb
@@ -95,13 +104,17 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: fission
-rubygems_version: 1.8.2
+rubygems_version: 1.8.5
 signing_key: 
 specification_version: 3
 summary: Tool to clone VMware fusion VMs
 test_files: 
 - spec/fission/cli_spec.rb
 - spec/fission/command/clone_spec.rb
+- spec/fission/command/start_spec.rb
+- spec/fission/command/status_spec.rb
+- spec/fission/command/stop_spec.rb
+- spec/fission/command/suspend_spec.rb
 - spec/fission/command_spec.rb
 - spec/fission/config_spec.rb
 - spec/fission/ui_spec.rb