padrino-cache 0.16.1 → 0.17.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4de7f247e914cd078c49862002f8ef8148a0fc0b181c19e423f2eba3b69823b8
-  data.tar.gz: 07e51c4ef2e8f367045aaa0a42399e8601a71cb0d0e5b24af0e86ed5f59782cf
+  metadata.gz: debf35ad318b1406cca39e13b78606d3a6c907871c86f80f3597dd8f3e102f7b
+  data.tar.gz: 6d0348ce0bb9c75715968b52c45cbf0f3ddfba3ccfd6c46bfc5788efe8bcb47a
 SHA512:
-  metadata.gz: 1b2facd8f8494a3b419e5e018a52b698f1c6aec5f371b295700553365786e98321d47189c6cbde8ac2d6cde4cd51426f3023c1569f71ecdef55e37ee460a864b
-  data.tar.gz: 33236b44a6d640beaffdbe0589f75c46da33a5797531fe2a80d72d99476a84bf6c43d11c4ba14bfab3e52dc5ff5c42986806de6b1a37643dab322ac5c1d8bcf5
+  metadata.gz: '08d3aff0107b8308cbef75f856c186210843d0338b0948376c859817b20961ec37dc86bba22f05a99012cb8d4f4fd8d907e8d34d6cfddd822c5d1a55e597e844'
+  data.tar.gz: 69227bd4456285c96c3074c0b55dd839a7ce4f22eeef12b5c8514606db18bc5a70710f41de316a419ec7ac786d8d80bca90f5c45eb10bd372430e15df4373c2f

data/lib/padrino-cache/helpers/cache_object.rb

@@ -7,11 +7,11 @@ module Padrino
             began_at = Time.now
             if settings.cache.key?(key.to_s)
               value = settings.cache[key.to_s]
-              logger.debug 'GET Object', began_at, key.to_s if defined?(logger)
+              logger.bench 'GET Object', began_at, key.to_s if defined?(logger) && logger.respond_to?(:bench) && logger.debug?
             else
               value = yield
               settings.cache.store(key.to_s, value, opts)
-              logger.debug 'SET Object', began_at, key.to_s if defined?(logger)
+              logger.bench 'SET Object', began_at, key.to_s if defined?(logger) && logger.respond_to?(:bench) && logger.debug?
             end
             value
           else

data/lib/padrino-cache/helpers/fragment.rb

@@ -53,12 +53,12 @@ module Padrino
             began_at = Time.now
             if settings.cache.key?(key.to_s)
               value = settings.cache[key.to_s]
-              logger.debug 'GET Fragment', began_at, key.to_s if defined?(logger)
+              logger.bench 'GET Fragment', began_at, key.to_s if defined?(logger) && logger.respond_to?(:bench) && logger.debug?
               concat_content(value.to_s.html_safe)
             else
               value = capture_html(&block)
               settings.cache.store(key.to_s, value, opts)
-              logger.debug 'SET Fragment', began_at, key.to_s if defined?(logger)
+              logger.bench 'SET Fragment', began_at, key.to_s if defined?(logger) && logger.respond_to?(:bench) && logger.debug?
               concat_content(value)
             end
           else

data/lib/padrino-cache/helpers/page.rb

@@ -117,7 +117,7 @@ module Padrino
           route_cache_key = resolve_cache_key || env['PATH_INFO']
 
           value = settings.cache[route_cache_key]
-          logger.debug 'GET Cache', began_at, route_cache_key if defined?(logger) && value
+          logger.bench 'GET Cache', began_at, route_cache_key if defined?(logger) && logger.respond_to?(:bench) && logger.debug? && value
 
           value
         end
@@ -135,7 +135,7 @@ module Padrino
 
           settings.cache.store(route_cache_key, content, expires: cache_expires)
 
-          logger.debug 'SET Cache', began_at, route_cache_key if defined?(logger)
+          logger.bench 'SET Cache', began_at, route_cache_key if defined?(logger) && logger.respond_to?(:bench) && logger.debug?
         end
 
         ##

data/test/helper.rb

@@ -2,6 +2,11 @@ ENV['RACK_ENV'] = 'test'
 PADRINO_ROOT = __dir__ unless defined?(PADRINO_ROOT)
 
 require 'minitest/autorun'
+begin
+  require 'minitest/mock'
+rescue LoadError
+  # minitest/mock is built-in for older minitest versions
+end
 require 'minitest/pride'
 require 'rack/test'
 require 'padrino-cache'

data/test/test_padrino_cache.rb

@@ -22,6 +22,51 @@ describe 'PadrinoCache' do
     refute_equal called, false
   end
 
+  it 'should cache with a logger without Padrino::Logger::Extensions' do
+    log = StringIO.new
+    save_logger = Padrino.logger
+    Padrino.logger = Logger.new(log)
+    mock_app do
+      register Padrino::Cache
+      enable :caching
+      get('/foo') { cache(:test) { 'test fragment' } }
+      get('/bar', cache: true) { cache_object(:object) { 'test object' } }
+    end
+    2.times do
+      get '/foo'
+      assert_equal 200, status
+      assert_equal 'test fragment', body
+      get '/bar'
+      assert_equal 200, status
+      assert_equal 'test object', body
+    end
+    ['SET FRAGMENT', 'GET FRAGMENT', 'SET OBJECT', 'SET CACHE', 'GET CACHE'].each do |action|
+      assert_match(/#{action} \(\d+\.\d+s\)/, log.string)
+    end
+  ensure
+    Padrino.logger = save_logger
+  end
+
+  it 'should cache with a logger that lacks bench' do
+    log = StringIO.new
+    mock_app do
+      register Padrino::Cache
+      enable :caching
+      helpers { define_method(:logger) { @logger ||= Logger.new(log).freeze } }
+      get('/foo') { cache(:test) { 'test fragment' } }
+      get('/bar', cache: true) { cache_object(:object) { 'test object' } }
+      get('/baz') { cache_object(:uncached_page) { 'object only' } }
+    end
+    2.times do
+      get '/foo'
+      assert_equal 'test fragment', body
+      get '/bar'
+      assert_equal 'test object', body
+      get '/baz'
+      assert_equal 'object only', body
+    end
+  end
+
   it 'should cache a page' do
     called = false
     mock_app do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: padrino-cache
 version: !ruby/object:Gem::Version
-  version: 0.16.1
+  version: 0.17.0
 platform: ruby
 authors:
 - Padrino Team
@@ -11,7 +11,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-03-27 00:00:00.000000000 Z
+date: 2026-10-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: padrino-core
@@ -19,28 +19,28 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.16.1
+        version: 0.17.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.16.1
+        version: 0.17.0
 - !ruby/object:Gem::Dependency
   name: padrino-helpers
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.16.1
+        version: 0.17.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.16.1
+        version: 0.17.0
 - !ruby/object:Gem::Dependency
   name: moneta
   requirement: !ruby/object:Gem::Requirement