mirror of
https://github.com/inspec/inspec
synced 2024-11-23 21:23:29 +00:00
Update resource based on PR feedback
Signed-off-by: Chris Redekop <chris.redekop@d2l.com>
This commit is contained in:
parent
3f22a54aa6
commit
0ab4ae7d6e
2 changed files with 11 additions and 15 deletions
|
@ -18,9 +18,7 @@ class AwsIamAccessKey < Inspec.resource(1)
|
|||
|
||||
def exists?
|
||||
!access_key.nil?
|
||||
rescue AccessKeyNotFoundError
|
||||
false
|
||||
rescue Aws::IAM::Errors::NoSuchEntity
|
||||
rescue AccessKeyNotFoundError, Aws::IAM::Errors::NoSuchEntity
|
||||
false
|
||||
end
|
||||
|
||||
|
|
|
@ -85,31 +85,29 @@ class AwsIamAccessKeyTest < Minitest::Test
|
|||
def test_get_access_key_raises_when_no_access_keys_found
|
||||
validator = mock_validator
|
||||
|
||||
begin
|
||||
e = assert_raises AwsIamAccessKey::AccessKeyNotFoundError do
|
||||
iam_client_decorator(validator).get_access_key(Username, Id)
|
||||
flunk
|
||||
rescue AwsIamAccessKey::AccessKeyNotFoundError => e
|
||||
assert_match(/.*access key not found.*/, e.message)
|
||||
assert_match(/.*#{Username}.*/, e.message)
|
||||
assert_match(/.*#{Id}.*/, e.message)
|
||||
end
|
||||
|
||||
assert_match(/.*access key not found.*/, e.message)
|
||||
assert_match(/.*#{Username}.*/, e.message)
|
||||
assert_match(/.*#{Id}.*/, e.message)
|
||||
|
||||
validator.verify
|
||||
end
|
||||
|
||||
def test_get_access_key_raises_when_matching_access_key_not_found
|
||||
validator = mock_validator
|
||||
|
||||
begin
|
||||
e = assert_raises AwsIamAccessKey::AccessKeyNotFoundError do
|
||||
iam_client_decorator(validator, [stub_access_key(id: 'Foo')])
|
||||
.get_access_key(Username, Id)
|
||||
flunk
|
||||
rescue AwsIamAccessKey::AccessKeyNotFoundError => e
|
||||
assert_match(/.*access key not found.*/, e.message)
|
||||
assert_match(/.*#{Username}.*/, e.message)
|
||||
assert_match(/.*#{Id}.*/, e.message)
|
||||
end
|
||||
|
||||
assert_match(/.*access key not found.*/, e.message)
|
||||
assert_match(/.*#{Username}.*/, e.message)
|
||||
assert_match(/.*#{Id}.*/, e.message)
|
||||
|
||||
validator.verify
|
||||
end
|
||||
|
||||
|
|
Loading…
Reference in a new issue