rubyipmi 0.7.0 → 0.8.1

data/lib/rubyipmi/freeipmi/connection.rb

@@ -11,10 +11,11 @@ module Rubyipmi
 
     class Connection
 
-      attr_accessor :options
+      attr_accessor :options, :debug
 
 
-      def initialize(user, pass, host,debug=false)
+      def initialize(user, pass, host, opts)
+        @debug = opts[:debug]
         @options = Rubyipmi::ObservableHash.new
         raise("Must provide a host to connect to") unless host
         @options["hostname"] = host
@@ -22,12 +23,24 @@ module Rubyipmi
         # So they are not required
         @options["username"] = user if user
         @options["password"] = pass if pass
-        #@options["driver-type"] = 'LAN_2_0'
-        #getWorkArounds
+        if opts.has_key?(:privilege)
+          @options["privilege-level"] = opts[:privilege]        # privilege type
+        end
+        # Note: rubyipmi should auto detect which driver to use so its unnecessary to specify the driver unless
+        #       the user really wants to.
+        @options['driver-type'] = drivers_map[opts[:driver]] unless drivers_map[opts[:driver]].nil?
+      end
+
+      def drivers_map
+        {
+          'lan15' => 'LAN',
+          'lan20' => 'LAN_2_0',
+          'open'  => 'OPENIPMI'
+        }
       end
 
       def provider
-        return "freeipmi"
+        'freeipmi'
       end
 
       def fru
@@ -63,3 +76,4 @@ module Rubyipmi
     end
   end
 end
+

data/lib/rubyipmi/ipmitool/commands/power.rb

@@ -16,10 +16,10 @@ module Rubyipmi::Ipmitool
 
     # Turn on the system
     def on
-      if off?
-        command("on")
+      if on?
+        return true
       else
-         return true
+        command("on")
       end
     end
 

data/lib/rubyipmi/ipmitool/connection.rb

@@ -13,11 +13,10 @@ module Rubyipmi
     class Connection
 
       attr_accessor :options, :debug
-      attr_reader :debug
 
 
-      def initialize(user, pass, host,debug_value=false)
-        @debug = debug_value
+      def initialize(user, pass, host, opts)
+        @debug = opts[:debug]
         @options = Rubyipmi::ObservableHash.new
         raise("Must provide a host to connect to") unless host
         @options["H"] = host
@@ -25,11 +24,20 @@ module Rubyipmi
         # So they are not required
         @options["U"] = user if user
         @options["P"] = pass if pass
-        # default to IPMI 2.0 communication, this means that older devices will not work
-        # Those old servers should be recycled by now, as the 1.0, 1.5 spec came out in 2005ish and is 2013.
-        #@options["I"] = "lanplus"
+        if opts.has_key?(:privilege)
+          @options["L"] = opts[:privilege]
+        end
+        # Note: rubyipmi should auto detect which driver to use so its unnecessary to specify the driver unless
+        #  the user really wants to.
+        @options['I'] = drivers_map[opts[:driver]] unless drivers_map[opts[:driver]].nil?
+      end
 
-        #getWorkArounds
+      def drivers_map
+        {
+          'lan15' => 'lan',
+          'lan20' => 'lanplus',
+          'open'  => 'open'
+        }
       end
 
       def fru
@@ -37,7 +45,7 @@ module Rubyipmi
       end
 
       def provider
-        return "ipmitool"
+        'ipmitool'
       end
 
       def bmc

data/lib/rubyipmi.rb

@@ -1,113 +1,164 @@
+# Copyright (C) 2014 Corey Osman
+#
+#     This library is free software; you can redistribute it and/or
+#     modify it under the terms of the GNU Lesser General Public
+#     License as published by the Free Software Foundation; either
+#     version 2.1 of the License, or (at your option) any later version.
+#
+#     This library is distributed in the hope that it will be useful,
+#     but WITHOUT ANY WARRANTY; without even the implied warranty of
+#     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+#     Lesser General Public License for more details.
+#
+#     You should have received a copy of the GNU Lesser General Public
+#     License along with this library; if not, write to the Free Software
+#     Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301
+#     USA
+#
+
+
 require 'rubyipmi/ipmitool/connection'
 require 'rubyipmi/freeipmi/connection'
 
 
 module Rubyipmi
+  PRIV_TYPES = ['CALLBACK', 'USER', 'OPERATOR', 'ADMINISTRATOR']
 
+  def self.valid_drivers
+    ['auto', "lan15", "lan20", "open"]
+  end
 
   # The connect method will create a connection object based the provider type passed in
   # If provider is left blank the function will use the first available provider
 
-    def self.connect(user, pass, host, provider="any",debug=false)
-
-      # use this variable to reduce cmd calls
-      installed = false
-
-      # use the first available provider
-      if provider == "any"
-        if is_provider_installed?("freeipmi")
-          provider = "freeipmi"
-          installed = true
-        elsif is_provider_installed?("ipmitool")
-          provider = "ipmitool"
-          installed = true
-        else
-          raise "No IPMI provider is installed, please install freeipmi or ipmitool"
-        end
-      end
+  def self.connect(user, pass, host, provider='any', opts={:driver => 'auto',
+                                                           :timeout => 'default', :debug => false})
 
-      # If the provider is available create a connection object
-      if installed or is_provider_installed?(provider)
-        if provider == "freeipmi"
-          @conn = Rubyipmi::Freeipmi::Connection.new(user, pass, host, debug)
-        elsif provider == "ipmitool"
-          @conn = Rubyipmi::Ipmitool::Connection.new(user,pass,host, debug)
-        else
-          raise "Incorrect provider given, must use freeipmi or ipmitool"
-        end
-      else
-        # Can't find the provider command line tool, maybe try other provider?
-        raise "The IPMI provider: #{provider} is not installed"
+    # use this variable to reduce cmd calls
+    installed = false
 
-      end
+    if provider.is_a?(Hash)
+      opts = provider
+      provider = 'any'
     end
 
-    def self.connection
-      return @conn if @conn
-      raise "No Connection available, please use the connect method"
-    end
+    # Verify options just in case user passed in a incomplete hash
+    opts[:driver]  ||= 'auto'
+    opts[:timeout] ||= 'default'
+    opts[:debug]   = false if opts[:debug] != true
 
-    # method used to find the command which also makes it easier to mock with
-    def self.locate_command(commandname)
-      location = `which #{commandname}`.strip
-      if not $?.success?
-        location = nil
+    if ! opts[:privilege].nil?
+      if ! supported_privilege_type?(opts[:privilege])
+        raise "Invalid privilege type :#{opts[:privilege]}, must be one of: #{PRIV_TYPES.join("\n")}"
       end
-      return location
     end
 
-    # Return true or false if the provider is available
-    def self.is_provider_installed?(provider)
-      case provider
-        when "freeipmi"
-          cmdpath = locate_command('ipmipower')
-        when "ipmitool"
-          cmdpath = locate_command('ipmitool')
-        else
-          raise "Invalid BMC provider type"
+    # use the first available provider
+    if provider == 'any'
+      if is_provider_installed?("freeipmi")
+        provider = "freeipmi"
+        installed = true
+      elsif is_provider_installed?("ipmitool")
+        provider = "ipmitool"
+        installed = true
+      else
+        raise "No IPMI provider is installed, please install freeipmi or ipmitool"
       end
-      # return false if command was not found
-      return ! cmdpath.nil?
     end
 
-    def self.providers
-      ["freeipmi", "ipmitool"]
+    # Support multiple drivers
+    # Note: these are just generic names of drivers that need to be specified for each provider
+    unless valid_drivers.include?(opts[:driver])
+      raise "You must specify a valid driver: #{valid_drivers.join(',')}"
     end
 
-    # returns true if any of the providers are installed
-    def self.provider_installed?
-      providers_installed?.length > 0
-    end
+    # If the provider is available create a connection object
+    if installed or is_provider_installed?(provider)
+      if provider == "freeipmi"
+        @conn = Rubyipmi::Freeipmi::Connection.new(user, pass, host, opts)
+      elsif provider == "ipmitool"
+        @conn = Rubyipmi::Ipmitool::Connection.new(user,pass,host, opts)
+      else
+        raise "Incorrect provider given, must use freeipmi or ipmitool"
+      end
+    else
+      # Can't find the provider command line tool, maybe try other provider?
+      raise "The IPMI provider: #{provider} is not installed"
 
-    def self.providers_installed?
-      available = []
-      providers.each do |prov|
-        if is_provider_installed?(prov)
-          available << prov
-        end
+    end
+  end
+
+  # returns boolean true if privilege type is valid
+  def self.supported_privilege_type?(type)
+     PRIV_TYPES.include?(type)
+  end
+
+  def self.connection
+    return @conn if @conn
+    raise "No Connection available, please use the connect method"
+  end
+
+  # method used to find the command which also makes it easier to mock with
+  def self.locate_command(commandname)
+    location = `which #{commandname}`.strip
+    if not $?.success?
+      location = nil
+    end
+    return location
+  end
+
+  # Return true or false if the provider is available
+  def self.is_provider_installed?(provider)
+    case provider
+      when "freeipmi"
+        cmdpath = locate_command('ipmipower')
+      when "ipmitool"
+        cmdpath = locate_command('ipmitool')
+      else
+        raise "Invalid BMC provider type"
+    end
+    # return false if command was not found
+    return ! cmdpath.nil?
+  end
+
+  def self.providers
+    ["freeipmi", "ipmitool"]
+  end
+
+  # returns true if any of the providers are installed
+  def self.provider_installed?
+    providers_installed?.length > 0
+  end
+
+  def self.providers_installed?
+    available = []
+    providers.each do |prov|
+      if is_provider_installed?(prov)
+        available << prov
       end
-      return available
     end
-
-    # gets data from the bmc device and puts in a hash for diagnostics
-    def self.get_diag(user, pass, host)
-      data = {}
-
-      if Rubyipmi.is_provider_installed?('freeipmi')
-        @freeconn = Rubyipmi::connect(user, pass, host, 'freeipmi')
-        if @freeconn
-          puts "Retrieving freeipmi data"
-          data['freeipmi'] = @freeconn.get_diag
-        end
+    return available
+  end
+
+  # gets data from the bmc device and puts in a hash for diagnostics
+  def self.get_diag(user, pass, host)
+    data = {}
+
+    if Rubyipmi.is_provider_installed?('freeipmi')
+      @freeconn = Rubyipmi::connect(user, pass, host, 'freeipmi')
+      if @freeconn
+        puts "Retrieving freeipmi data"
+        data['freeipmi'] = @freeconn.get_diag
       end
-      if Rubyipmi.is_provider_installed?('ipmitool')
-        @ipmiconn = Rubyipmi::connect(user, pass, host, 'ipmitool')
-        if @ipmiconn
-          puts "Retrieving ipmitool data"
-          data['ipmitool'] = @ipmiconn.get_diag
-        end
+    end
+    if Rubyipmi.is_provider_installed?('ipmitool')
+      @ipmiconn = Rubyipmi::connect(user, pass, host, 'ipmitool')
+      if @ipmiconn
+        puts "Retrieving ipmitool data"
+        data['ipmitool'] = @ipmiconn.get_diag
       end
-      return data
     end
+    return data
+  end
 
 end

data/rubyipmi.gemspec

@@ -2,14 +2,16 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
+# stub: rubyipmi 0.8.1 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "rubyipmi"
-  s.version = "0.7.0"
+  s.version = "0.8.1"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
+  s.require_paths = ["lib"]
   s.authors = ["Corey Osman"]
-  s.date = "2013-10-17"
+  s.date = "2014-10-02"
   s.description = "Provides a library for controlling IPMI devices using pure ruby code"
   s.email = "corey@logicminds.biz"
   s.extra_rdoc_files = [
@@ -22,6 +24,7 @@ Gem::Specification.new do |s|
     "LICENSE.txt",
     "README.md",
     "README.rdoc",
+    "RELEASE_NOTES.md",
     "Rakefile",
     "VERSION",
     "lib/rubyipmi.rb",
@@ -102,8 +105,7 @@ Gem::Specification.new do |s|
   ]
   s.homepage = "http://github.com/logicminds/rubyipmi"
   s.licenses = ["GPLv3"]
-  s.require_paths = ["lib"]
-  s.rubygems_version = "2.0.7"
+  s.rubygems_version = "2.2.1"
   s.summary = "A ruby wrapper for ipmi command line tools that supports ipmitool and freeipmi"
 
   if s.respond_to? :specification_version then
@@ -114,7 +116,6 @@ Gem::Specification.new do |s|
       s.add_development_dependency(%q<rdoc>, ["~> 3.12"])
       s.add_development_dependency(%q<bundler>, [">= 1.1.5"])
       s.add_development_dependency(%q<jeweler>, ["~> 1.8.4"])
-      s.add_development_dependency(%q<rcov>, ["< 1.0.0"])
       s.add_development_dependency(%q<highline>, [">= 0"])
       s.add_development_dependency(%q<rake>, [">= 0"])
     else
@@ -122,7 +123,6 @@ Gem::Specification.new do |s|
       s.add_dependency(%q<rdoc>, ["~> 3.12"])
       s.add_dependency(%q<bundler>, [">= 1.1.5"])
       s.add_dependency(%q<jeweler>, ["~> 1.8.4"])
-      s.add_dependency(%q<rcov>, ["< 1.0.0"])
       s.add_dependency(%q<highline>, [">= 0"])
       s.add_dependency(%q<rake>, [">= 0"])
     end
@@ -131,7 +131,6 @@ Gem::Specification.new do |s|
     s.add_dependency(%q<rdoc>, ["~> 3.12"])
     s.add_dependency(%q<bundler>, [">= 1.1.5"])
     s.add_dependency(%q<jeweler>, ["~> 1.8.4"])
-    s.add_dependency(%q<rcov>, ["< 1.0.0"])
     s.add_dependency(%q<highline>, [">= 0"])
     s.add_dependency(%q<rake>, [">= 0"])
   end

data/spec/unit/freeipmi/bmc-info_spec.rb

@@ -15,7 +15,7 @@ describe "Bmc" do
     host = "ipmihost"
     Rubyipmi.stub(:locate_command).with('ipmipower').and_return("#{@path}/ipmipower")
 
-    @conn = Rubyipmi.connect(user, pass, host, provider, true)
+    @conn = Rubyipmi.connect(user, pass, host, provider, {:debug => true})
     @bmcinfo = @conn.bmc.information
     data = nil
     File.open("spec/fixtures/#{provider}/bmc_info.txt",'r') do |file|
@@ -26,17 +26,14 @@ describe "Bmc" do
     $?.stub(:success?).and_return(true)
   end
 
- it "cmd should be bmc-info with correct number of arguments" do
+  it "cmd should be bmc-info with correct number of arguments" do
     @bmcinfo.retrieve
     verify_freeipmi_command(@bmcinfo, 2, "#{@path}/bmc-info")
- end
+  end
 
- it "cmd should be bmc-info with correct number of arguments" do
+  it "cmd should be bmc-info with correct number of arguments" do
     @bmcinfo.guid
     verify_freeipmi_command(@bmcinfo, 3, "#{@path}/bmc-info")
- end
-
-
-
+  end
 
 end

data/spec/unit/freeipmi/bmc_spec.rb

@@ -15,7 +15,7 @@ describe "Bmc" do
     host = "ipmihost"
     Rubyipmi.stub(:locate_command).with('ipmipower').and_return("#{@path}/ipmipower")
 
-    @conn = Rubyipmi.connect(user, pass, host, provider, true)
+    @conn = Rubyipmi.connect(user, pass, host, provider,{:debug => true})
     @bmc = @conn.bmc
     data = nil
     File.open("spec/fixtures/#{provider}/bmc_info.txt",'r') do |file|

data/spec/unit/freeipmi/connection_spec.rb

@@ -7,17 +7,18 @@ describe "Bmc" do
 
   before :all do
     @path = '/usr/local/bin'
+    @provider = "freeipmi"
+    @user = "ipmiuser"
+    @pass = "impipass"
+    @host = "ipmihost"
   end
 
  before :each do
 
-    provider = "freeipmi"
-    user = "ipmiuser"
-    pass = "impipass"
-    host = "ipmihost"
+
     Rubyipmi.stub(:locate_command).with('ipmipower').and_return("#{@path}/ipmipower")
 
-    @conn = Rubyipmi.connect(user, pass, host, provider, true)
+    @conn = Rubyipmi.connect(@user, @pass, @host, @provider,{:debug => true})
 
 
   end
@@ -52,5 +53,50 @@ describe "Bmc" do
 
   end
 
+  it 'object should have debug set to true' do
+    @conn.debug.should be_true
+  end
+
+  it 'object should have driver set to auto if not specified' do
+    @conn.options.has_key?('driver-type').should be_false
+  end
+
+  it 'object should have driver set to auto if not specified' do
+    @conn = Rubyipmi.connect(@user, @pass, @host, @provider,{:debug => true, :driver => 'auto'})
+    @conn.options.has_key?('driver-type').should be_false
+  end
+
+  it 'object should have priv type set to ADMINISTRATOR if not specified' do
+    @conn = Rubyipmi.connect(@user, @pass, @host, @provider,{:debug => true, :driver => 'auto'})
+    @conn.options.has_key?('privilege-level').should be_false
+  end
+
+  it 'object should have priv type set to USER ' do
+    @conn = Rubyipmi.connect(@user, @pass, @host, @provider,{:privilege => 'USER', :debug => true, :driver => 'auto'})
+    @conn.options.fetch('privilege-level').should eq('USER')
+  end
+
+  it 'should raise exception if invalid privilege type' do
+    expect{Rubyipmi.connect(@user, @pass, @host, @provider,{:privilege => 'BLAH',:debug => true, :driver => 'auto'})}.to raise_error(RuntimeError)
+  end
+
+  it 'should raise exception if invalid driver type' do
+    expect{Rubyipmi.connect(@user, @pass, @host, @provider,{:debug => true, :driver => 'foo'})}.to raise_error(RuntimeError)
+  end
+
+  it 'object should have driver set to lan_2_0' do
+    @conn = Rubyipmi.connect(@user, @pass, @host, @provider,{:debug => true, :driver => 'lan20'})
+    @conn.options['driver-type'].should eq('LAN_2_0')
+  end
+
+  it 'object should have driver set to lan' do
+    @conn = Rubyipmi.connect(@user, @pass, @host, @provider,{:debug => true, :driver => 'lan15'})
+    @conn.options['driver-type'].should eq('LAN')
+  end
+
+  it 'object should have driver set to openipmi' do
+    @conn = Rubyipmi.connect(@user, @pass, @host, @provider,{:debug => true, :driver => 'open'})
+    @conn.options['driver-type'].should eq('OPENIPMI')
+  end
 
 end

data/spec/unit/freeipmi/fru_spec.rb

@@ -16,7 +16,7 @@ describe :Fru do
     host = "ipmihost"
     Rubyipmi.stub(:locate_command).with('ipmipower').and_return("#{@path}/ipmipower")
 
-    @conn = Rubyipmi.connect(user, pass, host, provider, true)
+    @conn = Rubyipmi.connect(user, pass, host, provider, {:debug => true})
     @fru = @conn.fru
     File.open("spec/fixtures/#{provider}/fru.txt",'r') do |file|
       data = file.read

data/spec/unit/freeipmi/sensors_spec.rb

@@ -14,7 +14,7 @@ describe :Sensors do
     # this stub allows us to mock the command that would be used to verify provider installation
     Rubyipmi.stub(:locate_command).with('ipmipower').and_return("#{@path}/ipmipower")
 
-    @conn = Rubyipmi.connect(user, pass, host, provider, true)
+    @conn = Rubyipmi.connect(user, pass, host, provider, {:debug => true})
     @sensors = @conn.sensors
     File.open("spec/fixtures/#{provider}/sensors.txt",'r') do |file|
       data = file.read

data/spec/unit/ipmitool/bmc_spec.rb

@@ -15,7 +15,7 @@ describe "Bmc" do
     pass = "impipass"
     host = "ipmihost"
     Rubyipmi.stub(:locate_command).with('ipmitool').and_return("#{@path}/ipmitool")
-    @conn = Rubyipmi.connect(user, pass, host, provider, true)
+    @conn = Rubyipmi.connect(user, pass, host, provider, {:debug => true})
     @bmc = @conn.bmc
     data = nil
     File.open("spec/fixtures/#{provider}/bmc_info.txt",'r') do |file|

data/spec/unit/ipmitool/connection_spec.rb

@@ -4,15 +4,16 @@ describe :Connection do
 
   before :all do
     @path = '/usr/local/bin'
+    @provider = "ipmitool"
+    @user = "ipmiuser"
+    @pass = "impipass"
+    @host = "ipmihost"
   end
 
   before :each do
-    provider = "ipmitool"
-    user = "ipmiuser"
-    pass = "impipass"
-    host = "ipmihost"
+
     Rubyipmi.stub(:locate_command).with('ipmitool').and_return("#{@path}/ipmitool")
-    @conn = Rubyipmi.connect(user, pass, host, provider, true)
+    @conn = Rubyipmi.connect(@user, @pass, @host, @provider,{:debug => true})
 
   end
 
@@ -53,6 +54,46 @@ describe :Connection do
     @conn.debug.should be_true
   end
 
+  it 'object should have driver set to auto if not specified' do
+    @conn.options.has_key?('driver-type').should be_false
+  end
 
+  it 'object should have driver set to auto if not specified' do
+    @conn = Rubyipmi.connect(@user, @pass, @host, @provider,{:debug => true, :driver => 'auto'})
+    @conn.options.has_key?('I').should be_false
+  end
+
+  it 'should raise exception if invalid driver type' do
+    expect{Rubyipmi.connect(@user, @pass, @host, @provider,{:debug => true, :driver => 'foo'})}.to raise_error(RuntimeError)
+  end
+
+  it 'object should have priv type set to ADMINISTRATOR if not specified' do
+    @conn = Rubyipmi.connect(@user, @pass, @host, @provider,{:debug => true, :driver => 'auto'})
+    @conn.options.has_key?('L').should be_false
+  end
+
+  it 'object should have priv type set to USER ' do
+    @conn = Rubyipmi.connect(@user, @pass, @host, @provider,{:privilege => 'USER', :debug => true, :driver => 'auto'})
+    @conn.options.fetch('L').should eq('USER')
+  end
+
+  it 'should raise exception if invalid privilege type' do
+    expect{Rubyipmi.connect(@user, @pass, @host, @provider,{:privilege => 'BLAH',:debug => true, :driver => 'auto'})}.to raise_error(RuntimeError)
+  end
+
+  it 'object should have driver set to lanplus' do
+    @conn = Rubyipmi.connect(@user, @pass, @host, @provider,{:debug => true, :driver => 'lan20'})
+    @conn.options['I'].should eq('lanplus')
+  end
+
+  it 'object should have driver set to lanplus' do
+    @conn = Rubyipmi.connect(@user, @pass, @host, @provider,{:debug => true, :driver => 'lan15'})
+    @conn.options['I'].should eq('lan')
+  end
+
+  it 'object should have driver set to open' do
+    @conn = Rubyipmi.connect(@user, @pass, @host, @provider,{:debug => true, :driver => 'open'})
+    @conn.options['I'].should eq('open')
+  end
 
 end

data/spec/unit/ipmitool/fru_spec.rb

@@ -15,7 +15,7 @@ describe :Fru do
     host = "ipmihost"
     Rubyipmi.stub(:locate_command).with('ipmitool').and_return("#{@path}/ipmitool")
 
-    @conn = Rubyipmi.connect(user, pass, host, provider, true)
+    @conn = Rubyipmi.connect(user, pass, host, provider, {:debug => true})
     @fru = @conn.fru
 
     File.open("spec/fixtures/#{provider}/fru.txt",'r') do |file|

data/spec/unit/ipmitool/lan_spec.rb

@@ -12,7 +12,7 @@ describe "Lan" do
     pass = "impipass"
     host = "ipmihost"
     Rubyipmi.stub(:locate_command).with('ipmitool').and_return("#{@path}/ipmitool")
-    @conn = Rubyipmi.connect(user, pass, host, provider, true)
+    @conn = Rubyipmi.connect(user, pass, host, provider, {:debug => true})
     @lan = @conn.bmc.lan
     data = nil
     File.open("spec/fixtures/#{provider}/lan.txt",'r') do |file|

data/spec/unit/ipmitool/sensors_spec.rb

@@ -14,7 +14,7 @@ describe :Sensors do
     host = "ipmihost"
     Rubyipmi.stub(:locate_command).with('ipmitool').and_return("#{@path}/ipmitool")
 
-    @conn = Rubyipmi.connect(user, pass, host, provider, true)
+    @conn = Rubyipmi.connect(user, pass, host, provider, {:debug => true})
     @sensors = @conn.sensors
     File.open("spec/fixtures/#{provider}/sensors.txt",'r') do |file|
       data = file.read