diff options
Diffstat (limited to 'spec/unit/policy_builder')
-rw-r--r-- | spec/unit/policy_builder/dynamic_spec.rb | 10 | ||||
-rw-r--r-- | spec/unit/policy_builder/expand_node_object_spec.rb | 14 | ||||
-rw-r--r-- | spec/unit/policy_builder/policyfile_spec.rb | 31 |
3 files changed, 23 insertions, 32 deletions
diff --git a/spec/unit/policy_builder/dynamic_spec.rb b/spec/unit/policy_builder/dynamic_spec.rb index 19ad057863..f91b0ba7d2 100644 --- a/spec/unit/policy_builder/dynamic_spec.rb +++ b/spec/unit/policy_builder/dynamic_spec.rb @@ -22,8 +22,8 @@ require "chef/policy_builder" describe Chef::PolicyBuilder::Dynamic do let(:node_name) { "joe_node" } - let(:ohai_data) { {"platform" => "ubuntu", "platform_version" => "13.04", "fqdn" => "joenode.example.com"} } - let(:json_attribs) { {"custom_attr" => "custom_attr_value"} } + let(:ohai_data) { { "platform" => "ubuntu", "platform_version" => "13.04", "fqdn" => "joenode.example.com" } } + let(:json_attribs) { { "custom_attr" => "custom_attr_value" } } let(:override_runlist) { nil } let(:events) { Chef::EventDispatch::Dispatcher.new } @@ -123,7 +123,7 @@ describe Chef::PolicyBuilder::Dynamic do context "and no policyfile attributes are present in json_attribs" do - let(:json_attribs) { {"foo" => "bar"} } + let(:json_attribs) { { "foo" => "bar" } } it "uses the ExpandNodeObject implementation" do expect(implementation).to be_a(Chef::PolicyBuilder::ExpandNodeObject) @@ -171,7 +171,7 @@ describe Chef::PolicyBuilder::Dynamic do context "and policyfile attributes are present in json_attribs" do - let(:json_attribs) { {"policy_name" => "example-policy", "policy_group" => "testing"} } + let(:json_attribs) { { "policy_name" => "example-policy", "policy_group" => "testing" } } it "uses the Policyfile implementation" do expect(implementation).to be_a(Chef::PolicyBuilder::Policyfile) @@ -207,7 +207,6 @@ describe Chef::PolicyBuilder::Dynamic do context "when not running chef solo" do - context "when successful" do before do @@ -243,7 +242,6 @@ describe Chef::PolicyBuilder::Dynamic do expect(implementation).to receive(:finish_load_node).and_raise("oops") end - it "sends a node_load_failed event and re-raises" do expect(events).to receive(:node_load_failed) expect { policy_builder.load_node }.to raise_error("oops") diff --git a/spec/unit/policy_builder/expand_node_object_spec.rb b/spec/unit/policy_builder/expand_node_object_spec.rb index 2a1a85e605..8667532d72 100644 --- a/spec/unit/policy_builder/expand_node_object_spec.rb +++ b/spec/unit/policy_builder/expand_node_object_spec.rb @@ -22,8 +22,8 @@ require "chef/policy_builder" describe Chef::PolicyBuilder::ExpandNodeObject do let(:node_name) { "joe_node" } - let(:ohai_data) { {"platform" => "ubuntu", "platform_version" => "13.04", "fqdn" => "joenode.example.com"} } - let(:json_attribs) { {"run_list" => []} } + let(:ohai_data) { { "platform" => "ubuntu", "platform_version" => "13.04", "fqdn" => "joenode.example.com" } } + let(:json_attribs) { { "run_list" => [] } } let(:override_runlist) { "recipe[foo::default]" } let(:events) { Chef::EventDispatch::Dispatcher.new } let(:policy_builder) { Chef::PolicyBuilder::ExpandNodeObject.new(node_name, ohai_data, json_attribs, override_runlist, events) } @@ -155,8 +155,8 @@ describe Chef::PolicyBuilder::ExpandNodeObject do let(:override_runlist) { nil } let(:primary_runlist) { ["recipe[primary::default]"] } - let(:original_default_attrs) { {"default_key" => "default_value"} } - let(:original_override_attrs) { {"override_key" => "override_value"} } + let(:original_default_attrs) { { "default_key" => "default_value" } } + let(:original_override_attrs) { { "override_key" => "override_value" } } let(:node) do node = Chef::Node.new @@ -227,7 +227,7 @@ describe Chef::PolicyBuilder::ExpandNodeObject do context "when JSON attributes are given on the command line" do - let(:json_attribs) { {"run_list" => ["recipe[json_attribs::default]"], "json_attribs_key" => "json_attribs_value" } } + let(:json_attribs) { { "run_list" => ["recipe[json_attribs::default]"], "json_attribs_key" => "json_attribs_value" } } it "sets the run list according to the given JSON" do expect(node.run_list).to eq(["recipe[json_attribs::default]"]) @@ -267,7 +267,7 @@ describe Chef::PolicyBuilder::ExpandNodeObject do let(:configured_environment) { environment.name } let(:environment) do - environment = Chef::Environment.new.tap {|e| e.name("prod") } + environment = Chef::Environment.new.tap { |e| e.name("prod") } expect(Chef::Environment).to receive(:load).with("prod").and_return(environment) environment end @@ -293,7 +293,7 @@ describe Chef::PolicyBuilder::ExpandNodeObject do let(:chef_http) { double("Chef::ServerAPI") } let(:cookbook_resolve_url) { "environments/#{node.chef_environment}/cookbook_versions" } - let(:cookbook_resolve_post_data) { {:run_list=>["first::default", "second::default"]} } + let(:cookbook_resolve_post_data) { { :run_list => ["first::default", "second::default"] } } # cookbook_hash is just a hash, but since we're passing it between mock # objects, we get a little better test strictness by using a double (which diff --git a/spec/unit/policy_builder/policyfile_spec.rb b/spec/unit/policy_builder/policyfile_spec.rb index 1425fe2b70..6b71b4d43e 100644 --- a/spec/unit/policy_builder/policyfile_spec.rb +++ b/spec/unit/policy_builder/policyfile_spec.rb @@ -22,8 +22,8 @@ require "chef/policy_builder" describe Chef::PolicyBuilder::Policyfile do let(:node_name) { "joe_node" } - let(:ohai_data) { {"platform" => "ubuntu", "platform_version" => "13.04", "fqdn" => "joenode.example.com"} } - let(:json_attribs) { {"custom_attr" => "custom_attr_value"} } + let(:ohai_data) { { "platform" => "ubuntu", "platform_version" => "13.04", "fqdn" => "joenode.example.com" } } + let(:json_attribs) { { "custom_attr" => "custom_attr_value" } } let(:override_runlist) { nil } let(:events) { Chef::EventDispatch::Dispatcher.new } let(:policy_builder) { Chef::PolicyBuilder::Policyfile.new(node_name, ohai_data, json_attribs, override_runlist, events) } @@ -40,11 +40,10 @@ describe Chef::PolicyBuilder::Policyfile do major = sha1_id[0...14] minor = sha1_id[14...28] patch = sha1_id[28..40] - decimal_integers =[major, minor, patch].map {|hex| hex.to_i(16) } + decimal_integers = [major, minor, patch].map { |hex| hex.to_i(16) } decimal_integers.join(".") end - let(:example1_lock_data) do # based on https://github.com/danielsdeleo/chef-workflow2-prototype/blob/master/skeletons/basic_policy/Policyfile.lock.json { @@ -53,10 +52,10 @@ describe Chef::PolicyBuilder::Policyfile do # NOTE: for compatibility mode we include the dotted id in the policyfile to enhance discoverability. "dotted_decimal_identifier" => id_to_dotted("168d2102fb11c9617cd8a981166c8adc30a6e915"), "source" => { "path" => "./cookbooks/demo" }, - "scm_identifier"=> { - "vcs"=> "git", - "rev_id"=> "9d5b09026470c322c3cb5ca8a4157c4d2f16cef3", - "remote"=> nil, + "scm_identifier" => { + "vcs" => "git", + "rev_id" => "9d5b09026470c322c3cb5ca8a4157c4d2f16cef3", + "remote" => nil, }, } end @@ -71,8 +70,8 @@ describe Chef::PolicyBuilder::Policyfile do } end - let(:policyfile_default_attributes) { {"policyfile_default_attr" => "policyfile_default_value"} } - let(:policyfile_override_attributes) { {"policyfile_override_attr" => "policyfile_override_value"} } + let(:policyfile_default_attributes) { { "policyfile_default_attr" => "policyfile_default_value" } } + let(:policyfile_override_attributes) { { "policyfile_override_attr" => "policyfile_override_value" } } let(:policyfile_run_list) { ["recipe[example1::default]", "recipe[example2::server]"] } @@ -132,7 +131,7 @@ describe Chef::PolicyBuilder::Policyfile do end context "when json_attribs contains a run_list" do - let(:json_attribs) { {"run_list" => []} } + let(:json_attribs) { { "run_list" => [] } } it "errors on create" do expect { initialize_pb }.to raise_error(err_namespace::UnsupportedFeature) @@ -158,8 +157,8 @@ describe Chef::PolicyBuilder::Policyfile do let(:override_runlist) { nil } let(:primary_runlist) { nil } - let(:original_default_attrs) { {"default_key" => "default_value"} } - let(:original_override_attrs) { {"override_key" => "override_value"} } + let(:original_default_attrs) { { "default_key" => "default_value" } } + let(:original_override_attrs) { { "override_key" => "override_value" } } let(:node) do node = Chef::Node.new @@ -268,7 +267,6 @@ describe Chef::PolicyBuilder::Policyfile do end - describe "building policy from the policyfile" do before do @@ -304,7 +302,6 @@ describe Chef::PolicyBuilder::Policyfile do expect(policy_builder.expand_run_list.roles).to eq([]) end - describe "validating the Policyfile.lock" do it "errors if the policyfile json contains any non-recipe items" do @@ -447,7 +444,6 @@ describe Chef::PolicyBuilder::Policyfile do node end - it "prefers the policy_name and policy_group from the node json" do expect(policy_builder.policy_name).to eq("policy_name_from_node_json") expect(policy_builder.policy_group).to eq("policy_group_from_node_json") @@ -592,7 +588,6 @@ describe Chef::PolicyBuilder::Policyfile do end end - describe "fetching the desired cookbook set" do let(:example1_cookbook_data) { double("CookbookVersion Hash for example1 cookbook") } @@ -716,7 +711,6 @@ describe Chef::PolicyBuilder::Policyfile do include_examples "fetching cookbooks when they don't exist" end - context "when the cookbooks exist on the server" do before do @@ -735,7 +729,6 @@ describe Chef::PolicyBuilder::Policyfile do end - end end |