fix(document-handling): fixes after Simon comment (#3811)

* fix(document-handling): fixes after Simon comment * fix(document-handling): fixes after Simon comment 2
camunda · Dec 16, 2024 · 0efd81c · 0efd81c
1 parent b10324a
commit 0efd81c
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 16 deletions.
diff --git a/.../test/src/main/java/io/camunda/connector/test/inbound/InboundConnectorContextBuilder.java b/.../test/src/main/java/io/camunda/connector/test/inbound/InboundConnectorContextBuilder.java
@@ -55,12 +55,8 @@ public class InboundConnectorContextBuilder {
   protected DocumentFactory documentFactory =
       new DocumentFactoryImpl(InMemoryDocumentStore.INSTANCE);
   protected ObjectMapper objectMapper =
-      ConnectorsObjectMapperSupplier.getCopy()
-          .registerModule(
-              new JacksonModuleDocument(
-                  this.documentFactory,
-                  null,
-                  JacksonModuleDocument.DocumentModuleSettings.create()));
+      ConnectorsObjectMapperSupplier.getCopy(
+          this.documentFactory, JacksonModuleDocument.DocumentModuleSettings.create());
 
   public static InboundConnectorContextBuilder create() {
     return new InboundConnectorContextBuilder();

diff --git a/...est/src/main/java/io/camunda/connector/test/outbound/OutboundConnectorContextBuilder.java b/...est/src/main/java/io/camunda/connector/test/outbound/OutboundConnectorContextBuilder.java
@@ -46,12 +46,8 @@ public class OutboundConnectorContextBuilder {
   protected DocumentFactory documentFactory =
       new DocumentFactoryImpl(InMemoryDocumentStore.INSTANCE);
   private ObjectMapper objectMapper =
-      ConnectorsObjectMapperSupplier.getCopy()
-          .registerModule(
-              new JacksonModuleDocument(
-                  this.documentFactory,
-                  null,
-                  JacksonModuleDocument.DocumentModuleSettings.create()));
+      ConnectorsObjectMapperSupplier.getCopy(
+          this.documentFactory, JacksonModuleDocument.DocumentModuleSettings.create());
 
   /**
    * @return a new instance of the {@link OutboundConnectorContextBuilder}

diff --git a/...-test-rabbitmq/src/test/java/io/camunda/connector/e2e/RabbitMqInboundStartEventTests.java b/...-test-rabbitmq/src/test/java/io/camunda/connector/e2e/RabbitMqInboundStartEventTests.java
@@ -19,6 +19,7 @@
 import static io.camunda.connector.e2e.BpmnFile.replace;
 import static org.mockito.Mockito.when;
 
+import com.fasterxml.jackson.databind.ObjectMapper;
 import com.rabbitmq.client.Channel;
 import com.rabbitmq.client.Connection;
 import com.rabbitmq.client.ConnectionFactory;
@@ -64,6 +65,7 @@ public class RabbitMqInboundStartEventTests extends BaseRabbitMqTest {
   private static String PORT;
   private static RabbitMQContainer rabbitMQContainer;
   private static ConnectionFactory factory;
+  private final ObjectMapper objectMapper = ConnectorsObjectMapperSupplier.getCopy();
 
   @Autowired ProcessStateStore processStateStore;
 
@@ -131,10 +133,9 @@ public void uriAuthenticationReceiveMessageTest() throws Exception {
 
   private void assertIntermediateCatchEventUsingModel(BpmnModelInstance model) throws Exception {
     Object expectedJsonResponse =
-        ConnectorsObjectMapperSupplier.getCopy()
-            .readValue(
-                "{\"message\":{\"consumerTag\":\"myConsumerTag\",\"body\":{\"foo\": {\"bar\": \"barValue\"}},\"properties\":{}}}",
-                Object.class);
+        this.objectMapper.readValue(
+            "{\"message\":{\"consumerTag\":\"myConsumerTag\",\"body\":{\"foo\": {\"bar\": \"barValue\"}},\"properties\":{}}}",
+            Object.class);
 
     processStateStore.update(mockProcessDefinition(model));