RDBC-812 Don't send 'Id' property to the server

Author: poissoncorp

ravendb/documents/session/entity_to_json.py

@@ -38,7 +38,7 @@ def convert_entity_to_json(self, entity: object, document_info: DocumentInfo) ->
             self._session.before_conversion_to_document_invoke(
                 BeforeConversionToDocumentEventArgs(document_info.key, entity, self._session)
             )
-        document = EntityToJson._convert_entity_to_json_internal(self, entity, document_info)
+        document = EntityToJson._convert_entity_to_json_internal(self, entity, document_info, True)
         if document_info is not None:
             self._session.after_conversion_to_document_invoke(
                 AfterConversionToDocumentEventArgs(self._session, document_info.key, entity, document)

ravendb/tests/issue_tests/test_issue_198.py

@@ -1,3 +1,5 @@
+from ravendb.infrastructure.orders import Company
+from ravendb.primitives import constants
 from ravendb.tests.test_base import TestBase
 
 
@@ -22,6 +24,28 @@ def test_create_document_with_server_generated_guid(self):
         with self.store.open_session() as session:
             product = session.load(guid_from_server)
             self.assertEqual("Test", product.name)
+            metadata_id = session.advanced.get_metadata_for(product)["@id"]
+            identity = session.advanced.get_document_id(product)
+            id_prop = product.Id
+
+            self.assertEqual(id_prop, identity)
+            self.assertEqual(id_prop, metadata_id)
+            self.assertEqual(id_prop, guid_from_server)
+
+        with self.store.open_session() as session:
+            product = session.load(guid_from_server)
+            product.Id = ""
+            session.save_changes()
+
+        with self.store.open_session() as session:
+            product = session.load(guid_from_server)
+            metadata_id1 = session.advanced.get_metadata_for(product)[constants.Documents.Metadata.ID]
+            identity1 = session.advanced.get_document_id(product)
+            id_prop1 = product.Id
+
+            self.assertEqual(id_prop, id_prop1)
+            self.assertEqual(id_prop, metadata_id1)
+            self.assertEqual(id_prop, identity1)
 
     def test_can_create_multiple_documents_with_server_generated_guid(self):
         with self.store.open_session() as session:
@@ -45,3 +69,14 @@ def test_can_create_multiple_documents_with_server_generated_guid(self):
 
             product3 = session.load(guid_from_server3)
             self.assertEqual("Test3", product3.name)
+
+    def test_preserve_id_field(self):
+        with self.store.open_session() as session:
+            company = Company(Id="companies/222", name="Apple Inc.")
+            session.store(company)
+            session.save_changes()
+
+        with self.store.open_session() as session:
+            company = session.load("companies/222", Company)
+            self.assertEqual("companies/222", company.Id)
+            self.assertEqual("Apple Inc.", company.name)

ravendb/tests/jvm_migrated_tests/crud_tests/test_basic_document.py

@@ -13,7 +13,7 @@ def setUp(self):
         super(TestBasicDocument, self).setUp()
 
     def test_get(self):
-        dummy = {"name": None, "age": None, "Id": None}
+        dummy = {"name": None, "age": None}
         with self.store.open_session() as session:
             user1 = User("jacus", None)
             user2 = User("arek", None)

ravendb/tests/jvm_migrated_tests/issues_tests/test_RDBC_387.py

@@ -25,5 +25,5 @@ def test_should_store_single_id(self):
             self.assertIsNone(session_user.get("id", None))
             self.assertIsNone(bulk_insert_user.get("id", None))
 
-            self.assertIsNotNone(session_user.get("Id", None))
+            self.assertIsNone(session_user.get("Id", None))
             self.assertIsNone(bulk_insert_user.get("Id", None))

ravendb/tests/jvm_migrated_tests/server_tests/patching/test_advanced_patching.py

@@ -46,7 +46,7 @@ def to_json(self) -> Dict[str, Any]:
     @classmethod
     def from_json(cls, json_dict: Dict[str, Any]) -> CustomType:
         return cls(
-            json_dict["Id"],
+            json_dict["Id"] if "Id" in json_dict else None,
             json_dict["owner"],
             json_dict["value"],
             json_dict["comments"],

ravendb/tools/utils.py

@@ -450,7 +450,16 @@ def fill_with_nested_object_types(entity: object, nested_object_types: Dict[str,
     @staticmethod
     def initialize_object(obj: dict, object_type: Type[_T], convert_to_snake_case: bool = None) -> _T:
         initialize_dict, set_needed = Utils.make_initialize_dict(obj, object_type.__init__, convert_to_snake_case)
-        o = object_type(**initialize_dict)
+        try:
+            o = object_type(**initialize_dict)
+        except Exception as e:
+            if "Id" not in initialize_dict:
+                initialize_dict["Id"] = None
+                o = object_type(**initialize_dict)
+            else:
+                raise TypeError(
+                    f"Couldn't initialize object of type '{object_type.__name__}' using dict '{obj}'"
+                ) from e
         if set_needed:
             for key, value in obj.items():
                 setattr(o, key, value)