diff options
author | Bryan McLellan <btm@loftninjas.org> | 2018-03-16 15:41:50 -0400 |
---|---|---|
committer | Bryan McLellan <btm@loftninjas.org> | 2018-03-19 13:52:37 -0400 |
commit | aa26322ef48b1b3a77ff4d6b7fe15693d5ac9800 (patch) | |
tree | 4b0c37f3e41c0275718efe7b4867b2479921001e | |
parent | 4f54c7b443f55a0c2dcfaf4d193aaa037a35009d (diff) | |
download | chef-aa26322ef48b1b3a77ff4d6b7fe15693d5ac9800.tar.gz |
Fix regression in #6980, add functional tests
The logging refactor in #6980 should have been a class variable. This fixes that.
Also adds functional tests that would catch that and the original #6980 bug.
Signed-off-by: Bryan McLellan <btm@loftninjas.org>
-rw-r--r-- | lib/chef/win32/security.rb | 2 | ||||
-rw-r--r-- | spec/functional/win32/security_spec.rb | 35 |
2 files changed, 36 insertions, 1 deletions
diff --git a/lib/chef/win32/security.rb b/lib/chef/win32/security.rb index a58fa48399..feefdfc19a 100644 --- a/lib/chef/win32/security.rb +++ b/lib/chef/win32/security.rb @@ -646,7 +646,7 @@ class Chef Token.new(Handle.new(token.read_pointer)) end - def test_and_raise_lsa_nt_status(result) + def self.test_and_raise_lsa_nt_status(result) win32_error = LsaNtStatusToWinError(result) if win32_error != 0 Chef::ReservedNames::Win32::Error.raise!(nil, win32_error) diff --git a/spec/functional/win32/security_spec.rb b/spec/functional/win32/security_spec.rb index 40ae99bfa4..bd30bf805f 100644 --- a/spec/functional/win32/security_spec.rb +++ b/spec/functional/win32/security_spec.rb @@ -97,4 +97,39 @@ describe "Chef::Win32::Security", :windows_only do end end end + + describe ".get_account_right" do + let(:username) { ENV["USERNAME"] } + + context "when given a valid username" do + it "returns an array of account right constants" do + Chef::ReservedNames::Win32::Security.add_account_right(username, "SeBatchLogonRight") + expect(Chef::ReservedNames::Win32::Security.get_account_right(username)).to include("SeBatchLogonRight") + end + + it "passes an FFI::Pointer to LsaFreeMemory" do + Chef::ReservedNames::Win32::Security.add_account_right(username, "SeBatchLogonRight") # otherwise we return an empty array before LsaFreeMemory + expect(Chef::ReservedNames::Win32::Security).to receive(:LsaFreeMemory).with(instance_of(FFI::Pointer)).and_return(0) # not FFI::MemoryPointer + Chef::ReservedNames::Win32::Security.get_account_right(username) + end + end + + context "when given an invalid username" do + let(:username) { "noooooooooope" } + + it "raises an exception" do + expect { Chef::ReservedNames::Win32::Security.get_account_right(username) }.to raise_error(Chef::Exceptions::Win32APIError) + end + end + end + + describe ".test_and_raise_lsa_nt_status" do + # NTSTATUS code: 0xC0000001 / STATUS_UNSUCCESSFUL + # Windows Error: ERROR_GEN_FAILURE / 31 / 0x1F / A device attached to the system is not functioning. + let(:status_unsuccessful) { 0xC0000001 } + + it "raises an exception with the Win Error if the win32 result is not 0" do + expect { Chef::ReservedNames::Win32::Security.test_and_raise_lsa_nt_status(status_unsuccessful) }.to raise_error(Chef::Exceptions::Win32APIError) + end + end end |