chef 18.8.11 → 18.8.46

data/spec/unit/resource/registry_key_spec.rb

@@ -1,215 +0,0 @@
-#
-# Author:: Lamont Granquist (<lamont@chef.io>)
-# Copyright:: Copyright (c) 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::RegistryKey, "initialize" do
-  let(:resource) { Chef::Resource::RegistryKey.new("HKCU\\Software\\Raxicoricofallapatorius") }
-
-  it "sets the resource_name to :registry_key" do
-    expect(resource.resource_name).to eql(:registry_key)
-  end
-
-  it "the key property is the name_property" do
-    expect(resource.key).to eql("HKCU\\Software\\Raxicoricofallapatorius")
-  end
-
-  it "sets the default action as :create" do
-    expect(resource.action).to eql([:create])
-  end
-
-  it "supports :create, :create_if_missing, :delete, :delete_key actions" do
-    expect { resource.action :create }.not_to raise_error
-    expect { resource.action :create_if_missing }.not_to raise_error
-    expect { resource.action :delete }.not_to raise_error
-    expect { resource.action :delete_key }.not_to raise_error
-  end
-
-  it "defaults recursive to false" do
-    expect(resource.recursive).to eql(false)
-  end
-
-  it "defaults architecture to :machine" do
-    expect(resource.architecture).to eql(:machine)
-  end
-
-  it "sets action to :create" do
-    expect(resource.action).to eql([:create])
-  end
-
-  %w{create create_if_missing delete delete_key}.each do |action|
-    it "allows action #{action}" do
-      expect(resource.allowed_actions.detect { |a| a == action.to_sym }).to eql(action.to_sym)
-    end
-  end
-end
-
-describe Chef::Resource::RegistryKey, "key" do
-  let(:resource) { Chef::Resource::RegistryKey.new("HKCU\\Software\\Raxicoricofallapatorius") }
-
-  it "allows a string" do
-    resource.key "HKCU\\Software\\Poosh"
-    expect(resource.key).to eql("HKCU\\Software\\Poosh")
-  end
-
-  it "does not allow an integer" do
-    expect { resource.send(:key, 100) }.to raise_error(ArgumentError)
-  end
-
-  it "does not allow a hash" do
-    expect { resource.send(:key, { sonic: "screwdriver" }) }.to raise_error(ArgumentError)
-  end
-end
-
-describe Chef::Resource::RegistryKey, "values" do
-  let(:resource) { Chef::Resource::RegistryKey.new("HKCU\\Software\\Raxicoricofallapatorius") }
-
-  it "allows a single proper hash of registry values" do
-    resource.values( { name: "poosh", type: :string, data: "carmen" } )
-    expect(resource.values).to eql([ { name: "poosh", type: :string, data: "carmen" } ])
-  end
-
-  it "allows an array of proper hashes of registry values" do
-    resource.values [ { name: "poosh", type: :string, data: "carmen" } ]
-    expect(resource.values).to eql([ { name: "poosh", type: :string, data: "carmen" } ])
-  end
-
-  it "returns checksummed data if the type is unsafe" do
-    resource.values( { name: "poosh", type: :binary, data: 255.chr * 1 })
-    expect(resource.values).to eql([ { name: "poosh", type: :binary, data: "a8100ae6aa1940d0b663bb31cd466142ebbdbd5187131b92d93818987832eb89" } ])
-  end
-
-  it "raises an exception if the name field is missing" do
-    expect { resource.values [ { type: :string, data: "carmen" } ] }.to raise_error(ArgumentError)
-  end
-
-  it "raises an exception if extra fields are present" do
-    expect { resource.values [ { name: "poosh", type: :string, data: "carmen", screwdriver: "sonic" } ] }.to raise_error(ArgumentError)
-  end
-
-  it "does not allow a string" do
-    expect { resource.send(:values, "souffle") }.to raise_error(ArgumentError)
-  end
-
-  it "does not allow an integer" do
-    expect { resource.send(:values, 100) }.to raise_error(ArgumentError)
-  end
-
-  it "raises an exception if type of name is not string" do
-    expect { resource.values([ { name: 123, type: :string, data: "carmen" } ]) }.to raise_error(ArgumentError)
-  end
-
-  it "does not raise an exception if type of name is string" do
-    expect { resource.values([ { name: "123", type: :string, data: "carmen" } ]) }.to_not raise_error
-  end
-
-  context "type key not given" do
-    it "does not raise an exception" do
-      expect { resource.values([ { name: "123", data: "carmen" } ]) }.to_not raise_error
-    end
-  end
-
-  context "type key given" do
-    it "raises an exception if type of type is not symbol" do
-      expect { resource.values([ { name: "123", type: "string", data: "carmen" } ]) }.to raise_error(ArgumentError)
-    end
-
-    it "does not raise an exception if type of type is symbol" do
-      expect { resource.values([ { name: "123", type: :string, data: "carmen" } ]) }.to_not raise_error
-    end
-  end
-
-  it "does not raise an exception if keys are in string format" do
-    expect { resource.values([ { "name" => "123", "type" => :string, "data" => "carmen" } ]) }.to_not raise_error
-  end
-end
-
-describe Chef::Resource::RegistryKey, "recursive" do
-  let(:resource) { Chef::Resource::RegistryKey.new("HKCU\\Software\\Raxicoricofallapatorius") }
-
-  it "allows a boolean" do
-    resource.recursive(true)
-    expect(resource.recursive).to eql(true)
-  end
-
-  it "does not allow a hash" do
-    expect { resource.recursive({ sonic: :screwdriver }) }.to raise_error(ArgumentError)
-  end
-
-  it "does not allow an array" do
-    expect { resource.recursive(%i{nose chin}) }.to raise_error(ArgumentError)
-  end
-
-  it "does not allow a string" do
-    expect { resource.recursive("souffle") }.to raise_error(ArgumentError)
-  end
-
-  it "does not allow an integer" do
-    expect { resource.recursive(100) }.to raise_error(ArgumentError)
-  end
-end
-
-describe Chef::Resource::RegistryKey, "architecture" do
-  let(:resource) { Chef::Resource::RegistryKey.new("HKCU\\Software\\Raxicoricofallapatorius") }
-
-  %i{i386 x86_64 machine}.each do |arch|
-    it "allows #{arch} as a symbol" do
-      resource.architecture(arch)
-      expect(resource.architecture).to eql(arch)
-    end
-  end
-
-  it "does not allow other symbols" do
-    expect { resource.architecture(:nope) }.to raise_error(ArgumentError)
-  end
-
-  it "does not allow a hash" do
-    expect { resource.architecture({ sonic: :screwdriver }) }.to raise_error(ArgumentError)
-  end
-
-  it "does not allow an array" do
-    expect { resource.architecture(%i{nose chin}) }.to raise_error(ArgumentError)
-  end
-
-  it "does not allow a string" do
-    expect { resource.architecture("souffle") }.to raise_error(ArgumentError)
-  end
-
-  it "does not allow an integer" do
-    expect { resource.architecture(100) }.to raise_error(ArgumentError)
-  end
-end
-
-describe Chef::Resource::RegistryKey, ":unscrubbed_values" do
-  let(:resource) { Chef::Resource::RegistryKey.new("HKCU\\Software\\Raxicoricofallapatorius") }
-
-  it "returns unsafe data as-is" do
-    key_values = [ { name: "poosh", type: :binary, data: 255.chr * 1 } ]
-    resource.values(key_values)
-    expect(resource.unscrubbed_values).to eql(key_values)
-  end
-end
-
-describe Chef::Resource::RegistryKey, "state" do
-  let(:resource) { Chef::Resource::RegistryKey.new("HKCU\\Software\\Raxicoricofallapatorius") }
-
-  it "returns scrubbed values" do
-    resource.values([ { name: "poosh", type: :binary, data: 255.chr * 1 } ])
-    expect(resource.state_for_resource_reporter[:values]).to eql( [{ name: "poosh", type: :binary, data: "a8100ae6aa1940d0b663bb31cd466142ebbdbd5187131b92d93818987832eb89" }] )
-  end
-end

data/spec/unit/resource/remote_directory_spec.rb

@@ -1,118 +0,0 @@
-#
-# Author:: Adam Jacob (<adam@chef.io>)
-# Copyright:: Copyright (c) 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::RemoteDirectory do
-
-  let(:resource) { Chef::Resource::RemoteDirectory.new("/etc/dunk") }
-
-  it "the path property is the name_property" do
-    expect(resource.path).to eql("/etc/dunk")
-  end
-
-  it "sets the default action as :create" do
-    expect(resource.action).to eql([:create])
-  end
-
-  it "supports :create, :create_if_missing, :delete actions" do
-    expect { resource.action :create }.not_to raise_error
-    expect { resource.action :create_if_missing }.not_to raise_error
-    expect { resource.action :delete }.not_to raise_error
-  end
-
-  it "accepts a String for the cookbook property" do
-    resource.cookbook "foo"
-    expect(resource.cookbook).to eql("foo")
-  end
-
-  it "accepts a String for the source property" do
-    resource.source "foo"
-    expect(resource.source).to eql("foo")
-  end
-
-  it "uses the basename of the pat property as the default value of the source property" do
-    resource.path "/foo/bar"
-    expect(resource.source).to eql("bar")
-  end
-
-  it "files_backup property defaults to 5" do
-    expect(resource.files_backup).to eql(5)
-  end
-
-  it "accepts an Integer for the files_backup property" do
-    resource.files_backup 1
-    expect(resource.files_backup).to eql(1)
-  end
-
-  it "accepts false for the files_backup property" do
-    resource.files_backup false
-    expect(resource.files_backup).to eql(false)
-  end
-
-  it "accepts 3 or 4 digits for the files_mode property" do
-    resource.files_mode 100
-    expect(resource.files_mode).to eql(100)
-    resource.files_mode 1000
-    expect(resource.files_mode).to eql(1000)
-  end
-
-  it "accepts a String or number for the files_group property" do
-    resource.files_group "heart"
-    expect(resource.files_group).to eql("heart")
-    resource.files_group 1000
-    expect(resource.files_group).to eql(1000)
-  end
-
-  it "accepts a String or number for the files_owner property" do
-    resource.files_owner "heart"
-    expect(resource.files_owner).to eql("heart")
-    resource.files_owner 1000
-    expect(resource.files_owner).to eql(1000)
-  end
-
-  it "overwrite property has the default value of true" do
-    expect(resource.overwrite).to be true
-  end
-
-  it "recursive property has the default value of true" do
-    expect(resource.recursive).to be true
-  end
-
-  describe "when it has cookbook, files owner, files mode, and source" do
-    before do
-      resource.path("/var/path/")
-      resource.cookbook("pokemon.rb")
-      resource.files_owner("root")
-      resource.files_group("supergroup")
-      resource.files_mode("0664")
-      resource.source("/var/source/")
-    end
-
-    it "describes its state" do
-      state = resource.state_for_resource_reporter
-      expect(state[:files_owner]).to eq("root")
-      expect(state[:files_group]).to eq("supergroup")
-      expect(state[:files_mode]).to eq("0664")
-    end
-
-    it "returns the path as its identity" do
-      expect(resource.identity).to eq("/var/path/")
-    end
-  end
-end

data/spec/unit/resource/remote_file_spec.rb

@@ -1,220 +0,0 @@
-#
-# Author:: Adam Jacob (<adam@chef.io>)
-# Author:: Tyler Cloke (<tyler@chef.io>)
-# Copyright:: Copyright (c) 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::RemoteFile do
-
-  let(:resource) { Chef::Resource::RemoteFile.new("fakey_fakerton") }
-
-  describe "name_property" do
-    it "the path property is the name_property" do
-      expect(resource.path).to eql("fakey_fakerton")
-    end
-  end
-
-  describe "Actions" do
-    it "sets the default action as :create" do
-      expect(resource.action).to eql([:create])
-    end
-
-    it "supports :create, :create_if_missing, :delete, :touch actions" do
-      expect { resource.action :create }.not_to raise_error
-      expect { resource.action :create_if_missing }.not_to raise_error
-      expect { resource.action :delete }.not_to raise_error
-      expect { resource.action :touch }.not_to raise_error
-    end
-  end
-
-  describe "initialize" do
-    it "is a subclass of Chef::Resource::File" do
-      expect(resource).to be_a_kind_of(Chef::Resource::File)
-    end
-  end
-
-  it "says its provider is RemoteFile when the source is an absolute URI" do
-    resource.source("http://www.google.com/robots.txt")
-    expect(resource.provider_for_action(:create)).to be_kind_of(Chef::Provider::RemoteFile)
-  end
-
-  it "says its provider is RemoteFile when the source is a network share" do
-    resource.source("\\\\fakey\\fakerton\\fake.txt")
-    expect(resource.provider_for_action(:create)).to be_kind_of(Chef::Provider::RemoteFile)
-  end
-
-  describe "source" do
-    it "does not have a default value for 'source'" do
-      expect(resource.source).to eql([])
-    end
-
-    it "accepts a URI for the remote file source" do
-      resource.source "http://opscode.com/"
-      expect(resource.source).to eql([ "http://opscode.com/" ])
-    end
-
-    it "accepts a windows network share source" do
-      resource.source "\\\\fakey\\fakerton\\fake.txt"
-      expect(resource.source).to eql([ "\\\\fakey\\fakerton\\fake.txt" ])
-    end
-
-    it "accepts file URIs with spaces" do
-      resource.source("file:///C:/foo bar")
-      expect(resource.source).to eql(["file:///C:/foo bar"])
-    end
-
-    it "accepts a delayed evaluator (string) for the remote file source" do
-      resource.source Chef::DelayedEvaluator.new { "http://opscode.com/" }
-      expect(resource.source).to eql([ "http://opscode.com/" ])
-    end
-
-    it "accepts an array of URIs for the remote file source" do
-      resource.source([ "http://opscode.com/", "http://puppetlabs.com/" ])
-      expect(resource.source).to eql([ "http://opscode.com/", "http://puppetlabs.com/" ])
-    end
-
-    it "accepts a delated evaluator (array) for the remote file source" do
-      resource.source Chef::DelayedEvaluator.new { [ "http://opscode.com/", "http://puppetlabs.com/" ] }
-      expect(resource.source).to eql([ "http://opscode.com/", "http://puppetlabs.com/" ])
-    end
-
-    it "accepts an multiple URIs as arguments for the remote file source" do
-      resource.source("http://opscode.com/", "http://puppetlabs.com/")
-      expect(resource.source).to eql([ "http://opscode.com/", "http://puppetlabs.com/" ])
-    end
-
-    it "only accept a single argument if a delayed evalutor is used" do
-      expect do
-        resource.source("http://opscode.com/", Chef::DelayedEvaluator.new { "http://opscode.com/" })
-      end.to raise_error(Chef::Exceptions::InvalidRemoteFileURI)
-    end
-
-    it "only accept a single array item if a delayed evalutor is used" do
-      expect do
-        resource.source(["http://opscode.com/", Chef::DelayedEvaluator.new { "http://opscode.com/" }])
-      end.to raise_error(Chef::Exceptions::InvalidRemoteFileURI)
-    end
-
-    it "does not accept a non-URI as the source" do
-      expect { resource.source("not-a-uri") }.to raise_error(Chef::Exceptions::InvalidRemoteFileURI)
-    end
-
-    it "does not accept a non-URI as the source when read from a delayed evaluator" do
-      expect do
-        resource.source(Chef::DelayedEvaluator.new { "not-a-uri" })
-        resource.source
-      end.to raise_error(Chef::Exceptions::InvalidRemoteFileURI)
-    end
-
-    it "raises an exception when source is an empty array" do
-      expect { resource.source([]) }.to raise_error(ArgumentError)
-    end
-
-  end
-
-  describe "checksum" do
-    it "accepts a string for the checksum object" do
-      resource.checksum "asdf"
-      expect(resource.checksum).to eql("asdf")
-    end
-
-    it "defaults to nil" do
-      expect(resource.checksum).to eq(nil)
-    end
-  end
-
-  describe "ftp_active_mode" do
-    it "accepts a boolean for the ftp_active_mode object" do
-      resource.ftp_active_mode true
-      expect(resource.ftp_active_mode).to be_truthy
-    end
-
-    it "defaults to false" do
-      expect(resource.ftp_active_mode).to be_falsey
-    end
-  end
-
-  describe "conditional get options" do
-    it "defaults to using etags and last modified" do
-      expect(resource.use_etags).to be_truthy
-      expect(resource.use_last_modified).to be_truthy
-    end
-
-    it "enable or disables etag and last modified options as a group" do
-      resource.use_conditional_get(false)
-      expect(resource.use_etags).to be_falsey
-      expect(resource.use_last_modified).to be_falsey
-
-      resource.use_conditional_get(true)
-      expect(resource.use_etags).to be_truthy
-      expect(resource.use_last_modified).to be_truthy
-    end
-
-    it "disables etags individually" do
-      resource.use_etags(false)
-      expect(resource.use_etags).to be_falsey
-      expect(resource.use_last_modified).to be_truthy
-    end
-
-    it "disables last modified individually" do
-      resource.use_last_modified(false)
-      expect(resource.use_last_modified).to be_falsey
-      expect(resource.use_etags).to be_truthy
-    end
-
-  end
-
-  describe "when it has group, mode, owner, source, and checksum" do
-    before do
-      if ChefUtils.windows?
-        resource.path("C:/temp/origin/file.txt")
-        resource.rights(:read, "Everyone")
-        resource.deny_rights(:full_control, "Clumsy_Sam")
-      else
-        resource.path("/this/path/")
-        resource.group("pokemon")
-        resource.mode("0664")
-        resource.owner("root")
-      end
-      resource.source("https://www.google.com/images/srpr/logo3w.png")
-      resource.checksum("1" * 26)
-    end
-
-    it "describes its state" do
-      state = resource.state_for_resource_reporter
-      if ChefUtils.windows?
-        puts state
-        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("pokemon")
-        expect(state[:mode]).to eq("0664")
-        expect(state[:owner]).to eq("root")
-        expect(state[:checksum]).to eq("1" * 26)
-      end
-    end
-
-    it "returns the path as its identity" do
-      if ChefUtils.windows?
-        expect(resource.identity).to eq("C:/temp/origin/file.txt")
-      else
-        expect(resource.identity).to eq("/this/path/")
-      end
-    end
-  end
-end