knife 18.2.7 → 18.3.0

data/spec/unit/knife/user_password_spec.rb

@@ -1,64 +1,64 @@
-#
-# Author:: Snehal Dwivedi (<sdwivedi@msystechnologies.com>)
-# Copyright:: Copyright 2014-2016 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 "knife_spec_helper"
-require "chef/org"
-
-Chef::Knife::UserDelete.load_deps
-
-describe Chef::Knife::UserPassword do
-
-  let(:root_rest) { double("Chef::ServerAPI") }
-
-  before :each do
-    @knife = Chef::Knife::UserPassword.new
-    @user_name = "foobar"
-    @password = "abc123"
-    @user = double("Chef::User")
-    allow(@user).to receive(:root_rest).and_return(root_rest)
-    @key = "You don't come into cooking to get rich - Ramsay"
-  end
-
-  describe "should change user's password" do
-    before :each do
-      @knife.name_args << @user_name << @password
-    end
-
-    it "with username and password" do
-      result = { "password" => [], "recovery_authentication_enabled" => true }
-      allow(@user).to receive(:[]).with("organization")
-
-      expect(Chef::ServerAPI).to receive(:new).with(Chef::Config[:chef_server_root]).and_return(root_rest)
-      expect(@user.root_rest).to receive(:get).with("users/#{@user_name}").and_return(result)
-      expect(@user.root_rest).to receive(:put).with("users/#{@user_name}", result)
-      expect(@knife.ui).to receive(:msg).with("Authentication info updated for #{@user_name}.")
-
-      @knife.run
-    end
-  end
-
-  describe "should not change user's password" do
-
-    it "ails with an informative message" do
-      expect(@knife).to receive(:show_usage)
-      expect(@knife.ui).to receive(:fatal).with("You must pass two arguments")
-      expect(@knife.ui).to receive(:fatal).with("Note that --enable-external-auth cannot be passed with a password")
-      expect { @knife.run }.to raise_error(SystemExit)
-    end
-  end
-end
+#
+# Author:: Snehal Dwivedi (<sdwivedi@msystechnologies.com>)
+# Copyright:: Copyright 2014-2016 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 "knife_spec_helper"
+require "chef/org"
+
+Chef::Knife::UserDelete.load_deps
+
+describe Chef::Knife::UserPassword do
+
+  let(:root_rest) { double("Chef::ServerAPI") }
+
+  before :each do
+    @knife = Chef::Knife::UserPassword.new
+    @user_name = "foobar"
+    @password = "abc123"
+    @user = double("Chef::User")
+    allow(@user).to receive(:root_rest).and_return(root_rest)
+    @key = "You don't come into cooking to get rich - Ramsay"
+  end
+
+  describe "should change user's password" do
+    before :each do
+      @knife.name_args << @user_name << @password
+    end
+
+    it "with username and password" do
+      result = { "password" => [], "recovery_authentication_enabled" => true }
+      allow(@user).to receive(:[]).with("organization")
+
+      expect(Chef::ServerAPI).to receive(:new).with(Chef::Config[:chef_server_root]).and_return(root_rest)
+      expect(@user.root_rest).to receive(:get).with("users/#{@user_name}").and_return(result)
+      expect(@user.root_rest).to receive(:put).with("users/#{@user_name}", result)
+      expect(@knife.ui).to receive(:msg).with("Authentication info updated for #{@user_name}.")
+
+      @knife.run
+    end
+  end
+
+  describe "should not change user's password" do
+
+    it "ails with an informative message" do
+      expect(@knife).to receive(:show_usage)
+      expect(@knife.ui).to receive(:fatal).with("You must pass two arguments")
+      expect(@knife.ui).to receive(:fatal).with("Note that --enable-external-auth cannot be passed with a password")
+      expect { @knife.run }.to raise_error(SystemExit)
+    end
+  end
+end

data/spec/unit/knife/user_reregister_spec.rb

@@ -1,56 +1,56 @@
-#
-# Author:: Steven Danna (<steve@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 "knife_spec_helper"
-
-describe Chef::Knife::UserReregister do
-  let(:knife) { Chef::Knife::UserReregister.new }
-  let(:user_mock) { double("user_mock", private_key: "private_key") }
-  let(:stdout) { StringIO.new }
-
-  before do
-    Chef::Knife::UserReregister.load_deps
-    knife.name_args = [ "a_user" ]
-    allow(Chef::UserV1).to receive(:load).and_return(user_mock)
-    allow(knife.ui).to receive(:stdout).and_return(stdout)
-    allow(knife.ui).to receive(:stderr).and_return(stdout)
-    allow(user_mock).to receive(:username).and_return("a_user")
-  end
-
-  it "prints usage and exits when a user 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
-
-  it "reregisters the user and prints the key" do
-    expect(user_mock).to receive(:reregister).and_return(user_mock)
-    knife.run
-    expect(stdout.string).to match( /private_key/ )
-  end
-
-  it "writes the private key to a file when --file is specified" do
-    expect(user_mock).to receive(:reregister).and_return(user_mock)
-    knife.config[:file] = "/tmp/a_file"
-    filehandle = StringIO.new
-    expect(File).to receive(:open).with("/tmp/a_file", "w").and_yield(filehandle)
-    knife.run
-    expect(filehandle.string).to eq("private_key")
-  end
-end
+#
+# Author:: Steven Danna (<steve@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 "knife_spec_helper"
+
+describe Chef::Knife::UserReregister do
+  let(:knife) { Chef::Knife::UserReregister.new }
+  let(:user_mock) { double("user_mock", private_key: "private_key") }
+  let(:stdout) { StringIO.new }
+
+  before do
+    Chef::Knife::UserReregister.load_deps
+    knife.name_args = [ "a_user" ]
+    allow(Chef::UserV1).to receive(:load).and_return(user_mock)
+    allow(knife.ui).to receive(:stdout).and_return(stdout)
+    allow(knife.ui).to receive(:stderr).and_return(stdout)
+    allow(user_mock).to receive(:username).and_return("a_user")
+  end
+
+  it "prints usage and exits when a user 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
+
+  it "reregisters the user and prints the key" do
+    expect(user_mock).to receive(:reregister).and_return(user_mock)
+    knife.run
+    expect(stdout.string).to match( /private_key/ )
+  end
+
+  it "writes the private key to a file when --file is specified" do
+    expect(user_mock).to receive(:reregister).and_return(user_mock)
+    knife.config[:file] = "/tmp/a_file"
+    filehandle = StringIO.new
+    expect(File).to receive(:open).with("/tmp/a_file", "w").and_yield(filehandle)
+    knife.run
+    expect(filehandle.string).to eq("private_key")
+  end
+end

data/spec/unit/knife/user_show_spec.rb

@@ -1,91 +1,91 @@
-#
-# Author:: Steven Danna (<steve@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 "knife_spec_helper"
-require "chef/org"
-
-Chef::Knife::UserShow.load_deps
-
-describe Chef::Knife::UserShow do
-  let(:knife) { Chef::Knife::UserShow.new }
-  let(:user_mock) { double("user_mock") }
-  let(:root_rest) { double("Chef::ServerAPI") }
-
-  before :each do
-    @user_name = "foobar"
-    @password = "abc123"
-    @user = double("Chef::User")
-    allow(@user).to receive(:root_rest).and_return(root_rest)
-    # allow(Chef::User).to receive(:new).and_return(@user)
-    @key = "You don't come into cooking to get rich - Ramsay"
-  end
-
-  describe "withot organisation argument" do
-    before do
-      knife.name_args = [ "my_user" ]
-      allow(user_mock).to receive(:username).and_return("my_user")
-    end
-
-    it "should load the user" do
-      expect(Chef::ServerAPI).to receive(:new).with(Chef::Config[:chef_server_root]).and_return(root_rest)
-      expect(@user.root_rest).to receive(:get).with("users/my_user")
-      knife.run
-    end
-
-    it "loads and displays the user" do
-      expect(Chef::ServerAPI).to receive(:new).with(Chef::Config[:chef_server_root]).and_return(root_rest)
-      expect(@user.root_rest).to receive(:get).with("users/my_user")
-      expect(knife).to receive(:format_for_display)
-      knife.run
-    end
-
-    it "prints usage and exits when a user 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
-
-  describe "with organisation argument" do
-    before :each do
-      @user_name = "foobar"
-      @org_name = "abc_org"
-      knife.name_args << @user_name << @org_name
-      @org = double("Chef::Org")
-      allow(Chef::Org).to receive(:new).and_return(@org)
-      @key = "You don't come into cooking to get rich - Ramsay"
-    end
-
-    let(:orgs) do
-      [@org]
-    end
-
-    it "should load the user with organisation" do
-
-      result = { "organizations" => [] }
-      knife.config[:with_orgs] = true
-
-      expect(Chef::ServerAPI).to receive(:new).with(Chef::Config[:chef_server_root]).and_return(root_rest)
-      allow(@org).to receive(:[]).with("organization").and_return({ "name" => "test" })
-      expect(@user.root_rest).to receive(:get).with("users/#{@user_name}").and_return(result)
-      expect(@user.root_rest).to receive(:get).with("users/#{@user_name}/organizations").and_return(orgs)
-      knife.run
-    end
-  end
-end
+#
+# Author:: Steven Danna (<steve@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 "knife_spec_helper"
+require "chef/org"
+
+Chef::Knife::UserShow.load_deps
+
+describe Chef::Knife::UserShow do
+  let(:knife) { Chef::Knife::UserShow.new }
+  let(:user_mock) { double("user_mock") }
+  let(:root_rest) { double("Chef::ServerAPI") }
+
+  before :each do
+    @user_name = "foobar"
+    @password = "abc123"
+    @user = double("Chef::User")
+    allow(@user).to receive(:root_rest).and_return(root_rest)
+    # allow(Chef::User).to receive(:new).and_return(@user)
+    @key = "You don't come into cooking to get rich - Ramsay"
+  end
+
+  describe "withot organisation argument" do
+    before do
+      knife.name_args = [ "my_user" ]
+      allow(user_mock).to receive(:username).and_return("my_user")
+    end
+
+    it "should load the user" do
+      expect(Chef::ServerAPI).to receive(:new).with(Chef::Config[:chef_server_root]).and_return(root_rest)
+      expect(@user.root_rest).to receive(:get).with("users/my_user")
+      knife.run
+    end
+
+    it "loads and displays the user" do
+      expect(Chef::ServerAPI).to receive(:new).with(Chef::Config[:chef_server_root]).and_return(root_rest)
+      expect(@user.root_rest).to receive(:get).with("users/my_user")
+      expect(knife).to receive(:format_for_display)
+      knife.run
+    end
+
+    it "prints usage and exits when a user 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
+
+  describe "with organisation argument" do
+    before :each do
+      @user_name = "foobar"
+      @org_name = "abc_org"
+      knife.name_args << @user_name << @org_name
+      @org = double("Chef::Org")
+      allow(Chef::Org).to receive(:new).and_return(@org)
+      @key = "You don't come into cooking to get rich - Ramsay"
+    end
+
+    let(:orgs) do
+      [@org]
+    end
+
+    it "should load the user with organisation" do
+
+      result = { "organizations" => [] }
+      knife.config[:with_orgs] = true
+
+      expect(Chef::ServerAPI).to receive(:new).with(Chef::Config[:chef_server_root]).and_return(root_rest)
+      allow(@org).to receive(:[]).with("organization").and_return({ "name" => "test" })
+      expect(@user.root_rest).to receive(:get).with("users/#{@user_name}").and_return(result)
+      expect(@user.root_rest).to receive(:get).with("users/#{@user_name}/organizations").and_return(orgs)
+      knife.run
+    end
+  end
+end