From 92e2ed78d1db8385d6bbb0b3446653ad7362f9c5 Mon Sep 17 00:00:00 2001 From: Pau Perez Date: Tue, 18 Dec 2018 17:26:37 +0100 Subject: [PATCH] Test active distributor as well --- .../api/cached_enterprise_serializer_spec.rb | 26 ++++++++++++++----- 1 file changed, 20 insertions(+), 6 deletions(-) diff --git a/spec/serializers/api/cached_enterprise_serializer_spec.rb b/spec/serializers/api/cached_enterprise_serializer_spec.rb index 92b62ea919..39c3a5fde1 100644 --- a/spec/serializers/api/cached_enterprise_serializer_spec.rb +++ b/spec/serializers/api/cached_enterprise_serializer_spec.rb @@ -30,9 +30,6 @@ describe Api::CachedEnterpriseSerializer do let(:shop) { create(:distributor_enterprise) } let(:options) { { data: enterprise_injection_data } } - let(:enterprise_injection_data) do - instance_double(OpenFoodNetwork::EnterpriseInjectionData, active_distributors: []) - end let(:property) { create(:property, presentation: 'One') } let(:duplicate_property) { create(:property, presentation: 'One') } @@ -51,9 +48,26 @@ describe Api::CachedEnterpriseSerializer do ) end - it 'does not duplicate properties' do - properties = cached_enterprise_serializer.distributed_properties - expect(properties).to eq([property]) + context 'when the enterprise is not an active distributor' do + let(:enterprise_injection_data) do + instance_double(OpenFoodNetwork::EnterpriseInjectionData, active_distributors: []) + end + + it 'does not duplicate properties' do + properties = cached_enterprise_serializer.distributed_properties + expect(properties).to eq([property]) + end + end + + context 'when the enterprise is an active distributor' do + let(:enterprise_injection_data) do + instance_double(OpenFoodNetwork::EnterpriseInjectionData, active_distributors: [shop]) + end + + it 'does not duplicate properties' do + properties = cached_enterprise_serializer.distributed_properties + expect(properties).to eq([property]) + end end end end