Merge pull request #44 from chef/lcg/add-spec-test

add spec test for unusual behavior

1 files changed, 14 insertions, 0 deletions

diff --git a/spec/mixlib/config_spec.rb b/spec/mixlib/config_spec.rb
index 400d367..daaf5b4 100644
--- a/spec/mixlib/config_spec.rb
+++ b/spec/mixlib/config_spec.rb
@@ -796,6 +796,20 @@ describe Mixlib::Config do
       expect(@klass.blah.x).to eql(10)
       expect(@klass.save).to eql({ :blah => { :x => 10 } })
     end
+
+    # this tests existing (somewhat bizzare) behavior of mixlib-config where testing to
+    # see if a key exists is equivalent to testing if the key has been set -- we can still
+    # retrieve the default value if it was set.  the code in chef/chef which merges
+    # knife config values into cli values will be sensitive to this behavior.
+    it "defaults values do not show up when querying with #has_key?" do
+      expect(@klass.blah.has_key?(:x)).to be false
+      expect(@klass.blah.x).to be 5
+    end
+
+    it "if we assign the values, they show up when querying with #has_key?" do
+      @klass.blah.x = 5
+      expect(@klass.blah.has_key?(:x)).to be true
+    end
   end
 
   describe "When a configurable exists with a nested context" do