chef-dk 0.6.2 → 0.7.0

data/spec/unit/command/export_spec.rb

@@ -59,6 +59,19 @@ describe ChefDK::Command::Export do
       end
     end
 
+    context "when archive mode is set" do
+
+      let(:params) { [ "path/to/export", "-a" ] }
+
+      it "enables archiving the exported repo" do
+        expect(command.archive?).to be(true)
+      end
+
+      it "configures the export service to archive" do
+        expect(command.export_service.archive?).to be(true)
+      end
+    end
+
     context "when the path to the exported repo is given" do
 
       let(:params) { [ "path/to/export" ] }

data/spec/unit/command/provision_spec.rb

@@ -195,6 +195,62 @@ describe ChefDK::Command::Provision do
 
         end
 
+        context "with --target" do
+
+          let(:extra_params) { %w[ -t 192.168.255.123 ] }
+
+          it "sets the target host to the given value" do
+            expect(context.target).to eq("192.168.255.123")
+          end
+
+        end
+
+        context "with --opt" do
+          context "with one user-specified option" do
+            let(:extra_params) { %w[ --opt color=ebfg ] }
+
+            it "sets the given option name to the given value" do
+              expect(context.opts.color).to eq("ebfg")
+            end
+          end
+
+          context "with an option given as a quoted arg with spaces" do
+
+            let(:extra_params) { [ '--opt', 'color = ebfg' ] }
+
+            it "sets the given option name to the given value" do
+              expect(context.opts.color).to eq("ebfg")
+            end
+          end
+
+          context "with an option with an '=' in it" do
+
+            let(:extra_params) { [ '--opt', 'api_key=abcdef==' ] }
+
+            it "sets the given option name to the given value" do
+              expect(context.opts.api_key).to eq("abcdef==")
+            end
+          end
+
+          context "with an option with a space in it" do
+
+            let(:extra_params) { [ '--opt', 'full_name=Bobo T. Clown' ] }
+
+            it "sets the given option name to the given value" do
+              expect(context.opts.full_name).to eq("Bobo T. Clown")
+            end
+          end
+
+          context "with multiple options given" do
+            let(:extra_params) { %w[ --opt color=ebfg --opt nope=seppb ] }
+
+            it "sets the given option name to the given value" do
+              expect(context.opts.color).to eq("ebfg")
+              expect(context.opts.nope).to eq("seppb")
+            end
+          end
+        end
+
         context "with -d" do
 
           let(:extra_params) { %w[ -d ] }

data/spec/unit/command/push_archive_spec.rb

@@ -0,0 +1,153 @@
+#
+# 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 'shared/command_with_ui_object'
+require 'chef-dk/command/push_archive'
+
+describe ChefDK::Command::PushArchive do
+
+  subject(:command) { described_class.new }
+
+  let(:policy_group) { "staging" }
+
+  let(:archive_path) { "mypolicy-abc123.tgz" }
+
+  it_behaves_like "a command with a UI object"
+
+  let(:ui) { TestHelpers::TestUI.new }
+
+  let(:chef_config_loader) { instance_double("Chef::WorkstationConfigLoader") }
+
+  let(:config_arg) { nil }
+
+  before do
+    allow(Chef::WorkstationConfigLoader).to receive(:new).with(config_arg).and_return(chef_config_loader)
+  end
+
+  describe "evaluating params" do
+
+    before do
+      command.ui = ui
+    end
+
+    context "when given no arguments" do
+
+      it "prints the banner message and exits non-zero" do
+        expect(command.run([])).to eq(1)
+        expect(ui.output).to include(described_class.banner)
+      end
+
+    end
+
+    context "when the archive path is omitted" do
+
+      it "prints the banner message and exits non-zero" do
+        expect(command.run([policy_group])).to eq(1)
+        expect(ui.output).to include(described_class.banner)
+      end
+
+    end
+
+    context "when all required arguments are given" do
+
+      let(:params) { [ policy_group, archive_path ] }
+
+      before do
+        command.apply_params!(params)
+      end
+
+      it "disables debug by default" do
+        expect(command.debug?).to be(false)
+      end
+
+      context "and debug mode is set" do
+
+        let(:params) { [ policy_group, archive_path, "-D" ] }
+
+        it "enables debug messages" do
+          expect(command.debug?).to be(true)
+        end
+
+      end
+
+      describe "configuring settings that depend on the chef config file" do
+
+        before do
+          expect(chef_config_loader).to receive(:load)
+        end
+
+        it "sets the policy group" do
+          expect(command.policy_group).to eq(policy_group)
+          expect(command.push_archive_service.policy_group).to eq(policy_group)
+        end
+
+        it "sets the path to the archive file" do
+          expect(command.archive_path).to eq(archive_path)
+          expect(command.push_archive_service.archive_file).to eq(File.expand_path(archive_path))
+        end
+      end
+
+    end
+
+  end
+
+  describe "running the push operation" do
+
+    let(:push_archive_service) { instance_double("ChefDK::PolicyfileServices::PushArchive") }
+
+    let(:params) { [ policy_group, archive_path ] }
+
+    before do
+      allow(command).to receive(:push_archive_service).and_return(push_archive_service)
+      command.ui = ui
+    end
+
+    context "when the push is successful" do
+
+      it "exits 0" do
+        expect(push_archive_service).to receive(:run)
+        expect(command.run(params)).to eq(0)
+      end
+
+    end
+
+    context "when the push in not successful" do
+
+      let(:backtrace) { caller[0...3] }
+
+      let(:cause) do
+        e = StandardError.new("some operation failed")
+        e.set_backtrace(backtrace)
+        e
+      end
+
+      let(:exception) do
+        ChefDK::PolicyfilePushArchiveError.new("push failed", cause)
+      end
+
+      before do
+        expect(push_archive_service).to receive(:run).and_raise(exception)
+      end
+
+      it "exits non-zero" do
+        expect(command.run(params)).to eq(1)
+      end
+
+    end
+  end
+end

data/spec/unit/command/show_policy_spec.rb

@@ -0,0 +1,235 @@
+#
+# 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 'shared/command_with_ui_object'
+require 'chef-dk/command/show_policy'
+
+describe ChefDK::Command::ShowPolicy do
+
+  it_behaves_like "a command with a UI object"
+
+  subject(:command) do
+    described_class.new
+  end
+
+  let(:show_policy_service) { command.show_policy_service }
+
+  let(:chef_config_loader) { instance_double("Chef::WorkstationConfigLoader") }
+
+  let(:chef_config) { double("Chef::Config") }
+
+  # nil means the config loader will do the default path lookup
+  let(:config_arg) { nil }
+
+  before do
+    stub_const("Chef::Config", chef_config)
+    allow(Chef::WorkstationConfigLoader).to receive(:new).with(config_arg).and_return(chef_config_loader)
+  end
+
+  describe "parsing args and options" do
+    let(:params) { [] }
+
+    before do
+      command.apply_params!(params)
+    end
+
+    context "when given a path to the config" do
+
+      let(:params) { %w[ -c ~/otherstuff/config.rb ] }
+
+      let(:config_arg) { "~/otherstuff/config.rb" }
+
+      before do
+        expect(chef_config_loader).to receive(:load)
+      end
+
+      it "reads the chef/knife config" do
+        expect(Chef::WorkstationConfigLoader).to receive(:new).with(config_arg).and_return(chef_config_loader)
+        expect(command.chef_config).to eq(chef_config)
+        expect(show_policy_service.chef_config).to eq(chef_config)
+      end
+
+    end
+
+    describe "settings that require loading chef config" do
+
+      before do
+        allow(chef_config_loader).to receive(:load)
+      end
+
+      context "with no params" do
+
+        it "disables debug by default" do
+          expect(command.debug?).to be(false)
+        end
+
+        it "is configured to show all policies across all groups" do
+          expect(show_policy_service.show_all_policies?).to be(true)
+        end
+
+        it "disables displaying orphans" do
+          expect(command.show_orphans?).to be(false)
+          expect(show_policy_service.show_orphans?).to be(false)
+        end
+
+      end
+
+      context "when debug mode is set" do
+
+        let(:params) { [ "-D" ] }
+
+        it "enables debug" do
+          expect(command.debug?).to be(true)
+        end
+
+      end
+
+      context "when --show-orphans is given" do
+
+        let(:params) { %w[ -o ] }
+
+        it "enables displaying orphans" do
+          expect(command.show_orphans?).to be(true)
+          expect(show_policy_service.show_orphans?).to be(true)
+        end
+
+      end
+
+      context "when given a policy name" do
+
+        let(:params) { %w[ appserver ] }
+
+        it "is not configured to show all policies" do
+          expect(show_policy_service.show_all_policies?).to be(false)
+        end
+
+        it "is configured to show the given policy" do
+          expect(command.policy_name).to eq("appserver")
+          expect(show_policy_service.policy_name).to eq("appserver")
+        end
+
+      end
+
+      context "when given a policy name and a policy group name" do
+
+        let(:params) { %w[ appserver production ] }
+
+        it "is not configured to show all policies" do
+          expect(show_policy_service.show_all_policies?).to be(false)
+        end
+
+        it "is configured to show the given policy" do
+          expect(command.policy_name).to eq("appserver")
+          expect(show_policy_service.policy_name).to eq("appserver")
+        end
+
+        it "is configured to display the exact revision for the given policy+group" do
+          expect(command.policy_group).to eq("production")
+          expect(show_policy_service.policy_group).to eq("production")
+        end
+
+      end
+
+    end
+  end
+
+  describe "running the command" do
+
+    let(:ui) { TestHelpers::TestUI.new }
+
+    before do
+      allow(chef_config_loader).to receive(:load)
+      command.ui = ui
+    end
+
+    context "when given too many arguments" do
+
+      let(:params) { %w[ appserver policygroup wut-is-this ] }
+
+      it "shows usage and exits" do
+        expect(command.run(params)).to eq(1)
+      end
+
+    end
+
+    context "when the list service raises an exception" do
+
+      let(:backtrace) { caller[0...3] }
+
+      let(:cause) do
+        e = StandardError.new("some operation failed")
+        e.set_backtrace(backtrace)
+        e
+      end
+
+      let(:exception) do
+        ChefDK::PolicyfileListError.new("Failed to list policies", cause)
+      end
+
+      before do
+        allow(command.show_policy_service).to receive(:run).and_raise(exception)
+      end
+
+      it "prints a debugging message and exits non-zero" do
+        expect(command.run([])).to eq(1)
+
+        expected_output=<<-E
+Error: Failed to list policies
+Reason: (StandardError) some operation failed
+
+E
+
+        expect(ui.output).to eq(expected_output)
+      end
+
+      context "when debug is enabled" do
+
+        it "includes the backtrace in the error" do
+
+          command.run(%w[ -D ])
+
+          expected_output=<<-E
+Error: Failed to list policies
+Reason: (StandardError) some operation failed
+
+
+E
+          expected_output << backtrace.join("\n") << "\n"
+
+          expect(ui.output).to eq(expected_output)
+        end
+
+      end
+
+    end
+
+    context "when the list service executes successfully" do
+
+      before do
+        expect(command.show_policy_service).to receive(:run)
+      end
+
+      it "exits 0" do
+        expect(command.run([])).to eq(0)
+      end
+
+    end
+
+  end
+end
+