Allow custom serialization based on the operation

Author: benedikt

lib/userlist/push/operations/create.rb

@@ -5,8 +5,9 @@ module Create
         module ClassMethods
           def create(payload = {}, config = self.config)
             return false unless resource = from_payload(payload, config)
+            return false unless resource.create?
 
-            strategy.call(:post, endpoint, resource) if resource.create?
+            strategy.call(:post, endpoint, resource.for_context(:create))
           end
 
           alias push create

lib/userlist/push/operations/delete.rb

@@ -5,8 +5,9 @@ module Delete
         module ClassMethods
           def delete(payload = {}, config = self.config)
             return false unless resource = from_payload(payload, config)
+            return false unless resource.delete?
 
-            strategy.call(:delete, endpoint, resource) if resource.delete?
+            strategy.call(:delete, endpoint, resource.for_context(:delete))
           end
         end
 

lib/userlist/push/resource.rb

@@ -58,13 +58,14 @@ def generated_methods
         end
       end
 
-      attr_reader :payload, :config
+      attr_reader :payload, :config, :context
 
       def initialize(payload = {}, config = Userlist.config)
         raise Userlist::ArgumentError, 'Missing required payload' unless payload
 
         @payload = payload
         @config = config
+        @context = :push
       end
 
       def respond_to_missing?(method, include_private = false)
@@ -73,7 +74,7 @@ def respond_to_missing?(method, include_private = false)
       end
 
       def to_hash
-        Serializer.serialize(self, context: :push)
+        Serializer.serialize(self, context: context)
       end
       alias to_h to_hash
 
@@ -106,6 +107,12 @@ def push?
         true
       end
 
+      def for_context(context)
+        dup.tap do |instance|
+          instance.instance_variable_set(:@context, context)
+        end
+      end
+
     private
 
       def method_missing(method, *args, &block)

spec/userlist/push/operations/create_spec.rb

@@ -36,6 +36,25 @@
         expect(strategy).to receive(:call).with(:post, '/users', resource)
         relation.create(payload)
       end
+
+      it 'should set the context to :create' do
+        expect(strategy).to receive(:call).with(:post, '/users', satisfy { |r| r.context == :create })
+        relation.create(payload)
+      end
+    end
+
+    context 'when given a resource instance' do
+      let(:payload) { resource_type.from_payload({ identifier: 'identifier' }) }
+
+      it 'should send the request to the endpoint' do
+        expect(strategy).to receive(:call).with(:post, '/users', resource)
+        relation.create(payload)
+      end
+
+      it 'should set the context to :create' do
+        expect(strategy).to receive(:call).with(:post, '/users', satisfy { |r| r.context == :create })
+        relation.create(payload)
+      end
     end
 
     context 'when given an identifier' do

spec/userlist/push/operations/delete_spec.rb

@@ -17,6 +17,11 @@
         expect(strategy).to receive(:call).with(:delete, '/users', resource)
         relation.delete(payload)
       end
+
+      it 'should set the context to :create' do
+        expect(strategy).to receive(:call).with(:delete, '/users', satisfy { |r| r.context == :delete })
+        relation.delete(payload)
+      end
     end
 
     context 'when given a payload hash' do
@@ -28,6 +33,25 @@
         expect(strategy).to receive(:call).with(:delete, '/users', resource)
         relation.delete(payload)
       end
+
+      it 'should set the context to :create' do
+        expect(strategy).to receive(:call).with(:delete, '/users', satisfy { |r| r.context == :delete })
+        relation.delete(payload)
+      end
+    end
+
+    context 'when given a resource instance' do
+      let(:payload) { resource_type.from_payload({ identifier: identifier }) }
+
+      it 'should send the request to the endpoint' do
+        expect(strategy).to receive(:call).with(:delete, '/users', resource)
+        relation.delete(payload)
+      end
+
+      it 'should set the context to :create' do
+        expect(strategy).to receive(:call).with(:delete, '/users', satisfy { |r| r.context == :delete })
+        relation.delete(payload)
+      end
     end
 
     context 'when given nil' do