chef 11.14.0.alpha.2-x86-mingw32 → 11.14.0.alpha.3-x86-mingw32

data/spec/unit/provider/package/{freebsd_spec.rb → freebsd/pkg_spec.rb}

@@ -20,7 +20,7 @@
 require 'spec_helper'
 require 'ostruct'
 
-describe Chef::Provider::Package::Freebsd, "load_current_resource" do
+describe Chef::Provider::Package::Freebsd::Pkg, "load_current_resource" do
   before(:each) do
     @node = Chef::Node.new
     @events = Chef::EventDispatch::Dispatcher.new
@@ -28,7 +28,7 @@ describe Chef::Provider::Package::Freebsd, "load_current_resource" do
     @new_resource     = Chef::Resource::Package.new("zsh")
     @current_resource = Chef::Resource::Package.new("zsh")
 
-    @provider = Chef::Provider::Package::Freebsd.new(@new_resource, @run_context)
+    @provider = Chef::Provider::Package::Freebsd::Pkg.new(@new_resource, @run_context)
     @provider.current_resource = @current_resource
     ::File.stub(:exist?).with('/usr/ports/Makefile').and_return(false)
   end
@@ -39,7 +39,7 @@ describe Chef::Provider::Package::Freebsd, "load_current_resource" do
     end
 
     it "should create a current resource with the name of the new_resource" do
-      current_resource = Chef::Provider::Package::Freebsd.new(@new_resource, @run_context).current_resource
+      current_resource = Chef::Provider::Package::Freebsd::Pkg.new(@new_resource, @run_context).current_resource
       current_resource.name.should == "zsh"
     end
 
@@ -66,7 +66,7 @@ describe Chef::Provider::Package::Freebsd, "load_current_resource" do
     before do
       #@new_resource = Chef::Resource::Package.new("zsh")
 
-      #@provider = Chef::Provider::Package::Freebsd.new(@node, @new_resource)
+      #@provider = Chef::Provider::Package::Freebsd::Pkg.new(@node, @new_resource)
 
       #@status = double("Status", :exitstatus => 0)
       #@stdin = double("STDIN", :null_object => true)
@@ -101,7 +101,7 @@ describe Chef::Provider::Package::Freebsd, "load_current_resource" do
     # Not happy with the form of these tests as they are far too closely tied to the implementation and so very fragile.
     it "should return the ports candidate version when given a valid port path" do
       @provider.stub(:port_path).and_return("/usr/ports/shells/zsh")
-      make_v = OpenStruct.new(:stdout => "4.3.6\n")
+      make_v = OpenStruct.new(:stdout => "4.3.6\n", :exitstatus => 0)
       @provider.should_receive(:shell_out!).with("make -V PORTVERSION", {:cwd=>"/usr/ports/shells/zsh", :returns=>[0, 1], :env=>nil}).and_return(make_v)
       @provider.ports_candidate_version.should == "4.3.6"
     end
@@ -109,14 +109,14 @@ describe Chef::Provider::Package::Freebsd, "load_current_resource" do
     it "should figure out the package name when we have ports" do
       ::File.stub(:exist?).with('/usr/ports/Makefile').and_return(true)
       @provider.stub(:port_path).and_return("/usr/ports/shells/zsh")
-      make_v = OpenStruct.new(:stdout => "zsh-4.3.6_7\n")
+      make_v = OpenStruct.new(:stdout => "zsh-4.3.6_7\n", :exitstatus => 0)
       @provider.should_receive(:shell_out!).with("make -V PKGNAME", {:cwd=>"/usr/ports/shells/zsh", :env=>nil, :returns=>[0, 1]}).and_return(make_v)
       #@provider.should_receive(:ports_makefile_variable_value).with("PKGNAME").and_return("zsh-4.3.6_7")
       @provider.package_name.should == "zsh"
     end
   end
 
-  describe Chef::Provider::Package::Freebsd, "install_package" do
+  describe Chef::Provider::Package::Freebsd::Pkg, "install_package" do
     before(:each) do
       @cmd_result = OpenStruct.new(:status => true)
 
@@ -130,21 +130,14 @@ describe Chef::Provider::Package::Freebsd, "load_current_resource" do
       @provider.should_receive(:shell_out!).with("pkg_add -r zsh", :env => nil).and_return(@cmd_result)
       @provider.install_package("zsh", "4.3.6_7")
     end
-
-    it "should run make install when installing from ports" do
-      @new_resource.stub(:source).and_return("ports")
-      @provider.should_not_receive(:shell_out!).with("make -DBATCH -f /usr/ports/shells/zsh/Makefile install", :timeout => 1200, :env=>nil)
-      @provider.should_receive(:shell_out!).with("make -DBATCH install", :timeout => 1200, :env=>nil, :cwd => @provider.port_path).and_return(@cmd_result)
-      @provider.install_package("zsh", "4.3.6_7")
-    end
   end
 
-  describe Chef::Provider::Package::Freebsd, "port path" do
+  describe Chef::Provider::Package::Freebsd::Pkg, "port path" do
     before do
       #@node = Chef::Node.new
       @new_resource = Chef::Resource::Package.new("zsh")
       @new_resource.cookbook_name = "adventureclub"
-      @provider = Chef::Provider::Package::Freebsd.new(@new_resource, @run_context)
+      @provider = Chef::Provider::Package::Freebsd::Pkg.new(@new_resource, @run_context)
     end
 
     it "should figure out the port path from the package_name using whereis" do
@@ -155,7 +148,7 @@ describe Chef::Provider::Package::Freebsd, "load_current_resource" do
 
     it "should use the package_name as the port path when it starts with /" do
       new_resource = Chef::Resource::Package.new("/usr/ports/www/wordpress")
-      provider = Chef::Provider::Package::Freebsd.new(new_resource, @run_context)
+      provider = Chef::Provider::Package::Freebsd::Pkg.new(new_resource, @run_context)
       provider.should_not_receive(:popen4)
       provider.port_path.should == "/usr/ports/www/wordpress"
     end
@@ -165,17 +158,17 @@ describe Chef::Provider::Package::Freebsd, "load_current_resource" do
       #                       :package_name => "www/wordpress",
       #                       :cookbook_name => "xenoparadox")
       new_resource = Chef::Resource::Package.new("www/wordpress")
-      provider = Chef::Provider::Package::Freebsd.new(new_resource, @run_context)
+      provider = Chef::Provider::Package::Freebsd::Pkg.new(new_resource, @run_context)
       provider.should_not_receive(:popen4)
       provider.port_path.should == "/usr/ports/www/wordpress"
     end
   end
 
-  describe Chef::Provider::Package::Freebsd, "ruby-iconv (package with a dash in the name)" do
+  describe Chef::Provider::Package::Freebsd::Pkg, "ruby-iconv (package with a dash in the name)" do
     before(:each) do
       @new_resource     = Chef::Resource::Package.new("ruby-iconv")
       @current_resource = Chef::Resource::Package.new("ruby-iconv")
-      @provider = Chef::Provider::Package::Freebsd.new(@new_resource, @run_context)
+      @provider = Chef::Provider::Package::Freebsd::Pkg.new(@new_resource, @run_context)
       @provider.current_resource = @current_resource
       @provider.stub(:port_path).and_return("/usr/ports/converters/ruby-iconv")
       @provider.stub(:package_name).and_return("ruby18-iconv")
@@ -188,15 +181,9 @@ describe Chef::Provider::Package::Freebsd, "load_current_resource" do
       @provider.should_receive(:shell_out!).with("pkg_add -r ruby18-iconv", :env => nil).and_return(@install_result)
       @provider.install_package("ruby-iconv", "1.0")
     end
-
-    it "should run make install when installing from ports" do
-      @new_resource.stub(:source).and_return("ports")
-      @provider.should_receive(:shell_out!).with("make -DBATCH install", :timeout => 1200, :env=>nil, :cwd => @provider.port_path).and_return(@install_result)
-      @provider.install_package("ruby-iconv", "1.0")
-    end
   end
 
-  describe Chef::Provider::Package::Freebsd, "remove_package" do
+  describe Chef::Provider::Package::Freebsd::Pkg, "remove_package" do
     before(:each) do
       @pkg_delete = OpenStruct.new(:status => true)
       @new_resource.version "4.3.6_7"
@@ -216,11 +203,11 @@ describe Chef::Provider::Package::Freebsd, "load_current_resource" do
   # version of a package is currently installed and to get the port_path.
   #  Example package name: bonnie++
 
-  describe Chef::Provider::Package::Freebsd, "bonnie++ (package with a plus in the name :: CHEF-4371)" do
+  describe Chef::Provider::Package::Freebsd::Pkg, "bonnie++ (package with a plus in the name :: CHEF-4371)" do
     before(:each) do
       @new_resource     = Chef::Resource::Package.new("bonnie++")
       @current_resource = Chef::Resource::Package.new("bonnie++")
-      @provider = Chef::Provider::Package::Freebsd.new(@new_resource, @run_context)
+      @provider = Chef::Provider::Package::Freebsd::Pkg.new(@new_resource, @run_context)
       @provider.current_resource = @current_resource
     end
 
@@ -256,11 +243,11 @@ describe Chef::Provider::Package::Freebsd, "load_current_resource" do
   # The variable LATEST_LINK is named that way because the directory that "pkg_add -r" downloads from is called "Latest" and
   # contains the "latest" versions of package as symbolic links to the files in the "All" directory.
 
-  describe Chef::Provider::Package::Freebsd, "install_package latest link fixes" do
+  describe Chef::Provider::Package::Freebsd::Pkg, "install_package latest link fixes" do
     it "should install the perl binary package with the correct name" do
       @new_resource = Chef::Resource::Package.new("perl5.8")
       @current_resource = Chef::Resource::Package.new("perl5.8")
-      @provider = Chef::Provider::Package::Freebsd.new(@new_resource, @run_context)
+      @provider = Chef::Provider::Package::Freebsd::Pkg.new(@new_resource, @run_context)
       @provider.current_resource = @current_resource
       @provider.stub(:package_name).and_return("perl")
       @provider.stub(:latest_link_name).and_return("perl")
@@ -274,7 +261,7 @@ describe Chef::Provider::Package::Freebsd, "load_current_resource" do
 
       @new_resource     = Chef::Resource::Package.new("mysql50-server")
       @current_resource = Chef::Resource::Package.new("mysql50-server")
-      @provider = Chef::Provider::Package::Freebsd.new(@new_resource, @run_context)
+      @provider = Chef::Provider::Package::Freebsd::Pkg.new(@new_resource, @run_context)
       @provider.current_resource = @current_resource
       @provider.stub(:package_name).and_return("mysql-server")
       @provider.stub(:latest_link_name).and_return("mysql50-server")

data/spec/unit/provider/package/freebsd/pkgng_spec.rb

@@ -0,0 +1,155 @@
+#
+# Authors:: Richard Manyanza (liseki@nyikacraftsmen.com)
+# Copyright:: Copyright (c) 2014 Richard Manyanza
+# 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::Package::Freebsd::Port 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("zsh")
+    @provider = Chef::Provider::Package::Freebsd::Pkgng.new(@new_resource, @run_context)
+  end
+
+
+  describe "initialization" do
+    it "should create a current resource with the name of the new resource" do
+      @provider.current_resource.is_a?(Chef::Resource::Package).should be_true
+      @provider.current_resource.name.should == 'zsh'
+    end
+  end
+
+
+  describe "loading current resource" do
+    before(:each) do
+      @provider.stub(:current_installed_version)
+      @provider.stub(:candidate_version)
+    end
+
+    it "should set the package name" do
+      @provider.load_current_resource
+      @provider.current_resource.package_name.should == "zsh"
+    end
+
+    it "should set the current version" do
+      @provider.should_receive(:current_installed_version).and_return("5.0.2")
+      @provider.load_current_resource
+      @provider.current_resource.version.should == "5.0.2"
+    end
+
+    it "should set the candidate version" do
+      @provider.should_receive(:candidate_version).and_return("5.0.5")
+      @provider.load_current_resource
+      @provider.instance_variable_get(:"@candidate_version").should == "5.0.5"
+    end
+  end
+
+
+  describe "determining current installed version" do
+    before(:each) do
+      @provider.stub(:supports_pkgng?)
+      @pkg_info = OpenStruct.new(:stdout => "zsh-3.1.7\n")
+    end
+
+    it "should query pkg database" do
+      @provider.should_receive(:shell_out!).with('pkg info "zsh"', :env => nil, :returns => [0,70]).and_return(@pkg_info)
+      @provider.current_installed_version.should == "3.1.7"
+    end
+  end
+
+
+  describe "determining candidate version" do
+    it "should query repository" do
+      pkg_query = OpenStruct.new(:stdout => "5.0.5\n", :exitstatus => 0)
+      @provider.should_receive(:shell_out!).with("pkg rquery '%v' zsh", :env => nil).and_return(pkg_query)
+      @provider.candidate_version.should == "5.0.5"
+    end
+
+    it "should query specified repository when given option" do
+      @provider.new_resource.options('-r LocalMirror') # This requires LocalMirror repo configuration.
+      pkg_query = OpenStruct.new(:stdout => "5.0.3\n", :exitstatus => 0)
+      @provider.should_receive(:shell_out!).with("pkg rquery -r LocalMirror '%v' zsh", :env => nil).and_return(pkg_query)
+      @provider.candidate_version.should == "5.0.3"
+    end
+
+    it "should return candidate version from file when given a file" do
+      @provider.new_resource.source("/nas/pkg/repo/zsh-5.0.1.txz")
+      @provider.candidate_version.should == "5.0.1"
+    end
+  end
+
+
+  describe "installing a binary package" do
+    before(:each) do
+      @install_result = OpenStruct.new(:status => true)
+    end
+
+    it "should handle package source from file" do
+      @provider.new_resource.source("/nas/pkg/repo/zsh-5.0.1.txz")
+      @provider.should_receive(:shell_out!).
+        with("pkg add /nas/pkg/repo/zsh-5.0.1.txz", :env => { 'LC_ALL' => nil }).
+        and_return(@install_result)
+      @provider.install_package("zsh", "5.0.1")
+    end
+
+    it "should handle package source over ftp or http" do
+      @provider.new_resource.source("http://repo.example.com/zsh-5.0.1.txz")
+      @provider.should_receive(:shell_out!).
+        with("pkg add http://repo.example.com/zsh-5.0.1.txz", :env => { 'LC_ALL' => nil }).
+        and_return(@install_result)
+      @provider.install_package("zsh", "5.0.1")
+    end
+
+    it "should handle a package name" do
+      @provider.should_receive(:shell_out!).
+        with("pkg install -y zsh", :env => { 'LC_ALL' => nil }).and_return(@install_result)
+      @provider.install_package("zsh", "5.0.1")
+    end
+
+    it "should handle a package name with a specified repo" do
+      @provider.new_resource.options('-r LocalMirror') # This requires LocalMirror repo configuration.
+      @provider.should_receive(:shell_out!).
+        with("pkg install -y -r LocalMirror zsh", :env => { 'LC_ALL' => nil }).and_return(@install_result)
+      @provider.install_package("zsh", "5.0.1")
+    end
+  end
+
+
+  describe "removing a binary package" do
+    before(:each) do
+      @install_result = OpenStruct.new(:status => true)
+    end
+
+    it "should call pkg delete" do
+      @provider.should_receive(:shell_out!).
+        with("pkg delete -y zsh-5.0.1", :env => nil).and_return(@install_result)
+      @provider.remove_package("zsh", "5.0.1")
+    end
+
+    it "should not include repo option in pkg delete" do
+      @provider.new_resource.options('-r LocalMirror') # This requires LocalMirror repo configuration.
+      @provider.should_receive(:shell_out!).
+        with("pkg delete -y zsh-5.0.1", :env => nil).and_return(@install_result)
+      @provider.remove_package("zsh", "5.0.1")
+    end
+  end
+end

data/spec/unit/provider/package/freebsd/port_spec.rb

@@ -0,0 +1,160 @@
+#
+# Authors:: Richard Manyanza (liseki@nyikacraftsmen.com)
+# Copyright:: Copyright (c) 2014 Richard Manyanza
+# 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::Package::Freebsd::Port 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("zsh")
+    @provider = Chef::Provider::Package::Freebsd::Port.new(@new_resource, @run_context)
+  end
+
+
+  describe "initialization" do
+    it "should create a current resource with the name of the new resource" do
+      @provider.current_resource.is_a?(Chef::Resource::Package).should be_true
+      @provider.current_resource.name.should == 'zsh'
+    end
+  end
+
+
+  describe "loading current resource" do
+    before(:each) do
+      @provider.stub(:current_installed_version)
+      @provider.stub(:candidate_version)
+    end
+
+    it "should set the package name" do
+      @provider.load_current_resource
+      @provider.current_resource.package_name.should == "zsh"
+    end
+
+    it "should set the current version" do
+      @provider.should_receive(:current_installed_version).and_return("5.0.2")
+      @provider.load_current_resource
+      @provider.current_resource.version.should == "5.0.2"
+    end
+
+    it "should set the candidate version" do
+      @provider.should_receive(:candidate_version).and_return("5.0.5")
+      @provider.load_current_resource
+      @provider.instance_variable_get(:"@candidate_version").should == "5.0.5"
+    end
+  end
+
+
+  describe "determining current installed version" do
+    before(:each) do
+      @provider.stub(:supports_pkgng?)
+      @pkg_info = OpenStruct.new(:stdout => "zsh-3.1.7\n")
+    end
+
+    it "should check 'pkg_info' if system uses pkg_* tools" do
+      @provider.should_receive(:supports_pkgng?).and_return(false)
+      @provider.should_receive(:shell_out!).with('pkg_info -E "zsh*"', :env => nil, :returns => [0,1]).and_return(@pkg_info)
+      @provider.current_installed_version.should == "3.1.7"
+    end
+
+    it "should check 'pkg info' if system uses pkgng" do
+      @provider.should_receive(:supports_pkgng?).and_return(true)
+      @provider.should_receive(:shell_out!).with('pkg info "zsh"', :env => nil, :returns => [0,70]).and_return(@pkg_info)
+      @provider.current_installed_version.should == "3.1.7"
+    end
+  end
+
+
+  describe "determining candidate version" do
+    before(:each) do
+      @port_version = OpenStruct.new(:stdout => "5.0.5\n", :exitstatus => 0)
+    end
+
+    it "should return candidate version if port exists" do
+      ::File.stub(:exist?).with('/usr/ports/Makefile').and_return(true)
+      @provider.stub(:port_dir).and_return('/usr/ports/shells/zsh')
+      @provider.should_receive(:shell_out!).with("make -V PORTVERSION", :cwd => "/usr/ports/shells/zsh", :env => nil, :returns => [0,1]).
+        and_return(@port_version)
+      @provider.candidate_version.should == "5.0.5"
+    end
+
+    it "should raise exception if ports tree not found" do
+      ::File.stub(:exist?).with('/usr/ports/Makefile').and_return(false)
+      expect { @provider.candidate_version }.to raise_error(Chef::Exceptions::Package, "Ports collection could not be found")
+    end
+  end
+
+
+  describe "determining port directory" do
+    it "should return name if package name is absolute path" do
+      @provider.new_resource.stub(:package_name).and_return("/var/ports/shells/zsh")
+      @provider.port_dir.should == "/var/ports/shells/zsh"
+    end
+
+    it "should return full ports path given package name and category" do
+      @provider.new_resource.stub(:package_name).and_return("shells/zsh")
+      @provider.port_dir.should == "/usr/ports/shells/zsh"
+    end
+
+    it "should query system for path given just a name" do
+      whereis = OpenStruct.new(:stdout => "zsh: /usr/ports/shells/zsh\n")
+      @provider.should_receive(:shell_out!).with("whereis -s zsh", :env => nil).and_return(whereis)
+      @provider.port_dir.should == "/usr/ports/shells/zsh"
+    end
+
+    it "should raise exception if not found" do
+      whereis = OpenStruct.new(:stdout => "zsh:\n")
+      @provider.should_receive(:shell_out!).with("whereis -s zsh", :env => nil).and_return(whereis)
+      expect { @provider.port_dir }.to raise_error(Chef::Exceptions::Package, "Could not find port with the name zsh")
+    end
+  end
+
+
+  describe "building a binary package" do
+    before(:each) do
+      @install_result = OpenStruct.new(:status => true)
+    end
+
+    it "should run make install in port directory" do
+      @provider.stub(:port_dir).and_return("/usr/ports/shells/zsh")
+      @provider.should_receive(:shell_out!).
+        with("make -DBATCH install clean", :timeout => 1800, :cwd => "/usr/ports/shells/zsh", :env => nil).
+        and_return(@install_result)
+      @provider.install_package("zsh", "5.0.5")
+    end
+  end
+
+
+  describe "removing a binary package" do
+    before(:each) do
+      @install_result = OpenStruct.new(:status => true)
+    end
+
+    it "should run make deinstall in port directory" do
+      @provider.stub(:port_dir).and_return("/usr/ports/shells/zsh")
+      @provider.should_receive(:shell_out!).
+        with("make deinstall", :timeout => 300, :cwd => "/usr/ports/shells/zsh", :env => nil).
+        and_return(@install_result)
+      @provider.remove_package("zsh", "5.0.5")
+    end
+  end
+end