chef 14.2.0 → 14.3.37

data/spec/unit/resource/conditional_spec.rb

@@ -22,7 +22,7 @@ require "ostruct"
 describe Chef::Resource::Conditional do
   before do
     allow_any_instance_of(Mixlib::ShellOut).to receive(:run_command).and_return(nil)
-    @status = OpenStruct.new(:success? => true)
+    @status = OpenStruct.new(success?: true)
     allow_any_instance_of(Mixlib::ShellOut).to receive(:status).and_return(@status)
     @parent_resource = Chef::Resource.new("", Chef::Node.new)
   end

data/spec/unit/resource/cookbook_file_spec.rb

@@ -2,7 +2,7 @@
 # Author:: Daniel DeLeo (<dan@chef.io>)
 # Author:: Tyler Cloke (<tyler@chef.io>)
 # Copyright:: Copyright 2010-2017, Chef Software Inc.
-#p License:: Apache License, Version 2.0
+# p 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.
@@ -72,8 +72,8 @@ describe Chef::Resource::CookbookFile do
       state = resource.state_for_resource_reporter
       if Chef::Platform.windows?
         puts state
-        expect(state[:rights]).to eq([{ :permissions => :read, :principals => "Everyone" }])
-        expect(state[:deny_rights]).to eq([{ :permissions => :full_control, :principals => "Clumsy_Sam" }])
+        expect(state[:rights]).to eq([{ permissions: :read, principals: "Everyone" }])
+        expect(state[:deny_rights]).to eq([{ permissions: :full_control, principals: "Clumsy_Sam" }])
       else
         expect(state[:group]).to eq("wheel")
         expect(state[:mode]).to eq("0664")

data/spec/unit/resource/dnf_package_spec.rb

@@ -59,19 +59,19 @@ describe Chef::Resource::DnfPackage, "flush_cache" do
   let(:resource) { Chef::Resource::DnfPackage.new("foo") }
 
   it "defaults the flush timing to false" do
-    flush_hash = { :before => false, :after => false }
+    flush_hash = { before: false, after: false }
     expect(resource.flush_cache).to eq(flush_hash)
   end
 
   it "allows you to set the flush timing with an array" do
     flush_array = [ :before, :after ]
-    flush_hash = { :before => true, :after => true }
+    flush_hash = { before: true, after: true }
     resource.flush_cache(flush_array)
     expect(resource.flush_cache).to eq(flush_hash)
   end
 
   it "allows you to set the flush timing with a hash" do
-    flush_hash = { :before => true, :after => true }
+    flush_hash = { before: true, after: true }
     resource.flush_cache(flush_hash)
     expect(resource.flush_cache).to eq(flush_hash)
   end

data/spec/unit/resource/dsc_script_spec.rb

@@ -45,27 +45,27 @@ describe Chef::Resource::DscScript do
       expect { resource.action :run }.not_to raise_error
     end
 
-    it "allows the code attribute to be set" do
+    it "allows the code property to be set" do
       resource.code(configuration_code)
       expect(resource.code).to eq(configuration_code)
     end
 
-    it "allows the command attribute to be set" do
+    it "allows the command property to be set" do
       resource.command(configuration_path)
       expect(resource.command).to eq(configuration_path)
     end
 
-    it "allows the configuration_name attribute to be set" do
+    it "allows the configuration_name property to be set" do
       resource.configuration_name(configuration_name)
       expect(resource.configuration_name).to eq(configuration_name)
     end
 
-    it "allows the configuration_data attribute to be set" do
+    it "allows the configuration_data property to be set" do
       resource.configuration_data(configuration_data)
       expect(resource.configuration_data).to eq(configuration_data)
     end
 
-    it "allows the configuration_data_script attribute to be set" do
+    it "allows the configuration_data_script property to be set" do
       resource.configuration_data_script(configuration_data_script)
       expect(resource.configuration_data_script).to eq(configuration_data_script)
     end
@@ -106,27 +106,27 @@ describe Chef::Resource::DscScript do
       end
     end
 
-    it "raises an ArgumentError exception if an attempt is made to set the code attribute when the command attribute is already set" do
+    it "raises an ArgumentError exception if an attempt is made to set the code property when the command property is already set" do
       resource.command(configuration_path)
       expect { resource.code(configuration_code) }.to raise_error(ArgumentError)
     end
 
-    it "raises an ArgumentError exception if an attempt is made to set the command attribute when the code attribute is already set" do
+    it "raises an ArgumentError exception if an attempt is made to set the command property when the code property is already set" do
       resource.code(configuration_code)
       expect { resource.command(configuration_path) }.to raise_error(ArgumentError)
     end
 
-    it "raises an ArgumentError exception if an attempt is made to set the configuration_name attribute when the code attribute is already set" do
+    it "raises an ArgumentError exception if an attempt is made to set the configuration_name property when the code property is already set" do
       resource.code(configuration_code)
       expect { resource.configuration_name(configuration_name) }.to raise_error(ArgumentError)
     end
 
-    it "raises an ArgumentError exception if an attempt is made to set the configuration_data attribute when the configuration_data_script attribute is already set" do
+    it "raises an ArgumentError exception if an attempt is made to set the configuration_data property when the configuration_data_script property is already set" do
       resource.configuration_data_script(configuration_data_script)
       expect { resource.configuration_data(configuration_data) }.to raise_error(ArgumentError)
     end
 
-    it "raises an ArgumentError exception if an attempt is made to set the configuration_data_script attribute when the configuration_data attribute is already set" do
+    it "raises an ArgumentError exception if an attempt is made to set the configuration_data_script property when the configuration_data property is already set" do
       resource.configuration_data(configuration_data)
       expect { resource.configuration_data_script(configuration_data_script) }.to raise_error(ArgumentError)
     end

data/spec/unit/resource/execute_spec.rb

@@ -109,7 +109,7 @@ describe Chef::Resource::Execute do
   shared_examples_for "a consumer of the Execute resource" do
     context "when running on Windows" do
       before do
-        allow(resource).to receive(:node).and_return({ :platform_family => "windows" })
+        allow(resource).to receive(:node).and_return({ platform_family: "windows" })
       end
 
       context "when no user, domain, or password is specified" do
@@ -211,7 +211,7 @@ describe Chef::Resource::Execute do
 
     context "when not running on Windows" do
       before do
-        allow(resource).to receive(:node).and_return({ :platform_family => "ubuntu" })
+        allow(resource).to receive(:node).and_return({ platform_family: "ubuntu" })
       end
 
       context "when no user, domain, or password is specified" do

data/spec/unit/resource/file/verification/systemd_unit_spec.rb

@@ -20,7 +20,7 @@ require "spec_helper"
 
 describe Chef::Resource::File::Verification::SystemdUnit do
   let(:command) { "#{systemd_analyze_path} verify %{path}" }
-  let(:opts) { { :future => true } }
+  let(:opts) { { future: true } }
   let(:parent_resource) { Chef::Resource.new("llama") }
   let(:systemd_analyze_path) { "/usr/bin/systemd-analyze" }
   let(:systemd_dir) { "/etc/systemd/system" }

data/spec/unit/resource/file/verification_spec.rb

@@ -48,7 +48,7 @@ describe Chef::Resource::File::Verification do
 
     it "accepts an options hash" do
       v = Chef::Resource::File::Verification.new(parent_resource, nil, {}) {}
-      expect { v.verify("/foo/bar", { :future => true }) }.to_not raise_error
+      expect { v.verify("/foo/bar", { future: true }) }.to_not raise_error
     end
 
     context "with a verification block" do

data/spec/unit/resource/file_spec.rb

@@ -113,15 +113,15 @@ describe Chef::Resource::File do
 
   end
 
-  describe "when access controls are set on windows", :windows_only => true do
+  describe "when access controls are set on windows", windows_only: true do
     before do
       resource.rights :read, "Everyone"
       resource.rights :full_control, "DOMAIN\User"
     end
     it "describes its state including windows ACL attributes" do
       state = resource.state_for_resource_reporter
-      expect(state[:rights]).to eq([ { :permissions => :read, :principals => "Everyone" },
-                               { :permissions => :full_control, :principals => "DOMAIN\User" } ])
+      expect(state[:rights]).to eq([ { permissions: :read, principals: "Everyone" },
+                               { permissions: :full_control, principals: "DOMAIN\User" } ])
     end
   end
 end

data/spec/unit/resource/freebsd_package_spec.rb

@@ -79,8 +79,8 @@ describe Chef::Resource::FreebsdPackage do
 
     describe "if pkgng enabled" do
       it "is Freebsd::Pkgng" do
-        pkg_enabled = OpenStruct.new(:stdout => "yes\n")
-        allow(resource).to receive(:shell_out!).with("make", "-V", "WITH_PKGNG", :env => nil).and_return(pkg_enabled)
+        pkg_enabled = OpenStruct.new(stdout: "yes\n")
+        allow(resource).to receive(:shell_out!).with("make", "-V", "WITH_PKGNG", env: nil).and_return(pkg_enabled)
         resource.after_created
         expect(resource.provider).to eq(Chef::Provider::Package::Freebsd::Pkgng)
       end
@@ -88,11 +88,12 @@ describe Chef::Resource::FreebsdPackage do
 
     describe "if freebsd_version is less than 1000017 and pkgng not enabled" do
       it "is Freebsd::Pkg" do
-        pkg_enabled = OpenStruct.new(:stdout => "\n")
-        allow(resource).to receive(:shell_out!).with("make", "-V", "WITH_PKGNG", :env => nil).and_return(pkg_enabled)
+        pkg_enabled = OpenStruct.new(stdout: "\n")
+        allow(resource).to receive(:shell_out!).with("make", "-V", "WITH_PKGNG", env: nil).and_return(pkg_enabled)
 
         [1000016, 1000000, 901503, 902506, 802511].each do |freebsd_version|
           node.automatic_attrs[:os_version] = freebsd_version
+          expect(Chef).to receive(:deprecated).with(:freebsd_package_provider, kind_of(String))
           resource.after_created
           expect(resource.provider).to eq(Chef::Provider::Package::Freebsd::Pkg)
         end

data/spec/unit/resource/group_spec.rb

@@ -72,7 +72,7 @@ describe Chef::Resource::Group, "group_name" do
   end
 
   it "does not allow a hash" do
-    expect { resource.send(:group_name, { :aj => "is freakin awesome" }) }.to raise_error(ArgumentError)
+    expect { resource.send(:group_name, { aj: "is freakin awesome" }) }.to raise_error(ArgumentError)
   end
 end
 
@@ -85,7 +85,7 @@ describe Chef::Resource::Group, "gid" do
   end
 
   it "does not allow a hash" do
-    expect { resource.send(:gid, { :aj => "is freakin awesome" }) }.to raise_error(ArgumentError)
+    expect { resource.send(:gid, { aj: "is freakin awesome" }) }.to raise_error(ArgumentError)
   end
 end
 
@@ -109,7 +109,7 @@ describe Chef::Resource::Group, "members" do
     end
 
     it "(#{method}) does not allow a hash" do
-      expect { resource.send(method, { :aj => "is freakin awesome" }) }.to raise_error(ArgumentError)
+      expect { resource.send(method, { aj: "is freakin awesome" }) }.to raise_error(ArgumentError)
     end
   end
 end
@@ -127,7 +127,7 @@ describe Chef::Resource::Group, "append" do
   end
 
   it "does not allow a hash" do
-    expect { resource.send(:gid, { :aj => "is freakin awesome" }) }.to raise_error(ArgumentError)
+    expect { resource.send(:gid, { aj: "is freakin awesome" }) }.to raise_error(ArgumentError)
   end
 
   describe "when it has members" do

data/spec/unit/resource/kernel_module_spec.rb

@@ -0,0 +1,48 @@
+#
+# Copyright:: Copyright 2018, Chef Software, 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::Resource::KernelModule do
+  let(:resource) { Chef::Resource::KernelModule.new("foo") }
+
+  it "sets resource name as :kernel_module" do
+    expect(resource.resource_name).to eql(:kernel_module)
+  end
+
+  it "is not a preview resource in Chef 15" do
+    pending("Chef 15") unless Chef::VERSION.start_with?("15")
+    expect(resource.class.preview_resource).to be_falsey
+  end
+
+  it "sets the default action as :install" do
+    expect(resource.action).to eql([:install])
+  end
+
+  it "sets the modname property as its name property" do
+    expect(resource.modname).to eql("foo")
+  end
+
+  it "supports :create and :flush actions" do
+    expect { resource.action :install }.not_to raise_error
+    expect { resource.action :uninstall }.not_to raise_error
+    expect { resource.action :blacklist }.not_to raise_error
+    expect { resource.action :load }.not_to raise_error
+    expect { resource.action :unload }.not_to raise_error
+    expect { resource.action :delete }.to raise_error(ArgumentError)
+  end
+end

data/spec/unit/resource/mount_spec.rb

@@ -44,6 +44,16 @@ describe Chef::Resource::Mount do
     expect(resource.device).to eql("/dev/sdb3")
   end
 
+  it "allows you to set mount_point property" do
+    resource.mount_point "U:"
+    expect(resource.mount_point).to eql("U:")
+  end
+
+  it "raises error when mount_point property is not set" do
+    resource.mount_point nil
+    expect { resource.mounted("poop") }.to raise_error(ArgumentError)
+  end
+
   it "sets fsck_device to '-' by default" do
     expect(resource.fsck_device).to eql("-")
   end
@@ -134,19 +144,19 @@ describe Chef::Resource::Mount do
   end
 
   it "defaults all feature support to false" do
-    support_hash = { :remount => false }
+    support_hash = { remount: false }
     expect(resource.supports).to eq(support_hash)
   end
 
   it "allows you to set feature support as an array" do
     support_array = [ :remount ]
-    support_hash = { :remount => true }
+    support_hash = { remount: true }
     resource.supports(support_array)
     expect(resource.supports).to eq(support_hash)
   end
 
   it "allows you to set feature support as a hash" do
-    support_hash = { :remount => true }
+    support_hash = { remount: true }
     resource.supports(support_hash)
     expect(resource.supports).to eq(support_hash)
   end

data/spec/unit/resource/package_spec.rb

@@ -56,8 +56,8 @@ describe Chef::Resource::Package do
   end
 
   it "accepts a hash for response file template variables" do
-    resource.response_file_variables({ :variables => true })
-    expect(resource.response_file_variables).to eql({ :variables => true })
+    resource.response_file_variables({ variables: true })
+    expect(resource.response_file_variables).to eql({ variables: true })
   end
 
   it "accepts a string for the source" do

data/spec/unit/resource/powershell_package_source_spec.rb

@@ -0,0 +1,219 @@
+# Author:: Tor Magnus Rakvåg (tm@intility.no)
+# Copyright:: 2018, Intility AS
+# 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::Resource::PowershellPackageSource do
+  let(:resource) { Chef::Resource::PowershellPackageSource.new("MyGallery") }
+  let(:provider) { resource.provider_for_action(:enable) }
+
+  it "has a resource name of :powershell_package_source" do
+    expect(resource.resource_name).to eql(:powershell_package_source)
+  end
+
+  it "the name_property is 'name'" do
+    expect(resource.source_name).to eql("MyGallery")
+  end
+
+  it "the default action is :register" do
+    expect(resource.action).to eql([:register])
+  end
+
+  it "supports :register and :unregister actions" do
+    expect { resource.action :register }.not_to raise_error
+    expect { resource.action :unregister }.not_to raise_error
+  end
+
+  it "the url property accepts strings" do
+    resource.url("https://mygallery.company.co/api/v2/")
+    expect(resource.url).to eql("https://mygallery.company.co/api/v2/")
+  end
+
+  it "the trusted property accepts true and false" do
+    resource.trusted(false)
+    expect(resource.trusted).to eql(false)
+    resource.trusted(true)
+    expect(resource.trusted).to eql(true)
+  end
+
+  it "trusted defaults to false" do
+    expect(resource.trusted).to eql(false)
+  end
+
+  it "provider_name accepts 'Programs', 'msi', 'NuGet', 'msu', 'PowerShellGet', 'psl', 'chocolatey'" do
+    expect { resource.provider_name("Programs") }.not_to raise_error
+    expect { resource.provider_name("msi") }.not_to raise_error
+    expect { resource.provider_name("NuGet") }.not_to raise_error
+    expect { resource.provider_name("msu") }.not_to raise_error
+    expect { resource.provider_name("PowerShellGet") }.not_to raise_error
+    expect { resource.provider_name("psl") }.not_to raise_error
+    expect { resource.provider_name("chocolatey") }.not_to raise_error
+  end
+
+  it "the publish_location property accepts strings" do
+    resource.publish_location("https://mygallery.company.co/api/v2/package")
+    expect(resource.publish_location).to eql("https://mygallery.company.co/api/v2/package")
+  end
+
+  it "the script_source_location property accepts strings" do
+    resource.publish_location("https://mygallery.company.co/api/v2/scripts")
+    expect(resource.publish_location).to eql("https://mygallery.company.co/api/v2/scripts")
+  end
+
+  it "the script_publish_location property accepts strings" do
+    resource.publish_location("https://mygallery.company.co/api/v2/scripts")
+    expect(resource.publish_location).to eql("https://mygallery.company.co/api/v2/scripts")
+  end
+
+  describe "#build_ps_repository_command" do
+    before do
+      resource.source_name("MyGallery")
+      resource.url("https://mygallery.company.co/api/v2/")
+    end
+
+    context "#register" do
+      it "builds a minimal command" do
+        expect(provider.build_ps_repository_command("Register", resource)).to eql("Register-PSRepository -Name 'MyGallery' -SourceLocation 'https://mygallery.company.co/api/v2/' -InstallationPolicy 'Untrusted'")
+      end
+
+      it "builds a command with trusted set to true" do
+        resource.trusted(true)
+        expect(provider.build_ps_repository_command("Register", resource)).to eql("Register-PSRepository -Name 'MyGallery' -SourceLocation 'https://mygallery.company.co/api/v2/' -InstallationPolicy 'Trusted'")
+      end
+
+      it "builds a command with a publish location" do
+        resource.publish_location("https://mygallery.company.co/api/v2/package")
+        expect(provider.build_ps_repository_command("Register", resource)).to eql("Register-PSRepository -Name 'MyGallery' -SourceLocation 'https://mygallery.company.co/api/v2/' -InstallationPolicy 'Untrusted' -PublishLocation 'https://mygallery.company.co/api/v2/package'")
+      end
+
+      it "builds a command with a script source location" do
+        resource.script_source_location("https://mygallery.company.co/api/v2/scripts")
+        expect(provider.build_ps_repository_command("Register", resource)).to eql("Register-PSRepository -Name 'MyGallery' -SourceLocation 'https://mygallery.company.co/api/v2/' -InstallationPolicy 'Untrusted' -ScriptSourceLocation 'https://mygallery.company.co/api/v2/scripts'")
+      end
+
+      it "builds a command with a script publish location" do
+        resource.script_publish_location("https://mygallery.company.co/api/v2/scripts/package")
+        expect(provider.build_ps_repository_command("Register", resource)).to eql("Register-PSRepository -Name 'MyGallery' -SourceLocation 'https://mygallery.company.co/api/v2/' -InstallationPolicy 'Untrusted' -ScriptPublishLocation 'https://mygallery.company.co/api/v2/scripts/package'")
+      end
+    end
+
+    context "#set" do
+      it "builds a minimal command" do
+        expect(provider.build_ps_repository_command("Set", resource)).to eql("Set-PSRepository -Name 'MyGallery' -SourceLocation 'https://mygallery.company.co/api/v2/' -InstallationPolicy 'Untrusted'")
+      end
+
+      it "builds a command to change the url" do
+        resource.url("https://othergallery.company.co/api/v2/")
+        expect(provider.build_ps_repository_command("Set", resource)).to eql("Set-PSRepository -Name 'MyGallery' -SourceLocation 'https://othergallery.company.co/api/v2/' -InstallationPolicy 'Untrusted'")
+      end
+
+      it "builds a command with trusted set to true" do
+        resource.trusted(true)
+        expect(provider.build_ps_repository_command("Set", resource)).to eql("Set-PSRepository -Name 'MyGallery' -SourceLocation 'https://mygallery.company.co/api/v2/' -InstallationPolicy 'Trusted'")
+      end
+
+      it "builds a command with a publish location" do
+        resource.publish_location("https://mygallery.company.co/api/v2/package")
+        expect(provider.build_ps_repository_command("Set", resource)).to eql("Set-PSRepository -Name 'MyGallery' -SourceLocation 'https://mygallery.company.co/api/v2/' -InstallationPolicy 'Untrusted' -PublishLocation 'https://mygallery.company.co/api/v2/package'")
+      end
+
+      it "builds a command with a script source location" do
+        resource.script_source_location("https://mygallery.company.co/api/v2/scripts")
+        expect(provider.build_ps_repository_command("Set", resource)).to eql("Set-PSRepository -Name 'MyGallery' -SourceLocation 'https://mygallery.company.co/api/v2/' -InstallationPolicy 'Untrusted' -ScriptSourceLocation 'https://mygallery.company.co/api/v2/scripts'")
+      end
+
+      it "builds a command with a script publish location" do
+        resource.script_publish_location("https://mygallery.company.co/api/v2/scripts/package")
+        expect(provider.build_ps_repository_command("Set", resource)).to eql("Set-PSRepository -Name 'MyGallery' -SourceLocation 'https://mygallery.company.co/api/v2/' -InstallationPolicy 'Untrusted' -ScriptPublishLocation 'https://mygallery.company.co/api/v2/scripts/package'")
+      end
+    end
+  end
+
+  describe "#build_package_source_command" do
+    before do
+      resource.source_name("NuGet")
+      resource.url("http://nuget.org/api/v2/")
+    end
+
+    context "#register" do
+      it "builds a minimal command" do
+        expect(provider.build_package_source_command("Register", resource)).to eql("Register-PackageSource -Name 'NuGet' -Location 'http://nuget.org/api/v2/' -Trusted:$false -ProviderName 'NuGet'")
+      end
+
+      it "builds a command with trusted set to true" do
+        resource.trusted(true)
+        expect(provider.build_package_source_command("Register", resource)).to eql("Register-PackageSource -Name 'NuGet' -Location 'http://nuget.org/api/v2/' -Trusted:$true -ProviderName 'NuGet'")
+      end
+
+      it "builds a command with a different provider" do
+        resource.source_name("choco")
+        resource.url("https://chocolatey.org/api/v2/")
+        resource.provider_name("chocolatey")
+        expect(provider.build_package_source_command("Register", resource)).to eql("Register-PackageSource -Name 'choco' -Location 'https://chocolatey.org/api/v2/' -Trusted:$false -ProviderName 'chocolatey'")
+      end
+    end
+
+    context "#set" do
+      it "builds a minimal command" do
+        expect(provider.build_package_source_command("Set", resource)).to eql("Set-PackageSource -Name 'NuGet' -Location 'http://nuget.org/api/v2/' -Trusted:$false -ProviderName 'NuGet'")
+      end
+
+      it "builds a command to change the url" do
+        resource.url("https://nuget.company.co/api/v2/")
+        expect(provider.build_package_source_command("Set", resource)).to eql("Set-PackageSource -Name 'NuGet' -Location 'https://nuget.company.co/api/v2/' -Trusted:$false -ProviderName 'NuGet'")
+      end
+
+      it "builds a command with trusted set to true" do
+        resource.trusted(true)
+        expect(provider.build_package_source_command("Set", resource)).to eql("Set-PackageSource -Name 'NuGet' -Location 'http://nuget.org/api/v2/' -Trusted:$true -ProviderName 'NuGet'")
+      end
+
+      it "builds a command with a different provider" do
+        resource.source_name("choco")
+        resource.url("https://chocolatey.org/api/v2/")
+        resource.provider_name("chocolatey")
+        expect(provider.build_package_source_command("Set", resource)).to eql("Set-PackageSource -Name 'choco' -Location 'https://chocolatey.org/api/v2/' -Trusted:$false -ProviderName 'chocolatey'")
+      end
+    end
+  end
+
+  describe "#psrepository_cmdlet_appropriate?" do
+    it "returns true if the provider_name is 'PowerShellGet'" do
+      resource.provider_name("PowerShellGet")
+      expect(provider.psrepository_cmdlet_appropriate?).to eql(true)
+    end
+
+    it "returns false if the provider_name is something else" do
+      resource.provider_name("NuGet")
+      expect(provider.psrepository_cmdlet_appropriate?).to eql(false)
+    end
+  end
+
+  describe "#package_source_exists?" do
+    it "returns true if it exists" do
+      allow(provider).to receive(:powershell_out!).with("(Get-PackageSource -Name 'MyGallery').Name").and_return(double("powershell_out!", stdout: "MyGallery\r\n"))
+      resource.source_name("MyGallery")
+      expect(provider.package_source_exists?).to eql(true)
+    end
+
+    it "returns false if it doesn't exist" do
+      allow(provider).to receive(:powershell_out!).with("(Get-PackageSource -Name 'MyGallery').Name").and_return(double("powershell_out!", stdout: ""))
+      resource.source_name("MyGallery")
+      expect(provider.package_source_exists?).to eql(false)
+    end
+  end
+end