chef 12.6.0-universal-mingw32 → 12.7.2-universal-mingw32

data/spec/unit/knife/tag_list_spec.rb

@@ -1,4 +1,4 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe Chef::Knife::TagList do
   before(:each) do
@@ -14,7 +14,7 @@ describe Chef::Knife::TagList do
 
   describe "run" do
     it "can list tags on a node" do
-      expected = %w(sadtag happytag)
+      expected = %w{sadtag happytag}
       expect(@node.tags).to eq(expected)
       expect(@knife).to receive(:output).with(expected)
       @knife.run

data/spec/unit/knife/user_create_spec.rb

@@ -1,7 +1,7 @@
 #
 # Author:: Steven Danna (<steve@chef.io>)
 # Author:: Tyler Cloke (<tyler@chef.io>)
-# Copyright:: Copyright (c) 2012, 2015 Chef Software, Inc.
+# Copyright:: Copyright 2012-2016, Chef Software, Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -17,7 +17,7 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
+require "spec_helper"
 
 Chef::Knife::UserCreate.load_deps
 
@@ -41,18 +41,18 @@ describe Chef::Knife::UserCreate do
   # delete this once OSC11 support is gone
   context "when only one name_arg is passed" do
     before do
-      knife.name_args = ['some_user']
+      knife.name_args = ["some_user"]
       allow(knife).to receive(:run_osc_11_user_create).and_raise(SystemExit)
     end
 
     it "displays the osc warning" do
       expect(knife.ui).to receive(:warn).with(knife.osc_11_warning)
-      expect{ knife.run }.to raise_error(SystemExit)
+      expect { knife.run }.to raise_error(SystemExit)
     end
 
     it "calls knife osc_user create" do
       expect(knife).to receive(:run_osc_11_user_create)
-      expect{ knife.run }.to raise_error(SystemExit)
+      expect { knife.run }.to raise_error(SystemExit)
     end
 
   end
@@ -61,7 +61,7 @@ describe Chef::Knife::UserCreate do
     # from spec/support/shared/unit/knife_shared.rb
     it_should_behave_like "mandatory field missing" do
       let(:name_args) { [] }
-      let(:fieldname) { 'username' }
+      let(:fieldname) { "username" }
     end
   end
 
@@ -79,38 +79,38 @@ describe Chef::Knife::UserCreate do
   context "when FIRST_NAME isn't specified" do
     # from spec/support/shared/unit/knife_shared.rb
     it_should_behave_like "mandatory field missing" do
-      let(:name_args) { ['some_user', 'some_display_name'] }
-      let(:fieldname) { 'first name' }
+      let(:name_args) { ["some_user", "some_display_name"] }
+      let(:fieldname) { "first name" }
     end
   end
 
   context "when LAST_NAME isn't specified" do
     # from spec/support/shared/unit/knife_shared.rb
     it_should_behave_like "mandatory field missing" do
-      let(:name_args) { ['some_user', 'some_display_name', 'some_first_name'] }
-      let(:fieldname) { 'last name' }
+      let(:name_args) { ["some_user", "some_display_name", "some_first_name"] }
+      let(:fieldname) { "last name" }
     end
   end
 
   context "when EMAIL isn't specified" do
     # from spec/support/shared/unit/knife_shared.rb
     it_should_behave_like "mandatory field missing" do
-      let(:name_args) { ['some_user', 'some_display_name', 'some_first_name', 'some_last_name'] }
-      let(:fieldname) { 'email' }
+      let(:name_args) { ["some_user", "some_display_name", "some_first_name", "some_last_name"] }
+      let(:fieldname) { "email" }
     end
   end
 
   context "when PASSWORD isn't specified" do
     # from spec/support/shared/unit/knife_shared.rb
     it_should_behave_like "mandatory field missing" do
-      let(:name_args) { ['some_user', 'some_display_name', 'some_first_name', 'some_last_name', 'some_email'] }
-      let(:fieldname) { 'password' }
+      let(:name_args) { ["some_user", "some_display_name", "some_first_name", "some_last_name", "some_email"] }
+      let(:fieldname) { "password" }
     end
   end
 
   context "when all mandatory fields are validly specified" do
     before do
-      knife.name_args = ['some_user', 'some_display_name', 'some_first_name', 'some_last_name', 'some_email', 'some_password']
+      knife.name_args = ["some_user", "some_display_name", "some_first_name", "some_last_name", "some_email", "some_password"]
       allow(knife).to receive(:edit_data).and_return(knife.user.to_hash)
       allow(knife).to receive(:create_user_from_hash).and_return(knife.user)
     end
@@ -122,12 +122,12 @@ describe Chef::Knife::UserCreate do
 
     it "sets all the mandatory fields" do
       knife.run
-      expect(knife.user.username).to eq('some_user')
-      expect(knife.user.display_name).to eq('some_display_name')
-      expect(knife.user.first_name).to eq('some_first_name')
-      expect(knife.user.last_name).to eq('some_last_name')
-      expect(knife.user.email).to eq('some_email')
-      expect(knife.user.password).to eq('some_password')
+      expect(knife.user.username).to eq("some_user")
+      expect(knife.user.display_name).to eq("some_display_name")
+      expect(knife.user.first_name).to eq("some_first_name")
+      expect(knife.user.last_name).to eq("some_last_name")
+      expect(knife.user.email).to eq("some_email")
+      expect(knife.user.password).to eq("some_password")
     end
 
     context "when user_key and prevent_keygen are passed" do
@@ -166,14 +166,14 @@ describe Chef::Knife::UserCreate do
 
     context "when --user-key is passed" do
       before do
-        knife.config[:user_key] = 'some_key'
-        allow(File).to receive(:read).and_return('some_key')
+        knife.config[:user_key] = "some_key"
+        allow(File).to receive(:read).and_return("some_key")
         allow(File).to receive(:expand_path)
       end
 
       it "sets user.public_key" do
         knife.run
-        expect(knife.user.public_key).to eq('some_key')
+        expect(knife.user.public_key).to eq("some_key")
       end
     end
 
@@ -186,25 +186,25 @@ describe Chef::Knife::UserCreate do
 
     context "when a private_key is returned" do
       before do
-        allow(knife).to receive(:create_user_from_hash).and_return(Chef::UserV1.from_hash(knife.user.to_hash.merge({"private_key" => "some_private_key"})))
+        allow(knife).to receive(:create_user_from_hash).and_return(Chef::UserV1.from_hash(knife.user.to_hash.merge({ "private_key" => "some_private_key" })))
       end
 
       context "when --file is passed" do
         before do
-          knife.config[:file] = '/some/path'
+          knife.config[:file] = "/some/path"
         end
 
         it "creates a new file of the path passed" do
-          filehandle = double('filehandle')
-          expect(filehandle).to receive(:print).with('some_private_key')
-          expect(File).to receive(:open).with('/some/path', 'w').and_yield(filehandle)
+          filehandle = double("filehandle")
+          expect(filehandle).to receive(:print).with("some_private_key")
+          expect(File).to receive(:open).with("/some/path", "w").and_yield(filehandle)
           knife.run
         end
       end
 
       context "when --file is not passed" do
         it "prints the private key to stdout" do
-          expect(knife.ui).to receive(:msg).with('some_private_key')
+          expect(knife.ui).to receive(:msg).with("some_private_key")
           knife.run
         end
       end

data/spec/unit/knife/user_delete_spec.rb

@@ -1,6 +1,6 @@
 #
-# Author:: Steven Danna (<steve@opscode.com>)
-# Copyright:: Copyright (c) 2012 Opscode, Inc
+# Author:: Steven Danna (<steve@chef.io>)
+# Copyright:: Copyright 2012-2016, Chef Software Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -16,18 +16,18 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Chef::Knife::UserDelete do
   let(:knife) { Chef::Knife::UserDelete.new }
-  let(:user) { double('user_object') }
+  let(:user) { double("user_object") }
   let(:stdout) { StringIO.new }
 
   before(:each) do
     Chef::Knife::UserDelete.load_deps
-    knife.name_args = [ 'my_user' ]
+    knife.name_args = [ "my_user" ]
     allow(Chef::UserV1).to receive(:load).and_return(user)
-    allow(user).to receive(:username).and_return('my_user')
+    allow(user).to receive(:username).and_return("my_user")
     allow(knife.ui).to receive(:stderr).and_return(stdout)
     allow(knife.ui).to receive(:stdout).and_return(stdout)
   end
@@ -41,22 +41,22 @@ describe Chef::Knife::UserDelete do
 
     it "displays the osc warning" do
       expect(knife.ui).to receive(:warn).with(knife.osc_11_warning)
-      expect{ knife.run }.to raise_error(SystemExit)
+      expect { knife.run }.to raise_error(SystemExit)
     end
 
     it "forwards the command to knife osc_user edit" do
       expect(knife).to receive(:run_osc_11_user_delete)
-      expect{ knife.run }.to raise_error(SystemExit)
+      expect { knife.run }.to raise_error(SystemExit)
     end
   end
 
-  it 'deletes the user' do
+  it "deletes the user" do
     #expect(knife).to receive(:delete_object).with(Chef::UserV1, 'my_user')
-    expect(knife).to receive(:delete_object).with('my_user')
+    expect(knife).to receive(:delete_object).with("my_user")
     knife.run
   end
 
-  it 'prints usage and exits when a user name is not provided' do
+  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)

data/spec/unit/knife/user_edit_spec.rb

@@ -1,6 +1,6 @@
 #
-# Author:: Steven Danna (<steve@opscode.com>)
-# Copyright:: Copyright (c) 2012 Opscode, Inc
+# Author:: Steven Danna (<steve@chef.io>)
+# Copyright:: Copyright 2012-2016, Chef Software Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -16,7 +16,7 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Chef::Knife::UserEdit do
   let(:knife) { Chef::Knife::UserEdit.new }
@@ -28,7 +28,7 @@ describe Chef::Knife::UserEdit do
     Chef::Knife::UserEdit.load_deps
     allow(knife.ui).to receive(:stderr).and_return(@stderr)
     allow(knife.ui).to receive(:stdout).and_return(@stdout)
-    knife.name_args = [ 'my_user' ]
+    knife.name_args = [ "my_user" ]
     knife.config[:disable_editing] = true
   end
 
@@ -36,28 +36,28 @@ describe Chef::Knife::UserEdit do
   context "when the username field is not supported by the server" do
     before do
       allow(knife).to receive(:run_osc_11_user_edit).and_raise(SystemExit)
-      allow(Chef::UserV1).to receive(:load).and_return({"username" => nil})
+      allow(Chef::UserV1).to receive(:load).and_return({ "username" => nil })
     end
 
     it "displays the osc warning" do
       expect(knife.ui).to receive(:warn).with(knife.osc_11_warning)
-      expect{ knife.run }.to raise_error(SystemExit)
+      expect { knife.run }.to raise_error(SystemExit)
     end
 
     it "forwards the command to knife osc_user edit" do
       expect(knife).to receive(:run_osc_11_user_edit)
-      expect{ knife.run }.to raise_error(SystemExit)
+      expect { knife.run }.to raise_error(SystemExit)
     end
   end
 
-  it 'loads and edits the user' do
+  it "loads and edits the user" do
     data = { "username" => "my_user" }
     allow(Chef::UserV1).to receive(:load).with("my_user").and_return(data)
     expect(knife).to receive(:edit_data).with(data).and_return(data)
     knife.run
   end
 
-  it 'prints usage and exits when a user name is not provided' do
+  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)

data/spec/unit/knife/user_list_spec.rb

@@ -1,6 +1,6 @@
 #
 # Author:: Steven Danna
-# Copyright:: Copyright (c) 2012 Opscode, Inc
+# Copyright:: Copyright 2012-2016, Chef Software Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -16,7 +16,7 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Chef::Knife::UserList do
   let(:knife) { Chef::Knife::UserList.new }
@@ -28,7 +28,7 @@ describe Chef::Knife::UserList do
     allow(knife.ui).to receive(:stdout).and_return(stdout)
   end
 
-  it 'lists the users' do
+  it "lists the users" do
     expect(Chef::UserV1).to receive(:list)
     expect(knife).to receive(:format_list_for_display)
     knife.run

data/spec/unit/knife/user_reregister_spec.rb

@@ -1,6 +1,6 @@
 #
-# Author:: Steven Danna (<steve@opscode.com>)
-# Copyright:: Copyright (c) 2012 Opscode, Inc
+# Author:: Steven Danna (<steve@chef.io>)
+# Copyright:: Copyright 2012-2016, Chef Software Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -16,20 +16,20 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Chef::Knife::UserReregister do
   let(:knife) { Chef::Knife::UserReregister.new }
-  let(:user_mock) { double('user_mock', :private_key => "private_key") }
+  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' ]
+    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')
+    allow(user_mock).to receive(:username).and_return("a_user")
   end
 
   # delete this once OSC11 support is gone
@@ -41,33 +41,33 @@ describe Chef::Knife::UserReregister do
 
     it "displays the osc warning" do
       expect(knife.ui).to receive(:warn).with(knife.osc_11_warning)
-      expect{ knife.run }.to raise_error(SystemExit)
+      expect { knife.run }.to raise_error(SystemExit)
     end
 
     it "forwards the command to knife osc_user edit" do
       expect(knife).to receive(:run_osc_11_user_reregister)
-      expect{ knife.run }.to raise_error(SystemExit)
+      expect { knife.run }.to raise_error(SystemExit)
     end
   end
 
-  it 'prints usage and exits when a user name is not provided' do
+  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
+  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
+  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'
+    knife.config[:file] = "/tmp/a_file"
     filehandle = StringIO.new
-    expect(File).to receive(:open).with('/tmp/a_file', 'w').and_yield(filehandle)
+    expect(File).to receive(:open).with("/tmp/a_file", "w").and_yield(filehandle)
     knife.run
     expect(filehandle.string).to eq("private_key")
   end

data/spec/unit/knife/user_show_spec.rb

@@ -1,6 +1,6 @@
 #
-# Author:: Steven Danna (<steve@opscode.com>)
-# Copyright:: Copyright (c) 2012 Opscode, Inc
+# Author:: Steven Danna (<steve@chef.io>)
+# Copyright:: Copyright 2012-2016, Chef Software Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -16,17 +16,17 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
+require "spec_helper"
 
 describe Chef::Knife::UserShow do
   let(:knife) { Chef::Knife::UserShow.new }
-  let(:user_mock) { double('user_mock') }
+  let(:user_mock) { double("user_mock") }
   let(:stdout) { StringIO.new }
 
   before do
     Chef::Knife::UserShow.load_deps
-    knife.name_args = [ 'my_user' ]
-    allow(user_mock).to receive(:username).and_return('my_user')
+    knife.name_args = [ "my_user" ]
+    allow(user_mock).to receive(:username).and_return("my_user")
     allow(knife.ui).to receive(:stderr).and_return(stdout)
     allow(knife.ui).to receive(:stdout).and_return(stdout)
   end
@@ -35,28 +35,28 @@ describe Chef::Knife::UserShow do
   context "when the username field is not supported by the server" do
     before do
       allow(knife).to receive(:run_osc_11_user_show).and_raise(SystemExit)
-      allow(Chef::UserV1).to receive(:load).with('my_user').and_return(user_mock)
+      allow(Chef::UserV1).to receive(:load).with("my_user").and_return(user_mock)
       allow(user_mock).to receive(:username).and_return(nil)
     end
 
     it "displays the osc warning" do
       expect(knife.ui).to receive(:warn).with(knife.osc_11_warning)
-      expect{ knife.run }.to raise_error(SystemExit)
+      expect { knife.run }.to raise_error(SystemExit)
     end
 
     it "forwards the command to knife osc_user edit" do
       expect(knife).to receive(:run_osc_11_user_show)
-      expect{ knife.run }.to raise_error(SystemExit)
+      expect { knife.run }.to raise_error(SystemExit)
     end
   end
 
-  it 'loads and displays the user' do
-    expect(Chef::UserV1).to receive(:load).with('my_user').and_return(user_mock)
+  it "loads and displays the user" do
+    expect(Chef::UserV1).to receive(:load).with("my_user").and_return(user_mock)
     expect(knife).to receive(:format_for_display).with(user_mock)
     knife.run
   end
 
-  it 'prints usage and exits when a user name is not provided' do
+  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)