chef 16.11.7 → 17.2.29

data/spec/unit/knife/client_list_spec.rb

@@ -1,34 +0,0 @@
-#
-# Author:: Thomas Bishop (<bishop.thomas@gmail.com>)
-# Copyright:: Copyright 2011-2016, Thomas Bishop
-# 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::Knife::ClientList do
-  before(:each) do
-    @knife = Chef::Knife::ClientList.new
-    @knife.name_args = [ "adam" ]
-  end
-
-  describe "run" do
-    it "should list the clients" do
-      expect(Chef::ApiClientV1).to receive(:list)
-      expect(@knife).to receive(:format_list_for_display)
-      @knife.run
-    end
-  end
-end

data/spec/unit/knife/client_reregister_spec.rb

@@ -1,62 +0,0 @@
-#
-# Author:: Thomas Bishop (<bishop.thomas@gmail.com>)
-# Copyright:: Copyright 2011-2016, Thomas Bishop
-# 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::Knife::ClientReregister do
-  before(:each) do
-    @knife = Chef::Knife::ClientReregister.new
-    @knife.name_args = [ "adam" ]
-    @client_mock = double("client_mock", private_key: "foo_key")
-    @stdout = StringIO.new
-    allow(@knife.ui).to receive(:stdout).and_return(@stdout)
-  end
-
-  context "when no client name is given on the command line" do
-    before do
-      @knife.name_args = []
-    end
-
-    it "should print usage and exit when a client name is not provided" do
-      expect(@knife).to receive(:show_usage)
-      expect(@knife.ui).to receive(:fatal)
-      expect { @knife.run }.to raise_error(SystemExit)
-    end
-  end
-
-  context "when not configured for file output" do
-    it "reregisters the client and prints the key" do
-      expect(Chef::ApiClientV1).to receive(:reregister).with("adam").and_return(@client_mock)
-      @knife.run
-      expect(@stdout.string).to match( /foo_key/ )
-    end
-  end
-
-  context "when configured for file output" do
-    it "should write the private key to a file" do
-      expect(Chef::ApiClientV1).to receive(:reregister).with("adam").and_return(@client_mock)
-
-      @knife.config[:file] = "/tmp/monkeypants"
-      filehandle = StringIO.new
-      expect(File).to receive(:open).with("/tmp/monkeypants", "w").and_yield(filehandle)
-      @knife.run
-      expect(filehandle.string).to eq("foo_key")
-    end
-  end
-
-end

data/spec/unit/knife/client_show_spec.rb

@@ -1,52 +0,0 @@
-#
-# Author:: Thomas Bishop (<bishop.thomas@gmail.com>)
-# Copyright:: Copyright 2011-2016, Thomas Bishop
-# 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::Knife::ClientShow do
-  before(:each) do
-    @knife = Chef::Knife::ClientShow.new
-    @knife.name_args = [ "adam" ]
-    @client_mock = double("client_mock")
-  end
-
-  describe "run" do
-    it "should list the client" do
-      expect(Chef::ApiClientV1).to receive(:load).with("adam").and_return(@client_mock)
-      expect(@knife).to receive(:format_for_display).with(@client_mock)
-      @knife.run
-    end
-
-    it "should pretty print json" do
-      @knife.config[:format] = "json"
-      @stdout = StringIO.new
-      allow(@knife.ui).to receive(:stdout).and_return(@stdout)
-      fake_client_contents = { "foo" => "bar", "baz" => "qux" }
-      expect(Chef::ApiClientV1).to receive(:load).with("adam").and_return(fake_client_contents)
-      @knife.run
-      expect(@stdout.string).to eql("{\n  \"foo\": \"bar\",\n  \"baz\": \"qux\"\n}\n")
-    end
-
-    it "should print usage and exit when a client name is not provided" do
-      @knife.name_args = []
-      expect(@knife).to receive(:show_usage)
-      expect(@knife.ui).to receive(:fatal)
-      expect { @knife.run }.to raise_error(SystemExit)
-    end
-  end
-end

data/spec/unit/knife/configure_client_spec.rb

@@ -1,81 +0,0 @@
-#
-# Author:: Thomas Bishop (<bishop.thomas@gmail.com>)
-# Copyright:: Copyright 2011-2016, Thomas Bishop
-# 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::Knife::ConfigureClient do
-  before do
-    @knife = Chef::Knife::ConfigureClient.new
-    Chef::Config[:chef_server_url] = "https://chef.example.com"
-    Chef::Config[:validation_client_name] = "chef-validator"
-    Chef::Config[:validation_key] = "/etc/chef/validation.pem"
-
-    @stderr = StringIO.new
-    allow(@knife.ui).to receive(:stderr).and_return(@stderr)
-  end
-
-  describe "run" do
-    it "should print usage and exit when a directory is not provided" do
-      expect(@knife).to receive(:show_usage)
-      expect(@knife.ui).to receive(:fatal).with(/must provide the directory/)
-      expect do
-        @knife.run
-      end.to raise_error SystemExit
-    end
-
-    describe "when specifing a directory" do
-      before do
-        @knife.name_args = ["/home/bob/.chef"]
-        @client_file = StringIO.new
-        @validation_file = StringIO.new
-        expect(File).to receive(:open).with("/home/bob/.chef/client.rb", "w")
-          .and_yield(@client_file)
-        expect(File).to receive(:open).with("/home/bob/.chef/validation.pem", "w")
-          .and_yield(@validation_file)
-        expect(IO).to receive(:read).and_return("foo_bar_baz")
-      end
-
-      it "should recursively create the directory" do
-        expect(FileUtils).to receive(:mkdir_p).with("/home/bob/.chef")
-        @knife.run
-      end
-
-      it "should write out the config file" do
-        allow(FileUtils).to receive(:mkdir_p)
-        @knife.run
-        expect(@client_file.string).to match %r{chef_server_url\s+'https\://chef\.example\.com'}
-        expect(@client_file.string).to match(/validation_client_name\s+'chef-validator'/)
-      end
-
-      it "should write out the validation.pem file" do
-        allow(FileUtils).to receive(:mkdir_p)
-        @knife.run
-        expect(@validation_file.string).to match(/foo_bar_baz/)
-      end
-
-      it "should print information on what is being configured" do
-        allow(FileUtils).to receive(:mkdir_p)
-        @knife.run
-        expect(@stderr.string).to match(/creating client configuration/i)
-        expect(@stderr.string).to match(/writing client\.rb/i)
-        expect(@stderr.string).to match(/writing validation\.pem/i)
-      end
-    end
-  end
-
-end

data/spec/unit/knife/configure_spec.rb

@@ -1,190 +0,0 @@
-require "spec_helper"
-
-describe Chef::Knife::Configure do
-  before do
-    Chef::Log.logger = Logger.new(StringIO.new)
-
-    Chef::Config[:node_name] = "webmonkey.example.com"
-    @knife = Chef::Knife::Configure.new
-    @rest_client = double("null rest client", post: { result: :true })
-    allow(@knife).to receive(:rest).and_return(@rest_client)
-
-    @out = StringIO.new
-    allow(@knife.ui).to receive(:stdout).and_return(@out)
-    @knife.config[:config_file] = "/home/you/.chef/knife.rb"
-
-    @in = StringIO.new("\n" * 7)
-    allow(@knife.ui).to receive(:stdin).and_return(@in)
-
-    @err = StringIO.new
-    allow(@knife.ui).to receive(:stderr).and_return(@err)
-
-    allow(Ohai::System).to receive(:new).and_return(ohai)
-  end
-
-  let(:fqdn) { "foo.example.org" }
-
-  let(:ohai) do
-    o = {}
-    allow(o).to receive(:all_plugins).with(%w{ os hostname fqdn })
-    o[:fqdn] = fqdn
-    o
-  end
-
-  let(:default_admin_key) { "/etc/chef-server/admin.pem" }
-  let(:default_admin_key_win32) { File.expand_path(default_admin_key) }
-
-  let(:default_validator_key) { "/etc/chef-server/chef-validator.pem" }
-  let(:default_validator_key_win32) { File.expand_path(default_validator_key) }
-
-  let(:default_server_url) { "https://#{fqdn}/organizations/myorg" }
-
-  it "asks the user for the URL of the chef server" do
-    @knife.ask_user_for_config
-    expect(@out.string).to match(Regexp.escape("Please enter the chef server URL: [#{default_server_url}]"))
-    expect(@knife.chef_server).to eq(default_server_url)
-  end
-
-  it "asks the user for the clientname they want for the new client if -i is specified" do
-    @knife.config[:initial] = true
-    allow(Etc).to receive(:getlogin).and_return("a-new-user")
-    @knife.ask_user_for_config
-    expect(@out.string).to match(Regexp.escape("Please enter a name for the new user: [a-new-user]"))
-    expect(@knife.new_client_name).to eq(Etc.getlogin)
-  end
-
-  it "should not ask the user for the clientname they want for the new client if -i and --node_name are specified" do
-    @knife.config[:initial] = true
-    @knife.config[:node_name] = "testnode"
-    allow(Etc).to receive(:getlogin).and_return("a-new-user")
-    @knife.ask_user_for_config
-    expect(@out.string).not_to match(Regexp.escape("Please enter a name for the new user"))
-    expect(@knife.new_client_name).to eq("testnode")
-  end
-
-  it "asks the user for the existing API username or clientname if -i is not specified" do
-    allow(Etc).to receive(:getlogin).and_return("a-new-user")
-    @knife.ask_user_for_config
-    expect(@out.string).to match(Regexp.escape("Please enter an existing username or clientname for the API: [a-new-user]"))
-    expect(@knife.new_client_name).to eq(Etc.getlogin)
-  end
-
-  it "asks the user for the existing admin client's name if -i is specified" do
-    @knife.config[:initial] = true
-    @knife.ask_user_for_config
-    expect(@out.string).to match(Regexp.escape("Please enter the existing admin name: [admin]"))
-    expect(@knife.admin_client_name).to eq("admin")
-  end
-
-  it "should not ask the user for the existing admin client's name if -i and --admin-client_name are specified" do
-    @knife.config[:initial] = true
-    @knife.config[:admin_client_name] = "my-webui"
-    @knife.ask_user_for_config
-    expect(@out.string).not_to match(Regexp.escape("Please enter the existing admin:"))
-    expect(@knife.admin_client_name).to eq("my-webui")
-  end
-
-  it "should not ask the user for the existing admin client's name if -i is not specified" do
-    @knife.ask_user_for_config
-    expect(@out.string).not_to match(Regexp.escape("Please enter the existing admin: [admin]"))
-    expect(@knife.admin_client_name).not_to eq("admin")
-  end
-
-  it "asks the user for the location of the existing admin key if -i is specified" do
-    @knife.config[:initial] = true
-    @knife.ask_user_for_config
-    expect(@out.string).to match(Regexp.escape("Please enter the location of the existing admin's private key: [#{default_admin_key}]"))
-    if windows?
-      expect(@knife.admin_client_key.capitalize).to eq(default_admin_key_win32.capitalize)
-    else
-      expect(@knife.admin_client_key).to eq(default_admin_key)
-    end
-  end
-
-  it "should not ask the user for the location of the existing admin key if -i and --admin_client_key are specified" do
-    @knife.config[:initial] = true
-    @knife.config[:admin_client_key] = "/home/you/.chef/my-webui.pem"
-    @knife.ask_user_for_config
-    expect(@out.string).not_to match(Regexp.escape("Please enter the location of the existing admin client's private key:"))
-    if windows?
-      expect(@knife.admin_client_key).to match %r{^[A-Za-z]:/home/you/\.chef/my-webui\.pem$}
-    else
-      expect(@knife.admin_client_key).to eq("/home/you/.chef/my-webui.pem")
-    end
-  end
-
-  it "should not ask the user for the location of the existing admin key if -i is not specified" do
-    @knife.ask_user_for_config
-    expect(@out.string).not_to match(Regexp.escape("Please enter the location of the existing admin client's private key: [#{default_admin_key}]"))
-    if windows?
-      expect(@knife.admin_client_key).not_to eq(default_admin_key_win32)
-    else
-      expect(@knife.admin_client_key).not_to eq(default_admin_key)
-    end
-  end
-
-  it "should not ask the user for anything if -i and all other properties are specified" do
-    @knife.config[:initial] = true
-    @knife.config[:chef_server_url] = "http://localhost:5000"
-    @knife.config[:node_name] = "testnode"
-    @knife.config[:admin_client_name] = "my-webui"
-    @knife.config[:admin_client_key] = "/home/you/.chef/my-webui.pem"
-    @knife.config[:client_key] = "/home/you/a-new-user.pem"
-    allow(Etc).to receive(:getlogin).and_return("a-new-user")
-
-    @knife.ask_user_for_config
-    expect(@out.string).to match(/\s*/)
-
-    expect(@knife.new_client_name).to eq("testnode")
-    expect(@knife.chef_server).to eq("http://localhost:5000")
-    expect(@knife.admin_client_name).to eq("my-webui")
-    if windows?
-      expect(@knife.admin_client_key).to match %r{^[A-Za-z]:/home/you/\.chef/my-webui\.pem$}
-      expect(@knife.new_client_key).to match %r{^[A-Za-z]:/home/you/a-new-user\.pem$}
-    else
-      expect(@knife.admin_client_key).to eq("/home/you/.chef/my-webui.pem")
-      expect(@knife.new_client_key).to eq("/home/you/a-new-user.pem")
-    end
-  end
-
-  it "writes the new data to a config file" do
-    allow(Chef::Util::PathHelper).to receive(:home).with(".chef").and_return("/home/you/.chef")
-    allow(File).to receive(:expand_path).with("/home/you/.chef/credentials").and_return("/home/you/.chef/credentials")
-    allow(File).to receive(:expand_path).with("/home/you/.chef/#{Etc.getlogin}.pem").and_return("/home/you/.chef/#{Etc.getlogin}.pem")
-    allow(File).to receive(:expand_path).with(default_admin_key).and_return(default_admin_key)
-    expect(FileUtils).to receive(:mkdir_p).with("/home/you/.chef")
-    config_file = StringIO.new
-    expect(::File).to receive(:open).with("/home/you/.chef/credentials", "w").and_yield config_file
-    @knife.config[:repository] = "/home/you/chef-repo"
-    @knife.run
-    expect(config_file.string).to match(/^client_name\s+=\s+'#{Etc.getlogin}'$/)
-    expect(config_file.string).to match(%r{^client_key\s+=\s+'/home/you/.chef/#{Etc.getlogin}.pem'$})
-    expect(config_file.string).to match(/^chef_server_url\s+=\s+'#{default_server_url}'$/)
-  end
-
-  it "creates a new client when given the --initial option" do
-    allow(Chef::Util::PathHelper).to receive(:home).with(".chef").and_return("/home/you/.chef")
-    expect(File).to receive(:expand_path).with("/home/you/.chef/credentials").and_return("/home/you/.chef/credentials")
-    expect(File).to receive(:expand_path).with("/home/you/.chef/a-new-user.pem").and_return("/home/you/.chef/a-new-user.pem")
-    allow(File).to receive(:expand_path).with(default_admin_key).and_return(default_admin_key)
-    Chef::Config[:node_name] = "webmonkey.example.com"
-
-    user_command = Chef::Knife::UserCreate.new
-    expect(user_command).to receive(:run)
-
-    allow(Etc).to receive(:getlogin).and_return("a-new-user")
-
-    allow(Chef::Knife::UserCreate).to receive(:new).and_return(user_command)
-    expect(FileUtils).to receive(:mkdir_p).with("/home/you/.chef")
-    expect(::File).to receive(:open).with("/home/you/.chef/credentials", "w")
-    @knife.config[:initial] = true
-    @knife.config[:user_password] = "blah"
-    @knife.run
-    expect(user_command.name_args).to eq(Array("a-new-user"))
-    expect(user_command.config[:user_password]).to eq("blah")
-    expect(user_command.config[:admin]).to be_truthy
-    expect(user_command.config[:file]).to eq("/home/you/.chef/a-new-user.pem")
-    expect(user_command.config[:yes]).to be_truthy
-    expect(user_command.config[:disable_editing]).to be_truthy
-  end
-end

data/spec/unit/knife/cookbook_bulk_delete_spec.rb

@@ -1,87 +0,0 @@
-#
-# Author:: Stephen Delano (<stephen@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::Knife::CookbookBulkDelete do
-  before(:each) do
-    Chef::Log.logger = Logger.new(StringIO.new)
-
-    Chef::Config[:node_name] = "webmonkey.example.com"
-    @knife = Chef::Knife::CookbookBulkDelete.new
-    @knife.config = { print_after: nil }
-    @knife.name_args = ["."]
-    @stdout = StringIO.new
-    @stderr = StringIO.new
-    allow(@knife.ui).to receive(:stdout).and_return(@stdout)
-    allow(@knife.ui).to receive(:stderr).and_return(@stderr)
-    allow(@knife.ui).to receive(:confirm).and_return(true)
-    @cookbooks = {}
-    %w{cheezburger pizza lasagna}.each do |cookbook_name|
-      cookbook = Chef::CookbookVersion.new(cookbook_name)
-      @cookbooks[cookbook_name] = cookbook
-    end
-    @rest = double("Chef::ServerAPI")
-    allow(@rest).to receive(:get).and_return(@cookbooks)
-    allow(@rest).to receive(:delete).and_return(true)
-    allow(@knife).to receive(:rest).and_return(@rest)
-    allow(Chef::CookbookVersion).to receive(:list).and_return(@cookbooks)
-
-  end
-
-  describe "when there are several cookbooks on the server" do
-    before do
-      @cheezburger = { "cheezburger" => { "url" => "file:///dev/null", "versions" => [{ "url" => "file:///dev/null-cheez", "version" => "1.0.0" }] } }
-      allow(@rest).to receive(:get).with("cookbooks/cheezburger").and_return(@cheezburger)
-      @pizza = { "pizza" => { "url" => "file:///dev/null", "versions" => [{ "url" => "file:///dev/null-pizza", "version" => "2.0.0" }] } }
-      allow(@rest).to receive(:get).with("cookbooks/pizza").and_return(@pizza)
-      @lasagna = { "lasagna" => { "url" => "file:///dev/null", "versions" => [{ "url" => "file:///dev/null-lasagna", "version" => "3.0.0" }] } }
-      allow(@rest).to receive(:get).with("cookbooks/lasagna").and_return(@lasagna)
-    end
-
-    it "should print the cookbooks you are about to delete" do
-      expected = @knife.ui.list(@cookbooks.keys.sort, :columns_down)
-      @knife.run
-      expect(@stdout.string).to match(/#{expected}/)
-    end
-
-    it "should confirm you really want to delete them" do
-      expect(@knife.ui).to receive(:confirm)
-      @knife.run
-    end
-
-    it "should delete each cookbook" do
-      { "cheezburger" => "1.0.0", "pizza" => "2.0.0", "lasagna" => "3.0.0" }.each do |cookbook_name, version|
-        expect(@rest).to receive(:delete).with("cookbooks/#{cookbook_name}/#{version}")
-      end
-      @knife.run
-    end
-
-    it "should only delete cookbooks that match the regex" do
-      @knife.name_args = ["cheezburger"]
-      expect(@rest).to receive(:delete).with("cookbooks/cheezburger/1.0.0")
-      @knife.run
-    end
-  end
-
-  it "should exit if the regex is not provided" do
-    @knife.name_args = []
-    expect { @knife.run }.to raise_error(SystemExit)
-  end
-
-end