mirror of
https://github.com/inspec/inspec
synced 2025-02-17 06:28:40 +00:00
Fix for default null values compatible for both windows and non-windows system
Signed-off-by: Nikita Mathur <nikita.mathur@chef.io>
This commit is contained in:
parent
457e544724
commit
7693363a1a
2 changed files with 6 additions and 2 deletions
|
@ -118,7 +118,7 @@ module Inspec::Resources
|
||||||
end
|
end
|
||||||
|
|
||||||
def members
|
def members
|
||||||
flatten_entry(group_info, "members") || []
|
flatten_entry(group_info, "members") || empty_value_for_members
|
||||||
end
|
end
|
||||||
|
|
||||||
def members_array
|
def members_array
|
||||||
|
@ -152,6 +152,10 @@ module Inspec::Resources
|
||||||
group = @group.dup
|
group = @group.dup
|
||||||
@groups_cache ||= inspec.groups.where { name == group }
|
@groups_cache ||= inspec.groups.where { name == group }
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def empty_value_for_members
|
||||||
|
inspec.os.windows? ? [] : ""
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
class GroupInfo
|
class GroupInfo
|
||||||
|
|
|
@ -108,6 +108,6 @@ describe "Inspec::Resources::Group" do
|
||||||
it "verify non-existent group member on centos" do
|
it "verify non-existent group member on centos" do
|
||||||
resource = MockLoader.new(:centos7).load_resource("group", "root")
|
resource = MockLoader.new(:centos7).load_resource("group", "root")
|
||||||
_(resource.exists?).must_equal true
|
_(resource.exists?).must_equal true
|
||||||
_(resource.members).must_equal []
|
_(resource.members).must_equal ""
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
Loading…
Add table
Reference in a new issue