chef 10.34.6-x86-mingw32 → 11.6.2-x86-mingw32

data/spec/unit/provider/ifconfig/debian_spec.rb

@@ -0,0 +1,89 @@
+#
+# Author:: Xabier de Zuazo (xabier@onddo.com)
+# Copyright:: Copyright (c) 2013 Onddo Labs, SL.
+# License:: Apache License, Version 2.0
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+require 'spec_helper'
+require 'chef/exceptions'
+
+describe Chef::Provider::Ifconfig::Debian do
+  before do
+    @node = Chef::Node.new
+    @cookbook_collection = Chef::CookbookCollection.new([])
+    @events = Chef::EventDispatch::Dispatcher.new
+    @run_context = Chef::RunContext.new(@node, @cookbook_collection, @events)
+    #This new_resource can be called anything --> it is not the same as in ifconfig.rb
+    @new_resource = Chef::Resource::Ifconfig.new("10.0.0.1", @run_context)
+    @new_resource.mask "255.255.254.0"
+    @new_resource.metric "1"
+    @new_resource.mtu "1500"
+    @new_resource.device "eth0"
+    @provider = Chef::Provider::Ifconfig::Debian.new(@new_resource, @run_context)
+    @current_resource = Chef::Resource::Ifconfig.new("10.0.0.1", @run_context)
+
+    status = mock("Status", :exitstatus => 0)
+    @provider.instance_variable_set("@status", status)
+    @provider.current_resource = @current_resource
+    @provider.stub!(:load_current_resource)
+    @provider.stub!(:run_command)
+
+    @config_filename_ifaces = "/etc/network/interfaces"
+    @config_filename_ifcfg = "/etc/network/interfaces.d/ifcfg-#{@new_resource.device}"
+  end
+
+  describe "generate_config for action_add" do
+   before do
+    @config_file_ifaces = StringIO.new
+    @config_file_ifcfg = StringIO.new
+    FileUtils.should_receive(:cp)
+    File.should_receive(:new).with(@config_filename_ifaces).and_return(StringIO.new)
+    File.should_receive(:open).with(@config_filename_ifaces, "w").and_yield(@config_file_ifaces)
+    File.should_receive(:new).with(@config_filename_ifcfg, "w").and_return(@config_file_ifcfg)
+    File.should_receive(:exist?).with(@config_filename_ifaces).and_return(true)
+   end
+
+   it "should create network-scripts directory" do
+    File.should_receive(:directory?).with(File.dirname(@config_filename_ifcfg)).and_return(false)
+    Dir.should_receive(:mkdir).with(File.dirname(@config_filename_ifcfg))
+    @provider.run_action(:add)
+   end
+
+   it "should write configure network-scripts directory" do
+    File.should_receive(:directory?).with(File.dirname(@config_filename_ifcfg)).and_return(true)
+    @provider.run_action(:add)
+    @config_file_ifaces.string.should match(/^\s*source\s+\/etc\/network\/interfaces[.]d\/[*]\s*$/)
+   end
+
+   it "should write a network-script" do
+    File.should_receive(:directory?).with(File.dirname(@config_filename_ifcfg)).and_return(true)
+    @provider.run_action(:add)
+    @config_file_ifcfg.string.should match(/^iface eth0 inet static\s*$/)
+    @config_file_ifcfg.string.should match(/^\s+address 10\.0\.0\.1\s*$/)
+    @config_file_ifcfg.string.should match(/^\s+netmask 255\.255\.254\.0\s*$/)
+   end
+  end
+
+  describe "delete_config for action_delete" do
+
+    it "should delete network-script if it exists" do
+      @current_resource.device @new_resource.device
+      File.should_receive(:exist?).with(@config_filename_ifcfg).and_return(true)
+      FileUtils.should_receive(:rm_f).with(@config_filename_ifcfg, :verbose => false)
+
+      @provider.run_action(:delete)
+    end
+  end
+end

data/spec/unit/provider/ifconfig/redhat_spec.rb

@@ -0,0 +1,71 @@
+#
+# Author:: Xabier de Zuazo (xabier@onddo.com)
+# Copyright:: Copyright (c) 2013 Onddo Labs, SL.
+# License:: Apache License, Version 2.0
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+require 'spec_helper'
+require 'chef/exceptions'
+
+describe Chef::Provider::Ifconfig::Redhat do
+  before do
+    @node = Chef::Node.new
+    @cookbook_collection = Chef::CookbookCollection.new([])
+    @events = Chef::EventDispatch::Dispatcher.new
+    @run_context = Chef::RunContext.new(@node, @cookbook_collection, @events)
+    #This new_resource can be called anything --> it is not the same as in ifconfig.rb
+    @new_resource = Chef::Resource::Ifconfig.new("10.0.0.1", @run_context)
+    @new_resource.mask "255.255.254.0"
+    @new_resource.metric "1"
+    @new_resource.mtu "1500"
+    @new_resource.device "eth0"
+    @provider = Chef::Provider::Ifconfig::Redhat.new(@new_resource, @run_context)
+    @current_resource = Chef::Resource::Ifconfig.new("10.0.0.1", @run_context)
+
+    status = mock("Status", :exitstatus => 0)
+    @provider.instance_variable_set("@status", status)
+    @provider.current_resource = @current_resource
+ end
+
+  describe "generate_config for action_add" do
+
+     it "should write network-script for centos" do
+      @provider.stub!(:load_current_resource)
+      @provider.stub!(:run_command)
+      config_filename = "/etc/sysconfig/network-scripts/ifcfg-#{@new_resource.device}"
+      config_file = StringIO.new
+      File.should_receive(:new).with(config_filename, "w").and_return(config_file)
+
+      @provider.run_action(:add)
+      config_file.string.should match(/^\s*DEVICE=eth0\s*$/)
+      config_file.string.should match(/^\s*IPADDR=10\.0\.0\.1\s*$/)
+      config_file.string.should match(/^\s*NETMASK=255\.255\.254\.0\s*$/)
+     end
+  end
+
+  describe "delete_config for action_delete" do
+
+    it "should delete network-script if it exists for centos" do
+      @current_resource.device @new_resource.device
+      @provider.stub!(:load_current_resource)
+      @provider.stub!(:run_command)
+      config_filename =  "/etc/sysconfig/network-scripts/ifcfg-#{@new_resource.device}"
+      File.should_receive(:exist?).with(config_filename).and_return(true)
+      FileUtils.should_receive(:rm_f).with(config_filename, :verbose => false)
+
+      @provider.run_action(:delete)
+    end
+  end
+end

data/spec/unit/provider/ifconfig_spec.rb

@@ -177,37 +177,4 @@ describe Chef::Provider::Ifconfig do
       @new_resource.should_not be_updated
     end
   end
-  
-  describe Chef::Provider::Ifconfig, "generate_config for action_add" do
-   #%w[ centos redhat fedora ].each do |platform|
-     
-     it "should write network-script for centos" do
-      @provider.stub!(:load_current_resource)
-      @node[:platform] = "centos"
-      @provider.stub!(:run_command)
-      config_filename = "/etc/sysconfig/network-scripts/ifcfg-#{@new_resource.device}"
-      config_file = StringIO.new
-      File.should_receive(:new).with(config_filename, "w").and_return(config_file)
-
-      @provider.run_action(:add)
-      config_file.string.should match(/^\s*DEVICE=eth0\s*$/)
-      config_file.string.should match(/^\s*IPADDR=10.0.0.1\s*$/)
-      config_file.string.should match(/^\s*NETMASK=255.255.254.0\s*$/)
-     end
-  end
-
-  describe Chef::Provider::Ifconfig, "delete_config for action_delete" do
-
-    it "should delete network-script if it exists for centos" do
-      @node[:platform] = "centos"
-      @current_resource.device "eth0"
-      @provider.stub!(:load_current_resource)
-      @provider.stub!(:run_command)
-      config_filename =  "/etc/sysconfig/network-scripts/ifcfg-#{@new_resource.device}"
-      File.should_receive(:exist?).with(config_filename).and_return(true)
-      FileUtils.should_receive(:rm_f).with(config_filename, :verbose => false)
-
-      @provider.run_action(:delete)
-    end
-  end
 end

data/spec/unit/provider/mount/mount_spec.rb

@@ -108,13 +108,18 @@ describe Chef::Provider::Mount::Mount do
     it "does not expect the device to exist for tmpfs" do
       @new_resource.fstype("tmpfs")
       @new_resource.device("whatever")
-      lambda { @provider.load_current_resource() }.should_not raise_error
+      lambda { @provider.load_current_resource();@provider.mountable? }.should_not raise_error
     end
 
     it "does not expect the device to exist for Fuse filesystems" do
       @new_resource.fstype("fuse")
       @new_resource.device("nilfs#xxx")
-      lambda { @provider.load_current_resource() }.should_not raise_error
+      lambda { @provider.load_current_resource();@provider.mountable? }.should_not raise_error
+    end
+
+    it "does not expect the device to exist if it's none" do
+      @new_resource.device("none")
+      lambda { @provider.load_current_resource();@provider.mountable? }.should_not raise_error
     end
 
     it "should set mounted true if the mount point is found in the mounts list" do
@@ -216,6 +221,32 @@ describe Chef::Provider::Mount::Mount do
       @provider.load_current_resource
       @provider.current_resource.enabled.should be_false
     end
+
+    it "should not mangle the mount options if the device in fstab is a symlink" do
+      target = "/dev/mapper/target"
+      options = "rw,noexec,noauto"
+
+      ::File.stub!(:symlink?).with(@new_resource.device).and_return(true)
+      ::File.stub!(:readlink).with(@new_resource.device).and_return(target)
+
+      fstab = "#{@new_resource.device} #{@new_resource.mount_point} #{@new_resource.fstype} #{options} 1 2\n"
+      ::File.stub!(:foreach).with("/etc/fstab").and_yield fstab
+      @provider.load_current_resource
+      @provider.current_resource.options.should eq(options.split(','))
+    end
+
+    it "should not mangle the mount options if the symlink target is in fstab" do
+      target = "/dev/mapper/target"
+      options = "rw,noexec,noauto"
+
+      ::File.stub!(:symlink?).with(@new_resource.device).and_return(true)
+      ::File.stub!(:readlink).with(@new_resource.device).and_return(target)
+
+      fstab = "#{target} #{@new_resource.mount_point} #{@new_resource.fstype} #{options} 1 2\n"
+      ::File.stub!(:foreach).with("/etc/fstab").and_yield fstab
+      @provider.load_current_resource
+      @provider.current_resource.options.should eq(options.split(','))
+    end
   end
 
   context "after the mount's state has been discovered" do

data/spec/unit/provider/mount/windows_spec.rb

@@ -99,7 +99,10 @@ describe Chef::Provider::Mount::Windows do
       end
 
       it "should mount the filesystem if it is not mounted" do
-        @vol.should_receive(:add).with(@new_resource.device)
+        @vol.should_receive(:add).with(:remote => @new_resource.device,
+                                       :username => @new_resource.username,
+                                       :domainname => @new_resource.domain,
+                                       :password => @new_resource.password)
         @provider.mount_fs
       end
 

data/spec/unit/provider/mount_spec.rb

@@ -51,7 +51,7 @@ describe Chef::Provider::Mount do
 
     it "should not mount the filesystem if it is mounted" do
       @current_resource.stub!(:mounted).and_return(true)
-      @provider.should_not_receive(:mount_fs).and_return(true)
+      @provider.should_not_receive(:mount_fs)
       @provider.run_action(:mount)
       @new_resource.should_not be_updated_by_last_action
     end
@@ -68,7 +68,7 @@ describe Chef::Provider::Mount do
 
     it "should not umount the filesystem if it is not mounted" do
       @current_resource.stub!(:mounted).and_return(false)
-      @provider.should_not_receive(:umount_fs).and_return(true)
+      @provider.should_not_receive(:umount_fs)
       @provider.run_action(:umount)
       @new_resource.should_not be_updated_by_last_action
     end
@@ -108,14 +108,24 @@ describe Chef::Provider::Mount do
   describe "when enabling the filesystem to be mounted" do
     it "should enable the mount if it isn't enable" do
       @current_resource.stub!(:enabled).and_return(false)
-      @provider.should_receive(:enable_fs).with.and_return(true)
+      @provider.should_not_receive(:mount_options_unchanged?)
+      @provider.should_receive(:enable_fs).and_return(true)
       @provider.run_action(:enable)
       @new_resource.should be_updated_by_last_action
     end
 
-    it "should not enable the mount if it is enabled" do
+    it "should enable the mount if it is enabled and mount options have changed" do
       @current_resource.stub!(:enabled).and_return(true)
-      @provider.should_not_receive(:enable_fs).with.and_return(true)
+      @provider.should_receive(:mount_options_unchanged?).and_return(false)
+      @provider.should_receive(:enable_fs).and_return(true)
+      @provider.run_action(:enable)
+      @new_resource.should be_updated_by_last_action
+    end
+
+    it "should not enable the mount if it is enabled and mount options have not changed" do
+      @current_resource.stub!(:enabled).and_return(true)
+      @provider.should_receive(:mount_options_unchanged?).and_return(true)
+      @provider.should_not_receive(:enable_fs).and_return(true)
       @provider.run_action(:enable)
       @new_resource.should_not be_updated_by_last_action
     end
@@ -131,7 +141,7 @@ describe Chef::Provider::Mount do
 
     it "should not disable the mount if it isn't enabled" do
       @current_resource.stub!(:enabled).and_return(false)
-      @provider.should_not_receive(:disable_fs).with.and_return(true)
+      @provider.should_not_receive(:disable_fs)
       @provider.run_action(:disable)
       @new_resource.should_not be_updated_by_last_action
     end

data/spec/unit/provider/ohai_spec.rb

@@ -34,7 +34,7 @@ describe Chef::Provider::Ohai do
       :platform => @platform,
       :platform_version => @platform_version,
       :data => {
-        :origdata => "somevalue"      
+        :origdata => "somevalue"
       },
       :data2 => {
         :origdata => "somevalue",
@@ -49,7 +49,7 @@ describe Chef::Provider::Ohai do
     Chef::Platform.stub!(:find_platform_and_version).and_return({ "platform" => @platform,
                                                                   "platform_version" => @platform_version})
     # Fake node with a dummy save
-    @node = Chef::Node.new(@hostname)
+    @node = Chef::Node.new
     @node.name(@fqdn)
     @node.stub!(:save).and_return(@node)
     @events = Chef::EventDispatch::Dispatcher.new
@@ -57,14 +57,14 @@ describe Chef::Provider::Ohai do
     @new_resource = Chef::Resource::Ohai.new("ohai_reload")
     ohai = Ohai::System.new
     ohai.all_plugins
-    @node.process_external_attrs(ohai.data,{})
+    @node.consume_external_attrs(ohai.data,{})
 
     @provider = Chef::Provider::Ohai.new(@new_resource, @run_context)
   end
 
   describe "when reloading ohai" do
     before do
-      @node[:origdata] = 'somevalue'
+      @node.automatic_attrs[:origdata] = 'somevalue'
     end
 
     it "applies updated ohai data to the node" do

data/spec/unit/provider/package/apt_spec.rb

@@ -22,7 +22,6 @@ require 'ostruct'
 describe Chef::Provider::Package::Apt do
   before(:each) do
     @node = Chef::Node.new
-    @node.cookbook_collection = {}
     @events = Chef::EventDispatch::Dispatcher.new
     @run_context = Chef::RunContext.new(@node, {}, @events)
     @new_resource = Chef::Resource::Package.new("irssi", @run_context)

data/spec/unit/provider/package/ips_spec.rb

@@ -24,7 +24,6 @@ require 'ostruct'
 describe Chef::Provider::Package::Ips do
   before(:each) do
     @node = Chef::Node.new
-    @node.cookbook_collection = {}
     @events = Chef::EventDispatch::Dispatcher.new
     @run_context = Chef::RunContext.new(@node, {}, @events)
     @new_resource = Chef::Resource::Package.new("crypto/gnupg", @run_context)

data/spec/unit/provider/package/rpm_spec.rb

@@ -80,18 +80,6 @@ describe Chef::Provider::Package::Rpm do
       @provider.stub!(:popen4).and_return(status)
       lambda { @provider.run_action(:any) }.should raise_error(Chef::Exceptions::Package)
     end
-
-    it "should not detect the package name as version when not installed" do
-      @status = double("Status", :exitstatus => -1)
-      @stdout = StringIO.new("package openssh-askpass is not installed")
-      @new_resource = Chef::Resource::Package.new("openssh-askpass")
-      @new_resource.source 'openssh-askpass'
-      @provider = Chef::Provider::Package::Rpm.new(@new_resource, @run_context)
-      @provider.should_receive(:popen4).with("rpm -qp --queryformat '%{NAME} %{VERSION}-%{RELEASE}\n' openssh-askpass").and_yield(@pid, @stdin, @stdout, @stderr).and_return(@status)
-      @provider.should_receive(:popen4).with("rpm -q --queryformat '%{NAME} %{VERSION}-%{RELEASE}\n' openssh-askpass").and_return(@status)
-      @provider.load_current_resource
-      @provider.current_resource.version.should be_nil
-    end
   end
   
   describe "after the current resource is loaded" do

data/spec/unit/provider/package/rubygems_spec.rb

@@ -106,7 +106,7 @@ describe Chef::Provider::Package::Rubygems::CurrentGemEnvironment do
     dep_installer.should_receive(:find_gems_with_sources).with(dep).and_return(available_set)
     @gem_env.candidate_version_from_remote(Gem::Dependency.new('rspec', '>= 0')).should == Gem::Version.new('1.3.0')
   end
-  
+
   context "when rubygems was upgraded from 1.8->2.0" do
     # https://github.com/rubygems/rubygems/issues/404
     # tl;dr rubygems 1.8 and 2.0 can both be in the load path, which means that

data/spec/unit/provider/package/smartos_spec.rb

@@ -1,5 +1,6 @@
 #
 # Author:: Trevor O (trevoro@joyent.com)
+# Author:: Yukihiko Sawanobori (sawanoboriyu@higanworks.com)
 # Copyright:: Copyright (c) 2012 Opscode
 # License:: Apache License, Version 2.0
 #
@@ -72,8 +73,8 @@ describe Chef::Provider::Package::SmartOS, "load_current_resource" do
 	
 		it "run pkgin and install the package" do
 			out = OpenStruct.new(:stdout => nil)
-      @provider.should_receive(:shell_out!).with("pkg_info -E \"varnish*\"", {:env => nil, :returns=>[0,1]}).and_return(@shell_out)
-      @provider.should_receive(:shell_out!).with("pkgin -y install varnish-2.1.5nb2", {:env=>nil}).and_return(out)
+      @provider.should_receive(:shell_out!).with("/opt/local/sbin/pkg_info -E \"varnish*\"", {:env => nil, :returns=>[0,1]}).and_return(@shell_out)
+      @provider.should_receive(:shell_out!).with("/opt/local/bin/pkgin -y install varnish-2.1.5nb2", {:env=>nil}).and_return(out)
       @provider.load_current_resource
       @provider.install_package("varnish", "2.1.5nb2")
 		end

data/spec/unit/provider/package/zypper_spec.rb

@@ -50,7 +50,7 @@ describe Chef::Provider::Package::Zypper do
     end
 
     it "should run zypper info with the package name" do
-      @provider.should_receive(:popen4).with("zypper info #{@new_resource.package_name}").and_return(@status)
+      @provider.should_receive(:popen4).with("zypper --non-interactive info #{@new_resource.package_name}").and_return(@status)
       @provider.load_current_resource
     end
 
@@ -92,34 +92,99 @@ describe Chef::Provider::Package::Zypper do
 
   describe "install_package" do
     it "should run zypper install with the package name and version" do
-      @provider.should_receive(:run_command).with({
-          :command => "zypper -n --no-gpg-checks install -l  emacs=1.0",
-        })
+      Chef::Config.stub(:[]).with(:zypper_check_gpg).and_return(true)
+      @provider.should_receive(:shell_out!).with(
+        "zypper --non-interactive install --auto-agree-with-licenses emacs=1.0")
+      @provider.install_package("emacs", "1.0")
+    end
+    it "should run zypper install without gpg checks" do
+      Chef::Config.stub(:[]).with(:zypper_check_gpg).and_return(false)
+      @provider.should_receive(:shell_out!).with(
+        "zypper --non-interactive --no-gpg-checks install "+
+        "--auto-agree-with-licenses emacs=1.0")
+      @provider.install_package("emacs", "1.0")
+    end
+    it "should warn about gpg checks on zypper install" do
+      Chef::Log.should_receive(:warn).with(
+        /All packages will be installed without gpg signature checks/)
+      @provider.should_receive(:shell_out!).with(
+        "zypper --non-interactive --no-gpg-checks install "+
+        "--auto-agree-with-licenses emacs=1.0")
       @provider.install_package("emacs", "1.0")
     end
   end
 
   describe "upgrade_package" do
     it "should run zypper update with the package name and version" do
-      @provider.should_receive(:run_command).with({
-          :command => "zypper -n --no-gpg-checks install -l emacs=1.0",
-        })
+      Chef::Config.stub(:[]).with(:zypper_check_gpg).and_return(true)
+      @provider.should_receive(:shell_out!).with(
+        "zypper --non-interactive install --auto-agree-with-licenses emacs=1.0")
+      @provider.upgrade_package("emacs", "1.0")
+    end
+    it "should run zypper update without gpg checks" do
+      Chef::Config.stub(:[]).with(:zypper_check_gpg).and_return(false)
+      @provider.should_receive(:shell_out!).with(
+        "zypper --non-interactive --no-gpg-checks install "+
+        "--auto-agree-with-licenses emacs=1.0")
+      @provider.upgrade_package("emacs", "1.0")
+    end
+    it "should warn about gpg checks on zypper upgrade" do
+      Chef::Log.should_receive(:warn).with(
+        /All packages will be installed without gpg signature checks/)
+      @provider.should_receive(:shell_out!).with(
+        "zypper --non-interactive --no-gpg-checks install "+
+        "--auto-agree-with-licenses emacs=1.0")
+      @provider.upgrade_package("emacs", "1.0")
+    end
+    it "should run zypper upgrade without gpg checks" do
+      @provider.should_receive(:shell_out!).with(
+        "zypper --non-interactive --no-gpg-checks install "+
+        "--auto-agree-with-licenses emacs=1.0")
+
       @provider.upgrade_package("emacs", "1.0")
     end
   end
 
   describe "remove_package" do
     it "should run zypper remove with the package name" do
-      @provider.should_receive(:run_command).with({
-          :command => "zypper -n --no-gpg-checks remove  emacs=1.0",
-        })
+      Chef::Config.stub(:[]).with(:zypper_check_gpg).and_return(true)
+      @provider.should_receive(:shell_out!).with(
+        "zypper --non-interactive remove emacs=1.0")
+      @provider.remove_package("emacs", "1.0")
+    end
+    it "should run zypper remove without gpg checks" do
+      Chef::Config.stub(:[]).with(:zypper_check_gpg).and_return(false)
+      @provider.should_receive(:shell_out!).with(
+          "zypper --non-interactive --no-gpg-checks remove emacs=1.0")
+      @provider.remove_package("emacs", "1.0")
+    end
+    it "should warn about gpg checks on zypper remove" do
+      Chef::Log.should_receive(:warn).with(
+        /All packages will be installed without gpg signature checks/)
+      @provider.should_receive(:shell_out!).with(
+        "zypper --non-interactive --no-gpg-checks remove emacs=1.0")
+
       @provider.remove_package("emacs", "1.0")
     end
   end
 
   describe "purge_package" do
     it "should run remove_package with the name and version" do
-      @provider.should_receive(:remove_package).with("emacs", "1.0")
+      @provider.should_receive(:shell_out!).with(
+        "zypper --non-interactive --no-gpg-checks remove --clean-deps emacs=1.0")
+      @provider.purge_package("emacs", "1.0")
+    end
+    it "should run zypper purge without gpg checks" do
+      Chef::Config.stub(:[]).with(:zypper_check_gpg).and_return(false)
+      @provider.should_receive(:shell_out!).with(
+        "zypper --non-interactive --no-gpg-checks remove --clean-deps emacs=1.0")
+      @provider.purge_package("emacs", "1.0")
+    end
+    it "should warn about gpg checks on zypper purge" do
+      Chef::Log.should_receive(:warn).with(
+        /All packages will be installed without gpg signature checks/)
+      @provider.should_receive(:shell_out!).with(
+        "zypper --non-interactive --no-gpg-checks remove --clean-deps emacs=1.0")
       @provider.purge_package("emacs", "1.0")
     end
   end
@@ -131,27 +196,24 @@ describe Chef::Provider::Package::Zypper do
 
     describe "install_package" do
       it "should run zypper install with the package name and version" do
-        @provider.should_receive(:run_command).with({
-            :command => "zypper install -y emacs"
-          })
+        @provider.should_receive(:shell_out!).with(
+          "zypper --no-gpg-checks install --auto-agree-with-licenses -y emacs")
         @provider.install_package("emacs", "1.0")
       end
     end
-  
+
     describe "upgrade_package" do
       it "should run zypper update with the package name and version" do
-        @provider.should_receive(:run_command).with({
-            :command => "zypper install -y emacs"
-          })
+        @provider.should_receive(:shell_out!).with(
+          "zypper --no-gpg-checks install --auto-agree-with-licenses -y emacs")
         @provider.upgrade_package("emacs", "1.0")
       end
     end
-  
+
     describe "remove_package" do
       it "should run zypper remove with the package name" do
-        @provider.should_receive(:run_command).with({
-            :command => "zypper remove -y emacs"
-          })
+        @provider.should_receive(:shell_out!).with(
+           "zypper --no-gpg-checks remove -y emacs")
         @provider.remove_package("emacs", "1.0")
       end
     end