mirror of
https://github.com/ditkrg/active_model_serializers.git
synced 2026-01-23 06:16:50 +00:00
Clean up interface inconistency
This commit is contained in:
parent
535a33a33b
commit
e3888f0a40
@ -74,7 +74,7 @@ module ActiveModel
|
|||||||
|
|
||||||
class << self
|
class << self
|
||||||
# set peform caching like root
|
# set peform caching like root
|
||||||
def cache(value = true)
|
def cached(value = true)
|
||||||
self.perform_caching = value
|
self.perform_caching = value
|
||||||
end
|
end
|
||||||
|
|
||||||
@ -441,7 +441,7 @@ module ActiveModel
|
|||||||
end
|
end
|
||||||
|
|
||||||
def perform_caching?
|
def perform_caching?
|
||||||
perform_caching && cache && try(:cache_key)
|
perform_caching && cache && respond_to?(:cache_key)
|
||||||
end
|
end
|
||||||
|
|
||||||
def expand_cache_key(*args)
|
def expand_cache_key(*args)
|
||||||
|
|||||||
@ -38,12 +38,12 @@ class CachingTest < ActiveModel::TestCase
|
|||||||
def test_serializers_have_a_cache_store
|
def test_serializers_have_a_cache_store
|
||||||
ActiveModel::Serializer.cache = NullStore.new
|
ActiveModel::Serializer.cache = NullStore.new
|
||||||
|
|
||||||
assert ActiveModel::Serializer.cache
|
assert_kind_of NullStore, ActiveModel::Serializer.cache
|
||||||
end
|
end
|
||||||
|
|
||||||
def test_serializers_can_enable_caching
|
def test_serializers_can_enable_caching
|
||||||
serializer = Class.new(ActiveModel::Serializer) do
|
serializer = Class.new(ActiveModel::Serializer) do
|
||||||
cache true
|
cached true
|
||||||
end
|
end
|
||||||
|
|
||||||
assert serializer.perform_caching
|
assert serializer.perform_caching
|
||||||
@ -51,7 +51,7 @@ class CachingTest < ActiveModel::TestCase
|
|||||||
|
|
||||||
def test_serializers_cache_serializable_hash
|
def test_serializers_cache_serializable_hash
|
||||||
serializer = Class.new(ActiveModel::Serializer) do
|
serializer = Class.new(ActiveModel::Serializer) do
|
||||||
cache true
|
cached true
|
||||||
attributes :name, :skills
|
attributes :name, :skills
|
||||||
|
|
||||||
def self.to_s
|
def self.to_s
|
||||||
@ -76,7 +76,7 @@ class CachingTest < ActiveModel::TestCase
|
|||||||
|
|
||||||
def test_serializers_cache_to_json
|
def test_serializers_cache_to_json
|
||||||
serializer = Class.new(ActiveModel::Serializer) do
|
serializer = Class.new(ActiveModel::Serializer) do
|
||||||
cache true
|
cached true
|
||||||
attributes :name, :skills
|
attributes :name, :skills
|
||||||
|
|
||||||
def self.to_s
|
def self.to_s
|
||||||
@ -101,7 +101,7 @@ class CachingTest < ActiveModel::TestCase
|
|||||||
|
|
||||||
def test_can_use_defined_cache_key
|
def test_can_use_defined_cache_key
|
||||||
serializer = Class.new(ActiveModel::Serializer) do
|
serializer = Class.new(ActiveModel::Serializer) do
|
||||||
cache true
|
cached true
|
||||||
attributes :name, :skills
|
attributes :name, :skills
|
||||||
|
|
||||||
def self.to_s
|
def self.to_s
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user