From e601babb9266258f55580fcf2a91ea5ca4c5d368 Mon Sep 17 00:00:00 2001 From: Luke Kanies Date: Mon, 15 Dec 2008 11:03:21 -0600 Subject: Supporting a nil expirer on cacher objects. This works for those classes like Puppet::Type that use an expirer that isn't always present (they use their catalog as an expirer). The behaviour is now that if there is no expirer, expire() is a no-op and all data is considered expired all the time, so it's always fresh. Signed-off-by: Luke Kanies --- spec/unit/util/cacher.rb | 15 +++++++++++++++ 1 file changed, 15 insertions(+) (limited to 'spec') diff --git a/spec/unit/util/cacher.rb b/spec/unit/util/cacher.rb index 3e8d31a24..5a867c6c1 100755 --- a/spec/unit/util/cacher.rb +++ b/spec/unit/util/cacher.rb @@ -79,6 +79,21 @@ describe Puppet::Util::Cacher do @object.instance_cache.should_not equal(value) end + it "should be able to trigger expiration on its expirer" do + @expirer.expects(:expire) + @object.expire + end + + it "should do nothing when asked to expire when no expirer is available" do + cacher = CacheTest.new + class << cacher + def expirer + nil + end + end + lambda { cacher.expire }.should_not raise_error + end + it "should be able to cache false values" do @object.expects(:init_instance_cache).returns false @object.instance_cache.should be_false -- cgit