feat: make sure the header document id does not get overwritten by op…

…tions

lib/esse/lazy_document_header.rb

@@ -17,7 +17,7 @@ def self.coerce(value)
       if value.is_a?(Esse::LazyDocumentHeader)
         value
       elsif value.is_a?(Esse::Document)
-        new(id: value.id, type: value.type, routing: value.routing, **value.options)
+        new(**value.options, id: value.id, type: value.type, routing: value.routing)
       elsif value.is_a?(Hash)
         resp = value.transform_keys do |key|
           case key

spec/esse/lazzy_document_header_spec.rb

@@ -145,6 +145,26 @@
         expect(described_class.coerce(object)).to be_a(described_class)
       end
     end
+
+    context 'when value is a document with id in options' do
+      let(:object) do
+        Class.new(Esse::Document) do
+          def options
+            { id: 1 }
+          end
+
+          def id
+            2
+          end
+        end.new(nil)
+      end
+
+      it 'returns a LazyDocumentHeader instance with the proper id' do
+        instance = described_class.coerce(object)
+        expect(instance).to be_a(described_class)
+        expect(instance.id).to eq(2)
+      end
+    end
   end
 
   describe '.coerce_each' do
@@ -173,6 +193,11 @@
     it 'flattens the array' do
       expect(described_class.coerce_each([[{_id: 1}], {_id: 2}]).size).to eq(2)
     end
+
+    it 'coerces a list of Esse::Document instances' do
+      list = [Esse::HashDocument.new(_id: 1), Class.new(Esse::HashDocument).new(_id: 2)]
+      expect(described_class.coerce_each(list)).to all(be_a(described_class))
+    end
   end
 
   describe '#to_doc' do