diff options
author | Bapu L <bapu.labade@progress.com> | 2022-06-21 15:24:34 +0530 |
---|---|---|
committer | Bapu L <bapu.labade@progress.com> | 2022-06-21 15:24:34 +0530 |
commit | c9d1dd2c79fcce09531815d365861ec4e6aab5e9 (patch) | |
tree | 8e7a3d59a599b23c3bdfc1776e7606322638cefe | |
parent | e8086d2733311a17d37698bc6e9d6bdaa2b96068 (diff) | |
download | chef-c9d1dd2c79fcce09531815d365861ec4e6aab5e9.tar.gz |
Doc string message changes, test case cleanup
-rw-r--r-- | lib/chef/mixin/properties.rb | 2 | ||||
-rw-r--r-- | spec/unit/resource_spec.rb | 9 |
2 files changed, 9 insertions, 2 deletions
diff --git a/lib/chef/mixin/properties.rb b/lib/chef/mixin/properties.rb index 89cca2abac..4e00a09002 100644 --- a/lib/chef/mixin/properties.rb +++ b/lib/chef/mixin/properties.rb @@ -275,7 +275,7 @@ class Chef end # This method returns list of sensitive properties - # @return [Array<Property>] All identity properties. + # @return [Array<Property>] All sensitive properties. def sensitive_properties properties.values.empty? ? [] : properties.values.select(&:sensitive?) end diff --git a/spec/unit/resource_spec.rb b/spec/unit/resource_spec.rb index 6634390391..ec0026fba9 100644 --- a/spec/unit/resource_spec.rb +++ b/spec/unit/resource_spec.rb @@ -385,7 +385,6 @@ describe Chef::Resource do resource.foo = "some value" expect(resource.to_text).to match(/foo "\*sensitive value suppressed\*"/) end - it "suppresses that properties value for irrespective of desired state (false) " do resource_class = Class.new(Chef::Resource) { property :foo, String, sensitive: true, desired_state: false @@ -404,6 +403,14 @@ describe Chef::Resource do expect(resource.to_text).to match(/foo "\*sensitive value suppressed\*"/) end end + + context "when property is required" do + it "does not propagate validation errors" do + resource_class = Class.new(Chef::Resource) { property :foo, String, required: true } + resource = resource_class.new("required_property_tests") + expect { resource.to_text }.to_not raise_error + end + end end context "Documentation of resources" do |