From e55fbf80fc2b5d9d29df69c3b75c8e7a41268fc4 Mon Sep 17 00:00:00 2001 From: Pau Perez Date: Wed, 19 Dec 2018 11:52:07 +0100 Subject: [PATCH] Check property presentation attribute only in test The PropertySerializer contains :id, :name and :presentation attributes which leads to JSON objects like `{"id": 1, "name": "foo", "presentation": "foo"}`. Because of this, we don't care about object identity, just about their presentation attribute. --- .../api/cached_enterprise_serializer_spec.rb | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/spec/serializers/api/cached_enterprise_serializer_spec.rb b/spec/serializers/api/cached_enterprise_serializer_spec.rb index 0d3cd642bf..4401409cb2 100644 --- a/spec/serializers/api/cached_enterprise_serializer_spec.rb +++ b/spec/serializers/api/cached_enterprise_serializer_spec.rb @@ -55,15 +55,15 @@ describe Api::CachedEnterpriseSerializer do it 'does not duplicate properties' do properties = cached_enterprise_serializer.distributed_properties - expect(properties).to eq([property]) + expect(properties.map(&:presentation)).to eq([property.presentation]) end it 'fetches producer properties' do distributed_producer_properties = cached_enterprise_serializer .distributed_producer_properties - expect(distributed_producer_properties) - .to eq(producer.producer_properties.map(&:property)) + expect(distributed_producer_properties.map(&:presentation)) + .to eq(producer.producer_properties.map(&:property).map(&:presentation)) end end @@ -74,15 +74,15 @@ describe Api::CachedEnterpriseSerializer do it 'does not duplicate properties' do properties = cached_enterprise_serializer.distributed_properties - expect(properties).to eq([property]) + expect(properties.map(&:presentation)).to eq([property.presentation]) end it 'fetches producer properties' do distributed_producer_properties = cached_enterprise_serializer .distributed_producer_properties - expect(distributed_producer_properties) - .to eq(producer.producer_properties.map(&:property)) + expect(distributed_producer_properties.map(&:presentation)) + .to eq(producer.producer_properties.map(&:property).map(&:presentation)) end end end