chef 11.6.2 → 11.8.0.rc.1

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

@@ -0,0 +1,237 @@
+#
+# Author:: Kaustubh Deorukhkar (<kaustubh@clogeny.com>)
+# Copyright:: Copyright (c) 2013 OpsCode, Inc.
+# 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 'ostruct'
+
+describe Chef::Provider::Mount::Aix do
+
+  before(:all) do
+    @mounted_output = <<-MOUNT
+  node       mounted        mounted over    vfs       date        options
+-------- ---------------  ---------------  ------ ------------ ---------------
+         /dev/sdz1         /tmp/foo         jfs2   Jul 17 13:22 rw,log=/dev/hd8
+MOUNT
+
+    @unmounted_output = <<-UNMOUNTED
+  node       mounted        mounted over    vfs       date        options
+-------- ---------------  ---------------  ------ ------------ ---------------
+         /dev/sdz2         /                jfs2   Jul 17 13:22 rw,log=/dev/hd8
+UNMOUNTED
+
+    @conflict_mounted_output = <<-MOUNT
+  node       mounted        mounted over    vfs       date        options
+-------- ---------------  ---------------  ------ ------------ ---------------
+         /dev/sdz3         /tmp/foo         jfs2   Jul 17 13:22 rw,log=/dev/hd8
+MOUNT
+
+  @enabled_output = <<-ENABLED
+#MountPoint:Device:Vfs:Nodename:Type:Size:Options:AutoMount:Acct
+/tmp/foo:/dev/sdz1:jfs2::bootfs:10485760:rw:yes:no
+ENABLED
+  end
+
+  before(:each) do
+    @node = Chef::Node.new
+    @events = Chef::EventDispatch::Dispatcher.new
+    @run_context = Chef::RunContext.new(@node, {}, @events)
+
+    @new_resource = Chef::Resource::Mount.new("/tmp/foo")
+    @new_resource.device      "/dev/sdz1"
+    @new_resource.device_type :device
+    @new_resource.fstype      "jfs2"
+
+    @new_resource.supports :remount => false
+
+    @provider = Chef::Provider::Mount::Aix.new(@new_resource, @run_context)
+
+    ::File.stub!(:exists?).with("/dev/sdz1").and_return true
+    ::File.stub!(:exists?).with("/tmp/foo").and_return true
+  end
+
+  def stub_mounted(provider, mounted_output)
+    response = double("Mixlib::ShellOut command", :exitstatus => 0, :stdout => mounted_output, :stderr => "")
+    provider.should_receive(:shell_out!).with("mount").and_return(response)
+  end
+
+  def stub_enabled(provider, enabled_output)
+    response = double("Mixlib::ShellOut command", :exitstatus => 0, :stdout => enabled_output, :stderr => "")
+    provider.should_receive(:shell_out).with("lsfs -c #{@new_resource.mount_point}").and_return(response)
+  end
+
+  def stub_mounted_enabled(provider, mounted_output, enabled_output)
+    stub_mounted(provider, mounted_output)
+    stub_enabled(provider, enabled_output)
+  end
+
+  describe "when discovering the current fs state" do
+    it "should set current_resource.mounted to true if device is already mounted" do
+      stub_mounted_enabled(@provider, @mounted_output, "")
+      @provider.load_current_resource
+
+      expect(@provider.current_resource.mounted).to be_true
+    end
+
+    it "should set current_resource.mounted to false if device is not mounted" do
+      stub_mounted_enabled(@provider, @unmounted_output, "")
+
+      @provider.load_current_resource
+
+      expect(@provider.current_resource.mounted).to be_false
+    end
+
+    it "should set current_resource.mounted to false if the mount point is used for another device" do
+      stub_mounted_enabled(@provider, @conflict_mounted_output, "")
+
+      @provider.load_current_resource
+
+      expect(@provider.current_resource.mounted).to be_false
+    end
+  end
+
+  # tests for #enabled?
+  it "should load current_resource with properties if device is already mounted and enabled" do
+    stub_mounted_enabled(@provider, @mounted_output, @enabled_output)
+
+    @provider.load_current_resource
+
+    expect(@provider.current_resource.enabled).to be_true
+    expect(@provider.current_resource.mounted).to be_true
+    expect(@provider.current_resource.mount_point).to eql(@new_resource.mount_point)
+    expect(@provider.current_resource.fstype).to eql("jfs2")
+    expect(@provider.current_resource.options).to eql(['rw'])
+  end
+
+  describe "mount_fs" do
+    it "should mount resource if it is not mounted" do
+      stub_mounted_enabled(@provider, @unmounted_output, "")
+
+      @provider.should_receive(:shell_out!).with("mount -v #{@new_resource.fstype} #{@new_resource.device} #{@new_resource.mount_point}")
+
+      @provider.run_action(:mount)
+    end
+
+    it "should not mount resource if it is already mounted" do
+      stub_mounted_enabled(@provider, @mounted_output, "")
+
+      @provider.should_not_receive(:mount_fs)
+
+      @provider.run_action(:mount)
+    end
+  end
+
+  describe "umount_fs" do
+    it "should umount resource if it is already mounted" do
+      stub_mounted_enabled(@provider, @mounted_output, "")
+
+      @provider.should_receive(:shell_out!).with("umount #{@new_resource.mount_point}")
+
+      @provider.run_action(:umount)
+    end
+
+    it "should not umount resource if it is not mounted" do
+      stub_mounted_enabled(@provider, @unmounted_output, "")
+
+      @provider.should_not_receive(:umount_fs)
+
+      @provider.run_action(:umount)
+    end
+  end
+
+  describe "remount_fs" do
+    it "should remount resource if it is already mounted and it supports remounting" do
+      @new_resource.supports({:remount => true})
+      stub_mounted_enabled(@provider, @mounted_output, "")
+
+      @provider.should_receive(:shell_out!).with("mount -o remount #{@new_resource.device} #{@new_resource.mount_point}")
+
+      @provider.run_action(:remount)
+    end
+
+    it "should remount with new mount options if it is already mounted and it supports remounting" do
+      @new_resource.supports({:remount => true})
+      @new_resource.options("nodev,rw")
+      stub_mounted_enabled(@provider, @mounted_output, "")
+
+      @provider.should_receive(:shell_out!).with("mount -o remount,nodev,rw #{@new_resource.device} #{@new_resource.mount_point}")
+
+      @provider.run_action(:remount)
+    end
+  end
+
+  describe "enable_fs" do
+    it "should enable mount if it is mounted and not enabled" do
+      @new_resource.options("nodev,rw")
+      stub_mounted_enabled(@provider, @mounted_output, "")
+      filesystems = StringIO.new
+      ::File.stub!(:open).with("/etc/filesystems", "a").and_yield(filesystems)
+
+      @provider.run_action(:enable)
+
+      filesystems.string.should match(%r{^/tmp/foo:\n\tdev\t\t= /dev/sdz1\n\tvfs\t\t= jfs2\n\tmount\t\t= false\n\toptions\t\t= nodev,rw\n$})
+    end
+
+    it "should not enable mount if it is mounted and already enabled and mount options are unchanged" do
+      stub_mounted_enabled(@provider, @mounted_output, @enabled_output)
+      @new_resource.options "rw"
+
+      @provider.should_not_receive(:enable_fs)
+
+      @provider.run_action(:enable)
+    end
+  end
+
+  describe "disable_fs" do
+    it "should disable mount if it is mounted and enabled" do
+      stub_mounted_enabled(@provider, @mounted_output, @enabled_output)
+
+      ::File.stub!(:open).with("/etc/filesystems", "r").and_return(<<-ETCFILESYSTEMS)
+/tmp/foo:
+  dev   = /dev/sdz1
+  vfs   = jfs2
+  log   = /dev/hd8
+  mount   = true
+  check   = true
+  vol   = /opt
+  free    = false
+  quota   = no
+
+/tmp/abc:
+  dev   = /dev/sdz2
+  vfs   = jfs2
+  mount   = true
+  options   = rw
+ETCFILESYSTEMS
+
+      filesystems = StringIO.new
+      ::File.stub!(:open).with("/etc/filesystems", "w").and_yield(filesystems)
+
+      @provider.run_action(:disable)
+
+      filesystems.string.should match(%r{^/tmp/abc:\s+dev\s+= /dev/sdz2\s+vfs\s+= jfs2\s+mount\s+= true\s+options\s+= rw\n$})
+    end
+
+    it "should not disable mount if it is not mounted" do
+      stub_mounted_enabled(@provider, @unmounted_output, "")
+
+      @provider.should_not_receive(:disable_fs)
+
+      @provider.run_action(:disable)
+    end
+  end
+end

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

@@ -36,6 +36,8 @@ describe Chef::Provider::Mount::Mount do
 
     ::File.stub!(:exists?).with("/dev/sdz1").and_return true
     ::File.stub!(:exists?).with("/tmp/foo").and_return true
+    ::File.stub!(:realpath).with("/dev/sdz1").and_return "/dev/sdz1"
+    ::File.stub!(:realpath).with("/tmp/foo").and_return "/tmp/foo"
   end
 
   describe "when discovering the current fs state" do

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

@@ -6,9 +6,9 @@
 # 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.

data/spec/unit/provider/mount_spec.rb

@@ -23,13 +23,13 @@ describe Chef::Provider::Mount do
     @node = Chef::Node.new
     @events = Chef::EventDispatch::Dispatcher.new
     @run_context = Chef::RunContext.new(@node, {}, @events)
-    
+
     @new_resource = Chef::Resource::Mount.new('/tmp/foo')
     @new_resource.device      "/dev/sdz1"
     @new_resource.name        "/tmp/foo"
     @new_resource.mount_point "/tmp/foo"
     @new_resource.fstype      "ext3"
-    
+
     @current_resource = Chef::Resource::Mount.new('/tmp/foo')
     @current_resource.device      "/dev/sdz1"
     @current_resource.name        "/tmp/foo"
@@ -39,7 +39,7 @@ describe Chef::Provider::Mount do
     @provider = Chef::Provider::Mount.new(@new_resource, @run_context)
     @provider.current_resource = @current_resource
   end
-  
+
   describe "when the target state is a mounted filesystem" do
 
     it "should mount the filesystem if it isn't mounted" do
@@ -78,7 +78,7 @@ describe Chef::Provider::Mount do
     before do
       @new_resource.supports[:remount] = true
     end
-    
+
     it "should remount the filesystem if it is mounted" do
       @current_resource.stub!(:mounted).and_return(true)
       @provider.should_receive(:remount_fs).and_return(true)
@@ -93,8 +93,8 @@ describe Chef::Provider::Mount do
       @new_resource.should_not be_updated_by_last_action
     end
   end
-  describe "when the filesystem should be remounted and the resource does not support remounting" do 
-    before do 
+  describe "when the filesystem should be remounted and the resource does not support remounting" do
+    before do
       @new_resource.supports[:remount] = false
     end
 

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

@@ -0,0 +1,171 @@
+#
+# Author:: Deepali Jagtap (deepali.jagtap@clogeny.com)
+# Author:: Prabhu Das (prabhu.das@clogeny.com)
+# Copyright:: Copyright (c) 2013 Opscode, Inc.
+# 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'
+
+describe Chef::Provider::Package::Aix do
+  before(:each) do
+    @node = Chef::Node.new
+    @events = Chef::EventDispatch::Dispatcher.new
+    @run_context = Chef::RunContext.new(@node, {}, @events)
+
+    @new_resource = Chef::Resource::Package.new("samba.base")
+    @new_resource.source("/tmp/samba.base")
+
+    @provider = Chef::Provider::Package::Aix.new(@new_resource, @run_context)
+    ::File.stub!(:exists?).and_return(true)
+  end
+
+  describe "assessing the current package status" do
+    before do
+     @bffinfo ="/usr/lib/objrepos:samba.base:3.3.12.0::COMMITTED:I:Samba for AIX:
+ /etc/objrepos:samba.base:3.3.12.0::COMMITTED:I:Samba for AIX:"
+
+      @status = mock("Status", :exitstatus => 0)
+    end
+
+    it "should create a current resource with the name of new_resource" do
+      @provider.stub!(:popen4).and_return(@status)
+      @provider.load_current_resource
+      @provider.current_resource.name.should == "samba.base"
+    end
+
+    it "should set the current resource bff package name to the new resource bff package name" do
+      @provider.stub!(:popen4).and_return(@status)
+      @provider.load_current_resource
+      @provider.current_resource.package_name.should == "samba.base"
+    end
+
+    it "should raise an exception if a source is supplied but not found" do
+      @provider.stub!(:popen4).and_return(@status)
+      ::File.stub!(:exists?).and_return(false)
+      @provider.define_resource_requirements
+      @provider.load_current_resource
+      lambda { @provider.process_resource_requirements }.should raise_error(Chef::Exceptions::Package)
+    end
+
+
+    it "should get the source package version from lslpp if provided" do
+      @stdout = StringIO.new(@bffinfo)
+      @stdin, @stderr = StringIO.new, StringIO.new
+      @provider.should_receive(:popen4).with("installp -L -d /tmp/samba.base").and_yield(@pid, @stdin, @stdout, @stderr).and_return(@status)
+      @provider.should_receive(:popen4).with("lslpp -lcq samba.base").and_return(@status)
+      @provider.load_current_resource
+
+      @provider.current_resource.package_name.should == "samba.base"
+      @new_resource.version.should == "3.3.12.0"
+    end
+
+    it "should return the current version installed if found by lslpp" do
+      @stdout = StringIO.new(@bffinfo)
+      @stdin, @stderr = StringIO.new, StringIO.new
+      @provider.should_receive(:popen4).with("installp -L -d /tmp/samba.base").and_return(@status)
+      @provider.should_receive(:popen4).with("lslpp -lcq samba.base").and_yield(@pid, @stdin, @stdout, @stderr).and_return(@status)
+      @provider.load_current_resource
+      @provider.current_resource.version.should == "3.3.12.0"
+    end
+
+    it "should raise an exception if the source is not set but we are installing" do
+      @new_resource = Chef::Resource::Package.new("samba.base")
+      @provider = Chef::Provider::Package::Aix.new(@new_resource, @run_context)
+      @provider.stub!(:popen4).and_return(@status)
+      lambda { @provider.run_action(:install) }.should raise_error(Chef::Exceptions::Package)
+    end
+
+    it "should raise an exception if installp/lslpp fails to run" do
+      @status = mock("Status", :exitstatus => -1)
+      @provider.stub!(:popen4).and_return(@status)
+      lambda { @provider.load_current_resource }.should raise_error(Chef::Exceptions::Package)
+    end
+
+    it "should return a current resource with a nil version if the package is not found" do
+      @stdout = StringIO.new
+      @provider.should_receive(:popen4).with("installp -L -d /tmp/samba.base").and_return(@status)
+      @provider.should_receive(:popen4).with("lslpp -lcq samba.base").and_yield(@pid, @stdin, @stdout, @stderr).and_return(@status)
+      @provider.load_current_resource
+      @provider.current_resource.version.should be_nil
+    end
+  end
+
+  describe "candidate_version" do
+    it "should return the candidate_version variable if already setup" do
+      @provider.candidate_version = "3.3.12.0"
+      @provider.should_not_receive(:popen4)
+      @provider.candidate_version
+    end
+
+    it "should lookup the candidate_version if the variable is not already set" do
+      @status = mock("Status", :exitstatus => 0)
+      @provider.should_receive(:popen4).and_return(@status)
+      @provider.candidate_version
+    end
+
+    it "should throw and exception if the exitstatus is not 0" do
+      @status = mock("Status", :exitstatus => 1)
+      @provider.stub!(:popen4).and_return(@status)
+      lambda { @provider.candidate_version }.should raise_error(Chef::Exceptions::Package)
+    end
+
+  end
+
+  describe "install and upgrade" do
+    it "should run installp -aYF -d with the package source to install" do
+      @provider.should_receive(:run_command_with_systems_locale).with({
+        :command => "installp -aYF -d /tmp/samba.base samba.base"
+      })
+      @provider.install_package("samba.base", "3.3.12.0")
+    end
+
+    it "should run  when the package is a path to install" do
+      @new_resource = Chef::Resource::Package.new("/tmp/samba.base")
+      @provider = Chef::Provider::Package::Aix.new(@new_resource, @run_context)
+      @new_resource.source.should == "/tmp/samba.base"
+      @provider.should_receive(:run_command_with_systems_locale).with({
+        :command => "installp -aYF -d /tmp/samba.base /tmp/samba.base"
+      })
+      @provider.install_package("/tmp/samba.base", "3.3.12.0")
+    end
+
+    it "should run installp with -eLogfile option." do
+      @new_resource.stub!(:options).and_return("-e/tmp/installp.log")
+      @provider.should_receive(:run_command_with_systems_locale).with({
+        :command => "installp -aYF  -e/tmp/installp.log -d /tmp/samba.base samba.base"
+      })
+      @provider.install_package("samba.base", "3.3.12.0")
+    end
+  end
+
+  describe "remove" do
+    it "should run installp -u samba.base to remove the package" do
+      @provider.should_receive(:run_command_with_systems_locale).with({
+        :command => "installp -u samba.base"
+      })
+      @provider.remove_package("samba.base", "3.3.12.0")
+    end
+
+    it "should run installp -u -e/tmp/installp.log  with options -e/tmp/installp.log" do
+      @new_resource.stub!(:options).and_return("-e/tmp/installp.log")
+      @provider.should_receive(:run_command_with_systems_locale).with({
+        :command => "installp -u  -e/tmp/installp.log samba.base"
+      })
+      @provider.remove_package("samba.base", "3.3.12.0")
+    end
+
+  end
+end
+