chef-dk 0.6.2 → 0.7.0

data/spec/unit/command/verify_spec.rb

@@ -40,6 +40,7 @@ describe ChefDK::Command::Verify do
       "test-kitchen",
       "chef-client",
       "chef-dk",
+      "chef-provisioning",
       "chefspec",
       "rubocop",
       "fauxhai",

data/spec/unit/helpers_spec.rb

@@ -0,0 +1,68 @@
+#
+# Copyright:: Copyright (c) 2014 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'
+require 'chef-dk/helpers'
+
+describe ChefDK::Helpers do
+
+  let (:helpers) do
+    ChefDK::Helpers.send(:reset!)
+    ChefDK::Helpers
+  end
+
+  let(:env) { {} }
+
+  describe "chefdk_home" do
+    before do
+      allow(ENV).to receive(:[]) do |k|
+        env[k]
+      end
+      allow(Chef::Platform).to receive(:windows?).and_return(false)
+    end
+
+    context 'when CHEFDK_HOME is set' do
+      let(:env) { {'CHEFDK_HOME' => 'foo' } }
+      it "returns CHEFDK_HOME" do
+        expect(helpers.chefdk_home).to eq(env['CHEFDK_HOME'])
+      end
+    end
+
+    context 'when CHEFDK_HOME is not set' do
+      context 'on windows' do
+        before do
+          allow(Chef::Platform).to receive(:windows?).and_return(true)
+        end
+
+        let(:env) { { 'LOCALAPPDATA' => 'C:\\foo' } }
+
+        it 'uses LOCALAPPDATA' do
+          expect(File).to receive(:join).with(env['LOCALAPPDATA'], 'chefdk').and_return('chefdkdefaulthome')
+          expect(helpers.chefdk_home).to eq('chefdkdefaulthome')
+        end
+      end
+
+      context 'on *nix' do
+        it 'uses LOCALAPPDATA' do
+          expect(File).to receive(:expand_path).with('~/.chefdk').and_return('chefdkdefaulthome')
+          expect(helpers.chefdk_home).to eq('chefdkdefaulthome')
+        end
+      end
+    end
+
+  end
+end

data/spec/unit/policyfile/cookbook_locks_spec.rb

@@ -209,9 +209,11 @@ describe ChefDK::Policyfile::LocalCookbook do
 
   let(:storage_config) { ChefDK::Policyfile::StorageConfig.new }
 
+  let(:scm_profiler) { instance_double("ChefDK::CookbookProfiler::Git", profile_data: {}) }
+
   let(:cookbook_lock) do
     lock = described_class.new(cookbook_name, storage_config)
-    allow(lock).to receive(:scm_info).and_return({})
+    allow(lock).to receive(:scm_profiler).and_return(scm_profiler)
     lock
   end
 
@@ -254,6 +256,11 @@ describe ChefDK::Policyfile::LocalCookbook do
       path
     end
 
+    # everywhere else, #scm_profiler is stubbed, we need the unstubbed version
+    let(:cookbook_lock) do
+      described_class.new(cookbook_name, storage_config)
+    end
+
     before do
       cookbook_lock.source = cookbook_source_path
     end
@@ -342,6 +349,11 @@ describe ChefDK::Policyfile::LocalCookbook do
       expect(cookbook_lock.source_options).to eq(expected)
     end
 
+    it "doesn't refresh scm_data when #lock_data is called" do
+      allow(scm_profiler).to receive(:profile_data).and_raise("This shouldn't get called")
+      cookbook_lock.lock_data
+    end
+
     context "after the data has been refreshed" do
 
       before do
@@ -421,3 +433,97 @@ describe ChefDK::Policyfile::LocalCookbook do
   end
 
 end
+
+describe ChefDK::Policyfile::ArchivedCookbook do
+
+  let(:cookbook_name) { "nginx" }
+
+  let(:storage_config) { ChefDK::Policyfile::StorageConfig.new }
+
+  let(:wrapped_cookbook_lock_data) do
+    {
+      "identifier" => "abc123",
+      "dotted_decimal_identifier" => "111.222.333",
+      "version" => "1.2.3",
+      "source" => "../my_repo/nginx",
+      "source_options" => {
+        # when getting the cookbook location spec, source options needs to have
+        # symbolic keys, so a round trip via LocalCookbook#build_from_lock_data
+        # will result in this being a symbol
+        :path => "../my_repo/nginx"
+      },
+      "cache_key" => nil,
+      "scm_info" => {}
+    }
+  end
+
+  let(:wrapped_cookbook_lock) do
+    lock = ChefDK::Policyfile::LocalCookbook.new(cookbook_name, storage_config)
+    allow(lock).to receive(:scm_info).and_return({})
+    lock.build_from_lock_data(wrapped_cookbook_lock_data)
+    lock
+  end
+
+  let(:cookbook_lock) do
+    described_class.new(wrapped_cookbook_lock, storage_config)
+  end
+
+  let(:archived_cookbook_path) { File.join(storage_config.relative_paths_root, "cookbooks", "nginx-111.222.333") }
+
+  it "sets cookbook_path to the path within the archive" do
+    expect(cookbook_lock.cookbook_path).to eq(archived_cookbook_path)
+  end
+
+  it "implements build_from_lock_data" do
+    msg = "ArchivedCookbook cannot be built from lock data, it can only wrap an existing lock object"
+    expect { cookbook_lock.build_from_lock_data({}) }.to raise_error(NotImplementedError, msg)
+  end
+
+  it "implements validate!" do
+    expect(cookbook_lock.validate!).to be(true)
+  end
+
+  it "implements refresh!" do
+    expect(cookbook_lock.refresh!).to be(true)
+  end
+
+  it "implements installed?" do
+    allow(File).to receive(:exist?).with(archived_cookbook_path).and_return(false)
+    allow(File).to receive(:directory?).with(archived_cookbook_path).and_return(false)
+    expect(cookbook_lock.installed?).to be(false)
+    allow(File).to receive(:exist?).with(archived_cookbook_path).and_return(true)
+    allow(File).to receive(:directory?).with(archived_cookbook_path).and_return(true)
+    expect(cookbook_lock.installed?).to be(true)
+  end
+
+  describe "delegated behavior" do
+
+    it "sets the identifier" do
+      expect(cookbook_lock.identifier).to eq("abc123")
+    end
+
+    it "sets the dotted_decimal_identifier" do
+      expect(cookbook_lock.dotted_decimal_identifier).to eq("111.222.333")
+    end
+
+    it "sets the version" do
+      expect(cookbook_lock.version).to eq("1.2.3")
+    end
+
+    it "sets the source attribute" do
+      expect(cookbook_lock.source).to eq("../my_repo/nginx")
+    end
+
+    it "sets the source options, symbolizing keys so the data is compatible with CookbookLocationSpecification" do
+      expected = { path: "../my_repo/nginx" }
+      expect(cookbook_lock.source_options).to eq(expected)
+    end
+
+    it "returns unchanged data when calling to_lock" do
+      expect(cookbook_lock.to_lock).to eq(wrapped_cookbook_lock_data)
+    end
+
+
+  end
+
+end

data/spec/unit/policyfile/lister_spec.rb

@@ -0,0 +1,256 @@
+#
+# Copyright:: Copyright (c) 2015 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'
+require 'chef-dk/policyfile/lister'
+
+describe ChefDK::Policyfile::Lister do
+
+  def api_url(org_specific_path)
+    "https://chef.example/organizations/myorg/#{org_specific_path}"
+  end
+
+
+  let(:config) do
+    double("Chef::Config",
+           chef_server_url: "https://localhost:10443",
+           client_key: "/path/to/client/key.pem",
+           node_name: "deuce")
+  end
+
+  let(:http_client) { instance_double(ChefDK::AuthenticatedHTTP) }
+
+  subject(:info_fetcher) do
+    described_class.new(config: config)
+  end
+
+  it "configures an HTTP client" do
+    expect(ChefDK::AuthenticatedHTTP).to receive(:new).with("https://localhost:10443",
+                                                       signing_key_filename: "/path/to/client/key.pem",
+                                                       client_name: "deuce")
+    info_fetcher.http_client
+  end
+
+
+  context "when the data is fetched successfully from the server" do
+
+    before do
+      allow(info_fetcher).to receive(:http_client).and_return(http_client)
+
+      allow(http_client).to receive(:get).with("policy_groups").and_return(policy_group_list_data)
+      allow(http_client).to receive(:get).with("policies").and_return(policy_list_data)
+    end
+
+    context "when the server has no policies or groups" do
+
+      let(:policy_group_list_data) { {} }
+      let(:policy_list_data) { {} }
+
+      it "gives a Hash of policy revisions by policy name" do
+        expect(info_fetcher.policies_by_name).to eq({})
+      end
+
+      it "gives a Hash of policy revisions by policy group" do
+        expect(info_fetcher.policies_by_group).to eq({})
+      end
+
+      it "is empty" do
+        expect(info_fetcher).to be_empty
+      end
+
+      it "has no active revisions" do
+        expect(info_fetcher.active_revisions).to be_empty
+      end
+    end
+
+    context "when the server has policies and groups" do
+      ##
+      # Example API response data copied from oc-chef-pedant:
+
+      let(:policy_list_data) do
+        {
+          "appserver" => {
+            "uri" => api_url("policies/appserver"),
+            "revisions" => {
+              "1111111111111111111111111111111111111111" => {},
+              "2222222222222222222222222222222222222222" => {},
+              "3333333333333333333333333333333333333333" => {},
+              "4444444444444444444444444444444444444444" => {}
+            }
+          },
+          "db" => {
+            "uri" => api_url("policies/db"),
+            "revisions" => {
+              "6666666666666666666666666666666666666666" => {},
+              "7777777777777777777777777777777777777777" => {},
+              "8888888888888888888888888888888888888888" => {},
+              "9999999999999999999999999999999999999999" => {}
+            }
+          },
+          "cache" => {
+            "uri" => api_url("policies/cache"),
+            "revisions" => {
+              "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa" => {},
+              "bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb" => {}
+            }
+          }
+        }
+      end
+
+      let(:dev_group_data) do
+        {
+          "uri" => api_url("policy_groups/dev"),
+          "policies" => {
+            "db" => { "revision_id" => "6666666666666666666666666666666666666666" },
+            "appserver" => { "revision_id" => "1111111111111111111111111111111111111111" },
+            "cache" => { "revision_id" => "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa" }
+          }
+        }
+      end
+
+      let(:test_group_data) do
+        {
+          "uri" => api_url("policy_groups/test"),
+          "policies" => {
+            "db" => { "revision_id" => "7777777777777777777777777777777777777777" },
+            "appserver" => { "revision_id" => "2222222222222222222222222222222222222222" },
+            "cache" => { "revision_id" => "bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb" }
+          }
+        }
+      end
+
+      let(:prod_group_data) do
+        {
+          "uri" => api_url("policy_groups/prod"),
+          "policies" => {
+            "db" => { "revision_id" => "8888888888888888888888888888888888888888" },
+            "appserver" => { "revision_id" => "3333333333333333333333333333333333333333" }
+          }
+        }
+      end
+
+      let(:policy_group_list_data) do
+        {
+          "dev" => dev_group_data,
+          "test" => test_group_data,
+          "prod" => prod_group_data
+        }
+      end
+
+      let(:expected_policy_list) do
+        {
+          "appserver" => {
+            "1111111111111111111111111111111111111111" => {},
+            "2222222222222222222222222222222222222222" => {},
+            "3333333333333333333333333333333333333333" => {},
+            "4444444444444444444444444444444444444444" => {}
+          },
+          "db" => {
+            "6666666666666666666666666666666666666666" => {},
+            "7777777777777777777777777777777777777777" => {},
+            "8888888888888888888888888888888888888888" => {},
+            "9999999999999999999999999999999999999999" => {}
+          },
+          "cache" => {
+            "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa" => {},
+            "bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb" => {}
+          }
+        }
+      end
+
+      let(:expected_policy_group_list) do
+        {
+          "dev" => {
+            "db" => "6666666666666666666666666666666666666666",
+            "appserver" => "1111111111111111111111111111111111111111",
+            "cache" => "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa"
+          },
+          "test" => {
+            "db" => "7777777777777777777777777777777777777777",
+            "appserver" => "2222222222222222222222222222222222222222",
+            "cache" => "bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb"
+          },
+          "prod" => {
+            "db" =>"8888888888888888888888888888888888888888",
+            "appserver" => "3333333333333333333333333333333333333333"
+          }
+        }
+      end
+
+
+      it "gives a Hash of policy revisions by policy name" do
+        expect(info_fetcher.policies_by_name).to eq(expected_policy_list)
+      end
+
+      it "gives a Hash of policy revisions by policy group" do
+        expect(info_fetcher.policies_by_group).to eq(expected_policy_group_list)
+      end
+
+      it "is not empty" do
+        expect(info_fetcher).to_not be_empty
+      end
+
+      it "lists active revisions" do
+        expected_active_revisions = expected_policy_group_list.values.map(&:values).flatten
+
+        expected_active_revisions_set = Set.new(expected_active_revisions)
+        expect(info_fetcher.active_revisions).to eq(expected_active_revisions_set)
+      end
+
+      it "lists orphaned revisions for a given policy" do
+        expect(info_fetcher.orphaned_revisions("db")).to eq(%w[ 9999999999999999999999999999999999999999 ])
+        expect(info_fetcher.orphaned_revisions("appserver")).to eq(%w[ 4444444444444444444444444444444444444444 ])
+        expect(info_fetcher.orphaned_revisions("cache")).to eq([])
+      end
+
+      it "yields revision ids by group" do
+        map = {}
+
+        info_fetcher.revision_ids_by_group_for_each_policy do |policy_name, rev_id_by_group|
+          map[policy_name] = rev_id_by_group
+        end
+
+        appserver_rev_ids = {
+          "dev" => "1111111111111111111111111111111111111111",
+          "test" => "2222222222222222222222222222222222222222",
+          "prod" => "3333333333333333333333333333333333333333"
+        }
+
+        expect(map["appserver"].revision_ids_by_group).to eq(appserver_rev_ids)
+
+        db_rev_ids = {
+          "dev" => "6666666666666666666666666666666666666666",
+          "test" => "7777777777777777777777777777777777777777",
+          "prod" => "8888888888888888888888888888888888888888"
+        }
+
+        expect(map["db"].revision_ids_by_group).to eq(db_rev_ids)
+
+        cache_rev_ids = {
+          "dev" => "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa",
+          "test" => "bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb",
+          "prod" => nil
+        }
+        expect(map["cache"].revision_ids_by_group).to eq(cache_rev_ids)
+      end
+
+    end
+
+  end
+
+end
+