Merge pull request #127 from richmolj/defer_sideload

Defer serializer relationships when eager loading

lib/graphiti/resource/configuration.rb

@@ -10,7 +10,6 @@ def serializer=(val)
           if val
             if super(Class.new(val))
               apply_attributes_to_serializer
-              apply_sideloads_to_serializer
             end
           else
             super

spec/resource_spec.rb

@@ -67,6 +67,34 @@
         expect(klass.relationships_writable_by_default).to eq(true)
       end
 
+      context 'when rails' do
+        before do
+          rails = double \
+            application: double(config: double(eager_load: eager_load))
+          stub_const('Rails', rails)
+        end
+
+        context 'and eager loading' do
+          let(:eager_load) { true }
+
+          it 'does NOT assign relationships to the serializer' do
+            klass.has_many :positions, resource: PORO::PositionResource
+            expect(klass.serializer.relationship_blocks).to be_blank
+            Graphiti.setup!
+            expect(klass.serializer.relationship_blocks).to_not be_blank
+          end
+        end
+
+        context 'and NOT eager loading' do
+          let(:eager_load) { false }
+
+          it 'assigns relationships to the serializer' do
+            klass.has_many :positions, resource: PORO::PositionResource
+            expect(klass.serializer.relationship_blocks).to_not be_blank
+          end
+        end
+      end
+
       context "when assigning new adapter" do
         let(:adapter) do
           Class.new(Graphiti::Adapters::Abstract) do