fix: repository updateField ignores registered Gson TypeAdapters (res…

…olves gh-284)
redis · Jun 24, 2023 · 1288ece · 1288ece
1 parent 2d60af4
commit 1288ece
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/redis-om-spring/src/main/java/com/redis/om/spring/ops/json/JSONOperationsImpl.java b/redis-om-spring/src/main/java/com/redis/om/spring/ops/json/JSONOperationsImpl.java
@@ -89,7 +89,7 @@ public void set(K key, Object object) {
 
   @Override
   public void set(K key, Object object, Path path) {
-    client.clientForJSON().jsonSet(key.toString(), path, object);
+    client.clientForJSON().jsonSetWithPlainString(key.toString(), path, builder.gson().toJson(object));
   }
 
   @Override

diff --git a/...src/test/java/com/redis/om/spring/annotations/document/BasicRedisDocumentMappingTest.java b/...src/test/java/com/redis/om/spring/annotations/document/BasicRedisDocumentMappingTest.java
@@ -12,6 +12,7 @@
 import redis.clients.jedis.json.Path;
 
 import java.time.LocalDate;
+import java.time.LocalDateTime;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Optional;
@@ -35,6 +36,9 @@ class BasicRedisDocumentMappingTest extends AbstractBaseDocumentTest {
   @Autowired
   DocWithEnumRepository docWithEnumRepository;
 
+  @Autowired
+  SomeDocumentRepository someDocumentRepository;
+
   @BeforeEach
   void cleanUp() {
     flushSearchIndexFor(Company.class);
@@ -117,6 +121,18 @@ void testUpdateSingleField() {
     assertThat(maybeRedis).isPresent().map(Company::getName).contains("Redis");
   }
 
+  @Test
+  void testUpdateLocalDateTimeField() {
+    var now = LocalDateTime.now();
+    SomeDocument docWithDateTime = new SomeDocument();
+    docWithDateTime.setDocumentCreationDate(now);
+    docWithDateTime = someDocumentRepository.save(docWithDateTime);
+
+    someDocumentRepository.updateField(docWithDateTime, SomeDocument$.DOCUMENT_CREATION_DATE, now.minusDays(5));
+
+    assertThat(someDocumentRepository.findById(docWithDateTime.getId()).get().getDocumentCreationDate()).isEqualToIgnoringNanos(now.minusDays(5));
+  }
+
   @Test
   void testAuditAnnotations() {
     Company redis = repository.save(