virtualbox 0.5.1 → 0.5.2

data/VERSION

@@ -1 +1 @@
-0.5.1
+0.5.2

data/lib/virtualbox/command.rb

@@ -1,4 +1,8 @@
 module VirtualBox
+  def self.version
+    Command.version
+  end
+
   # Used by the rest of the virtualbox library to call shell commands.
   # It also can be used to change the path for your VBoxManage program.
   #

data/lib/virtualbox/vm.rb

@@ -329,21 +329,24 @@ module VirtualBox
     # attributes of the virtual machine. If any related attributes were saved
     # as well (such as storage controllers), those will be saved, too.
     def save(raise_errors=false)
-      # First save all the changed attributes in a single batch
-      saves = changes.inject([]) do |acc, kv|
-        key, values = kv
-        acc << ["--#{key}", values[1]]
-      end
+      if changed?
+        # First save all the changed attributes in a single batch
+        saves = changes.inject([]) do |acc, kv|
+          key, values = kv
+          acc << ["--#{key}", values[1]]
+        end
 
-      # Flatten to pass into vboxmanage
-      saves.flatten!
+        # Flatten to pass into vboxmanage
+        saves.flatten!
 
-      # Save all the attributes in one command
-      Command.vboxmanage("modifyvm", @original_name, *saves)
+        # Save all the attributes in one command
+        Command.vboxmanage("modifyvm", @original_name, *saves)
+
+        # Now clear attributes and call super so relationships are saved
+        @original_name = name
+        clear_dirty!
+      end
 
-      # Now clear attributes and call super so relationships are saved
-      @original_name = name
-      clear_dirty!
       super()
 
       # Force reload

data/test/virtualbox/command_test.rb

@@ -3,10 +3,15 @@ require File.join(File.dirname(__FILE__), '..', 'test_helper')
 class CommandTest < Test::Unit::TestCase
   context "getting the version" do
     setup do
-      VirtualBox::Command.stubs(:execute)
+      VirtualBox::Command.stubs(:execute).returns("foo")
       VirtualBox::Command.stubs(:success?).returns(true)
     end
 
+    should "be able to call version on VirtualBox and it calls the class method" do
+      version = "foo"
+      assert_equal version, VirtualBox.version
+    end
+
     should "run the command with the version flag" do
       VirtualBox::Command.expects(:execute).with("VBoxManage --version").once.returns("7")
       VirtualBox::Command.version

data/test/virtualbox/vm_test.rb

@@ -318,6 +318,12 @@ raw
       assert @vm.save
     end
 
+    should "do nothing if no attributes changed" do
+      VirtualBox::Command.expects(:vboxmanage).never
+      assert !@vm.changed?
+      assert @vm.save
+    end
+
     should "save name first if changed, then following values should modify new VM" do
       new_name = "foo2"
       VirtualBox::Command.expects(:vboxmanage).with("modifyvm", @name, "--ostype", "Zubuntu", "--name", new_name)

data/virtualbox.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = %q{virtualbox}
-  s.version = "0.5.1"
+  s.version = "0.5.2"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Mitchell Hashimoto"]

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 5
-  - 1
-  version: 0.5.1
+  - 2
+  version: 0.5.2
 platform: ruby
 authors: 
 - Mitchell Hashimoto