@@ -115,7 +115,12 @@ class APICache cache.get else APICache.logger.warn "Data not available in the cache or from API for key #{@key}" - raise e + if options.has_key?(:fail) + fail = options[:fail] + fail.respond_to?(:call) ? fail.call : fail + else + raise e + end end end end lib/api_cache.rb @@ -52,4 +52,22 @@ describe "api_cache" do end }.should_not raise_error end + + it "should return a default value rather than raising an exception if :fail passed" do + APICache.get('foo', :fail => "bar") do + raise 'foo' + end.should == 'bar' + end + + it "should accept a proc to fail" do + APICache.get('foo', :fail => lambda { "bar" }) do + raise 'foo' + end.should == 'bar' + end + + it "should accept nil values for :fail" do + APICache.get('foo', :fail => nil) do + raise 'foo' + end.should == nil + end end spec/integration_spec.rb 102804e1df5e879dd64b77040e3390048dadbcb1 Martyn Loughran me@mloughran.com http://github.com/mloughran/api_cache/commit/2a351cce1938dbab0cc98e8d9038011388c26c87 2a351cce1938dbab0cc98e8d9038011388c26c87 2009-08-20T17:55:30-07:00 2009-08-20T17:50:34-07:00 Support the :fail parameter - cleaner than catching exceptions in simple cases 428ca87fc57977fa1aed89a52f42f994e1d52ea8 Martyn Loughran me@mloughran.com