encapsulate

Author: khvn26

src/main/java/com/flagsmith/mappers/EngineMappers.java

@@ -243,10 +243,7 @@ private static Map<String, SegmentContext> mapIdentityOverridesToSegments(
       SegmentMetadata metadata = new SegmentMetadata();
       metadata.setSource(SegmentMetadata.Source.IDENTITY_OVERRIDES);
 
-      Map<String, Object> metadataMap = MapperFactory.getMapper()
-          .convertValue(metadata, 
-              new com.fasterxml.jackson.core.type.TypeReference<Map<String, Object>>() {
-              });
+      Map<String, Object> metadataMap = mapSegmentMetadataToMetadataMap(metadata);
 
       SegmentContext segmentContext = new SegmentContext()
           .withKey("") // Identity override segments never use % Split operator
@@ -416,10 +413,7 @@ private static SegmentContext mapSegmentToSegmentContext(SegmentModel segment) {
     metadata.setSource(SegmentMetadata.Source.API);
     metadata.setFlagsmithId(segment.getId());
 
-    Map<String, Object> metadataMap = MapperFactory.getMapper()
-          .convertValue(metadata,
-            new com.fasterxml.jackson.core.type.TypeReference<Map<String, Object>>() {
-            });
+    Map<String, Object> metadataMap = mapSegmentMetadataToMetadataMap(metadata);
 
     String segmentKey = String.valueOf(segment.getId());
     return new SegmentContext()
@@ -456,4 +450,10 @@ private static String getVirtualSegmentKey(
     // This is safer than using List.hashCode() as we control the string content
     return String.valueOf(keyBuilder.toString().hashCode());
   }
+
+  private static Map<String, Object> mapSegmentMetadataToMetadataMap(SegmentMetadata metadata) {
+    return MapperFactory.getMapper().convertValue(metadata,
+        new com.fasterxml.jackson.core.type.TypeReference<Map<String, Object>>() {
+        });
+  }
 }