virtualbox 0.7.4 → 0.7.5

data/VERSION

@@ -1 +1 @@
-0.7.4
+0.7.5

data/lib/virtualbox/com/mscom_interface.rb

@@ -18,7 +18,6 @@ module VirtualBox
 
         interface_dir = File.expand_path(File.join(File.dirname(__FILE__), "interface"))
         Dir[File.join(interface_dir, "*")].each do |f|
-          p "Checking: #{f}"
           return if File.directory?(f) && initialize_for_version(File.basename(f))
         end
       end
@@ -34,7 +33,6 @@ module VirtualBox
         # Check if they match or not.
         return false if vb_version.length == version.length
         (0...(version.length)).each do |i|
-          p "Checking: #{version[i,1]} to #{vb_version[i,1]}"
           next if version[i,1] == "x"
           return false if version[i,1] != vb_version[i,1]
         end

data/lib/virtualbox/version.rb

@@ -12,15 +12,23 @@ module VirtualBox
     end
 
     # Returns the version string of the VirtualBox installed, ex. "3.1.6"
-    def version
-      Lib.lib.virtualbox.version
+    # This string is cached since the version typically doesn't change
+    # during runtime. If you must refresh the version, send the boolean
+    # `true` as the first parameter.
+    def version(refresh=false)
+      @_version = Lib.lib.virtualbox.version if @_version.nil? || refresh
+      @_version
     rescue Exception
       nil
     end
 
     # Returns the revision string of the VirtualBox installed, ex. "51742"
-    def revision
-      Lib.lib.virtualbox.revision.to_s
+    # This string is cached since the revision doesn't typically change during
+    # runtime. If you must refresh the version, send the boolean `true` as the
+    # first parameter.
+    def revision(refresh=false)
+      @_revision = Lib.lib.virtualbox.revision.to_s if @_revision.nil? || refresh
+      @_revision
     end
   end
 end

data/test/virtualbox/version_test.rb

@@ -30,16 +30,30 @@ class VersionTest < Test::Unit::TestCase
   should "return the version" do
     version = mock("version")
     @vbox.expects(:version).returns(version)
-    assert_equal version, @module.version
+    assert_equal version, @module.version(true)
+  end
+
+  should "cache the version" do
+    @vbox.expects(:version).once.returns("v")
+    assert_equal "v", @module.version(true)
+    assert_equal "v", @module.version
+    assert_equal "v", @module.version
   end
 
   should "return nil if an error occurs" do
     @vbox.expects(:version).raises(LoadError)
-    assert @module.version.nil?
+    assert @module.version(true).nil?
   end
 
   should "return the revision" do
     @vbox.expects(:revision).returns(7)
+    assert_equal "7", @module.revision(true)
+  end
+
+  should "cache the revision" do
+    @vbox.expects(:revision).once.returns(7)
+    assert_equal "7", @module.revision(true)
+    assert_equal "7", @module.revision
     assert_equal "7", @module.revision
   end
 end

data/virtualbox.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{virtualbox}
-  s.version = "0.7.4"
+  s.version = "0.7.5"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Mitchell Hashimoto"]
-  s.date = %q{2010-07-19}
+  s.date = %q{2010-07-20}
   s.description = %q{Create and modify virtual machines in VirtualBox using pure ruby.}
   s.email = %q{mitchell.hashimoto@gmail.com}
   s.extra_rdoc_files = [

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: virtualbox
 version: !ruby/object:Gem::Version 
-  hash: 11
+  hash: 9
   prerelease: false
   segments: 
   - 0
   - 7
-  - 4
-  version: 0.7.4
+  - 5
+  version: 0.7.5
 platform: ruby
 authors: 
 - Mitchell Hashimoto
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-07-19 00:00:00 -07:00
+date: 2010-07-20 00:00:00 -07:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency